Re: [RFC][PATCH] packagegroup.bbclass: Drop build-time dependencies

[Paul Eggleton <paul.eggleton@linux.intel.com>]

Hi Martin,

On Friday 01 November 2013 13:12:12 Martin Jansa wrote:
> * RFC because:
>   packagegroup.bbclass inherits allarch and by default sets
>   PACKAGE_ARCH = "all", but then it has runtime-dependency on
>   TUNE_PKGARCH packages and different signatures, so instead
>   of setting INHIBIT_DEFAULT_DEPS in addition to inherit allarch
>   I would like to drop default allarch completely.

The fix looks OK, but I don't think we should be going further and removing the 
inherit. To my mind, the question of dependencies comes down to whether or not 
the packagegroup output changes depending on TUNE_PKGARCH, which in the 
general case it should not do.

If we need to do some optimisation to avoid the packagegroup recipe rebuilding 
when it shouldn't due to packages it "contains" changing, we can certainly 
look at that.

> * drop virtual/kernel dependency from packagegroup-core-boot
>   because kernel isn't needed to build packagegroup and building
>   it doesn't influence if it's included in image or not.

Yes, I agree this seems erroneous.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre