Re: [Spice-devel] Moving libcacard repo to spice fdo

2015-09-21 Thread Marc-André Lureau
Hi The repository is up http://cgit.freedesktop.org/spice/libcacard, let's prepare a release so qemu patch can be merged. -- Marc-André Lureau ___ Spice-devel mailing list Spice-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo

[Spice-devel] [PATCH 1/1] Allow missing capabilities from source host

2015-09-21 Thread Dr. David Alan Gilbert (git)
From: "Dr. David Alan Gilbert" When loading a USB redirection stream during a qemu migration, the source QEMU might be earlier and be missing a bunch of capabilities that are now available in a more modern version; allow this migration to work as long as the source isn't claiming any capabilities

[Spice-devel] [PATCH libcacard] Release v2.5

2015-09-21 Thread marcandre . lureau
From: Marc-André Lureau --- Makefile.am | 1 + NEWS| 6 ++ 2 files changed, 7 insertions(+) create mode 100644 NEWS diff --git a/Makefile.am b/Makefile.am index bf29337..2e737a9 100644 --- a/Makefile.am +++ b/Makefile.am @@ -49,6 +49,7 @@ vscclient_LDADD = libcacard.la $(CACARD_LIB

Re: [Spice-devel] [PATCH v5 17/20] spice-gtk: Add a GStreamer video decoder with support for MJPEG, VP8 and h264.

2015-09-21 Thread Christophe Fergeau
Hey, Couple of minor comments below, On Thu, Aug 27, 2015 at 09:03:06PM +0200, Francois Gouget wrote: > Based on a patch by Jeremy White. > > Signed-off-by: Francois Gouget > --- > > ***WARNNIG*** This is the first client patch that needs the protocol > constants defined in patch

Re: [Spice-devel] [PATCH v5 16/20] spice-gtk: Refactor the video decoding to use a more object oriented design.

2015-09-21 Thread Christophe Fergeau
Hey, On Thu, Aug 27, 2015 at 09:03:01PM +0200, Francois Gouget wrote: > The video decoder no longer stores its internal state in the display_stream > struct, or depend on it to know which frame to decode or store the decoded > frame. > This way adding alternative implementations will not pile al

Re: [Spice-devel] [PATCH v5 19/20] spice-gtk: Make it possible to disable support for the builtin MJPEG video decoder.

2015-09-21 Thread Christophe Fergeau
On Thu, Aug 27, 2015 at 09:03:13PM +0200, Francois Gouget wrote: > This makes it possible to test the GStreamer video decoder with MJPEG streams. > > Signed-off-by: Francois Gouget > --- > > Changes since take 2: > - None. > > > configure.ac | 8 > src/Makefile.am | 7

Re: [Spice-devel] [PATCH v5 20/20] spice-gtk: Use typefind and decodebin as fallbacks to set up the GStreamer pipeline.

2015-09-21 Thread Christophe Fergeau
On Thu, Aug 27, 2015 at 09:03:21PM +0200, Francois Gouget wrote: > Potentially this means future video codecs can be supported automatically. > One can also use force usage of typefind and decodebin by setting the > SPICE_GST_AUTO environment variable. If set to 'decodebin' then only > decodebin wi

Re: [Spice-devel] [PATCH v5 18/20] spice-gtk: Avoid copying the compressed message data in the GStreamer decoder.

2015-09-21 Thread Christophe Fergeau
On Thu, Aug 27, 2015 at 09:03:09PM +0200, Francois Gouget wrote: > Signed-off-by: Francois Gouget > --- > > Changes since take 2: > - None. > > src/channel-display-gst.c | 89 > --- > 1 file changed, 84 insertions(+), 5 deletions(-) > > diff --git

Re: [Spice-devel] [PATCH v5 02/20] server: Refactor the Spice server video encoding so alternative implementations can be added.

2015-09-21 Thread Christophe Fergeau
On Thu, Aug 27, 2015 at 09:00:42PM +0200, Francois Gouget wrote: > This patch simply replaces the mjpeg_encoder_xxx() call points with a more > object oriented design. > > Signed-off-by: Francois Gouget > --- > > Changes since take 4: > - Fixed the reported formatting issues. > > > server/M

Re: [Spice-devel] [PATCH v5 03/20] server: Add a GStreamer 0.10 MJPEG video encoder and use it by default.

2015-09-21 Thread Christophe Fergeau
Do we want to add gst 0.10 code at this point? It has been unmaintained upstream for 2.5 years http://lists.freedesktop.org/archives/gstreamer-announce/2013-March/000273.html Christophe On Thu, Aug 27, 2015 at 09:00:46PM +0200, Francois Gouget wrote: > The GStreamer video encoder supports both re

Re: [Spice-devel] [PATCH v5 03/20] server: Add a GStreamer 0.10 MJPEG video encoder and use it by default.

2015-09-21 Thread Jeremy White
Hey Christophe, Thanks for all the review. Francois and I were just off at the Wine conference, so forgive us if we're a little slow responding. On 09/21/2015 10:12 AM, Christophe Fergeau wrote: Do we want to add gst 0.10 code at this point? It has been unmaintained upstream for 2.5 years ht