Opened 4 years ago
Last modified 3 years ago
#61625 assigned defect
redis: wrong .conf file path in startupitem
Reported by: | ArtKoKo | Owned by: | dgilman (David Gilman) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 2.6.4 |
Keywords: | Cc: | ||
Port: | redis |
Description
The plist file for launchd is wrong : Redis is launched multiple times in an infinite loop.
The conf file path is unfinished : /opt/local/etc/redis.conf instead of /opt/local/etc/redis/redis.conf
I suggest in attach a replacement plist
Attachments (1)
Change History (7)
Changed 4 years ago by ArtKoKo
Attachment: | org.macports.redis.zip added |
---|
comment:1 Changed 4 years ago by jmroot (Joshua Root)
Keywords: | redis launchd removed |
---|---|
Summary: | Wrong plist / wrong path → redis: wrong .conf file path in startupitem |
comment:2 Changed 4 years ago by dgilman (David Gilman)
I'm not sure how you set up launchd here. We definitely install the Redis conf file to /opt/local/etc/redis.conf and not in the redis subdirectory. A "port load redis" is correctly setting up launchd with the correct paths.
comment:3 Changed 4 years ago by ArtKoKo
The conf path is a secondary and minor point, it's just to do like other port
The main problem is "port load redis" launch redis multiple times in an infinite loop because the call of daemondo is incorrect
I think the previous ticket is about this same issue
comment:4 Changed 3 years ago by dgilman (David Gilman)
I'd appreciate any more information on this. I've come back to this ticket and the other every now and then and I have never seen any of the behavior described. I'd appreciate any other information you have, your system configuration or anything else.
comment:5 Changed 3 years ago by ArtKoKo
I have to try to reinstall Redis on Mac OS X Mojave from a fresh install and come back to you with more information.
comment:6 Changed 3 years ago by ArtKoKo
I think I understood the problem: with a default installation everything works fine.
But if you change the PID path as suggested in the config file comment
# Note that on modern Linux systems "/run/redis.pid" is more conforming # and should be used instead.
then there is a conflict with the PID location statement given in the load .plist file :
<string>--pid=exec</string>
If you edit the .plist file the Redis upgrades overwrite the file...
I changed all the paths to respect the convention of other ports :
/opt/local/etc/redis/redis.conf /opt/local/var/log/redis/redis.log /opt/local/var/run/redis/redis.pid /opt/local/var/db/redis/
and I apply a patch as soon as an MacPorts upgrade is applied.
plist with wrapper