diff options
| author | MichaIng <micha@dietpi.com> | 2020-07-24 22:43:32 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-07-24 22:43:32 +0200 |
| commit | 1883ecd81716acb108424924711aed842dd6338a (patch) | |
| tree | 8cc0a6f0bc90d6b22a478fd5c70ace1ab00cc8b6 /debian | |
| parent | e3db0ac400e7a1a42e7a844a13ab052d26125612 (diff) | |
Fix left /usr/bin/jellyfin symlink on removal and typo
After removal of the symlink target file "/usr/lib/jellyfin/bin/jellyfin", file existence check on the symlink "[[ -f /usr/bin/jellyfin ]]" returns false. As a result the symlink is left in place on package purge. The correct check would be "[[ -L /usr/bin/jellyfin ]]", but since it could be a file in cases, e.g. manual fix on file systems with no symlink support or for any other reason, it is easiest to use "rm -f" to assure that it is removed in both cases and not return false even if it does not exist at all.
Additionally this fixes a typo on upstart script check.
Signed-off-by: MichaIng <micha@dietpi.com>
Diffstat (limited to 'debian')
| -rw-r--r-- | debian/postrm | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/debian/postrm b/debian/postrm index 1d00a984e..3d56a5f1e 100644 --- a/debian/postrm +++ b/debian/postrm @@ -25,7 +25,7 @@ case "$1" in purge) echo PURGE | debconf-communicate $NAME > /dev/null 2>&1 || true - if [[ -x "/etc/init.d/jellyfin" ]] || [[ -e "/etc/init/jellyfin.connf" ]]; then + if [[ -x "/etc/init.d/jellyfin" ]] || [[ -e "/etc/init/jellyfin.conf" ]]; then update-rc.d jellyfin remove >/dev/null 2>&1 || true fi @@ -54,7 +54,7 @@ case "$1" in rm -rf $PROGRAMDATA fi # Remove binary symlink - [[ -f /usr/bin/jellyfin ]] && rm /usr/bin/jellyfin + rm -f /usr/bin/jellyfin # Remove sudoers config [[ -f /etc/sudoers.d/jellyfin-sudoers ]] && rm /etc/sudoers.d/jellyfin-sudoers # Remove anything at the default locations; catches situations where the user moved the defaults |