summaryrefslogtreecommitdiff
path: root/src/analyze
diff options
context:
space:
mode:
authorLennart Poettering <lennart@poettering.net>2015-11-12 19:21:47 +0100
committerLennart Poettering <lennart@poettering.net>2015-11-12 19:27:24 +0100
commitf32b43bda454a70ae23d6802605d41b26dc24ce2 (patch)
tree6c673526e557006c0337eb90a6dfc639c4208b2d /src/analyze
parent05c450966a08ff85fa8348e2fe41b722612fa3dc (diff)
core: remove support for RequiresOverridable= and RequisiteOverridable=
As discussed at systemd.conf 2015 and on also raised on the ML: http://lists.freedesktop.org/archives/systemd-devel/2015-November/034880.html This removes the two XyzOverridable= unit dependencies, that were basically never used, and do not enhance user experience in any way. Most folks looking for the functionality this provides probably opt for the "ignore-dependencies" job mode, and that's probably a good idea. Hence, let's simplify systemd's dependency engine and remove these two dependency types (and their inverses). The unit file parser and the dbus property parser will now redirect the settings/properties to result in an equivalent non-overridable dependency. In the case of the unit file parser we generate a warning, to inform the user. The dbus properties for this unit type stay available on the unit objects, but they are now hidden from usual introspection and will always return the empty list when queried. This should provide enough compatibility for the few unit files that actually ever made use of this.
Diffstat (limited to 'src/analyze')
-rw-r--r--src/analyze/analyze.c6
1 files changed, 0 insertions, 6 deletions
diff --git a/src/analyze/analyze.c b/src/analyze/analyze.c
index 27ead903e9..b7e45dcfd6 100644
--- a/src/analyze/analyze.c
+++ b/src/analyze/analyze.c
@@ -1076,12 +1076,6 @@ static int graph_one(sd_bus *bus, const UnitInfo *u, char *patterns[], char *fro
r = graph_one_property(bus, u, "Requires", "black", patterns, from_patterns, to_patterns);
if (r < 0)
return r;
- r = graph_one_property(bus, u, "RequiresOverridable", "black", patterns, from_patterns, to_patterns);
- if (r < 0)
- return r;
- r = graph_one_property(bus, u, "RequisiteOverridable", "darkblue", patterns, from_patterns, to_patterns);
- if (r < 0)
- return r;
r = graph_one_property(bus, u, "Wants", "grey66", patterns, from_patterns, to_patterns);
if (r < 0)
return r;