core: make sure targets that get a default Conflicts=shutdown.target are also ordered...
authorLennart Poettering <lennart@poettering.net>
Tue, 29 Nov 2016 20:07:05 +0000 (21:07 +0100)
committerMartin Pitt <martin.pitt@ubuntu.com>
Wed, 14 Dec 2016 09:13:52 +0000 (10:13 +0100)
commit33e28180939bd1e71c3d8322b4e2c3fb338bc2fb
treee89570e3514366b3bb907278b723846e64fb91f7
parent4549fcdb1d1431a52c51fa8963a24dbb9f6ecaab
core: make sure targets that get a default Conflicts=shutdown.target are also ordered against it

Let's tweak the automatic dependency generation of target units: not only add a
Conflicts= towards shutdown.target but also an After= line for it, so that we
can be sure the new target is not started when the old target is still up.

Discovered in the context of #4733

(Also, exclude dependency generation if for shutdown.target itself. — This is
strictly speaking redundant, as unit_add_two_dependencies_by_name() detects
that and becomes a NOP, but let's make this explicit for readability.)
src/core/target.c