[meta-browser] New chromium warnings

[meta-browser] New chromium warnings

[Gary Thomas]

I'm now seeing these warnings with the latest OE-core:
   WARNING: chromium: invalid PACKAGECONFIG: component-build
   WARNING: chromium: invalid PACKAGECONFIG: use-gl
   WARNING: chromium: invalid PACKAGECONFIG: ignore-lost-context

I looked at the recipes and those config strings are not mentioned
in the PACKAGECONFIG setup which is what is triggering the error.

-- 
------------------------------------------------------------
Gary Thomas                 |  Consulting for the
MLB Associates              |    Embedded world
------------------------------------------------------------

Re: [meta-browser] New chromium warnings

[Burton, Ross]

On 30 June 2015 at 16:09, Gary Thomas <gary@mlbassoc.com> wrote:

> I'm now seeing these warnings with the latest OE-core:
>   WARNING: chromium: invalid PACKAGECONFIG: component-build
>   WARNING: chromium: invalid PACKAGECONFIG: use-gl
>   WARNING: chromium: invalid PACKAGECONFIG: ignore-lost-context
>
> I looked at the recipes and those config strings are not mentioned
> in the PACKAGECONFIG setup which is what is triggering the error.
>

Can you verify this with bitbake -e?  Maybe a conf file is adding them.

Ross

Re: [meta-browser] New chromium warnings

[Gary Thomas]

On 2015-06-30 09:32, Burton, Ross wrote:
> On 30 June 2015 at 16:09, Gary Thomas <gary@mlbassoc.com> wrote:
>
>> I'm now seeing these warnings with the latest OE-core:
>>    WARNING: chromium: invalid PACKAGECONFIG: component-build
>>    WARNING: chromium: invalid PACKAGECONFIG: use-gl
>>    WARNING: chromium: invalid PACKAGECONFIG: ignore-lost-context
>>
>> I looked at the recipes and those config strings are not mentioned
>> in the PACKAGECONFIG setup which is what is triggering the error.
>>
>
> Can you verify this with bitbake -e?  Maybe a conf file is adding them.

They are coming from meta-fsl-arm/browser-layer/recipes-browser/chromium/chromium-imx.inc
and my local.conf.  The chromium recipe checks the PACKAGECONFIG variable
for these settings, but is not using PACKAGECONFIG the normal way.  It's
just the warnings that are new.

I suspect the chromium recipe(s) need to be rewritten to play nice
with PACKAGECONFIG...

-- 
------------------------------------------------------------
Gary Thomas                 |  Consulting for the
MLB Associates              |    Embedded world
------------------------------------------------------------

Re: [meta-browser] New chromium warnings

[Burton, Ross]

On 30 June 2015 at 16:46, Gary Thomas <gary@mlbassoc.com> wrote:

> They are coming from
> meta-fsl-arm/browser-layer/recipes-browser/chromium/chromium-imx.inc
> and my local.conf.  The chromium recipe checks the PACKAGECONFIG variable
> for these settings, but is not using PACKAGECONFIG the normal way.  It's
> just the warnings that are new.
>
> I suspect the chromium recipe(s) need to be rewritten to play nice
> with PACKAGECONFIG...
>

I'm pleased we've got the warning, but recipes that are using a
PACKAGECONFIG variable to configure stuff but then not actually using the
PACKAGECONFIG machinery now need to add stub definitions for each of the
flags they're using:

PACKAGECONFIG[use-gl] = ""
etc

Ross

Re: [meta-browser] New chromium warnings

[Gary Thomas]

On 2015-06-30 13:10, Burton, Ross wrote:
> On 30 June 2015 at 16:46, Gary Thomas <gary@mlbassoc.com> wrote:
>
>> They are coming from
>> meta-fsl-arm/browser-layer/recipes-browser/chromium/chromium-imx.inc
>> and my local.conf.  The chromium recipe checks the PACKAGECONFIG variable
>> for these settings, but is not using PACKAGECONFIG the normal way.  It's
>> just the warnings that are new.
>>
>> I suspect the chromium recipe(s) need to be rewritten to play nice
>> with PACKAGECONFIG...
>>
>
> I'm pleased we've got the warning, but recipes that are using a
> PACKAGECONFIG variable to configure stuff but then not actually using the
> PACKAGECONFIG machinery now need to add stub definitions for each of the
> flags they're using:
>
> PACKAGECONFIG[use-gl] = ""
> etc
>

As I expected.  I'll send a patch for this.

-- 
------------------------------------------------------------
Gary Thomas                 |  Consulting for the
MLB Associates              |    Embedded world
------------------------------------------------------------