The original loop called fix_order() on each service immediately after
loading it, but fix_order() would reference other units which were not
loaded yet.
This resulted in bogus and unnecessary orderings based on the static
start priorities.
Therefore call load_sysv() for every init script when traversing them in
enumerate_sysv(). This ensures that all units are loaded when
fix_order() is called.
Bug-Debian: https://bugs.debian.org/771118
service->name = name;
service->path = fpath;
+ r = load_sysv(service);
+ if (r < 0)
+ continue;
+
r = hashmap_put(all_services, service->name, service);
if (r < 0)
return log_oom();
}
HASHMAP_FOREACH(service, all_services, j) {
- q = load_sysv(service);
- if (q < 0)
- continue;
-
q = fix_order(service, all_services);
if (q < 0)
continue;