[Spice-devel] [PATCH spice-server] syntax-check: Add missing contributors names to AUTHORS

2018-04-06 Thread Frediano Ziglio
Signed-off-by: Frediano Ziglio --- AUTHORS | 1 + 1 file changed, 1 insertion(+) diff --git a/AUTHORS b/AUTHORS index 99f6c187..8874a9d4 100644 --- a/AUTHORS +++ b/AUTHORS @@ -47,6 +47,7 @@ Patches also contributed by Jürg Billeter Li Zhijian Liang Guo +Lukáš Hrázký L

Re: [Spice-devel] [PATCH spice-server] syntax-check: Add missing contributors names to AUTHORS

2018-04-06 Thread Victor Toso
On Fri, Apr 06, 2018 at 08:23:45AM +0100, Frediano Ziglio wrote: > Signed-off-by: Frediano Ziglio Maybe it is better to do it just before a release so we can add all new contributors at the same time. Up to you. Acked-by: Victor Toso > --- > AUTHORS | 1 + > 1 file changed, 1 insertion(+) >

Re: [Spice-devel] [PATCH spice-server] syntax-check: Add missing contributors names to AUTHORS

2018-04-06 Thread Frediano Ziglio
> > On Fri, Apr 06, 2018 at 08:23:45AM +0100, Frediano Ziglio wrote: > > Signed-off-by: Frediano Ziglio > > Maybe it is better to do it just before a release so we can add > all new contributors at the same time. Up to you. > > Acked-by: Victor Toso > Merged. I think a CI loose values if you

[Spice-devel] [spice-gtk v1 1/2] Revert "channel-usbredir: Fix crash on channel-up"

2018-04-06 Thread Victor Toso
From: Victor Toso This reverts commit 291f3e4419e6fb4077ae43a5e09eb1c37b9dd729 Follow up patch should address rhbz#1399838 mentioned in commit above in a different way. Major reason to revert is that the SpiceUsbDeviceManager object is kept in SpiceSession as an easy way to share it between dif

[Spice-devel] [spice-gtk v1 2/2] channel-usbredir: Fix crash on channel-up

2018-04-06 Thread Victor Toso
From: Victor Toso By adding a guard to not handle channel-up on SpiceUsbredirChannel in case struct usbredirhost wasn't initialized yet. Same guard is in place for the generic usbredir_handle_msg() function to avoid handling Server's message while Client's initialization is not done. As mentione

Re: [Spice-devel] [PATCH spice-streaming-agent] Revert "build: Use pkgconfig to detect libjpeg"

2018-04-06 Thread Christophe Fergeau
Ah I thought this was done already, but yeah, Acked-by: Christophe Fergeau On Thu, Apr 05, 2018 at 06:03:15PM +0100, Frediano Ziglio wrote: > This reverts commit 5240f212ed364d5139f30810b14884f8e2c03535. > RHEL 7 does not provide a pkg-config module for libjpeg. > > Signed-off-by: Frediano Zigl

Re: [Spice-devel] [PATCH spice-streaming-agent] Use pkg-config to find jpeg library if available

2018-04-06 Thread Christophe Fergeau
With git diff -w this comes out as: diff --git a/configure.ac b/configure.ac index 958869f..699dc3b 100644 --- a/configure.ac +++ b/configure.ac @@ -36,6 +36,7 @@ AC_SUBST([SPICE_PROTOCOL_MIN_VER]) PKG_CHECK_MODULES(X11, x11) PKG_CHECK_MODULES(XFIXES, xfixes) +PKG_CHECK_MODULES(JPEG, libjpeg, ,

Re: [Spice-devel] [RFC spice-gtk 1/1] Gstreamer: Use GstVideoOverlay if possible

2018-04-06 Thread Frediano Ziglio
> > Currently when gstreamer is used to decode a full-screen > stream sent from the server, the decoding frames are being > forced to RBGA format and pushed using appsink to be scaled > and rendered to screen. > > Today most of the gstreamer sinks supports the GstVideoOverlay > interface which al