Re: [PATCH 0/1][v3] license.bbclass parsing changes rebased sans debug

[Elizabeth Flanagan <elizabeth.flanagan@intel.com>]

On 05/28/2011 01:57 PM, Khem Raj wrote:
> On 5/28/2011 10:54 AM, Elizabeth Flanagan wrote:
>> In this case, elfutils should probably be changes as well as
>> license.bbclass. As do_populate_lic is looking for valid python to make
>> a syntax tree from, this will fail. Solution:
>>
>> 1. Fix license.bbclass to check LICENSE and massage it a bit more.
>> 2. Fix elfutils to be GPLv2* (unless it's the GCC exception. I haven't
>> read the license yet).
>
> It does not like () around license values so simple solution is to replace
>
> -LICENSE = "(GPLv2&  Exception)"
> +LICENSE = "GPLv2&  Exception"
>
> Generally its preferable to fix eflutils first before license class in
> this case to keep the tree bisect'able.
>

Actually, the issue is my .replace("(", " ( "). It makes the string " ( 'GPLv2' & 'Exception' ) " which tosses an error 
because of the indented string. I have a fix I'm putting a pull request in (in about 10 minutes) for both this and the 
issue Koen found.

-b

---------------
Elizabeth Flanagan
Yocto Project
Release Engineer