Re: [client 07/11] build-sys: Allow simultaneous support for Pulse and GStreamer audio

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, Nov 08, 2015 at 07:39:11PM +0100, Francois Gouget wrote:
> On Fri, 6 Nov 2015, Christophe Fergeau wrote:
> [...]
> > > -AC_ARG_WITH([audio],
> > > -  AS_HELP_STRING([--with-audio=@<:@gstreamer/pulse/auto/no@:>@], [Select audio backend @<:@default=auto@:>@]),
> > > +AC_ARG_ENABLE([pulse],
> > > +  AS_HELP_STRING([--enable-pulse=@<:@yes/auto/no@:>@], [Enable the PulseAudio backend @<:@default=auto@:>@]),
> > 
> > Imo it would be a less disruptive change if we changed '--with-audio=auto' to
> > enable both GStreamer and PulseAudio if the needed .pc files are
> > available. Removing --with-audio and replacing it with
> > --enable-pulseaudio/--enable-gstreamer means anyone using --with-audio
> > will need to update its build scripts.
> 
> The drawback of --with-audio=auto is that it makes it impossible to 
> require having support for both PulseAudio and GStreamer. That is unlike 
> './configure --enable-pulse --enable-gstaudio' it will not print an 
> error if one of them is not available.

Yup, good point.

> 
> (and something like --with-audio=pulse,gstreamer feels wrong and would 
> be very non standard)

--with-audio=all could work...

> 
> Would keeping --with-audio as a temporary frontend for the two enable 
> options be ok? It could print a warning to remind developers it's 
> deprecated?

...but this works for me too.

Christophe

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]