[oe-commits] Richard Purdie : packagegroup: Set ALLOW_EMPTY on a per package basis

[git at git.openembedded.org]

Module: openembedded-core.git
Branch: master
Commit: 8981ed20234c42dc1f5cdef802ebe7214bd55238
URL:    http://git.openembedded.org/?p=openembedded-core.git&a=commit;h=8981ed20234c42dc1f5cdef802ebe7214bd55238

Author: Richard Purdie <richard.purdie at linuxfoundation.org>
Date:   Tue Mar  5 12:34:19 2013 +0000

packagegroup: Set ALLOW_EMPTY on a per package basis

ALLOW_EMPTY should be set on a per package basis, this updates the code
to do this, avoiding warnings.

Signed-off-by: Richard Purdie <richard.purdie at linuxfoundation.org>

---

 meta/classes/packagegroup.bbclass |   12 ++++--------
 1 files changed, 4 insertions(+), 8 deletions(-)

diff --git a/meta/classes/packagegroup.bbclass b/meta/classes/packagegroup.bbclass
index af6ee5d..201309c 100644
--- a/meta/classes/packagegroup.bbclass
+++ b/meta/classes/packagegroup.bbclass
@@ -1,9 +1,5 @@
 # Class for packagegroup (package group) recipes
 
-# packagegroup packages are only used to pull in other packages
-# via their dependencies. They are empty.
-ALLOW_EMPTY = "1"
-
 # By default, only the packagegroup package itself is in PACKAGES.
 # -dbg and -dev flavours are handled by the anonfunc below.
 # This means that packagegroup recipes used to build multiple packagegroup
@@ -18,16 +14,16 @@ inherit allarch
 # This automatically adds -dbg and -dev flavours of all PACKAGES
 # to the list. Their dependencies (RRECOMMENDS) are handled as usual
 # by package_depchains in a following step.
+# Also mark all packages as ALLOW_EMPTY
 python () {
-    if d.getVar('PACKAGEGROUP_DISABLE_COMPLEMENTARY', True) == '1':
-        return
-
     packages = d.getVar('PACKAGES', True).split()
     genpackages = []
     for pkg in packages:
+        d.setVar("ALLOW_EMPTY_%s" % pkg, "1")
         for postfix in ['-dbg', '-dev', '-ptest']:
             genpackages.append(pkg+postfix)
-    d.setVar('PACKAGES', ' '.join(packages+genpackages))
+    if d.getVar('PACKAGEGROUP_DISABLE_COMPLEMENTARY', True) != '1':
+        d.setVar('PACKAGES', ' '.join(packages+genpackages))
 }
 
 # We don't want to look at shared library dependencies for the