[systemd-devel] [PATCH] Don't pull in sysctl on shutdown
Zbigniew Jędrzejewski-Szmek
zbyszek at in.waw.pl
Fri Mar 22 20:05:39 PDT 2013
On Sat, Mar 23, 2013 at 03:11:41AM +0100, Lennart Poettering wrote:
> On Wed, 13.03.13 10:27, Umut Tezduyar (umut at tezduyar.com) wrote:
>
> > No need to try to stop systemd-sysctl on shutdown as
> > this service doesn't have ExecStop= anyways.
>
> It's kinda nice if all services are gone from the service list when we
> shut down, and doing this costs pretty much nothing.
>
> Also, the patch is incomplete, as shutdown.target is still listed in
> Before=.
>
> Anyway, I fail to see the reason why we would want to apply this patch...
Hi,
I applied the patch, because it simplifies things by a tiny bit, removing
one explicit relationship. But please do revert if what was there before
was better.
Zbyszek
>
> > ---
> > units/systemd-sysctl.service.in | 1 -
> > 1 files changed, 0 insertions(+), 1 deletions(-)
> >
> > diff --git a/units/systemd-sysctl.service.in b/units/systemd-sysctl.service.in
> > index 45e1ceb..d914553 100644
> > --- a/units/systemd-sysctl.service.in
> > +++ b/units/systemd-sysctl.service.in
> > @@ -9,7 +9,6 @@
> > Description=Apply Kernel Variables
> > Documentation=man:systemd-sysctl.service(8) man:sysctl.d(5)
> > DefaultDependencies=no
> > -Conflicts=shutdown.target
> > After=systemd-readahead-collect.service systemd-readahead-replay.service
> > Before=sysinit.target shutdown.target
> > ConditionPathIsReadWrite=/proc/sys/
More information about the systemd-devel
mailing list