[Spice-devel] [PATCH spice-gtk v3] main: Handle lack of free space for file transfer
Victor Toso
victortoso at redhat.com
Wed May 31 11:11:03 UTC 2017
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 at redhat.com>
> > ---
> > 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 at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/spice-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/spice-devel/attachments/20170531/6c4d1cca/attachment-0001.sig>
More information about the Spice-devel
mailing list