Re: [OE-core] [PATCH 0/2] Yocto Compatible 2.0 support code

2017-08-24 Thread Patrick Ohly
On Fri, 2017-06-09 at 10:12 +0200, Patrick Ohly wrote: > I also get for all recipes (i.e. the error is in the base > configuration): > > meta/conf/bitbake.conf:752: include/require/inherit > "conf/target/${TARGET_SYS}.conf" resulted in including > "conf/target/x86_64-oe-linux.conf" while parsing

Re: [OE-core] [PATCH 0/2] Yocto Compatible 2.0 support code

2017-06-09 Thread Patrick Ohly
On Fri, 2017-06-09 at 16:11 +0200, Patrick Ohly wrote: > On Fri, 2017-06-09 at 08:47 -0500, Joshua Watt wrote: > > Is there a currently known valid use for such a thing? > > Using LAYERDIR is one, although that particular example is probably > better caught by looking for that particular variable

Re: [OE-core] [PATCH 0/2] Yocto Compatible 2.0 support code

2017-06-09 Thread Patrick Ohly
On Fri, 2017-06-09 at 08:47 -0500, Joshua Watt wrote: > On Fri, 2017-06-09 at 10:12 +0200, Patrick Ohly wrote: > > On Thu, 2017-06-08 at 21:31 +0200, Patrick Ohly wrote: > > > On Thu, 2017-06-08 at 10:28 -0500, Joshua Watt wrote: > > > > Sure. I wouldn't suggest using an if statement for "just > >

Re: [OE-core] [PATCH 0/2] Yocto Compatible 2.0 support code

2017-06-09 Thread Patrick Ohly
On Fri, 2017-06-09 at 08:50 -0500, Joshua Watt wrote: > On Thu, 2017-06-08 at 21:31 +0200, Patrick Ohly wrote: > > There's some truth to that IMHO, but I'm uncertain whether it > > warrants > > introducing entirely new syntax. In refkit, I only ran into one > > particular case were an include file

Re: [OE-core] [PATCH 0/2] Yocto Compatible 2.0 support code

2017-06-09 Thread Joshua Watt
On Thu, 2017-06-08 at 21:31 +0200, Patrick Ohly wrote: > On Thu, 2017-06-08 at 10:28 -0500, Joshua Watt wrote: > > Sure. I wouldn't suggest using an if statement for "just anything", > > you > > can surely do terrible things that way. It would (by convention) be > > restricted to the same sorts of

Re: [OE-core] [PATCH 0/2] Yocto Compatible 2.0 support code

2017-06-09 Thread Joshua Watt
On Fri, 2017-06-09 at 10:12 +0200, Patrick Ohly wrote: > On Thu, 2017-06-08 at 21:31 +0200, Patrick Ohly wrote: > > On Thu, 2017-06-08 at 10:28 -0500, Joshua Watt wrote: > > > Sure. I wouldn't suggest using an if statement for "just > > > anything", you > > > can surely do terrible things that way.

Re: [OE-core] [PATCH 0/2] Yocto Compatible 2.0 support code

2017-06-09 Thread Patrick Ohly
On Thu, 2017-06-08 at 21:31 +0200, Patrick Ohly wrote: > On Thu, 2017-06-08 at 10:28 -0500, Joshua Watt wrote: > > Sure. I wouldn't suggest using an if statement for "just anything", you > > can surely do terrible things that way. It would (by convention) be > > restricted to the same sorts of thin

Re: [OE-core] [PATCH 0/2] Yocto Compatible 2.0 support code

2017-06-08 Thread Patrick Ohly
On Thu, 2017-06-08 at 10:28 -0500, Joshua Watt wrote: > Sure. I wouldn't suggest using an if statement for "just anything", you > can surely do terrible things that way. It would (by convention) be > restricted to the same sorts of things that the conditional includes > allow now. On a similar toke

Re: [OE-core] [PATCH 0/2] Yocto Compatible 2.0 support code

2017-06-08 Thread Joshua Watt
On Thu, 2017-06-08 at 15:33 +0100, Richard Purdie wrote: > On Thu, 2017-06-08 at 08:55 -0500, Joshua Watt wrote: > > On Thu, 2017-06-08 at 09:56 +0100, Richard Purdie wrote: > > > > > > On Wed, 2017-06-07 at 10:43 -0500, Joshua Watt wrote: > > > > > > > > On Wed, 2017-06-07 at 17:31 +0200, Patric

Re: [OE-core] [PATCH 0/2] Yocto Compatible 2.0 support code

2017-06-08 Thread Patrick Ohly
On Thu, 2017-06-08 at 15:33 +0100, Richard Purdie wrote: > Patrick for example mentioned IMAGE_FEATURES. This one is fraught with > problems since: > > a) Its a recipe level setting so using it in a base configuration > context would end badly We cannot even rely on DISTRO_FEATURES in the middle

Re: [OE-core] [PATCH 0/2] Yocto Compatible 2.0 support code

2017-06-08 Thread Richard Purdie
On Thu, 2017-06-08 at 08:55 -0500, Joshua Watt wrote: > On Thu, 2017-06-08 at 09:56 +0100, Richard Purdie wrote: > > > > On Wed, 2017-06-07 at 10:43 -0500, Joshua Watt wrote: > > > > > > On Wed, 2017-06-07 at 17:31 +0200, Patrick Ohly wrote: > > > > > > > > > > > > As discussed in the "[Openemb

Re: [OE-core] [PATCH 0/2] Yocto Compatible 2.0 support code

2017-06-08 Thread Joshua Watt
On Thu, 2017-06-08 at 09:56 +0100, Richard Purdie wrote: > On Wed, 2017-06-07 at 10:43 -0500, Joshua Watt wrote: > > On Wed, 2017-06-07 at 17:31 +0200, Patrick Ohly wrote: > > > > > > As discussed in the "[Openembedded-architecture] Yocto Compatible > > > 2.0 > > > + signature changes" mail thread

Re: [OE-core] [PATCH 0/2] Yocto Compatible 2.0 support code

2017-06-08 Thread Richard Purdie
On Wed, 2017-06-07 at 10:43 -0500, Joshua Watt wrote: > On Wed, 2017-06-07 at 17:31 +0200, Patrick Ohly wrote: > > > > As discussed in the "[Openembedded-architecture] Yocto Compatible > > 2.0 > > + signature changes" mail thread, changes in a .bbappend cannot be > > done unconditionally. Making _

Re: [OE-core] [PATCH 0/2] Yocto Compatible 2.0 support code

2017-06-07 Thread Joshua Watt
On Wed, 2017-06-07 at 17:31 +0200, Patrick Ohly wrote: > As discussed in the "[Openembedded-architecture] Yocto Compatible 2.0 > + signature changes" mail thread, changes in a .bbappend cannot be > done unconditionally. Making _append and _remove depend on overrides > which get set based on DISTRO_

[OE-core] [PATCH 0/2] Yocto Compatible 2.0 support code

2017-06-07 Thread Patrick Ohly
As discussed in the "[Openembedded-architecture] Yocto Compatible 2.0 + signature changes" mail thread, changes in a .bbappend cannot be done unconditionally. Making _append and _remove depend on overrides which get set based on DISTRO_FEATURES is one way of achieving this. The oe.utils.optional_i