linuxserver / docker-jellyfin

GNU General Public License v3.0
630 stars 94 forks source link

Don't `chown /data` #125

Closed kaysond closed 3 years ago

kaysond commented 3 years ago

linuxserver.io



Description:

Remove some commands from 30-config in cont-init.d to avoid changing permissions on /data

Benefits of this PR and context:

Some users manage their own media permissions and don't want them touched by the container. Documentation is clear that proper permissions are needed for the container to work. Addresses #62

How Has This Been Tested?

Built and launched container

Source / References:

LinuxServer-CI commented 3 years ago

I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/jellyfin/nightly-20210812.1-unstable-pkg-3941dd58-pr-125/index.html https://ci-tests.linuxserver.io/lspipepr/jellyfin/nightly-20210812.1-unstable-pkg-3941dd58-pr-125/shellcheck-result.xml