[systemd-commits] src/service.c src/unit.c

Lennart Poettering lennart at kemper.freedesktop.org
Mon Sep 27 14:24:23 PDT 2010


 src/service.c |    9 ++++++---
 src/unit.c    |    5 ++++-
 2 files changed, 10 insertions(+), 4 deletions(-)

New commits:
commit 9f151f29fd37d9fdd3aad303cf0489482522918f
Author: Lennart Poettering <lennart at poettering.net>
Date:   Mon Sep 27 23:24:17 2010 +0200

    service: don't create sysv order deps on merged units

diff --git a/src/service.c b/src/service.c
index 797f285..5706d7c 100644
--- a/src/service.c
+++ b/src/service.c
@@ -239,9 +239,9 @@ static char *sysv_translate_name(const char *name) {
                 /* Drop Debian-style .sh suffix */
                 strcpy(stpcpy(r, name) - 3, ".service");
 #ifdef TARGET_ARCH
-	else if (startswith(name, "@"))
-		/* Drop Arch-style background prefix */
-		strcpy(stpcpy(r, name + 1), ".service");
+        else if (startswith(name, "@"))
+                /* Drop Arch-style background prefix */
+                strcpy(stpcpy(r, name + 1), ".service");
 #endif
         else
                 /* Normal init scripts */
@@ -326,6 +326,9 @@ static int sysv_fix_order(Service *s) {
                 if (s == t)
                         continue;
 
+                if (t->meta.load_state != UNIT_LOADED)
+                        continue;
+
                 if (t->sysv_start_priority < 0)
                         continue;
 
diff --git a/src/unit.c b/src/unit.c
index 5a451c5..5960c27 100644
--- a/src/unit.c
+++ b/src/unit.c
@@ -578,7 +578,7 @@ const char *unit_description(Unit *u) {
         if (u->meta.description)
                 return u->meta.description;
 
-        return u->meta.id;
+        return strna(u->meta.id);
 }
 
 void unit_dump(Unit *u, FILE *f, const char *prefix) {
@@ -1403,6 +1403,9 @@ int unit_add_dependency(Unit *u, UnitDependency d, Unit *other, bool add_referen
         assert(d >= 0 && d < _UNIT_DEPENDENCY_MAX);
         assert(other);
 
+        u = unit_follow_merge(u);
+        other = unit_follow_merge(other);
+
         /* We won't allow dependencies on ourselves. We will not
          * consider them an error however. */
         if (u == other)


More information about the systemd-commits mailing list