Re: [Spice-devel] [client v2 01/10] gdk: Temporarily ignore clipboard_get()'s deprecation warnings

2016-10-21 Thread Jonathon Jongsma
On Fri, 2016-10-21 at 15:41 +0200, Francois Gouget wrote: > Signed-off-by: Francois Gouget > --- > > Of course a better solution would be to figure out whether this is > still  > needed, whether this executes in the main thread or not, etc. > >  src/spice-gtk-session.c | 2 ++ >  1 file changed,

Re: [Spice-devel] [client v2 01/10] gtk: Ignore GLib's too-new warnings where we explicitly check its version

2016-10-21 Thread Jonathon Jongsma
On Fri, 2016-10-21 at 15:40 +0200, Francois Gouget wrote: > Signed-off-by: Francois Gouget > --- > > I removed the extraneous hunk and moved it to its own patch. > I'll repost what remains of the whole series after the acked patches  > have been pushed. OK Acked-by: Jonathon Jongsma > >  

Re: [Spice-devel] [client v2 01/10] gtk: Ignore GLib's too-new warnings where we explicitly check its version

2016-10-21 Thread Victor Toso
Hi, On Fri, Oct 21, 2016 at 03:40:07PM +0200, Francois Gouget wrote: > Signed-off-by: Francois Gouget > --- > > I removed the extraneous hunk and moved it to its own patch. > I'll repost what remains of the whole series after the acked patches > have been pushed. They were pushed now. > > sr

Re: [Spice-devel] [client v2 01/10] gtk: Ignore GLib's too-new warnings where we explicitly check its version

2016-10-21 Thread Francois Gouget
Signed-off-by: Francois Gouget --- I removed the extraneous hunk and moved it to its own patch. I'll repost what remains of the whole series after the acked patches have been pushed. src/spice-gtk-session.c | 3 +++ src/spice-widget-egl.c | 3 +++ src/spice-widget.c | 6 ++ 3 files c

Re: [Spice-devel] [client v2 01/10] gdk: Temporarily ignore clipboard_get()'s deprecation warnings

2016-10-21 Thread Francois Gouget
Signed-off-by: Francois Gouget --- Of course a better solution would be to figure out whether this is still needed, whether this executes in the main thread or not, etc. src/spice-gtk-session.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/spice-gtk-session.c b/src/spice-gtk-sessio

[Spice-devel] [PATCH spice-server v2 4/6] Add an helper to test VideoEncoder

2016-10-21 Thread Frediano Ziglio
Add an utility to make possible to check various features of VideoEncoder. 2 GStreamer plugins are used in a chain like this: (1) input pipeline -> (2) video encoder -> (3) output pipeline While converting output from (1) is compared with output of (3) making sure the streaming is working correct

[Spice-devel] [PATCH spice-server v2] Compatibility for GStreamer 0.10 for test utility

2016-10-21 Thread Frediano Ziglio
Signed-off-by: Frediano Ziglio --- server/tests/gst-test.c | 55 + 1 file changed, 51 insertions(+), 4 deletions(-) Updated. Small rebase issue. diff --git a/server/tests/gst-test.c b/server/tests/gst-test.c index 0a68d7d..3277e04 100644 --- a/ser

[Spice-devel] [PATCH spice-server v2 5/6] Compatibility for GStreamer 0.10 for test utility

2016-10-21 Thread Frediano Ziglio
Signed-off-by: Frediano Ziglio --- server/tests/gst-test.c | 55 + 1 file changed, 51 insertions(+), 4 deletions(-) diff --git a/server/tests/gst-test.c b/server/tests/gst-test.c index 0a68d7d..38d0f1d 100644 --- a/server/tests/gst-test.c +++ b/ser

[Spice-devel] [PATCH spice-server v2 2/6] Handle top down bitmaps dumping

2016-10-21 Thread Frediano Ziglio
The top down flag can be specified using negative heights. Signed-off-by: Frediano Ziglio --- server/spice-bitmap-utils.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/server/spice-bitmap-utils.c b/server/spice-bitmap-utils.c index 72a9285..439f05d 100644 --- a/server/spice

[Spice-devel] [PATCH spice-server v2 1/6] gstreamer: Do not warn for tested formats

2016-10-21 Thread Frediano Ziglio
Signed-off-by: Frediano Ziglio --- server/gstreamer-encoder.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/server/gstreamer-encoder.c b/server/gstreamer-encoder.c index d575c67..53dfb98 100644 --- a/server/gstreamer-encoder.c +++ b/server/gstreamer-encoder.c @@ -757,7 +7

[Spice-devel] [PATCH spice-server v2 6/6] RFC EXAMPLE Some automatic check on video encoders

2016-10-21 Thread Frediano Ziglio
Signed-off-by: Frediano Ziglio --- server/tests/Makefile.am| 1 + server/tests/video-encoders | 21 + 2 files changed, 22 insertions(+) create mode 100755 server/tests/video-encoders diff --git a/server/tests/Makefile.am b/server/tests/Makefile.am index c799779..ef0e335

[Spice-devel] [PATCH spice-server v2 3/6] Simplify gstreamer 0.10 compatibility

2016-10-21 Thread Frediano Ziglio
Signed-off-by: Frediano Ziglio --- server/gstreamer-encoder.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/server/gstreamer-encoder.c b/server/gstreamer-encoder.c index 53dfb98..d8437e4 100644 --- a/server/gstreamer-encoder.c +++ b/server/gstreamer-encoder.c @@ -967,11

[Spice-devel] [PATCH spice-server v2 0/6] VideoEncoder testing and related

2016-10-21 Thread Frediano Ziglio
These patches attempt to provide some additional testing to VideoEncoder. Mainly "Add an helper to test VideoEncoder" add an utility that helps doing some test. The "RFC EXAMPLE Some automatic check on video encoders" provide a check script but require an additional video file. Other patches are mi