qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Gerd Hoffmann <kraxel@redhat.com>
To: qemu-devel@nongnu.org
Cc: "Brad Smith" <brad@comstyle.com>,
	"Philippe Mathieu-Daudé" <philmd@redhat.com>,
	"Gerd Hoffmann" <kraxel@redhat.com>
Subject: [Qemu-devel] [PATCH 4/4] [RfC] audio: probe audio drivers by default
Date: Wed, 23 Jan 2019 09:00:05 +0100	[thread overview]
Message-ID: <20190123080005.31078-5-kraxel@redhat.com> (raw)
In-Reply-To: <20190123080005.31078-1-kraxel@redhat.com>

Add the drivers listed in audio_possible_drivers to audio_drv_list,
using the try-* variants.  That way the probable drivers are compiled by
default if possible.

This is RfC because we might look at the ordering.  On linux we probably
want prefer alsa over oss.  Not sure about pulseaudio.  Most linux
distros use pulse by default on desktops.  On servers it'll probably not
be available.  Dunno how the situation looks like on the bsd family.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
 configure | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/configure b/configure
index bbadd83fea..26e7668028 100755
--- a/configure
+++ b/configure
@@ -790,13 +790,13 @@ MINGW32*)
 ;;
 GNU/kFreeBSD)
   bsd="yes"
-  audio_drv_list="oss"
+  audio_drv_list="oss try-sdl try-pa"
   audio_possible_drivers="oss sdl pa"
 ;;
 FreeBSD)
   bsd="yes"
   make="${MAKE-gmake}"
-  audio_drv_list="oss"
+  audio_drv_list="oss try-sdl try-pa"
   audio_possible_drivers="oss sdl pa"
   # needed for kinfo_getvmmap(3) in libutil.h
   LIBS="-lutil $LIBS"
@@ -809,14 +809,14 @@ FreeBSD)
 DragonFly)
   bsd="yes"
   make="${MAKE-gmake}"
-  audio_drv_list="oss"
+  audio_drv_list="oss try-sdl try-pa"
   audio_possible_drivers="oss sdl pa"
   HOST_VARIANT_DIR="dragonfly"
 ;;
 NetBSD)
   bsd="yes"
   make="${MAKE-gmake}"
-  audio_drv_list="oss"
+  audio_drv_list="oss try-sdl"
   audio_possible_drivers="oss sdl"
   oss_lib="-lossaudio"
   HOST_VARIANT_DIR="netbsd"
@@ -841,7 +841,7 @@ Darwin)
     LDFLAGS="-arch x86_64 $LDFLAGS"
   fi
   cocoa="yes"
-  audio_drv_list="coreaudio"
+  audio_drv_list="coreaudio try-sdl"
   audio_possible_drivers="coreaudio sdl"
   LDFLAGS="-framework CoreFoundation -framework IOKit $LDFLAGS"
   libs_softmmu="-F/System/Library/Frameworks -framework Cocoa -framework IOKit $libs_softmmu"
@@ -857,7 +857,7 @@ SunOS)
   install="${INSTALL-ginstall}"
   smbd="${SMBD-/usr/sfw/sbin/smbd}"
   if test -f /usr/include/sys/soundcard.h ; then
-    audio_drv_list="oss"
+    audio_drv_list="oss try-sdl"
   fi
   audio_possible_drivers="oss sdl"
 # needed for CMSG_ macros in sys/socket.h
@@ -875,7 +875,7 @@ Haiku)
   LIBS="-lposix_error_mapper -lnetwork $LIBS"
 ;;
 Linux)
-  audio_drv_list="oss"
+  audio_drv_list="oss try-alsa try-sdl try-pa"
   audio_possible_drivers="oss alsa sdl pa"
   linux="yes"
   linux_user="yes"
-- 
2.9.3

  parent reply	other threads:[~2019-01-23  8:00 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-01-23  8:00 [Qemu-devel] [PATCH 0/4] audio: rework driver probing Gerd Hoffmann
2019-01-23  8:00 ` [Qemu-devel] [PATCH 1/4] audio: use pkg-config Gerd Hoffmann
2019-01-23  8:30   ` Thomas Huth
2019-01-23  8:00 ` [Qemu-devel] [PATCH 2/4] audio: allow optional audio drivers Gerd Hoffmann
2019-01-23  8:00 ` [Qemu-devel] [PATCH 3/4] audio: use try-sdl and try-pa for openbsd Gerd Hoffmann
2019-01-23  8:00 ` Gerd Hoffmann [this message]
2019-01-23  8:27   ` [Qemu-devel] [PATCH 4/4] [RfC] audio: probe audio drivers by default Thomas Huth
2019-01-23  9:36     ` Daniel P. Berrangé
2019-01-23  9:50       ` Thomas Huth
2019-01-23 10:12         ` Daniel P. Berrangé
2019-01-23 10:36           ` Kamil Rytarowski
2019-01-23 10:59             ` Peter Maydell
2019-01-23 11:10               ` Kamil Rytarowski
2019-01-23 11:16                 ` Peter Maydell
2019-01-23 11:30                   ` Kamil Rytarowski
2019-01-23 12:20                 ` Gerd Hoffmann
2019-01-23 12:45                   ` Kamil Rytarowski
2019-01-23 13:59                     ` Gerd Hoffmann
2019-01-23 14:45                       ` Kamil Rytarowski
2019-01-23 14:54                       ` Brad Smith
2019-01-23 15:12                         ` Gerd Hoffmann
2019-01-23 15:20                           ` Kamil Rytarowski
2019-01-24  6:38                             ` Gerd Hoffmann
2019-01-24  6:51                               ` Thomas Huth
2019-01-24 10:06                               ` Kamil Rytarowski
2019-01-23 10:16         ` Kamil Rytarowski
2019-01-23 10:24     ` Gerd Hoffmann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190123080005.31078-5-kraxel@redhat.com \
    --to=kraxel@redhat.com \
    --cc=brad@comstyle.com \
    --cc=philmd@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).