[Spice-devel] [PATCH spice-gtk v2] configure: Try gstreamer audio backend if pulse is not available
Pavel Grunt
pgrunt at redhat.com
Mon Jun 15 04:44:14 PDT 2015
On Mon, 2015-06-15 at 07:37 -0400, Marc-André Lureau wrote:
> Hmm, it looks like it would compile fine with both gst & pulse, but at runtime
> you would only have gst backend, and a leak:
>
> #ifdef WITH_PULSE
> self = SPICE_AUDIO(spice_pulse_new(session, context, name));
> #endif
> #if defined(WITH_GSTAUDIO)
> self = SPICE_AUDIO(spice_gstaudio_new(session, context, name));
> #endif
Sorry about that. I'm fixing it.
Pavel
>
>
> ----- Original Message -----
> > Disable audio if no backend is available and configure is used with
> > the option '--with-audio=auto'
> > ---
> > v2: Removed AS_IF cascade, merged with "configure: Do not force audio by
> > default"
> > ---
> > configure.ac | 23 +++++++++++------------
> > 1 file changed, 11 insertions(+), 12 deletions(-)
> >
> > diff --git a/configure.ac b/configure.ac
> > index 1d8f4d0..dc8557a 100644
> > --- a/configure.ac
> > +++ b/configure.ac
> > @@ -296,19 +296,22 @@ AC_ARG_WITH([audio],
> > [],
> > [with_audio="auto"])
> >
> > -AS_IF([test "x$with_audio" = "xauto"], [
> > - AS_IF([test "x$os_win32" = "xyes"], [with_audio=gstreamer],
> > [with_audio=pulse])
> > -])
> > -
> > case "$with_audio" in
> > - gstreamer|pulse|no*)
> > + gstreamer|pulse|auto*)
> > + PKG_CHECK_MODULES(GST, gstreamer-1.0 gstreamer-base-1.0
> > gstreamer-app-1.0 gstreamer-audio-1.0, [have_gst=yes], [have_gst=no])
> > + PKG_CHECK_MODULES(PULSE, libpulse libpulse-mainloop-glib,
> > [have_pulse=yes], [have_pulse=no])
> > + ;;
> > + no*)
> > ;;
> > *) AC_MSG_ERROR(Unsupported audio backend)
> > esac
> >
> > -AS_IF([test "x$with_audio" = "xpulse"],
> > - [PKG_CHECK_MODULES(PULSE, libpulse libpulse-mainloop-glib,
> > [have_pulse=yes], [have_pulse=no])],
> > - [have_pulse=no])
> > +AS_IF([test "x$with_audio" = "xauto" && test "x$have_pulse" = "xyes"],
> > + [with_audio=pulse])
> > +
> > +AS_IF([test "x$with_audio" = "xauto" && test "x$have_gst" = "xyes"],
> > + [with_audio=gstreamer],
> > + [with_audio=no])
> >
> > AS_IF([test "x$have_pulse" = "xyes"],
> > [AC_DEFINE([WITH_PULSE], 1, [Have pulseaudio?])],
> > @@ -320,10 +323,6 @@ AM_CONDITIONAL([WITH_PULSE], [test "x$have_pulse" =
> > "xyes"])
> > AC_SUBST(PULSE_CFLAGS)
> > AC_SUBST(PULSE_LIBS)
> >
> > -AS_IF([test "x$with_audio" = "xgstreamer"],
> > - [PKG_CHECK_MODULES(GST, gstreamer-1.0 gstreamer-base-1.0
> > gstreamer-app-1.0 gstreamer-audio-1.0, [have_gst=yes], [have_gst=no])],
> > - [have_gst=no])
> > -
> > AS_IF([test "x$have_gst" = "xyes"],
> > [AC_DEFINE([WITH_GSTAUDIO], 1, [Have GStreamer 1.0?])],
> > [AS_IF([test "x$with_audio" = "xgstreamer"],
> > --
> > 2.4.3
> >
> > _______________________________________________
> > Spice-devel mailing list
> > Spice-devel at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/spice-devel
> >
More information about the Spice-devel
mailing list