Does this patch work on certain distros? On CentOS 6.3, if I replace 'start_daemon' with 'daemon' I get:
Starting CloudStack Usage Monitor cloud-usage/etc/init.d/cloud-usage: Usage: daemon [+/-nicelevel] {program} cat: /var/run/cloud-usage.pid: No such file or directory [FAILED] I actually have to replace "start_daemon -p" with "daemon --pidfile" for it to work. On Tue, Sep 25, 2012 at 4:46 AM, Wido den Hollander <w...@widodh.nl> wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/7248/#review11892 > ----------------------------------------------------------- > > Ship it! > > > Ship It! > > - Wido den Hollander > > > On Sept. 25, 2012, 11:45 a.m., Rohit Yadav wrote: >> >> ----------------------------------------------------------- >> This is an automatically generated e-mail. To reply, visit: >> https://reviews.apache.org/r/7248/ >> ----------------------------------------------------------- >> >> (Updated Sept. 25, 2012, 11:45 a.m.) >> >> >> Review request for cloudstack, Prasanna Santhanam, edison su, and Wido den >> Hollander. >> >> >> Description >> ------- >> >> Download original patch and git am <patch>: (CentOS and RHEL) >> http://bhaisaab.org/patches/cloudstack/0001-CLOUDSTACK-81-Use-non-lsb-function.patch >> >> For Fedora: >> http://bhaisaab.org/patches/cloudstack/0001-CLOUDSTACK-81-Fix-typo-in-function-name-for-script-i.patch >> >> >> Diffs >> ----- >> >> usage/distro/fedora/SYSCONFDIR/rc.d/init.d/cloud-usage.in 69f1b2c >> >> Diff: https://reviews.apache.org/r/7248/diff/ >> >> >> Testing >> ------- >> >> >> Thanks, >> >> Rohit Yadav >> >> >