Re: [PATCH 2/2] base: make feature backfilling happen earlier

Paul Eggleton
2013-01-28T16:13:51Z

On Monday 28 January 2013 16:44:15 Enrico Scholz wrote:
> Phil Blundell <pb@pbcl.net> writes:
> > This does seem rather like an abuse of DISTRO_FEATURES_BACKFILL. Can
> > you explain why you are doing it this way rather than just setting
> > DISTRO_FEATURES directly to what you wanted?
>
> I need a way to:
>
> 1. set some defaults on distribution base and avoid nasty details like
> mandatory ${DISTRO_FEATURES_LIBC} flags in the project configuration
>
> 2. allow to override these defaults on a per-project base
>
> 3. add features support by recipes/classes of my distribution
>
> afaik, DISTRO_FEATURES_BACKFILL + _CONSIDERED exist to allow the first
> two point without an '-=' operator which lacks in bitbake.

No they don't. They exist to allow adding new features that should be enabled
for all existing distro configs without each of those having to be changed, and
provide a means for distros to opt out of that enabling if they wish.

> Of course, I could reinvent the wheel and write my own _CONSIDERED
> mechanism. But until now (resp. without the '=' => '?=' change), it
> worked fine with DISTRO_FEATURES_BACKFILL.

We don't support using the backfill mechanism in this manner; so if it breaks
you get to keep both pieces.

It's still not clear to me why you could not just set DISTRO_FEATURES
directly. If you have values you're likely to want to remove, you can put them
in separate variables that you can clear out later.

Cheers,
Paul

--

Paul Eggleton
Intel Open Source Technology Centre