Re: [Spice-devel] [PATCH spice-gtk] main: Handle file-xfer detailed errors

2017-05-29 Thread Christophe Fergeau
Hey, On Mon, May 29, 2017 at 12:24:46PM +, Jakub Janků wrote: > Hi, > I copied the message concerning VD_AGENT_FILE_XFER_STATUS_NO_AGENT from the > linux vdagent. What message would you suggest instead? Perhaps something > like "Agent not connected, file-xfer cancelled."? It really depends on

Re: [Spice-devel] [PATCH spice-gtk] main: Handle file-xfer detailed errors

2017-05-29 Thread Jakub Janků
Hi, I copied the message concerning VD_AGENT_FILE_XFER_STATUS_NO_AGENT from the linux vdagent. What message would you suggest instead? Perhaps something like "Agent not connected, file-xfer cancelled."? Thanks On Mon, May 29, 2017 at 12:17 PM Christophe Fergeau wrote: > On Sun, May 28, 2017 at 1

Re: [Spice-devel] [PATCH spice-gtk] main: Handle file-xfer detailed errors

2017-05-29 Thread Christophe Fergeau
On Sun, May 28, 2017 at 11:36:34PM +0200, Jakub Janků wrote: > Log appropriate error messages when we receive > VD_AGENT_FILE_XFER_STATUS_SESSION_LOCKED or > VD_AGENT_FILE_XFER_STATUS_NO_AGENT. > --- > src/channel-main.c | 10 ++ > 1 file changed, 10 insertions(+) > > diff --git a/src/cha

[Spice-devel] [PATCH spice-gtk] main: Handle file-xfer detailed errors

2017-05-28 Thread Jakub Janků
Log appropriate error messages when we receive VD_AGENT_FILE_XFER_STATUS_SESSION_LOCKED or VD_AGENT_FILE_XFER_STATUS_NO_AGENT. --- src/channel-main.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/src/channel-main.c b/src/channel-main.c index af1350c..2a53a3f 100644 --- a/src/chan