Re: [PATCH spice-gtk 0/5] Require GStreamer, fix build warnings

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

 



Hi,

On Mon, Jan 07, 2019 at 11:30:55PM +0400, Marc-André Lureau wrote:
> > > In the future, yes! For now, use GStreamer by default if possible, and
> > > fallback on pulse.
> > >
> > > See also proposed implementation:
> > > "[PATCH spice-gtk 32/34] gst: check pulseaudio plugin version >= 1.15"
> > > "[PATCH spice-gtk 33/34] audio: use gstreamer by default"
> 
> We have only GStreamer support for video, so let's make it required.
> There is no modularity needed at this point, it can be added when
> needed (although I would rather have less modularity in general,
> because of testing matrix, maintainance..).
>
> We would like to remove PulseAudio audio backend in the future, as it
> will be replaced by pipewire, and unnecessarily make us maintain
> different audio backends: GStreamer handles various audio backends for
> us, and we know it is quite solid since it is being used with the
> Windows build for a long while.
>
> Since we would like to release v0.36 soonish, and there are valid
> concerns that such a change now is a bit too risky, I propose to:
> 
> - make GStreamer a blessed dependency in 0.36 for audio & video (since
> it is required for video decoding/overlay etc)
> - keep pulse audio backend as default, but add a warning at compile
> time that it will be deprecated in future releases
> - make GStreamer audio backend lower the rank of pulsesrc < 1.15 so
> alsa or other backends are chosen with higher priority

Not sure if that's good. Because of one pulsesrc bug we could
likely have potential different scenarios per user on their bug
reports for audio related issues.

> - after 0.36, make GStreamer audo backend the default, and disable the
> pulse backend (unless explicetly --enable-pulse).

Yes

> - later on, remove the pulse audio backend

Yep

> Any comments?

Besides being unsure about changing the rank of pulsesrc (I'd
simple prefer that distro's backport the fix or that we don't set
the volume before the audio stream is created).

We will need a spice-protocol release too...

Cheers,
Victor

Attachment: signature.asc
Description: PGP signature

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

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