[Qemu-devel] [PATCH] configure: Fix build with ALSA audio driver

[Qemu-devel] [PATCH] configure: Fix build with ALSA audio driver

[Stefan Weil]

Since commit 417c9d72d48275d19c60861896efd4962d21aca2,
all configure tests normally run with -Werror.

Some of these tests now fail because they raised a compiler warning.

Here a build breakage for ALSA (configure --audio-drv-list=alsa) is fixed.

Signed-off-by: Stefan Weil <sw@weilnetz.de>
---

One of the buildbot jobs should be configured with all sound options:

--audio-card-list=ac97,es1370,sb16,cs4231a,adlib,gus,hda
--audio-drv-list=alsa,sdl,oss,esd,pa
--enable-mixemu

That would have detected this bug (and more potential bugs in the future).

Regards

Stefan W.



 configure |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/configure b/configure
index 0a3896e..f745cc2 100755
--- a/configure
+++ b/configure
@@ -1888,7 +1888,7 @@ for drv in $audio_drv_list; do
     case $drv in
     alsa)
     audio_drv_probe $drv alsa/asoundlib.h -lasound \
-        "snd_pcm_t **handle; return snd_pcm_close(*handle);"
+        "return snd_pcm_close((snd_pcm_t *)0);"
     libs_softmmu="-lasound $libs_softmmu"
     ;;
 
-- 
1.7.10

Re: [Qemu-devel] [PATCH] configure: Fix build with ALSA audio driver

[Stefan Hajnoczi]

On Sun, Jul 15, 2012 at 03:49:13PM +0200, Stefan Weil wrote:
> One of the buildbot jobs should be configured with all sound options:
> 
> --audio-card-list=ac97,es1370,sb16,cs4231a,adlib,gus,hda
> --audio-drv-list=alsa,sdl,oss,esd,pa
> --enable-mixemu
> 
> That would have detected this bug (and more potential bugs in the future).

Daniel, please could you add these ./configure options to the qemu.git
buildbot?

I have tested on the yuzuki buildbot with Stefan Weil's patch applied.
Yuzuki has the library headers installed so we can build it all and get
better coverage.

Thanks,
Stefan

Re: [Qemu-devel] [PATCH] configure: Fix build with ALSA audio driver

[Daniel Gollub]

[-- Attachment #1: Type: text/plain, Size: 1178 bytes --]

Hi,

On Monday, July 16, 2012 12:07:58 PM Stefan Hajnoczi wrote:
> On Sun, Jul 15, 2012 at 03:49:13PM +0200, Stefan Weil wrote:
> > One of the buildbot jobs should be configured with all sound options:
> > 
> > --audio-card-list=ac97,es1370,sb16,cs4231a,adlib,gus,hda
> > --audio-drv-list=alsa,sdl,oss,esd,pa
> > --enable-mixemu
> > 
> > That would have detected this bug (and more potential bugs in the
> > future).
> Daniel, please could you add these ./configure options to the qemu.git
> buildbot?


I'll apply that tomorrow or the day after tomorrow once the build regression 
from last night got fixed. Otherwise we would not get notified if other build 
factories break due to this change ...

> 
> I have tested on the yuzuki buildbot with Stefan Weil's patch applied.
> Yuzuki has the library headers installed so we can build it all and get
> better coverage.


Best Regards,
Daniel

-- 
Daniel Gollub
Linux Consultant & Developer
Tel.: +49-160 47 73 970 
Mail: gollub@b1-systems.de

B1 Systems GmbH
Osterfeldstraße 7 / 85088 Vohburg / http://www.b1-systems.de
GF: Ralph Dehner / Unternehmenssitz: Vohburg / AG: Ingolstadt,HRB 3537

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]