Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Discussion] Ideas / Improvements / In-Use #34

Closed
gsantner opened this issue Jun 24, 2020 · 4 comments
Closed

[Discussion] Ideas / Improvements / In-Use #34

gsantner opened this issue Jun 24, 2020 · 4 comments
Labels
enhancement New feature or request

Comments

@gsantner
Copy link

Hello, thanks for this project. I'm using since a while now and pretty happy with it.

I thought to open a issue for discussion and sharing ideas to improve the project.
Myself having some dockerfile changes (and custom docker image), that I wanted to share, maybe the one or other might sense to be picked up by the project by default.

@gsantner
Copy link
Author

My overall changes to Dockerfile:

grafik

@ehough
What do you think about making these changes on master?:

  • Use DEBIAN_FRONTEND=noninteractive before apt calls, and export it as env var. I have run into cases where docker build would hang/freeze and waiting for input at apt.
  • Add apt pkg libnss3, required for DRM TV Addons to work.
  • Add apt pkg curl, helpful for health checks and some addons might use some executables that use normal-non-python curl.
  • For me, alsa-utils where rerquired for audio to work properly, not sure if it makes it work for others as well

Note, when libnss3 is not installed:

kodi-screenshot

@ehough ehough added the enhancement New feature or request label Jul 31, 2020
@ehough
Copy link
Owner

ehough commented Jul 31, 2020

Thanks so much for these great suggestions! And sorry for my late response.

Use DEBIAN_FRONTEND=noninteractive before apt calls, and export it as env var. I have run into cases where docker build would hang/freeze and waiting for input at apt.

Good idea. I'll add it as an ARG instead of ENV. See here for the rationale.

Add apt pkg libnss3, required for DRM TV Addons to work.

I'll be merging that PR (I promise!) as soon as I'm able to run some local testing. Did you also have to install libnss3-tools? Or just libnss3?

Add apt pkg curl, helpful for health checks and some addons might use some executables that use normal-non-python curl.

Any chance you have some examples of add-ons that invoke the curl binary? I know it's a fairly standard-issue tool to have installed, but I want to make sure that any package added to the base image has a compelling reason to be there.

For me, alsa-utils where rerquired for audio to work properly, not sure if it makes it work for others as well

Interesting. You don't remember what error message or behavior you saw, by chance? I'll investigate adding that package.

Last note for now: if you haven't considered it already, you could just extend this image to make your custom mods. e.g.

FROM erichough/kodi

RUN dpkg --add-architecture i386 && \
    sed -i ...
   ...

Know what I mean?

That way you wouldn't have to update your image every time I update the base image.

@gsantner
Copy link
Author

Thanks so much for these great suggestions! And sorry for my late response.

Use DEBIAN_FRONTEND=noninteractive before apt calls, and export it as env var. I have run into cases where docker build would hang/freeze and waiting for input at apt.

Good idea. I'll add it as an ARG instead of ENV. See here for the rationale.

don't really care here

Add apt pkg libnss3, required for DRM TV Addons to work.

I'll be merging that PR (I promise!) as soon as I'm able to run some local testing. Did you also have to install libnss3-tools? Or just libnss3?

I don't have tools.

Add apt pkg curl, helpful for health checks and some addons might use some executables that use normal-non-python curl.

Any chance you have some examples of add-ons that invoke the curl binary? I know it's a fairly standard-issue tool to have installed, but I want to make sure that any package added to the base image has a compelling reason to be there.

Often things don't work, and then you can bash into docker container. And if not even basics are there it's troublesome to debug why EPG streams.xml doesn't download.

For me, alsa-utils where rerquired for audio to work properly, not sure if it makes it work for others as well

Interesting. You don't remember what error message or behavior you saw, by chance? I'll investigate adding that package.

No error message. Just nothing with audio or any audio output channel was working for me.

Last note for now: if you haven't considered it already, you could just extend this image to make your custom mods. e.g.

FROM erichough/kodi

RUN dpkg --add-architecture i386 && \
    sed -i ...
   ...

Know what I mean?

That way you wouldn't have to update your image every time I update the base image.

Yeah but I found it easier to maintain forked Dockerfile for myself. For the reason, I can independently update/downgrade Kodi version if new update is out..or something doesn't work.

@gsantner
Copy link
Author

gsantner commented May 19, 2024

Closing. Don't use Kodi in Docker anymore. It's been 4 years and there are better options now (Flatpak/Snap) with similar goal...even supported by the Kodi project itself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants