Strange dependency

Strange dependency

[Gary Thomas]

I have a distribution which normally includes x11.  I just
ran a build with
   DISTRO_FEATURES_remove = "x11 wayland"

Why on earth did this require qemu-native to be rebuilt?
One would think that qemu-native would be independent of the
DISTRO_FEATURES??

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

Re: Strange dependency

[Paul Eggleton]

Hi Gary,

On Mon, 25 Apr 2016 15:06:14 Gary Thomas wrote:
> I have a distribution which normally includes x11.  I just
> ran a build with
>    DISTRO_FEATURES_remove = "x11 wayland"
> 
> Why on earth did this require qemu-native to be rebuilt?
> One would think that qemu-native would be independent of the
> DISTRO_FEATURES??

Looking at meta/recipes-devtools/qemu/qemu.inc it is explicitly looking at 
DISTRO_FEATURES for the native class. That seems wrong to me as well, but I 
would be surprised if it were the only place we were referring to 
DISTRO_FEATURES in the native context (especially implicitly - consider the 
defaults for PACKAGECONFIG in various recipes where we initialise it based on 
DISTRO_FEATURES and don't provide a special default for native). I guess it's 
something that someone will need to go through and clean up at some point.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre

Re: Strange dependency

[Burton, Ross]

[-- Attachment #1: Type: text/plain, Size: 950 bytes --]

On 27 April 2016 at 03:30, Paul Eggleton <paul.eggleton@linux.intel.com>
wrote:

> Looking at meta/recipes-devtools/qemu/qemu.inc it is explicitly looking at
> DISTRO_FEATURES for the native class. That seems wrong to me as well, but I
> would be surprised if it were the only place we were referring to
> DISTRO_FEATURES in the native context (especially implicitly - consider the
> defaults for PACKAGECONFIG in various recipes where we initialise it based
> on
> DISTRO_FEATURES and don't provide a special default for native). I guess
> it's
> something that someone will need to go through and clean up at some point.
>

Ah, that was a bit of cleanup I failed to do.  qemu.inc has a
PACKAGECONFIG_class-native to hardcode the options and not respect distro
features, but there were some dependencies that are picked based on
DISTRO_FEATURES.  This was a workaround for the previous odd linking, so
can be removed now.

Ross

[-- Attachment #2: Type: text/html, Size: 1359 bytes --]