On 20 March 2017 at 09:05, Gerd Hoffmann <kra...@redhat.com> wrote: > sdl is probed before audio, so we can simply look at $sdl so see > whenever we have support or not. Throw an error in case sdl audio > is requested without sdl being available. > > Reported-by: Peter Maydell <peter.mayd...@linaro.org> > Signed-off-by: Gerd Hoffmann <kra...@redhat.com> > --- > configure | 10 ++++++++-- > 1 file changed, 8 insertions(+), 2 deletions(-) > > diff --git a/configure b/configure > index 99d8bec..2d088c6 100755 > --- a/configure > +++ b/configure > @@ -2894,6 +2894,12 @@ for drv in $audio_drv_list; do > audio_pt_int="yes" > ;; > > + sdl) > + if test "$sdl" = "no"; then > + error_exit "sdl not found or disabled, can not use sdl audio driver" > + fi > + ;; > + > coreaudio) > libs_softmmu="-framework CoreAudio $libs_softmmu" > ;; > @@ -2907,8 +2913,8 @@ for drv in $audio_drv_list; do > libs_softmmu="$oss_lib $libs_softmmu" > ;; > > - sdl|wav) > - # XXX: Probes for CoreAudio, DirectSound, SDL(?) > + wav) > + # XXX: Probes for CoreAudio, DirectSound > ;; > > *) > -- > 1.8.3.1
Reviewed-by: Peter Maydell <peter.mayd...@linaro.org> Tested-by: Peter Maydell <peter.mayd...@linaro.org> I'll apply this to master as a buildfix in a bit. thanks -- PMM
