From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.chez-thomas.org (mail.mlbassoc.com [65.100.170.105]) by yocto-www.yoctoproject.org (Postfix) with ESMTP id 1CF9DE014F5 for ; Tue, 14 May 2013 04:22:33 -0700 (PDT) Received: by mail.chez-thomas.org (Postfix, from userid 1998) id A6E10F811DF; Tue, 14 May 2013 05:22:32 -0600 (MDT) X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on hermes.chez-thomas.org X-Spam-Level: X-Spam-Status: No, score=-2.9 required=4.0 tests=ALL_TRUSTED,BAYES_00 autolearn=unavailable version=3.3.2 Received: from [192.168.1.114] (zeus [192.168.1.114]) by mail.chez-thomas.org (Postfix) with ESMTP id 60DCDF81188; Tue, 14 May 2013 05:22:31 -0600 (MDT) Message-ID: <51921E78.4090708@mlbassoc.com> Date: Tue, 14 May 2013 05:22:32 -0600 From: Gary Thomas User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130329 Thunderbird/17.0.5 MIME-Version: 1.0 To: Martin Jansa References: <519215AE.3090608@mlbassoc.com> <20130514105619.GG3187@jama> In-Reply-To: <20130514105619.GG3187@jama> Cc: Yocto Project Subject: Re: IMAGE_FEATURES check broken X-BeenThere: yocto@yoctoproject.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: Discussion of all things Yocto Project List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 14 May 2013 11:22:33 -0000 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit On 2013-05-14 04:56, Martin Jansa wrote: > On Tue, May 14, 2013 at 04:45:02AM -0600, Gary Thomas wrote: >> This commit breaks systems where IMAGE_FEATURES includes "package-management": >> >> commit d63c8a1597c9bd0a838cb59ca461ac432b630a90 >> Author: Paul Eggleton >> Date: Thu May 9 17:24:25 2013 +0100 >> >> classes/image: show an error on invalid IMAGE_FEATURES >> >> A quick look at that class shows that package-management is being left >> out of the "validitems". This patch fixes it: >> >> diff --git a/meta/classes/image.bbclass b/meta/classes/image.bbclass >> index 2b42e12..4f07708 100644 >> --- a/meta/classes/image.bbclass >> +++ b/meta/classes/image.bbclass >> @@ -19,7 +19,7 @@ INHIBIT_DEFAULT_DEPS = "1" >> # IMAGE_FEATURES may contain any available package group >> IMAGE_FEATURES ?= "" >> IMAGE_FEATURES[type] = "list" >> -IMAGE_FEATURES[validitems] += "debug-tweaks read-only-rootfs" >> +IMAGE_FEATURES[validitems] += "debug-tweaks read-only-rootfs package-management" >> >> # rootfs bootstrap install >> ROOTFS_BOOTSTRAP_INSTALL = "${@base_contains("IMAGE_FEATURES", "package-management", "", "${ROOTFS_PKGMANAGE_BOOTSTR >> >> Am I missing something else, or was this just overlooked? > > The same patch is already on oe-core ML where it belongs: > http://lists.linuxtogo.org/pipermail/openembedded-core/2013-May/039541.html Thanks (I missed that one because it happened late Sunday over here) Note: I sent my email to this list because of the comment in the commit which referred directly to a Yocto bug. ... I have a hard time figuring out where Yocto ends and OE-core begins ... -- ------------------------------------------------------------ Gary Thomas | Consulting for the MLB Associates | Embedded world ------------------------------------------------------------