Re: [Spice-devel] [RFC spice-vdagent 03/18] vdagentd: use GMainLoop

2018-09-03 Thread Victor Toso
Hi, On Mon, Sep 03, 2018 at 06:38:58PM +0200, Jakub Janku wrote: > Hi Victor, > > On Tue, Aug 28, 2018 at 9:38 AM Victor Toso wrote: > > > > Hi, > > > > On Tue, Aug 14, 2018 at 08:53:37PM +0200, Jakub Janků wrote: > > > This is purely a preparatory patch as it renders > > > the vdagentd non-func

Re: [Spice-devel] [RFC spice-vdagent 00/18] GLib integration

2018-09-03 Thread Victor Toso
Hi, On Mon, Sep 03, 2018 at 09:03:38PM +0200, Jakub Janku wrote: > I do agree that turning VDAgentConnection into a GObject is > probably a good idea, however, I do not see how it could help > us in this situation. Am I missing something? I'll try to > explain the problem again: > > Let's consid

Re: [Spice-devel] [RFC spice-vdagent 02/18] vport: add by_user param to vdagent_virtio_port_disconnect_callback

2018-09-03 Thread Victor Toso
Hi, On Mon, Sep 03, 2018 at 06:20:44PM +0200, Jakub Janku wrote: > Hi, > > On Tue, Aug 28, 2018 at 7:52 AM Victor Toso wrote: > > > > Hi, > > > > Took me a while to get the rationale behind this so some more > > info in the commit log might help. > > OK, will do, sorry. No worries > > > > On

Re: [Spice-devel] [RFC spice-vdagent 01/18] vdagentd: parse argv using GLib

2018-09-03 Thread Victor Toso
Hi, On Mon, Sep 03, 2018 at 06:06:01PM +0200, Jakub Janku wrote: > Hi, > > > I think that after "vdagentd: use GMainLoop" patch, we should > > also include glib's command line options. > > Not sure what you are referring to here. > GLib uses environment variables instead of command line > option

Re: [Spice-devel] [RFC spice-vdagent 00/18] GLib integration

2018-09-03 Thread Jakub Janku
On Tue, Aug 28, 2018 at 9:33 AM Victor Toso wrote: > > Hi, > > First of all, thanks for your work on this :) > > I'm still looking into the glib-integration related patches so I > might still reply once more afterwards. To keep things easy to > review, I take we could split this series in three di

[Spice-devel] [PATCH spice-gtk 1/2] meson: Remove unecessary spice_gtk_link_args variable

2018-09-03 Thread Eduardo Lima (Etrunko)
Signed-off-by: Eduardo Lima (Etrunko) --- doc/reference/meson.build | 1 - meson.build | 1 - src/meson.build | 11 +++ 3 files changed, 3 insertions(+), 10 deletions(-) diff --git a/doc/reference/meson.build b/doc/reference/meson.build index 7ac703f..2ec333b 10

[Spice-devel] [PATCH spice-gtk 2/2] meson: Drop calls to declare_dependency() where appropriate

2018-09-03 Thread Eduardo Lima (Etrunko)
No need to use it when we can actually specify the parameters in the actual gnome.generate_gir() function calls. We still keep it in the case of gnome.gtkdoc(), as there is no way to specify the libraries to link with. Signed-off-by: Eduardo Lima (Etrunko) --- doc/reference/meson.build | 4 ++--

Re: [Spice-devel] [RFC spice-vdagent 05/18] add VDAgentConnection

2018-09-03 Thread Jakub Janku
Hey, On Tue, Aug 28, 2018 at 10:04 AM Victor Toso wrote: > > Hi, > > On Tue, Aug 14, 2018 at 08:53:39PM +0200, Jakub Janků wrote: > > Add a set of helper functions built around GIO that can be used to > > easily write messages to and read from the given FD. > > > > Since VDAgentConnection uses GI

Re: [Spice-devel] [RFC spice-vdagent 03/18] vdagentd: use GMainLoop

2018-09-03 Thread Jakub Janku
Hi Victor, On Tue, Aug 28, 2018 at 9:38 AM Victor Toso wrote: > > Hi, > > On Tue, Aug 14, 2018 at 08:53:37PM +0200, Jakub Janků wrote: > > This is purely a preparatory patch as it renders > > the vdagentd non-functional. > > I would rather not break it unless it really helps a lot. It was > possi

[Spice-devel] [PATCH spice-gtk] build: Make usb-acl-helper test work under Meson

2018-09-03 Thread Frediano Ziglio
Meson waits for pipes (stdout and stderr) to be not readable but our mocking helper does not close these handles causing Meson to give some problems. Signed-off-by: Frediano Ziglio --- tests/usb-acl-helper.c | 8 1 file changed, 8 insertions(+) diff --git a/tests/usb-acl-helper.c b/tes

Re: [Spice-devel] [RFC spice-vdagent 02/18] vport: add by_user param to vdagent_virtio_port_disconnect_callback

2018-09-03 Thread Jakub Janku
Hi, On Tue, Aug 28, 2018 at 7:52 AM Victor Toso wrote: > > Hi, > > Took me a while to get the rationale behind this so some more > info in the commit log might help. OK, will do, sorry. > > On Tue, Aug 14, 2018 at 08:53:36PM +0200, Jakub Janků wrote: > > If the virtio port is destroyed explicitl

Re: [Spice-devel] [RFC spice-vdagent 01/18] vdagentd: parse argv using GLib

2018-09-03 Thread Jakub Janku
Hi, On Tue, Aug 28, 2018 at 7:16 AM Victor Toso wrote: > > Hi, > > Just for reference, the output of spice-vdagentd -h before was: > > | $ spice-vdagentd -h > | Usage: spice-vdagentd [OPTIONS] > | > | Spice guest agent daemon, version 0.17.0. > | > | Options: > | -h print thi

Re: [Spice-devel] [PATCH spice-gtk] meson: Fix usage of add_project_arguments()

2018-09-03 Thread Frediano Ziglio
> > This function does accept a list of arguments, so there is no need to > iterate over it. > > Signed-off-by: Eduardo Lima (Etrunko) Acked-by: Frediano Ziglio But I'd like to see CI fixes merged before this one. I cannot understand why "meson test" fails, Meson fails with a weird Python sta

Re: [Spice-devel] [PATCH spice-gtk 2/2] build: Fix VAPI include directory

2018-09-03 Thread Eduardo Lima (Etrunko)
On 09/03/2018 09:46 AM, Frediano Ziglio wrote: > If gir files are not installed generate_vapi is not able to > find generated gir files, so look also in src generated files. > > Signed-off-by: Frediano Ziglio > --- > vapi/meson.build | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/vapi

Re: [Spice-devel] [PATCH spice-gtk 1/2] build: Fix gir dependencies

2018-09-03 Thread Eduardo Lima (Etrunko)
On 09/03/2018 09:46 AM, Frediano Ziglio wrote: > File generated should be depends on currently files built, not from > installed files, so use target objects for spice-gtk dependency > on spice-lib. > > Signed-off-by: Frediano Ziglio > --- > src/meson.build | 2 +- > 1 file changed, 1 insertion(

Re: [Spice-devel] [PATCH spice-gtk 2/2] build: Fix VAPI include directory

2018-09-03 Thread Eduardo Lima (Etrunko)
On 09/03/2018 10:30 AM, Eduardo Lima (Etrunko) wrote: > On 09/03/2018 09:46 AM, Frediano Ziglio wrote: >> If gir files are not installed generate_vapi is not able to >> find generated gir files, so look also in src generated files. >> >> Signed-off-by: Frediano Ziglio >> --- >> vapi/meson.build |

Re: [Spice-devel] [PATCH spice-gtk 2/2] build: Fix VAPI include directory

2018-09-03 Thread Eduardo Lima (Etrunko)
On 09/03/2018 09:46 AM, Frediano Ziglio wrote: > If gir files are not installed generate_vapi is not able to > find generated gir files, so look also in src generated files. > > Signed-off-by: Frediano Ziglio > --- > vapi/meson.build | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/vapi

[Spice-devel] [PATCH spice-gtk 1/2] build: Fix gir dependencies

2018-09-03 Thread Frediano Ziglio
File generated should be depends on currently files built, not from installed files, so use target objects for spice-gtk dependency on spice-lib. Signed-off-by: Frediano Ziglio --- src/meson.build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/meson.build b/src/meson.bui

[Spice-devel] [PATCH spice-gtk 2/2] build: Fix VAPI include directory

2018-09-03 Thread Frediano Ziglio
If gir files are not installed generate_vapi is not able to find generated gir files, so look also in src generated files. Signed-off-by: Frediano Ziglio --- vapi/meson.build | 1 + 1 file changed, 1 insertion(+) diff --git a/vapi/meson.build b/vapi/meson.build index 4737715..de9adb9 100644 ---

Re: [Spice-devel] [PATCH spice-common v2] meson: Make use of compiler.get_supported_arguments()

2018-09-03 Thread Frediano Ziglio
> > This avoids adding an extra build flag which may not be supported by the > compiler. > > Signed-off-by: Eduardo Lima (Etrunko) Acked-by: Frediano Ziglio Frediano > --- > meson.build | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/meson.build b/meson.build

Re: [Spice-devel] [PATCH spice-server] meson: Fix usage of add_project_arguments()

2018-09-03 Thread Frediano Ziglio
> > This function does accept a list of arguments, so there is no need to > iterate over it. > > Signed-off-by: Eduardo Lima (Etrunko) Acked-by: Frediano Ziglio --- > meson.build | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/meson.build b/meson.build > index 2