[PR #3690] Fix left /usr/bin/jellyfin symlink on removal and typo #9658

Closed
opened 2026-02-07 06:06:12 +03:00 by OVERLORD · 0 comments
Owner

Original Pull Request: https://github.com/jellyfin/jellyfin/pull/3690

State: closed
Merged: Yes


Changes
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.

**Original Pull Request:** https://github.com/jellyfin/jellyfin/pull/3690 **State:** closed **Merged:** Yes --- **Changes** 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.
OVERLORD added the pull-request label 2026-02-07 06:06:12 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#9658