Implement review feedback

6 years ago · 7774977cdd
parent aad34e62ca
commit 7774977cdd
2 changed files with 3 additions and 5 deletions
--- a/deployment/debian-package-x64/pkg-src/conf/jellyfin
+++ b/deployment/debian-package-x64/pkg-src/conf/jellyfin
@ -22,7 +22,7 @@ JELLYFIN_CACHE_DIRECTORY="/var/cache/jellyfin"
 JELLYFIN_RESTART_OPT="--restartpath /usr/lib/jellyfin/restart.sh"

 # [OPTIONAL] ffmpeg binary paths
-#JELLYFIN_FFMPEG_OPTS="-ffmpeg /usr/bin/ffmpeg -ffprobe /usr/bin/ffprobe"
+#JELLYFIN_FFMPEG_OPTS="--ffmpeg /usr/bin/ffmpeg --ffprobe /usr/bin/ffprobe"

 # [OPTIONAL] Additional user-defined options for the binary
 #JELLYFIN_ADD_OPTS=""
--- a/deployment/debian-package-x64/pkg-src/postinst
+++ b/deployment/debian-package-x64/pkg-src/postinst
@ -43,10 +43,8 @@ case "$1" in
      mkdir $CACHEDATA
    fi
    # Ensure permissions are correct on all config directories
-    chown -R jellyfin:jellyfin $PROGRAMDATA
-    chown -R jellyfin:jellyfin $CONFIGDATA
-    chown -R jellyfin:jellyfin $LOGDATA
-    chown -R jellyfin:jellyfin $CACHEDATA
+    chown -R jellyfin:jellyfin $PROGRAMDATA $CONFIGDATA $LOGDATA $CACHEDATA
+    chmod 0750 $PROGRAMDATA $CONFIGDATA $LOGDATA $CACHEDATA

    chmod +x /usr/lib/jellyfin/restart.sh > /dev/null 2>&1 || true