diff options
author | Lennart Poettering <lennart@poettering.net> | 2015-07-24 03:50:36 +0200 |
---|---|---|
committer | Lennart Poettering <lennart@poettering.net> | 2015-07-24 03:50:36 +0200 |
commit | fc1ba5d1933d83c73cf9ddb61e8a2da2b15e26e1 (patch) | |
tree | 445cbab04a90002255ee202e201d65e00324852f | |
parent | c5f44880ca35bc0246ddf90cbf2af45b10e4a4d5 (diff) |
core: print a nicer warning when two units have the same BusName= setting
This should make issues like #609 easier to debug.
-rw-r--r-- | src/core/service.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/core/service.c b/src/core/service.c index 39a2507b6f..b790ec98be 100644 --- a/src/core/service.c +++ b/src/core/service.c @@ -401,7 +401,6 @@ static int service_add_fd_store_set(Service *s, FDSet *fds) { r = service_add_fd_store(s, fd); if (r < 0) return log_unit_error_errno(UNIT(s), r, "Couldn't add fd to fd store: %m"); - if (r > 0) { log_unit_debug(UNIT(s), "Added fd to fd store."); fd = -1; @@ -576,8 +575,10 @@ static int service_add_extras(Service *s) { return r; r = unit_watch_bus_name(UNIT(s), s->bus_name); + if (r == -EEXIST) + return log_unit_error_errno(UNIT(s), r, "Two services allocated for the same bus name %s, refusing operation.", s->bus_name); if (r < 0) - return r; + return log_unit_error_errno(UNIT(s), r, "Cannot watch bus name %s: %m", s->bus_name); } if (UNIT(s)->default_dependencies) { |