[Spice-devel] [PATCH v6 00/26] Add GStreamer support for video streams

[Victor Toso]

Hi,

On Wed, Oct 14, 2015 at 05:30:28PM +0200, Francois Gouget wrote:
>
> This is the sixth iteration of these patches. I believe that I have
> taken into account all the remarks from the previous round, the
> exceptions being those I sent an email about.

I took a good look in the spice-gtk part for now and just minor comments
on it. I'll be doing some tests today just to be sure but client side
looks good to me.

>
> Let me know if there are changes that are needed for inclusion.
>

I'll take a look in the server side but I think others are best fit to
find issues there than me :)

>
> To simplify testing, one can also get the patches from the gst branch of 
> the GitHub repositories below:
> 
> spice:          https://github.com/fgouget/spice
> spice-gtk:      https://github.com/fgouget/spice-gtk
> xf86-video-qxl: https://github.com/fgouget/xf86-video-qxl
> spice-protocol: https://github.com/fgouget/spice-protocol
> 
> (spice-common no longer contains any modification)
> spice-common: https://github.com/fgouget/spice-common
> 
> (there's also 'extras' branches with more experimental/future patches 
> for the curious)
> 
> 
> For spice-html5 and QEMU one would have to refer to the patches posted 
> previously on spice-devel. They may need some updates but should still 
> work (that is as well as before) with this series.
> 
> QEMU:
>   http://lists.freedesktop.org/archives/spice-devel/2015-May/019771.html
> 
> Spice-html5 (note that there are known issues):
>   http://lists.freedesktop.org/archives/spice-devel/2015-May/019773.html
>   http://lists.freedesktop.org/archives/spice-devel/2015-May/019774.html
>
>
> --
> Francois Gouget <fgouget at codeweavers.com>

Many thanks for your patches and sorry for this delay on reviewing them.

Cheers,
  toso