[Merge] lp:~sdeziel/sysvinit/early-exit-ondemand into lp:ubuntu/vivid/sysvinit
Mathieu Trudel-Lapierre
mathieu.tl at gmail.com
Mon May 11 21:45:40 UTC 2015
On May 11, 2015 4:20 PM, "Simon Déziel" <simon.deziel at gmail.com> wrote:
>
> Thanks for the review Mathieu.
>
> > This seems to me like the kind of change that should be properly
documented
> > with a bug report. Have you opened one regarding this matter yet? Since
it
> > reportedly only avoids sleeping for 60 seconds before doing nothing
(exiting
> > because the governors aren't available), you should describe the net
effect of
> > this in a bug report.
>
> Good idea, LP: #1453946
>
Thanks.
[...]
>
> "start" just calls the same init script but with the "background"
argument so it ends up in the same code path.
I don't know. It seems to me like background does not call
start-stop-daemon at all, which is why I mentioned you're changing the
behavior of start by integrating it with background.
/ Matt
--
https://code.launchpad.net/~sdeziel/sysvinit/early-exit-ondemand/+merge/254111
Your team Ubuntu branches is subscribed to branch lp:ubuntu/vivid/sysvinit.
More information about the Ubuntu-reviews
mailing list