/pidgin/main: b02b90cece86: gtkxfer: add 'file://' scheme to pur...
Jakub Adam
jakub.adam at ktknet.cz
Sat Jun 18 22:10:08 EDT 2016
Changeset: b02b90cece86d5155382700b25422474d48b0891
Author: Jakub Adam <jakub.adam at ktknet.cz>
Date: 2016-06-17 07:11 +0200
Branch: release-2.x.y
URL: https://hg.pidgin.im/pidgin/main/rev/b02b90cece86
Description:
gtkxfer: add 'file://' scheme to purple_notify_uri() argument
The argument should be a URI, so make sure what we pass is one.
Not doing so causes trouble when the filename contains spaces or special
characters which may get escaped:
'/tmp/some file.txt' turns into '/tmp/some%20file.txt' which for example
xdg-open treats as a different and likely non-existent file.
'file:///tmp/some%20file.txt' is still a valid file URI though.
Backport of Pidgin 3 commits d9798c40b7ec and 9ab6c9a03ae5.
diffstat:
pidgin/gtkft.c | 4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diffs (14 lines):
diff --git a/pidgin/gtkft.c b/pidgin/gtkft.c
--- a/pidgin/gtkft.c
+++ b/pidgin/gtkft.c
@@ -504,7 +504,9 @@ open_button_cb(GtkButton *button, Pidgin
}
else
{
- purple_notify_uri(NULL, filename);
+ gchar *uri = g_strdup_printf("file://%s", filename);
+ purple_notify_uri(NULL, uri);
+ g_free(uri);
return;
}
More information about the Commits
mailing list