[Spice-devel] [PATCH spice-server v2] gstreamer: Include only needed fields in SpiceFormatForGStreamer structure

[Frediano Ziglio]

> 
> On Wed, Feb 01, 2017 at 03:09:36PM +0100, Victor Toso wrote:
> > Hi,
> > 
> > On Wed, Feb 01, 2017 at 05:48:32AM -0500, Frediano Ziglio wrote:
> > > >
> > > > Hi,
> > > >
> > > > On Fri, Jan 27, 2017 at 10:29:22AM +0000, Frediano Ziglio wrote:
> > > > > This structure is used to store format information for
> > > > > both Gstreamer 0.10 and 1.0 however the two format uses
> > > > > different fields from it.
> > > > > Use a macro to filter only needed fields.
> > > > > This currently also fixes a compile error using Gstreamer 0.10
> > > > > (GST_VIDEO_FORMAT_RGB15 not defined as not available).
> > > >
> > > > For how long is it intended to support gstreamer 0.10 in the server?
> > > >
> > > >   toso
> > >
> > > Till rhel6 is in some way supported I suppose.
> > 
> > In rhel 6, we have v0.12.4 + patches but no gstreamer.
> > 
> > If the reason for gstreamer 0.10 is due rhel6, we should at least target
> > support for it till next stable release (0.14.0) or so, and drop it
> > IMHO.
> 
> I believe François/Jeremy had some use for the GStreamer code on el6.
> I'd prefer that we keep el6 support in 0.14.0, and drop it in the next
> cycle.
> 
> Christophe
> 

So... ack ?

Frediano