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

2019-01-08 Thread Frediano Ziglio
> > Hi, > > On Tue, Jan 08, 2019 at 08:25:09AM -0500, Frediano Ziglio wrote: > > > We have only GStreamer support for video, so let's make it > > > required. > > > > There's also mjpeg for video. > > I've tested mjpeg from GStreamer and works well too 0:-) > > > > There is no modularity

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

2019-01-08 Thread Victor Toso
Hi, On Tue, Jan 08, 2019 at 08:25:09AM -0500, Frediano Ziglio wrote: > > We have only GStreamer support for video, so let's make it > > required. > > There's also mjpeg for video. I've tested mjpeg from GStreamer and works well too 0:-) > > There is no modularity needed at this point, it can

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

2019-01-08 Thread Frediano Ziglio
> > Hi > > On Mon, Jan 7, 2019 at 5:46 PM Frediano Ziglio wrote: > > > > > Hi > > > > > > On Mon, Jan 7, 2019 at 3:44 PM Frediano Ziglio > > > wrote: > > > > > > > > > Hi > > > > > > > > > > On Sat, Jan 5, 2019 at 9:00 PM Frediano Ziglio > > > > > wrote: > > > > > > > > > > > > > > > > > > >

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

2019-01-08 Thread Christophe Fergeau
On Mon, Jan 07, 2019 at 11:30:55PM +0400, Marc-André Lureau wrote: > 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

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

2019-01-08 Thread Victor Toso
On Tue, Jan 08, 2019 at 02:41:22PM +0400, Marc-André Lureau wrote: > Hi > > On Tue, Jan 8, 2019 at 1:15 PM Victor Toso wrote: > > > > 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 >

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

2019-01-08 Thread Marc-André Lureau
Hi On Tue, Jan 8, 2019 at 1:15 PM Victor Toso wrote: > > 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: > > > >

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

2019-01-08 Thread Victor Toso
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" >

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

2019-01-07 Thread Marc-André Lureau
Hi On Mon, Jan 7, 2019 at 5:46 PM Frediano Ziglio wrote: > > > Hi > > > > On Mon, Jan 7, 2019 at 3:44 PM Frediano Ziglio wrote: > > > > > > > Hi > > > > > > > > On Sat, Jan 5, 2019 at 9:00 PM Frediano Ziglio > > > > wrote: > > > > > > > > > > > > > > > > > Hi > > > > > > > > > > > > On Sat,

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

2019-01-07 Thread Frediano Ziglio
> Hi > > On Mon, Jan 7, 2019 at 3:44 PM Frediano Ziglio wrote: > > > > > Hi > > > > > > On Sat, Jan 5, 2019 at 9:00 PM Frediano Ziglio > > > wrote: > > > > > > > > > > > > > > Hi > > > > > > > > > > On Sat, Jan 5, 2019 at 1:59 PM Victor Toso > > > > > wrote: > > > > > > > > > > > > Hi, > > > >

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

2019-01-07 Thread Marc-André Lureau
Hi On Mon, Jan 7, 2019 at 3:44 PM Frediano Ziglio wrote: > > > Hi > > > > On Sat, Jan 5, 2019 at 9:00 PM Frediano Ziglio wrote: > > > > > > > > > > > Hi > > > > > > > > On Sat, Jan 5, 2019 at 1:59 PM Victor Toso > > > > wrote: > > > > > > > > > > Hi, > > > > > > > > > > On Fri, Jan 04, 2019

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

2019-01-07 Thread Christophe Fergeau
On Mon, Jan 07, 2019 at 03:12:45PM +0400, Marc-André Lureau wrote: > g_warning or g_debug, not a big difference to me here. > > But g_debug() is likely to be missed, since it is silent by default. Yes, and being silent for now is the goal since the vast majority of systems will be hitting that

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

2019-01-07 Thread Frediano Ziglio
> Hi > > On Sat, Jan 5, 2019 at 9:00 PM Frediano Ziglio wrote: > > > > > > > > Hi > > > > > > On Sat, Jan 5, 2019 at 1:59 PM Victor Toso wrote: > > > > > > > > Hi, > > > > > > > > On Fri, Jan 04, 2019 at 04:48:21PM +0400, marcandre.lur...@redhat.com > > > > wrote: > > > > > From: Marc-André

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

2019-01-07 Thread Marc-André Lureau
Hi On Mon, Jan 7, 2019 at 2:50 PM Christophe Fergeau wrote: > > On Mon, Jan 07, 2019 at 02:12:34PM +0400, Marc-André Lureau wrote: > > Hi > > > > On Mon, Jan 7, 2019 at 2:03 PM Christophe Fergeau > > wrote: > > > > > > On Sun, Jan 06, 2019 at 12:24:33AM +0400, Marc-André Lureau wrote: > > > >

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

2019-01-07 Thread Christophe Fergeau
On Mon, Jan 07, 2019 at 02:12:34PM +0400, Marc-André Lureau wrote: > Hi > > On Mon, Jan 7, 2019 at 2:03 PM Christophe Fergeau wrote: > > > > On Sun, Jan 06, 2019 at 12:24:33AM +0400, Marc-André Lureau wrote: > > > > > > > > > I'll work on a patch to remove pulse. > > > > > > > > > > > > > It

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

2019-01-07 Thread Marc-André Lureau
Hi On Mon, Jan 7, 2019 at 2:03 PM Christophe Fergeau wrote: > > On Sun, Jan 06, 2019 at 12:24:33AM +0400, Marc-André Lureau wrote: > > > > > > > I'll work on a patch to remove pulse. > > > > > > > > > > It seems too soon. > > > > yes, we haven't done enough testing of the gstreamer backend on

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

2019-01-07 Thread Christophe Fergeau
On Sun, Jan 06, 2019 at 12:24:33AM +0400, Marc-André Lureau wrote: > > > > > I'll work on a patch to remove pulse. > > > > > > > It seems too soon. > > yes, we haven't done enough testing of the gstreamer backend on Linux. > The recording path at least fails very often for me due to a race: >

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

2019-01-05 Thread Marc-André Lureau
Hi On Sat, Jan 5, 2019 at 9:00 PM Frediano Ziglio wrote: > > > > > Hi > > > > On Sat, Jan 5, 2019 at 1:59 PM Victor Toso wrote: > > > > > > Hi, > > > > > > On Fri, Jan 04, 2019 at 04:48:21PM +0400, marcandre.lur...@redhat.com > > > wrote: > > > > From: Marc-André Lureau > > > > > > > > Hi, > >

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

2019-01-05 Thread Frediano Ziglio
> > Hi > > On Sat, Jan 5, 2019 at 1:59 PM Victor Toso wrote: > > > > Hi, > > > > On Fri, Jan 04, 2019 at 04:48:21PM +0400, marcandre.lur...@redhat.com > > wrote: > > > From: Marc-André Lureau > > > > > > Hi, > > > > > > The series drops support for optional GStreamer dependency. Spice > > >

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

2019-01-05 Thread Victor Toso
Hi, On Sat, Jan 05, 2019 at 02:48:57PM +0400, Marc-André Lureau wrote: > Hi > > On Sat, Jan 5, 2019 at 1:59 PM Victor Toso wrote: > > > > Hi, > > > > On Fri, Jan 04, 2019 at 04:48:21PM +0400, marcandre.lur...@redhat.com wrote: > > > From: Marc-André Lureau > > > > > > Hi, > > > > > > The

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

2019-01-05 Thread Marc-André Lureau
Hi On Sat, Jan 5, 2019 at 1:59 PM Victor Toso wrote: > > Hi, > > On Fri, Jan 04, 2019 at 04:48:21PM +0400, marcandre.lur...@redhat.com wrote: > > From: Marc-André Lureau > > > > Hi, > > > > The series drops support for optional GStreamer dependency. Spice > > increasingly require GStreamer for

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

2019-01-05 Thread Victor Toso
Hi, On Fri, Jan 04, 2019 at 04:48:21PM +0400, marcandre.lur...@redhat.com wrote: > From: Marc-André Lureau > > Hi, > > The series drops support for optional GStreamer dependency. Spice > increasingly require GStreamer for audio and video support. GStreamer > is widely available, including in

[Spice-devel] [PATCH spice-gtk 0/5] Require GStreamer, fix build warnings

2019-01-04 Thread marcandre . lureau
From: Marc-André Lureau Hi, The series drops support for optional GStreamer dependency. Spice increasingly require GStreamer for audio and video support. GStreamer is widely available, including in embedded space. Marc-André Lureau (5): build-sys: remove autoconf --with-audio=.. build-sys: