[OE-core] [PATCH 0/3] Make pulseaudio a DISTRO_FEATURE
Paul Eggleton
paul.eggleton at linux.intel.com
Wed Nov 23 17:45:35 UTC 2011
On Wednesday 23 November 2011 17:09:08 Richard Purdie wrote:
> I'm wondering if we can do something in the core like:
>
> DISTRO_FEATURES_BACKFILLOPTS = "pulseaudio"
>
> and have the distro set:
>
> DISTRO_FEATURES_BACKFILLCONSIDERED = ""
>
> and then add some code which looks for anything in
> DISTRO_FEATURES_BACKFILLOPTS but not in
> DISTRO_FEATURES_BACKFILLCONSIDERED and adds it to DISTRO_FEATURES.
>
> Distros can then opt out of a given feature by adding it to
> DISTRO_FEATURES_BACKFILLCONSIDERED.
>
> This would let us maintain compatibility but also move forward and
> create new settings with names that make sense.
This will solve the issue, but the more I think about it the more I don't like
it - from the perspective of a new user it just puts a fairly arbitrary line
between old features and new ones. If you want control over the full range of
options you'll need to look at two places.
I think I'm right in assuming that when we introduce a DISTRO_FEATURES feature
we are almost always doing it to allow disabling some existing functionality,
rather than enabling something new. In that case, should we not be providing
the appropriate mechanism so that can exclude the features they don't want
rather than including the ones that they do? That would avoid breakage over
time in a much simpler way than the above, IMHO. (I'm aware that we have
oe_filter_out but I think that's a bit untidy.)
Cheers,
Paul
--
Paul Eggleton
Intel Open Source Technology Centre
More information about the Openembedded-core
mailing list