[v3,2/3] qt-mobility: Add pulseaudio dependency if DISTRO_FEATURES has pulseaudio

Submitted by eu@felipetonello.com on March 19, 2013, 9:54 p.m.

Details

Message ID 1363730053-7719-3-git-send-email-eu@felipetonello.com
State Accepted
Commit 12fc36be5796142b55edde67f1220531bb95cb29
Headers show

Commit Message

eu@felipetonello.com March 19, 2013, 9:54 p.m.
From: "Felipe F. Tonello" <ftonello@cercacor.com>

Signed-off-by: Felipe F. Tonello <ftonello@cercacor.com>
---
 meta/recipes-qt/qt4/qt-mobility_1.2.0.inc |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Patch hide | download patch | download mbox

diff --git a/meta/recipes-qt/qt4/qt-mobility_1.2.0.inc b/meta/recipes-qt/qt4/qt-mobility_1.2.0.inc
index e5e671e..fca4cb0 100644
--- a/meta/recipes-qt/qt4/qt-mobility_1.2.0.inc
+++ b/meta/recipes-qt/qt4/qt-mobility_1.2.0.inc
@@ -1,5 +1,5 @@ 
 HOMEPAGE = "http://qt.nokia.com"
-DEPENDS += "pulseaudio"
+DEPENDS += "${@base_contains('DISTRO_FEATURES', 'pulseaudio', 'pulseaudio', '', d)}"
 DEPENDS += "${@base_contains('DISTRO_FEATURES', 'bluetooth', 'bluez4', '', d)}"
 
 LICENSE = "LGPLv2.1"

Comments

Otavio Salvador March 20, 2013, 2:04 p.m.
On Tue, Mar 19, 2013 at 6:54 PM,  <eu@felipetonello.com> wrote:
> From: "Felipe F. Tonello" <ftonello@cercacor.com>
>
> Signed-off-by: Felipe F. Tonello <ftonello@cercacor.com>
> ---
>  meta/recipes-qt/qt4/qt-mobility_1.2.0.inc |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/recipes-qt/qt4/qt-mobility_1.2.0.inc b/meta/recipes-qt/qt4/qt-mobility_1.2.0.inc
> index e5e671e..fca4cb0 100644
> --- a/meta/recipes-qt/qt4/qt-mobility_1.2.0.inc
> +++ b/meta/recipes-qt/qt4/qt-mobility_1.2.0.inc
> @@ -1,5 +1,5 @@
>  HOMEPAGE = "http://qt.nokia.com"
> -DEPENDS += "pulseaudio"
> +DEPENDS += "${@base_contains('DISTRO_FEATURES', 'pulseaudio', 'pulseaudio', '', d)}"
>  DEPENDS += "${@base_contains('DISTRO_FEATURES', 'bluetooth', 'bluez4', '', d)}"

Is it possible to explicitly enable/disable pulseaudio support?
Ross Burton March 20, 2013, 2:56 p.m.
On 20 March 2013 14:04, Otavio Salvador <otavio@ossystems.com.br> wrote:
> Is it possible to explicitly enable/disable pulseaudio support?

Isn't PA one of those "all or nothing" options?  If your system is
using PA then you want support for it everywhere, if it's not then you
don't want it present at all.

Ross
Otavio Salvador March 20, 2013, 3:04 p.m.
On Wed, Mar 20, 2013 at 11:56 AM, Burton, Ross <ross.burton@intel.com> wrote:
> On 20 March 2013 14:04, Otavio Salvador <otavio@ossystems.com.br> wrote:
>> Is it possible to explicitly enable/disable pulseaudio support?
>
> Isn't PA one of those "all or nothing" options?  If your system is
> using PA then you want support for it everywhere, if it's not then you
> don't want it present at all.

Yes but if we don't explicitly disable/enable it we risk to do a host
contamination (for example building for same host arch)
Ross Burton March 20, 2013, 3:08 p.m.
On 20 March 2013 15:04, Otavio Salvador <otavio@ossystems.com.br> wrote:
> Yes but if we don't explicitly disable/enable it we risk to do a host
> contamination (for example building for same host arch)

Oh, right, your context wasn't clear.

Yes, totally agreed, and this is exactly why PACKAGECONFIG is so great.

Ross