Re: [Spice-devel] [PATCH spice-gtk 1/4] tests: add spice+unix:// URI checks

2018-02-15 Thread i iordanov
If one can infer the usage of unix sockets from other parameters passed anyway, then there may be no point having it in the first place. For the VNC URI, we could have gone with vnc+ssh://, or vnc+ssl:// (for stunnel), but there were ssh-related and stunnel parameters we could infer from so it beca

Re: [Spice-devel] [PATCH spice-gtk 1/4] tests: add spice+unix:// URI checks

2018-02-14 Thread Christophe de Dinechin
> On 13 Feb 2018, at 17:36, Marc-Andre Lureau wrote: > > I was not really fond of passing options via URI, but apparently this > is a standard practice. So is passing port number after a colon. Christophe ___ Spice-devel mailing list Spice-devel@li

Re: [Spice-devel] [PATCH spice-gtk 1/4] tests: add spice+unix:// URI checks

2018-02-13 Thread Frediano Ziglio
> > Hi > > On Tue, Feb 13, 2018 at 10:08 AM, Frediano Ziglio wrote: > >> > >> Hi > >> > >> On Thu, Feb 8, 2018 at 2:18 PM, Frediano Ziglio > >> wrote: > >> >> > >> >> From: Marc-André Lureau > >> >> > >> >> For some reason, the URIs test didn't include spice+unix:// checks, > >> >> probably be

Re: [Spice-devel] [PATCH spice-gtk 1/4] tests: add spice+unix:// URI checks

2018-02-13 Thread Marc-Andre Lureau
Hi On Tue, Feb 13, 2018 at 10:08 AM, Frediano Ziglio wrote: >> >> Hi >> >> On Thu, Feb 8, 2018 at 2:18 PM, Frediano Ziglio wrote: >> >> >> >> From: Marc-André Lureau >> >> >> >> For some reason, the URIs test didn't include spice+unix:// checks, >> >> probably because they came about the same t

Re: [Spice-devel] [PATCH spice-gtk 1/4] tests: add spice+unix:// URI checks

2018-02-13 Thread Frediano Ziglio
> > Hi > > On Thu, Feb 8, 2018 at 2:18 PM, Frediano Ziglio wrote: > >> > >> From: Marc-André Lureau > >> > >> For some reason, the URIs test didn't include spice+unix:// checks, > >> probably because they came about the same time. > >> > >> Signed-off-by: Marc-André Lureau > >> --- > >> tests

Re: [Spice-devel] [PATCH spice-gtk 1/4] tests: add spice+unix:// URI checks

2018-02-12 Thread Marc-André Lureau
Hi On Thu, Feb 8, 2018 at 2:18 PM, Frediano Ziglio wrote: >> >> From: Marc-André Lureau >> >> For some reason, the URIs test didn't include spice+unix:// checks, >> probably because they came about the same time. >> >> Signed-off-by: Marc-André Lureau >> --- >> tests/session.c | 28 +++

Re: [Spice-devel] [PATCH spice-gtk 1/4] tests: add spice+unix:// URI checks

2018-02-08 Thread Frediano Ziglio
> > From: Marc-André Lureau > > For some reason, the URIs test didn't include spice+unix:// checks, > probably because they came about the same time. > > Signed-off-by: Marc-André Lureau > --- > tests/session.c | 28 +--- > 1 file changed, 25 insertions(+), 3 deletions

[Spice-devel] [PATCH spice-gtk 1/4] tests: add spice+unix:// URI checks

2018-02-08 Thread marcandre . lureau
From: Marc-André Lureau For some reason, the URIs test didn't include spice+unix:// checks, probably because they came about the same time. Signed-off-by: Marc-André Lureau --- tests/session.c | 28 +--- 1 file changed, 25 insertions(+), 3 deletions(-) diff --git a/tes