[Spice-devel] [PATCH spice-gtk] display: mark get_window_handle as internal

2019-01-02 Thread marcandre . lureau
From: Marc-André Lureau Not strictly required, but slightly more clear and consistent with the rest of the code. Signed-off-by: Marc-André Lureau --- src/channel-display.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/channel-display.c b/src/channel-display.c index 14296d3..e5e5f56

[Spice-devel] [PATCH spice-gtk] display: remove needless SPICE_CHANNEL casts in reset_capabilities

2019-01-02 Thread marcandre . lureau
From: Marc-André Lureau The passed argument is already a SpiceChannel. Signed-off-by: Marc-André Lureau --- src/channel-display.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/src/channel-display.c b/src/channel-display.c index 7c663cb..14296d3

Re: [Spice-devel] [spice-gtk] gtk-session: do not request guest's clipboard data unnecessarily

2019-01-02 Thread Victor Toso
Hi, Thanks for taking a look! On Sun, Dec 30, 2018 at 10:23:02PM +0100, Jakub Janku wrote: > Hi, > > On Wed, Dec 19, 2018 at 3:30 PM Victor Toso wrote: > > > > From: Victor Toso > > > > If SpiceGtkSession is holding the keyboard, that's huge indication > > that we should not be requesting

Re: [Spice-devel] [PATCH spice-gtk] session: initialize USB device manager during init

2019-01-02 Thread Victor Toso
Hi, Have you tried with migration? On Wed, Jan 02, 2019 at 03:30:15PM +0400, marcandre.lur...@redhat.com wrote: > From: Marc-André Lureau > > Simple command line tools simply connect all channels, however > usbredir channels require that the manager was previously > initialized. Currently,

Re: [Spice-devel] [PATCH] use g_queue_free_full to free the queue

2019-01-02 Thread Victor Toso
Hi, On Wed, Jan 02, 2019 at 05:41:52PM +0200, Snir Sheriber wrote: > Signed-off-by: Snir Sheriber Acked-by: Victor Toso > --- > src/channel-display-gst.c | 6 +- > 1 file changed, 1 insertion(+), 5 deletions(-) > > diff --git a/src/channel-display-gst.c b/src/channel-display-gst.c >

Re: [Spice-devel] Always get Invalid password while trying to connect to spice server

2019-01-02 Thread Christophe Fergeau
On Wed, Jan 02, 2019 at 09:39:46AM -0500, Frediano Ziglio wrote: > > > > Hey, > > > > On Wed, Dec 26, 2018 at 01:03:38PM +0100, Niccolò Belli wrote: > > > I just noticed that the problem looks limited to special characters. > > > For example if I set "Password" as password it works, while if I

Re: [Spice-devel] [PATCH] use g_queue_free_full to free the queue

2019-01-02 Thread Snir Sheriber
[spice-gtk] On 1/2/19 5:41 PM, Snir Sheriber wrote: Signed-off-by: Snir Sheriber --- src/channel-display-gst.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/src/channel-display-gst.c b/src/channel-display-gst.c index 2c07f35..bd2a9ff 100644 ---

[Spice-devel] [PATCH] use g_queue_free_full to free the queue

2019-01-02 Thread Snir Sheriber
Signed-off-by: Snir Sheriber --- src/channel-display-gst.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/src/channel-display-gst.c b/src/channel-display-gst.c index 2c07f35..bd2a9ff 100644 --- a/src/channel-display-gst.c +++ b/src/channel-display-gst.c @@ -548,11

[Spice-devel] [PATCH spice-server 3/33 v2] build: Detect Windows build and change some definitions

2019-01-02 Thread Frediano Ziglio
Windows needs some specific setting to use network. Signed-off-by: Frediano Ziglio --- configure.ac | 20 +++- 1 file changed, 19 insertions(+), 1 deletion(-) Changes since v1: - all changes in configure.ac diff --git a/configure.ac b/configure.ac index 96e4e66c..afe0017d

[Spice-devel] [PATCH spice-server 1/33 v2] Use proper format strings for spice_log

2019-01-02 Thread Frediano Ziglio
Formatting string should be compatible with GLib. GLib uses formatting types compatible with GNU. For Linux this is not an issue as both systems (like a printf) and GLib one uses the same formatting type. However on Windows they differs potentially causing issues. This is also make worse as GLib

Re: [Spice-devel] Always get Invalid password while trying to connect to spice server

2019-01-02 Thread Frediano Ziglio
> > Hey, > > On Wed, Dec 26, 2018 at 01:03:38PM +0100, Niccolò Belli wrote: > > I just noticed that the problem looks limited to special characters. > > For example if I set "Password" as password it works, while if I set > > "Password%%" it doesn't. It's weird because both my server and my

Re: [Spice-devel] Always get Invalid password while trying to connect to spice server

2019-01-02 Thread Christophe Fergeau
On Wed, Jan 02, 2019 at 03:07:06PM +0100, Christophe Fergeau wrote: > Hey, > > On Wed, Dec 26, 2018 at 01:03:38PM +0100, Niccolò Belli wrote: > > I just noticed that the problem looks limited to special characters. > > For example if I set "Password" as password it works, while if I set > >

Re: [Spice-devel] Does SPICE have a solution on video redirection(or passthrough)?

2019-01-02 Thread Christophe Fergeau
Hey, On Tue, Dec 25, 2018 at 06:20:03PM +0800, 陈炤 wrote: > Hi, > > > Does SPICE have a solution to capture video stream, and pass-through it to > the client? > I want this feature for these reasons: > 1. Currently SPICE render video on server side, which would consume CPU of > the server. If

Re: [Spice-devel] Always get Invalid password while trying to connect to spice server

2019-01-02 Thread Christophe Fergeau
Hey, On Wed, Dec 26, 2018 at 01:03:38PM +0100, Niccolò Belli wrote: > I just noticed that the problem looks limited to special characters. > For example if I set "Password" as password it works, while if I set > "Password%%" it doesn't. It's weird because both my server and my desktop > client

Re: [Spice-devel] [PATCH spice-common] log: Use proper format attribute for spice_log

2019-01-02 Thread Christophe Fergeau
Fine with me, Acked-by: Christophe Fergeau On Wed, Jan 02, 2019 at 01:02:26PM +, Frediano Ziglio wrote: > SPICE_ATTR_PRINTF uses __printf__ format attribute. > spice_log internally uses g_string_append_vprintf which uses > G_GNUC_PRINTF attribute. > G_GNUC_PRINTF can be __printf__ or

Re: [Spice-devel] [PATCH spice-server 03/33] build: Detect Windows build and change some definitions

2019-01-02 Thread Christophe Fergeau
On Wed, Jan 02, 2019 at 08:07:22AM -0500, Frediano Ziglio wrote: > > > > On Sat, Dec 22, 2018 at 09:56:38AM -0500, Frediano Ziglio wrote: > > > > > > > Hi > > > > > > > > On Fri, Dec 21, 2018 at 4:03 PM Frediano Ziglio > > > > wrote: > > > > > > > > > > Windows needs some specific setting to

Re: [Spice-devel] [PATCH spice-gtk 3/3] meson: don't link with all dependencies

2019-01-02 Thread Frediano Ziglio
> > Hi > > On Wed, Jan 2, 2019 at 4:01 PM Frediano Ziglio wrote: > > > > > > > > From: Marc-André Lureau > > > > > > Split lib dependencies in 3 groups: glib, gtk, and acl. > > > > > > Fixes: > > > https://gitlab.freedesktop.org/spice/spice-gtk/issues/77 > > > > > > Signed-off-by: Marc-André

Re: [Spice-devel] [PATCH spice-gtk 3/3] meson: don't link with all dependencies

2019-01-02 Thread Marc-André Lureau
Hi On Wed, Jan 2, 2019 at 4:01 PM Frediano Ziglio wrote: > > > > > From: Marc-André Lureau > > > > Split lib dependencies in 3 groups: glib, gtk, and acl. > > > > Fixes: > > https://gitlab.freedesktop.org/spice/spice-gtk/issues/77 > > > > Signed-off-by: Marc-André Lureau > > --- > >

Re: [Spice-devel] [PATCH spice-server 03/33] build: Detect Windows build and change some definitions

2019-01-02 Thread Frediano Ziglio
> > On Sat, Dec 22, 2018 at 09:56:38AM -0500, Frediano Ziglio wrote: > > > > > Hi > > > > > > On Fri, Dec 21, 2018 at 4:03 PM Frediano Ziglio > > > wrote: > > > > > > > > Windows needs some specific setting to use network. > > > > > > > > Signed-off-by: Frediano Ziglio > > > > > > with neat

[Spice-devel] [PATCH spice-common] log: Use proper format attribute for spice_log

2019-01-02 Thread Frediano Ziglio
SPICE_ATTR_PRINTF uses __printf__ format attribute. spice_log internally uses g_string_append_vprintf which uses G_GNUC_PRINTF attribute. G_GNUC_PRINTF can be __printf__ or gnu_printf format which in some systems (currently Windows) can be different. GLib 2.58 changed G_GNUC_PRINTF on Windows from

Re: [Spice-devel] [PATCH spice-server 03/33] build: Detect Windows build and change some definitions

2019-01-02 Thread Christophe Fergeau
On Sat, Dec 22, 2018 at 09:56:38AM -0500, Frediano Ziglio wrote: > > > Hi > > > > On Fri, Dec 21, 2018 at 4:03 PM Frediano Ziglio wrote: > > > > > > Windows needs some specific setting to use network. > > > > > > Signed-off-by: Frediano Ziglio > > > > with neat below > > Reviewed-by:

Re: [Spice-devel] [PATCH spice-gtk 2/3] usb-device-widget: replace deprecated Gtk+ API usage

2019-01-02 Thread Frediano Ziglio
> > On Sat, Dec 22, 2018 at 06:53:25PM +0400, marcandre.lur...@redhat.com wrote: > > From: Marc-André Lureau > > > > Use widget properties instead, as recommended in the documentation. > > The UI doesn't seem to have changed after these modifications. > > I did not run with these changes, but

Re: [Spice-devel] [PATCH spice-server 01/33] Use PRIxPTR constant for string formatting

2019-01-02 Thread Christophe Fergeau
On Sat, Dec 22, 2018 at 10:02:36AM -0500, Frediano Ziglio wrote: > > > > On Fri, Dec 21, 2018 at 4:03 PM Frediano Ziglio wrote: > > > > > > These constants are meant to be used in format string for size_t > > > types. Use them for portability. > > > > PRI*PTR is for [u]intptr_t. > > > > There

Re: [Spice-devel] [PATCH spice-gtk 1/3] meson: use 0.49 support for pie

2019-01-02 Thread Christophe Fergeau
On Wed, Jan 02, 2019 at 06:30:50AM -0500, Frediano Ziglio wrote: > > On Sat, Dec 22, 2018 at 06:53:24PM +0400, marcandre.lur...@redhat.com wrote: > > > From: Marc-André Lureau > > > > > > Fixes > > > WARNING: Use the 'pie' kwarg instead of passing '-fpie' manually to > > >

Re: [Spice-devel] [PATCH spice-gtk 3/3] meson: don't link with all dependencies

2019-01-02 Thread Frediano Ziglio
> > From: Marc-André Lureau > > Split lib dependencies in 3 groups: glib, gtk, and acl. > > Fixes: > https://gitlab.freedesktop.org/spice/spice-gtk/issues/77 > > Signed-off-by: Marc-André Lureau > --- > meson.build | 46 ++ > src/meson.build |

[Spice-devel] [PATCH spice-gtk] session: initialize USB device manager during init

2019-01-02 Thread marcandre . lureau
From: Marc-André Lureau Simple command line tools simply connect all channels, however usbredir channels require that the manager was previously initialized. Currently, running spicy-stats on a VM with usbredir channels prints: (lt-spicy-stats:25224): GSpice-CRITICAL **: 14:30:54.724:

Re: [Spice-devel] [PATCH spice-gtk 1/3] meson: use 0.49 support for pie

2019-01-02 Thread Frediano Ziglio
> On Sat, Dec 22, 2018 at 06:53:24PM +0400, marcandre.lur...@redhat.com wrote: > > From: Marc-André Lureau > > > > Fixes > > WARNING: Use the 'pie' kwarg instead of passing '-fpie' manually to > > 'spice-client-glib-usb-acl-helper' > > > > Signed-off-by: Marc-André Lureau > > --- > >

[Spice-devel] [PATCH spice-server] Reuse SPICE_UPCAST instead of SPICE_CONTAINEROF where possible

2019-01-02 Thread Frediano Ziglio
Signed-off-by: Frediano Ziglio --- server/inputs-channel.c | 8 server/reds.c | 36 +++- server/reds.h | 2 +- 3 files changed, 24 insertions(+), 22 deletions(-) diff --git a/server/inputs-channel.c b/server/inputs-channel.c index

Re: [Spice-devel] [PATCH spice-gtk 1/3] meson: use 0.49 support for pie

2019-01-02 Thread Christophe Fergeau
On Sat, Dec 22, 2018 at 06:53:24PM +0400, marcandre.lur...@redhat.com wrote: > From: Marc-André Lureau > > Fixes > WARNING: Use the 'pie' kwarg instead of passing '-fpie' manually to > 'spice-client-glib-usb-acl-helper' > > Signed-off-by: Marc-André Lureau > --- > meson.build | 2 +- >

Re: [Spice-devel] [PATCH spice-gtk 2/3] usb-device-widget: replace deprecated Gtk+ API usage

2019-01-02 Thread Christophe Fergeau
On Sat, Dec 22, 2018 at 06:53:25PM +0400, marcandre.lur...@redhat.com wrote: > From: Marc-André Lureau > > Use widget properties instead, as recommended in the documentation. > The UI doesn't seem to have changed after these modifications. I did not run with these changes, but patch looks good

Re: [Spice-devel] [PATCH spice-gtk 1/3] meson: use 0.49 support for pie

2019-01-02 Thread Frediano Ziglio
> > Hi > > On Sat, Dec 22, 2018 at 7:06 PM Frediano Ziglio wrote: > > > > > > > > From: Marc-André Lureau > > > > > > Fixes > > > WARNING: Use the 'pie' kwarg instead of passing '-fpie' manually to > > > 'spice-client-glib-usb-acl-helper' > > > > > > Signed-off-by: Marc-André Lureau > > > >

Re: [Spice-devel] [PATCH spice-protocol] error_codes: Remove obsolete header file

2019-01-02 Thread Snir Sheriber
Hi, On 12/21/18 4:00 PM, Frediano Ziglio wrote: This file was used by the old client to hold return error codes. Signed-off-by: Frediano Ziglio --- spice/Makefile.am | 1 - spice/error_codes.h | 51 - spice/meson.build | 1 - 3 files

Re: [Spice-devel] [PATCH spice-gtk 2/3] usb-device-widget: replace deprecated Gtk+ API usage

2019-01-02 Thread Marc-André Lureau
ping On Sat, Dec 22, 2018 at 6:53 PM wrote: > > From: Marc-André Lureau > > Use widget properties instead, as recommended in the documentation. > The UI doesn't seem to have changed after these modifications. > > Signed-off-by: Marc-André Lureau > --- > src/usb-device-widget.c | 23

Re: [Spice-devel] [PATCH spice-gtk 3/3] meson: don't link with all dependencies

2019-01-02 Thread Marc-André Lureau
ping On Sat, Dec 22, 2018 at 6:53 PM wrote: > > From: Marc-André Lureau > > Split lib dependencies in 3 groups: glib, gtk, and acl. > > Fixes: > https://gitlab.freedesktop.org/spice/spice-gtk/issues/77 > > Signed-off-by: Marc-André Lureau > --- > meson.build | 46

Re: [Spice-devel] [PATCH spice-gtk 1/3] meson: use 0.49 support for pie

2019-01-02 Thread Marc-André Lureau
Hi On Sat, Dec 22, 2018 at 7:06 PM Frediano Ziglio wrote: > > > > > From: Marc-André Lureau > > > > Fixes > > WARNING: Use the 'pie' kwarg instead of passing '-fpie' manually to > > 'spice-client-glib-usb-acl-helper' > > > > Signed-off-by: Marc-André Lureau > > Patch looks good. > Technically