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
>>
>>
>

Reply via email to