Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 616328 - >=media-tv/plex-media-server-1.4.4 ignoring /etc/plex/plexmediaserver
Summary: >=media-tv/plex-media-server-1.4.4 ignoring /etc/plex/plexmediaserver
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Jonathan Vasquez (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-04-22 16:45 UTC by Camille Martel
Modified: 2017-04-25 03:08 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Camille Martel 2017-04-22 16:45:08 UTC
The media-tv/plex-media-server ebuild installs the /etc/default/plexmediaserver config file to /etc/plex/plexmediaserver, following gentoo philosophy. In order to redirect start_pms to this installed user config, a sed command was added to the ebuild. However, the result of that command is wrong:

sed -e "s#${CONFIG_VANILLA}#${CONFIG_PATH}#g" -i "${S}"/usr/sbin/start_pms || die

This effectively substitutes the /etc/default/plexmediaserver config to /etc/plex rather than /etc/plex/plexmediaserver as it should be. The proper substitution seems like it would be:

sed -e "s#${CONFIG_VANILLA}#${CONFIG_PATH}/${_APPNAME}#g" -i "${S}"/usr/sbin/start_pms || die

Or something similar.
Comment 1 Jonathan Vasquez (RETIRED) gentoo-dev 2017-04-25 03:08:05 UTC
Hey Camille,

Thanks for reporting this. You were right, the path was wrong. This has now been fixed and has been included with 1.4.4-r1, 1.5.3-r1, and also the new 1.5.5 ebuild.

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=aca88b815a7f5fb96f3a0a220e88db0d79e14a76

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=104defd9f31e572d095e60495cabcfaf45515bd3