[OE-core] [PATCH] gstreamer: make pulseaudio support dependent on DISTRO_FEATURES

[Paul Eggleton]

On Wednesday 26 September 2012 14:32:57 Martin Jansa wrote:
> On Wed, Sep 26, 2012 at 01:31:10PM +0100, Paul Eggleton wrote:
> > On Wednesday 26 September 2012 13:38:19 Martin Jansa wrote:
> > > On Wed, Sep 26, 2012 at 12:34:05PM +0100, Paul Eggleton wrote:
> > > > This should be no change to the previous situation unless you
> > > > explicitly have pulseaudio in DISTRO_FEATURES_BACKFILL_CONSIDERED
> > > > (currently).
> > > 
> > > Why not use PACKAGECONFIG for this?
> > 
> > Only because I didn't see the need complicate it just to be able to
> > configure it on a per-recipe basis - you almost certainly want pulseaudio
> > either on or off everywhere. If you want it for stylistic reasons, sure,
> > I guess... it is a bit ugly though.
> 
> Well it looked to me like ideal use-case for PACKAGECONFIG.

It does encapsulate the DEPENDS and EXTRA_OECONF into one piece, yes, but then 
when you make the PACKAGECONFIG option set from DISTRO_FEATURES anyway it gets 
ugly, and here I'm not convinced it adds anything useful.

FWIW, this also matches how the same configuration is implemented for Qt. If we 
want to switch everything over to PACKAGECONFIG later we can with no penalty.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre