[PATCH 0/1] Sanity check for duplicate PACKAGE_ARCHS

[PATCH 0/1] Sanity check for duplicate PACKAGE_ARCHS

[Joshua Lock]

From: Joshua Lock <josh@linux.intel.com>

I have had some pain this morning caused by duplicated entries in my
PACKAGE_ARCHS variable. After Richard helped me debug this he suggest we
add a sanity check for this issue.

Pull URL: git://git.openembedded.org/openembedded-core-contrib
  Branch: josh/pkgarch
  Browse: http://git.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=josh/pkgarch

Thanks,
    Joshua Lock <josh@linux.intel.com>
---


Joshua Lock (1):
  sanity.bbclass: check for duplicates in PACKAGE_ARCHS

 meta/classes/sanity.bbclass |   13 +++++++++++++
 1 files changed, 13 insertions(+), 0 deletions(-)

-- 
1.7.4

[PATCH 1/1] sanity.bbclass: check for duplicates in PACKAGE_ARCHS

[Joshua Lock]

From: Joshua Lock <josh@linux.intel.com>

Duplicate entries in PACKAGE_ARCHS causes problems with rootfs
generation. For example multiple architecture entries in opkg.conf
will confuse the opkg package manager.

Signed-off-by: Joshua Lock <josh@linux.intel.com>
---
 meta/classes/sanity.bbclass |   13 +++++++++++++
 1 files changed, 13 insertions(+), 0 deletions(-)

diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass
index 6e13d2a..639e1ea 100644
--- a/meta/classes/sanity.bbclass
+++ b/meta/classes/sanity.bbclass
@@ -338,6 +338,19 @@ def check_sanity(e):
     elif oeroot.find (' ') != -1:
         messages = messages + "Error, you have a space in your POKYBASE directory path. Please move Poky to a directory which doesn't include a space."
 
+    # Check that we don't have duplicate entries in PACKAGE_ARCHS
+    pkgarchs = data.getVar('PACKAGE_ARCHS', e.data, True)
+    seen = {}
+    dups = []
+
+    for pa in pkgarchs.split():
+    	if seen.get(pa, 0) == 1:
+	    dups.append(pa)
+	else:
+	    seen[pa] = 1
+    if len(dups):
+       messages = messages + "Error, the PACKAGE_ARCHS variable contains duplicates. The following archs are listed more than once: %s" % " ".join(dups)
+
     if messages != "":
         raise_sanity_error(messages)
 
-- 
1.7.4

[PATCH 0/1] Sanity check for duplicate PACKAGE_ARCHS

[Joshua Lock]

From: Joshua Lock <josh@linux.intel.com>

I have had some pain this morning caused by duplicated entries in my
PACKAGE_ARCHS variable. After Richard helped me debug this he suggest we
add a sanity check for this issue.

Pull URL: git://git.pokylinux.org/poky-contrib.git
  Branch: josh/bernard
  Browse: http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=josh/bernard

Thanks,
    Joshua Lock <josh@linux.intel.com>
---


Joshua Lock (1):
  sanity.bbclass: check for duplicates in PACKAGE_ARCHS

 meta/classes/sanity.bbclass |   13 +++++++++++++
 1 files changed, 13 insertions(+), 0 deletions(-)

-- 
1.7.4

Re: [PATCH 0/1] Sanity check for duplicate PACKAGE_ARCHS

[Richard Purdie]

On Fri, 2011-03-18 at 13:41 +0000, Joshua Lock wrote:
> From: Joshua Lock <josh@linux.intel.com>
> 
> I have had some pain this morning caused by duplicated entries in my
> PACKAGE_ARCHS variable. After Richard helped me debug this he suggest we
> add a sanity check for this issue.
> 
> Pull URL: git://git.pokylinux.org/poky-contrib.git
>   Branch: josh/bernard
>   Browse: http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=josh/bernard
> 
> Thanks,
>     Joshua Lock <josh@linux.intel.com>
> ---
> 
> 
> Joshua Lock (1):
>   sanity.bbclass: check for duplicates in PACKAGE_ARCHS
> 
>  meta/classes/sanity.bbclass |   13 +++++++++++++
>  1 files changed, 13 insertions(+), 0 deletions(-)

Merged into master, not sure this will make bernard at this point but
its good to have the check.

Cheers,

Richard