Bug#846350: bug can be closed

2016-12-01 Thread Chris Lamb
tags 846350 + pending thanks Peter Souter wrote: > Huh, that's pretty strange. > > When I test it with the standard Debian redis-server package, it > doesn't fail even if I delete the run directory and any pid files... Can reproduce this. This is probably due to systemd not needing the pidfile

Bug#846350: bug can be closed

2016-11-30 Thread Peter Souter
Huh, that's pretty strange. When I test it with the standard Debian redis-server package, it doesn't fail even if I delete the run directory and any pid files... $ apt-get -y install redis-server; $ systemctl stop redis-server; $ rm -rf /var/run/redis*; $ service start redis-server; $ redis-cli p

Bug#846350: bug can be closed

2016-11-30 Thread Chris Lamb
Peter Souter wrote: > This issue doesn't affect the Debian package as it doesn't specify pid > location, so it defaults to /var/run/redis.pid, the /var/run dir will > always exist. Are you sure? Under systemd we use: /var/run/redis/redis-server.pid (This bug cannot affect the SysVInit script

Bug#846350: bug can be closed

2016-11-30 Thread Peter Souter
My mistake, as can be seen, this is the dotdeb-redis package. I was testing multiple packages and I thought it affected both. I've fixed that upstream https://github.com/gplessis/dotdeb-redis/pull/9 This issue doesn't affect the Debian package as it doesn't specify pid location, so it defaults t