From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from dan.rpsys.net ([93.97.175.187]) by linuxtogo.org with esmtp (Exim 4.72) (envelope-from ) id 1UNt4f-0006v6-P1 for openembedded-core@lists.openembedded.org; Fri, 05 Apr 2013 00:56:07 +0200 Received: from localhost (dan.rpsys.net [127.0.0.1]) by dan.rpsys.net (8.14.4/8.14.4/Debian-2.1ubuntu1) with ESMTP id r34MnWN7023278; Thu, 4 Apr 2013 23:49:32 +0100 X-Virus-Scanned: Debian amavisd-new at dan.rpsys.net Received: from dan.rpsys.net ([127.0.0.1]) by localhost (dan.rpsys.net [127.0.0.1]) (amavisd-new, port 10024) with LMTP id 60pxP2rdR1uI; Thu, 4 Apr 2013 23:49:32 +0100 (BST) Received: from [192.168.3.10] (rpvlan0 [192.168.3.10]) (authenticated bits=0) by dan.rpsys.net (8.14.4/8.14.4/Debian-2.1ubuntu1) with ESMTP id r34MnSRT023221 (version=TLSv1/SSLv3 cipher=DHE-RSA-CAMELLIA256-SHA bits=256 verify=NOT); Thu, 4 Apr 2013 23:49:31 +0100 Message-ID: <1365115121.6526.121.camel@ted> From: Richard Purdie To: Martin Jansa Date: Thu, 04 Apr 2013 23:38:41 +0100 In-Reply-To: <20130404165547.GF1271@jama.dyndns-home.com> References: <1365093733-31674-1-git-send-email-Martin.Jansa@gmail.com> <1365094008.6526.115.camel@ted> <20130404165547.GF1271@jama.dyndns-home.com> X-Mailer: Evolution 3.6.2-0ubuntu0.1 Mime-Version: 1.0 Cc: openembedded-core@lists.openembedded.org Subject: Re: [PATCH 1/2] systemd: set INHIBIT_UPDATERCD_BBCLASS without sysvinit in features X-BeenThere: openembedded-core@lists.openembedded.org X-Mailman-Version: 2.1.11 Precedence: list List-Id: Patches and discussions about the oe-core layer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 04 Apr 2013 22:56:07 -0000 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit On Thu, 2013-04-04 at 18:55 +0200, Martin Jansa wrote: > On Thu, Apr 04, 2013 at 05:46:48PM +0100, Richard Purdie wrote: > > On Thu, 2013-04-04 at 18:42 +0200, Martin Jansa wrote: > > > * fixes udev configure in run-postinsts failing with: > > > update-rc.d: /etc/init.d/systemd-udev: file does not exist > > > because systemd-udev is installed only with sysvinit in features > > > but update-rc.d was always called from PN postinst > > > > > > Signed-off-by: Martin Jansa > > > --- > > > meta/recipes-core/systemd/systemd_199.bb | 6 ++++++ > > > 1 file changed, 6 insertions(+) > > > > > > diff --git a/meta/recipes-core/systemd/systemd_199.bb b/meta/recipes-core/systemd/systemd_199.bb > > > index ba1d133..e574548 100644 > > > --- a/meta/recipes-core/systemd/systemd_199.bb > > > +++ b/meta/recipes-core/systemd/systemd_199.bb > > > @@ -239,6 +239,12 @@ INITSCRIPT_PACKAGES = "udev" > > > INITSCRIPT_NAME_udev = "systemd-udevd" > > > INITSCRIPT_PARAMS_udev = "start 03 S ." > > > > > > +python __anonymous() { > > > + features = d.getVar("DISTRO_FEATURES", True).split() > > > + if "sysvinit" not in features: > > > + d.setVar("INHIBIT_UPDATERCD_BBCLASS", "1") > > > +} > > > + > > > # TODO: > > > # u-a for runlevel and telinit > > > > Would this make sense to be in systemd.bbclass? > > Similar logic is in systemd.bbclass already, but systemd is not inherited from > systemd and dbus recipes. Ok, fair enough. I hadn't realised that. > Also the version from systemd.bbclass does check also for systemd in > DISTRO_FEATURES, but that's not wanted here, because decision to install init.d > script is based only on sysvinit in DISTRO_FEATURES. > > Lot's of fun with all init systems sharing the same PN :/. :/. Cheers, Richard