From: Lennart Poettering <lennart@poettering.net>
Date: Fri, 16 Jul 2010 01:17:34 +0000 (+0200)
Subject: target: if the user configured a manual ordering between target units and the unit... 
X-Git-Tag: v4~48
X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?p=elogind.git;a=commitdiff_plain;h=772a599e4f118fd166d83e7476ed867decefba70

target: if the user configured a manual ordering between target units and the unit they require don't contradict that automatically
---

diff --git a/fixme b/fixme
index 581bc7fbd..c98ffec0f 100644
--- a/fixme
+++ b/fixme
@@ -39,7 +39,7 @@
 
 * timeout waiting for mount devices?
 
-* default logic for serial getty, ck logging, ssk readahead
+* default logic for serial getty, ck logging, ssh readahead
 
 * place /etc/inittab with explaining blurb.
 
@@ -59,12 +59,14 @@
 
 * /etc must always take precedence even if we follow symlinks!
 
-* when adding default deps to targets, check if they are already there and skip
-
 * /lib/init/rw
 
 * vielleicht implizit immer auf syslog dependen?
 
+* für selinux muss der socket wissen was für service geexeced wird.
+
+* debian deadlock when partition auf noauto is.
+
 External:
 
 * patch /etc/init.d/functions with:
diff --git a/src/target.c b/src/target.c
index f8df6fb75..778cc6e0f 100644
--- a/src/target.c
+++ b/src/target.c
@@ -55,18 +55,23 @@ static int target_add_default_dependencies(Target *t) {
         Unit *other;
         int r;
 
-        /* Imply ordering for requirement dependencies
-         * on target units. */
+        /* Imply ordering for requirement dependencies on target
+         * units. Note that when the user created a contradicting
+         * ordering manually we won't add anything in here to make
+         * sure we don't create a loop. */
 
         SET_FOREACH(other, t->meta.dependencies[UNIT_REQUIRES], i)
-                if ((r = unit_add_dependency(UNIT(t), UNIT_AFTER, other, true)) < 0)
-                        return r;
+                if (!set_get(t->meta.dependencies[UNIT_BEFORE], other))
+                        if ((r = unit_add_dependency(UNIT(t), UNIT_AFTER, other, true)) < 0)
+                                return r;
         SET_FOREACH(other, t->meta.dependencies[UNIT_REQUIRES_OVERRIDABLE], i)
-                if ((r = unit_add_dependency(UNIT(t), UNIT_AFTER, other, true)) < 0)
-                        return r;
+                if (!set_get(t->meta.dependencies[UNIT_BEFORE], other))
+                        if ((r = unit_add_dependency(UNIT(t), UNIT_AFTER, other, true)) < 0)
+                                return r;
         SET_FOREACH(other, t->meta.dependencies[UNIT_WANTS], i)
-                if ((r = unit_add_dependency(UNIT(t), UNIT_AFTER, other, true)) < 0)
-                        return r;
+                if (!set_get(t->meta.dependencies[UNIT_BEFORE], other))
+                        if ((r = unit_add_dependency(UNIT(t), UNIT_AFTER, other, true)) < 0)
+                                return r;
 
         return 0;
 }