> On Oct. 7, 2012, 8:12 a.m., Wido den Hollander wrote: > > Ship It!
The conformation from Marcus and Hugo were enough for me to commit this one. Still want LSB though to work on all platforms :( Init scripts should be distro unspecific! - Wido ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7460/#review12212 ----------------------------------------------------------- On Oct. 5, 2012, 10:42 p.m., Marcus Sorensen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/7460/ > ----------------------------------------------------------- > > (Updated Oct. 5, 2012, 10:42 p.m.) > > > Review request for cloudstack. > > > Description > ------- > > init script uses 'daemon -p', which returns : > > Usage: daemon [+/-nicelevel] {program} > > Fixing to 'daemon --pidfile' > > > Diffs > ----- > > usage/distro/centos/SYSCONFDIR/rc.d/init.d/cloud-usage.in 516a842 > usage/distro/fedora/SYSCONFDIR/rc.d/init.d/cloud-usage.in 516a842 > usage/distro/rhel/SYSCONFDIR/rc.d/init.d/cloud-usage.in 516a842 > > Diff: https://reviews.apache.org/r/7460/diff/ > > > Testing > ------- > > "service cloud-usage start" succeeded after this patch on CentOS 6.3 > > > Thanks, > > Marcus Sorensen > >
