[Buildroot] [PATCH] wine: Add dependency check for ALSA

[Buildroot] [PATCH] wine: Add dependency check for ALSA

[André Hentschel]

Signed-off-by: Andr? Hentschel <nerv@dawncrow.de>
---

This fixes the problem from http://autobuild.buildroot.org/results/018/018391c5e70d33bc0b5745ad7a2a273efa9be0d7/

 package/wine/wine.mk | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/package/wine/wine.mk b/package/wine/wine.mk
index 926ac99..47c7259 100644
--- a/package/wine/wine.mk
+++ b/package/wine/wine.mk
@@ -29,6 +29,13 @@ ifeq ($(BR2_TOOLCHAIN_EXTERNAL),y)
 WINE_CONF_OPTS += TARGETFLAGS="-b $(call qstrip,$(BR2_TOOLCHAIN_EXTERNAL_PREFIX))"
 endif
 
+ifeq ($(BR2_PACKAGE_ALSA_LIB)$(BR2_PACKAGE_ALSA_LIB_SEQ),yy)
+WINE_CONF_OPTS += --with-alsa
+WINE_DEPENDENCIES += alsa-lib
+else
+WINE_CONF_OPTS += --without-alsa
+endif
+
 ifeq ($(BR2_PACKAGE_CUPS),y)
 WINE_CONF_OPTS += --with-cups
 WINE_DEPENDENCIES += cups

[Buildroot] [PATCH] wine: Add dependency check for ALSA

[Ryan Barnett]

Andr? ,

On Thu, Mar 5, 2015 at 1:38 PM, Andr? Hentschel <nerv@dawncrow.de> wrote:

You should move the fixes line above your signed off line as follows
otherwise it will not be preserved in the commit message:

Fixes:
  http://autobuild.buildroot.org/results/018/018391c5e70d33bc0b5745ad7a2a273efa9be0d7/

> Signed-off-by: Andr? Hentschel <nerv@dawncrow.de>
> ---
>
> This fixes the problem from http://autobuild.buildroot.org/results/018/018391c5e70d33bc0b5745ad7a2a273efa9be0d7/
>
>  package/wine/wine.mk | 7 +++++++
>  1 file changed, 7 insertions(+)

Thanks,
-Ryan

-- 
Ryan Barnett / Sr Software Engineer
Airborne Information Systems / Security Systems and Software
MS 131-100, C Ave NE, Cedar Rapids, IA, 52498, USA
ryan.barnett at rockwellcollins.com
www.rockwellcollins.com

[Buildroot] [PATCH] wine: Add dependency check for ALSA

[Thomas Petazzoni]

Dear Andr? Hentschel,

On Thu, 05 Mar 2015 20:38:40 +0100, Andr? Hentschel wrote:
> Signed-off-by: Andr? Hentschel <nerv@dawncrow.de>

Thanks, applied, after moving the reference to the autobuilder failure
within the commit log itself, as suggested by Ryan.

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com