From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dan.rpsys.net (5751f4a1.skybroadband.com [87.81.244.161]) by mail.openembedded.org (Postfix) with ESMTP id 493EE6FEB0 for ; Tue, 17 Jun 2014 07:52:48 +0000 (UTC) Received: from localhost (dan.rpsys.net [127.0.0.1]) by dan.rpsys.net (8.14.4/8.14.4/Debian-2.1ubuntu4) with ESMTP id s5H7pjex009607; Tue, 17 Jun 2014 08:52:27 +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 HXgrqqJG0SbD; Tue, 17 Jun 2014 08:52:27 +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 s5H7qNfU009642 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES128-SHA bits=128 verify=NOT); Tue, 17 Jun 2014 08:52:25 +0100 Message-ID: <1402991538.562.84.camel@ted> From: Richard Purdie To: "Burton, Ross" Date: Tue, 17 Jun 2014 08:52:18 +0100 In-Reply-To: References: <539EDD51.9060508@digi.com> <539F08D0.7060602@digi.com> X-Mailer: Evolution 3.8.4-0ubuntu1 Mime-Version: 1.0 Cc: Javier Viguera , "openembedded-core@lists.openembedded.org" Subject: Re: [backport to Daisy] dbus: fix a hard dependency about dbus-ptest X-BeenThere: openembedded-core@lists.openembedded.org X-Mailman-Version: 2.1.12 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: Tue, 17 Jun 2014 07:52:58 -0000 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit On Mon, 2014-06-16 at 17:36 +0100, Burton, Ross wrote: > On 16 June 2014 16:10, Javier Viguera wrote: > > Attached the obvious cherry-pick of the commit from *master* with a basic > > conflict solved. But I don't think this is acceptable in an already-released > > stable branch "Daisy" as it involves a recipe rename (dbus-ptest_1.6.18.bb > > => dbus-test_1.6.18.bb) that probably will break the projects of current > > users of yocto Daisy. > > +PACKAGES += "${@base_contains('DISTRO_FEATURES', 'ptest', > 'dbus-ptest', '', d)}" > > This was changed in master to use PTEST_ENABLED: > > PACKAGES += "${@bb.utils.contains('PTEST_ENABLED', '1', 'dbus-ptest', '', d)}" > > If you could compare dbus.inc in this patch with master to ensure that > there's nothing else that should be merged that would be great. Also > remove the Conflicts line from the message. For what its worth I've pulled the two patches we need into the daisy branch for this. I'd appreciate someone confirming it does the right thing though. Cheers, Richard