Bug 273973 - multimedia/jellyfin adjust pkg-message to include proper jail parameter
Summary: multimedia/jellyfin adjust pkg-message to include proper jail parameter
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Robert Clausecker
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-09-20 14:56 UTC by FiLiS
Modified: 2023-09-22 05:15 UTC (History)
2 users (show)

See Also:
michiel: maintainer-feedback+

Attachments
change files/pkg-message.in (1.83 KB, patch)
2023-09-20 14:56 UTC, FiLiS 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description FiLiS 2023-09-20 14:56:12 UTC 
Created attachment 245064 [details]
change files/pkg-message.in

in order to run in a jail, jellyfin needs the jail's parameter `allow.mlock` to be set, the pkg-message.in mentions this to be `allow_mlock=1`, which won't work. 
The attached patch fixes this and also uses `service jellyfin enable` instead of the `sysrc` line.
Comment 1 Michiel van Baak Jansen 2023-09-20 15:45:51 UTC 
Sharp, thanks!
Comment 2 Michiel van Baak Jansen 2023-09-20 15:46:00 UTC 
Sharp, thanks!
Comment 3 commit-hook freebsd_committer freebsd_triage 2023-09-22 05:09:02 UTC 
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=67de7f3af9df240633f8f12ab5054d1aac55b227

commit 67de7f3af9df240633f8f12ab5054d1aac55b227
Author:     FiLiS <freebsdbugs@filis.org>
AuthorDate: 2023-09-21 13:39:19 +0000
Commit:     Robert Clausecker <fuz@FreeBSD.org>
CommitDate: 2023-09-22 04:56:48 +0000

    multimedia/jellyfin: adjust pkg-message to include proper jail parameters

    In order to run in a jail, jellyfin needs the jail's parameter
    `allow.mlock` to be set, the pkg-message.in mentions this to be
    `allow_mlock=1`, which won't work.  Fix this and also use
    `service jellyfin enable` instead of the `sysrc` line.

    PR:             273973
    Approved by:    michiel@vanbaak.eu

 multimedia/jellyfin/files/pkg-message.in | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
Comment 4 Robert Clausecker freebsd_committer freebsd_triage 2023-09-22 05:15:17 UTC 
Thank you for your contribution.