On Aug 1, 2011, at 8:03 AM, Richard Purdie wrote:

> On Mon, 2011-08-01 at 00:36 -0700, Saul Wold wrote:
>> Signed-off-by: Saul Wold <[email protected]>
>> ---
>> meta/classes/sanity.bbclass |    8 +++++---
>> 1 files changed, 5 insertions(+), 3 deletions(-)
>> 
>> diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass
>> index 999e15d..dc11c9e 100644
>> --- a/meta/classes/sanity.bbclass
>> +++ b/meta/classes/sanity.bbclass
>> @@ -383,12 +383,14 @@ def check_sanity(e):
>>     dups = []
>> 
>>     for pa in pkgarchs.split():
>> -            if seen.get(pa, 0) == 1:
>> +    if pa == tunepkg:
>> +        tunefound = True
>> +            if seen.get(pa, 0) == 1:
>> +               pkgarchs.remove(pa)
>> +            elif seen.get(pa, 0) == 1:
>>          dups.append(pa)
>>      else:
>>          seen[pa] = 1
>> -    if pa == tunepkg:
>> -        tunefound = True
>> 
>>     if len(dups):
>>        messages = messages + "Error, the PACKAGE_ARCHS variable contains 
>> duplicates. The following archs are listed more than once: %s" % " 
>> ".join(dups)
> 
> How about we don't put duplicates in there in the first place?
> 
> Which board/tune files is this occurring with?

Yeah, not sure about this one, I added the sanity check to find this early.  
Ran into issues with my PPC patch set and took a while to find (thus figured a 
sanity check made sense to check early on).

- k
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

Reply via email to