[Merge] lp:~jderose/ubuntu/wily/couchdb/fix-1457464 into lp:ubuntu/couchdb
Martin Pitt
martin.pitt at ubuntu.com
Thu Jul 9 04:27:27 UTC 2015
Review: Needs Fixing
Please keep the init.d script. It's necessary for insserv to calculate correct dependencies if some other init.d script depends on couchdb, and removing it is an unnecessary delta to Debian. It'll be inert if there's a couchdb.service.
In https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=723726 there is also a proposed systemd unit. That runs couchdb as root which is obviously bad, and it looks like it would suppress stdout/stderr; that's also rather harsh, as it's usually nice to have the output in the journal. That debdiff also adds a tmpfile snippet instead of just using RuntimeDirectory=. So the unit here is much better, can you please send that back to the Debian bug to prevent this worse one from getting included in Debian?
Thanks!
--
https://code.launchpad.net/~jderose/ubuntu/wily/couchdb/fix-1457464/+merge/264199
Your team Ubuntu branches is subscribed to branch lp:ubuntu/couchdb.
More information about the Ubuntu-reviews
mailing list