Hi, On Tue, May 30, 2017 at 05:56:56PM +0200, Pavel Grunt wrote: > On Wed, 2017-05-10 at 22:35 +0200, Jakub Janků wrote: > > Agent can send FileXferStatusMessage with result > > VD_AGENT_FILE_XFER_STATUS_NOT_ENOUGH_SPACE together with info about > > amount of available free space when the file to transfer is too > > large > > and the client announces support of this result by sending > > VD_AGENT_CAP_FILE_XFER_DETAILED_ERRORS > > Acked-by: Pavel Grunt <pgrunt@xxxxxxxxxx> > > --- > > src/channel-main.c | 12 ++++++++++++ > > 1 file changed, 12 insertions(+) > > > > diff --git a/src/channel-main.c b/src/channel-main.c > > index bbc5905..af1350c 100644 > > --- a/src/channel-main.c > > +++ b/src/channel-main.c > > @@ -1321,6 +1321,7 @@ static void > > agent_announce_caps(SpiceMainChannel *channel) > > VD_AGENT_SET_CAPABILITY(caps->caps, > > VD_AGENT_CAP_CLIPBOARD_BY_DEMAND); > > VD_AGENT_SET_CAPABILITY(caps->caps, > > VD_AGENT_CAP_CLIPBOARD_SELECTION); > > VD_AGENT_SET_CAPABILITY(caps->caps, > > VD_AGENT_CAP_MONITORS_CONFIG_POSITION); > > + VD_AGENT_SET_CAPABILITY(caps->caps, > > VD_AGENT_CAP_FILE_XFER_DETAILED_ERRORS); > > > > agent_msg_queue(channel, VD_AGENT_ANNOUNCE_CAPABILITIES, size, > > caps); > > g_free(caps); > > @@ -1870,6 +1871,17 @@ static void > > main_agent_handle_xfer_status(SpiceMainChannel *channel, > > error = g_error_new_literal(SPICE_CLIENT_ERROR, > > SPICE_CLIENT_ERROR_FAILED, > > _("The spice agent reported an > > error during the file transfer")); > > break; > > + case VD_AGENT_FILE_XFER_STATUS_NOT_ENOUGH_SPACE: { > > + uint64_t *free_space = (uint64_t *)(msg->data); > > + gchar *free_space_str = g_format_size(*free_space); > > + gchar *file_size_str = > > g_format_size(spice_file_transfer_task_get_total_bytes(xfer_task)); > > + error = g_error_new(SPICE_CLIENT_ERROR, > > SPICE_CLIENT_ERROR_FAILED, I think this string should be internationalized with _("") > > + "File transfer failed due to lack of > > free space on remote machine " > > + "(%s free, %s to transfer)", > > free_space_str, file_size_str); > > + g_free(free_space_str); > > + g_free(file_size_str); > > + break; > > + } > > case VD_AGENT_FILE_XFER_STATUS_SUCCESS: > > break; > > default: Besides that, looks good! > _______________________________________________ > Spice-devel mailing list > Spice-devel@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/spice-devel
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel