Re: [PATCH] gstreamer: make pulseaudio support dependent on DISTRO_FEATURES
From: Paul Eggleton <hidden>
Date: 2012-09-26 13:37:01
On Wednesday 26 September 2012 14:32:57 Martin Jansa wrote:
On Wed, Sep 26, 2012 at 01:31:10PM +0100, Paul Eggleton wrote:quoted
On Wednesday 26 September 2012 13:38:19 Martin Jansa wrote:quoted
On Wed, Sep 26, 2012 at 12:34:05PM +0100, Paul Eggleton wrote:quoted
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