]> git.ipfire.org Git - thirdparty/systemd.git/commit
units: express Conflict in syslog.socket instead of emergency.service
authorAlan Jenkins <alan.christopher.jenkins@gmail.com>
Fri, 29 Sep 2017 16:20:31 +0000 (17:20 +0100)
committerAlan Jenkins <alan.christopher.jenkins@gmail.com>
Fri, 29 Sep 2017 16:34:55 +0000 (17:34 +0100)
commit950d8fcdd9aa7e71ad32ea224e7d6fbe0f9f53de
tree075da5e438f5a04aa307c3421fff98d35eb71853
parenta3b22cc5962efef2f4cd57d49f59a67e0617e9e0
units: express Conflict in syslog.socket instead of emergency.service

Note this commit only changes how the code is expressed; it does not change
the existence of any dependency.

The `Conflicts=` was added in 3136ec90, "Stop syslog.socket when entering
emergency mode".  The discussion in the issue #266 raised concerns that
this might be needed for other units, but failed to point out why
syslog.socket is special.  The reason is that syslog.socket has
DefaultDepedencies=no, so it does not get Requires=sysinit.target like
other socket units do.  But syslog.service does require sysinit.target,
among other things.

We don't have many socket, path, or timer units with
DefaultDependencies=no, and I don't think any of the triggered services
have such additional hard dependencies as syslog.service does.

It is much less confusing if we keep this `Conflicts=` in the same file as
the `DefaultDependencies=no` which made it necessary.
units/emergency.service.in
units/syslog.socket