From: Lennart Poettering Date: Mon, 15 Nov 2010 17:26:37 +0000 (+0100) Subject: units: get rid of no_requires unit flag, and make crypto disks require cryptsetup... X-Git-Tag: v12~26 X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?p=elogind.git;a=commitdiff_plain;h=8c4dd542afde23d21d736b24b5ab66c25ab091e8;ds=sidebyside units: get rid of no_requires unit flag, and make crypto disks require cryptsetup service --- diff --git a/src/cryptsetup-generator.c b/src/cryptsetup-generator.c index b8db0d5e1..a2b398dd1 100644 --- a/src/cryptsetup-generator.c +++ b/src/cryptsetup-generator.c @@ -163,7 +163,7 @@ static int create_disk( to = NULL; e = unit_name_escape(name); - if (asprintf(&to, "%s/dev-mapper-%s.device.wants/%s", arg_dest, e, n) < 0) { + if (asprintf(&to, "%s/dev-mapper-%s.device.requires/%s", arg_dest, e, n) < 0) { r = -ENOMEM; goto fail; } diff --git a/src/device.c b/src/device.c index 7cb4ff699..5c18d996e 100644 --- a/src/device.c +++ b/src/device.c @@ -562,7 +562,6 @@ DEFINE_STRING_TABLE_LOOKUP(device_state, DeviceState); const UnitVTable device_vtable = { .suffix = ".device", - .no_requires = true, .no_instances = true, .no_snapshots = true, .no_isolate = true, diff --git a/src/unit.c b/src/unit.c index b3a8210a5..edc636412 100644 --- a/src/unit.c +++ b/src/unit.c @@ -1456,15 +1456,6 @@ int unit_add_dependency(Unit *u, UnitDependency d, Unit *other, bool add_referen if (u == other) return 0; - if (UNIT_VTABLE(u)->no_requires && - (d == UNIT_REQUIRES || - d == UNIT_REQUIRES_OVERRIDABLE || - d == UNIT_REQUISITE || - d == UNIT_REQUISITE_OVERRIDABLE || - d == UNIT_BIND_TO)) { - return -EINVAL; - } - if ((r = set_ensure_allocated(&u->meta.dependencies[d], trivial_hash_func, trivial_compare_func)) < 0) return r; diff --git a/src/unit.h b/src/unit.h index b260dd5b9..fbe88c2c7 100644 --- a/src/unit.h +++ b/src/unit.h @@ -353,11 +353,6 @@ struct UnitVTable { /* Can units of this type have multiple names? */ bool no_alias:1; - /* If true units of this types can never have "Requires" - * dependencies, because state changes can only be observed, - * not triggered */ - bool no_requires:1; - /* Instances make no sense for this type */ bool no_instances:1;