Merge lp:~noskcaj/ubuntu/trusty/thunar/1280641 into lp:ubuntu/trusty/thunar

Proposed by Jackson Doak
Status: Merged
Merge reported by: Martin Pitt
Merged at revision: not available
Proposed branch: lp:~noskcaj/ubuntu/trusty/thunar/1280641
Merge into: lp:ubuntu/trusty/thunar
Diff against target: 111 lines (+62/-5)
3 files modified
debian/changelog (+7/-0)
debian/patches/gtk3-bookmarks.patch (+38/-5)
thunar/thunar-shortcuts-model.c (+17/-0)
To merge this branch: bzr merge lp:~noskcaj/ubuntu/trusty/thunar/1280641
Reviewer Review Type Date Requested Status
Timo Jyrinki Approve
Ubuntu branches Pending
Review via email: mp+210692@code.launchpad.net

Description of the change

Bugfix for xubuntu 14.04

To post a comment you must log in.
Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :

Tested in XFCE via ppa:timo-jyrinki/ppa, good to upload as is. Thanks!

review: Approve
Revision history for this message
Martin Pitt (pitti) wrote :

Uploaded.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2014-02-07 16:54:48 +0000
+++ debian/changelog 2014-03-12 21:04:34 +0000
@@ -1,3 +1,10 @@
1thunar (1.6.3-1ubuntu4) trusty; urgency=medium
2
3 * Update gtk3-bookmarks.patch. LP: #1280641
4 Thanks Alistair Buxton
5
6 -- Jackson Doak <noskcaj@ubuntu.com> Thu, 13 Mar 2014 07:53:06 +1100
7
1thunar (1.6.3-1ubuntu3) trusty; urgency=medium8thunar (1.6.3-1ubuntu3) trusty; urgency=medium
29
3 * Add git-xfdesktop-4.11.patch. LP: #127714910 * Add git-xfdesktop-4.11.patch. LP: #1277149
411
=== modified file 'debian/patches/gtk3-bookmarks.patch'
--- debian/patches/gtk3-bookmarks.patch 2014-02-07 16:54:48 +0000
+++ debian/patches/gtk3-bookmarks.patch 2014-03-12 21:04:34 +0000
@@ -1,12 +1,12 @@
1Author: Alistair Buxton <a.j.buxton@gmail.com>1Author: Alistair Buxton <a.j.buxton@gmail.com>
2Date: Sun, 12 Jan 2014 17:27:30 +00002Description: Use the new Gtk bookmarks location.
3Description: [PATCH] Use the new Gtk bookmarks location.
4Bug: https://bugzilla.xfce.org/show_bug.cgi?id=106273Bug: https://bugzilla.xfce.org/show_bug.cgi?id=10627
54
6---5---
7 thunar/thunar-gio-extensions.c | 2 +-6 thunar/thunar-gio-extensions.c | 2 +-
8 thunar/thunar-util.c | 10 +++++++++-7 thunar/thunar-shortcuts-model.c | 17 +++++++++++++++++
9 2 files changed, 10 insertions(+), 2 deletions(-)8 thunar/thunar-util.c | 10 +++++++++-
9 3 files changed, 27 insertions(+), 2 deletions(-)
1010
11--- a/thunar/thunar-gio-extensions.c11--- a/thunar/thunar-gio-extensions.c
12+++ b/thunar/thunar-gio-extensions.c12+++ b/thunar/thunar-gio-extensions.c
@@ -19,6 +19,39 @@
19 bookmarks = g_file_new_for_path (filename);19 bookmarks = g_file_new_for_path (filename);
20 g_free (filename);20 g_free (filename);
21 21
22--- a/thunar/thunar-shortcuts-model.c
23+++ b/thunar/thunar-shortcuts-model.c
24@@ -1364,6 +1364,7 @@ thunar_shortcuts_model_save (ThunarShort
25 gchar *uri;
26 GList *lp;
27 GError *err = NULL;
28+ GFile *parent = NULL;
29
30 _thunar_return_if_fail (THUNAR_IS_SHORTCUTS_MODEL (model));
31
32@@ -1389,6 +1390,22 @@ thunar_shortcuts_model_save (ThunarShort
33 }
34 }
35
36+ /* create folder if it does not exist */
37+ parent = g_file_get_parent (model->bookmarks_file);
38+ if (!g_file_make_directory_with_parents (parent, NULL, &err))
39+ {
40+ if (g_error_matches (err, G_IO_ERROR, G_IO_ERROR_EXISTS))
41+ {
42+ g_clear_error (&err);
43+ }
44+ else
45+ {
46+ g_warning ("Failed to create bookmarks folder: %s", err->message);
47+ g_error_free (err);
48+ }
49+ }
50+ g_clear_object (&parent);
51+
52 /* write data to the disk */
53 bookmarks_path = g_file_get_path (model->bookmarks_file);
54 if (!g_file_set_contents (bookmarks_path, contents->str, contents->len, &err))
22--- a/thunar/thunar-util.c55--- a/thunar/thunar-util.c
23+++ b/thunar/thunar-util.c56+++ b/thunar/thunar-util.c
24@@ -84,6 +84,15 @@ thunar_util_load_bookmarks (GFile57@@ -84,6 +84,15 @@ thunar_util_load_bookmarks (GFile
2558
=== modified file 'thunar/thunar-shortcuts-model.c'
--- thunar/thunar-shortcuts-model.c 2013-08-15 21:37:17 +0000
+++ thunar/thunar-shortcuts-model.c 2014-03-12 21:04:34 +0000
@@ -1364,6 +1364,7 @@
1364 gchar *uri;1364 gchar *uri;
1365 GList *lp;1365 GList *lp;
1366 GError *err = NULL;1366 GError *err = NULL;
1367 GFile *parent = NULL;
13671368
1368 _thunar_return_if_fail (THUNAR_IS_SHORTCUTS_MODEL (model));1369 _thunar_return_if_fail (THUNAR_IS_SHORTCUTS_MODEL (model));
13691370
@@ -1389,6 +1390,22 @@
1389 }1390 }
1390 }1391 }
13911392
1393 /* create folder if it does not exist */
1394 parent = g_file_get_parent (model->bookmarks_file);
1395 if (!g_file_make_directory_with_parents (parent, NULL, &err))
1396 {
1397 if (g_error_matches (err, G_IO_ERROR, G_IO_ERROR_EXISTS))
1398 {
1399 g_clear_error (&err);
1400 }
1401 else
1402 {
1403 g_warning ("Failed to create bookmarks folder: %s", err->message);
1404 g_error_free (err);
1405 }
1406 }
1407 g_clear_object (&parent);
1408
1392 /* write data to the disk */1409 /* write data to the disk */
1393 bookmarks_path = g_file_get_path (model->bookmarks_file);1410 bookmarks_path = g_file_get_path (model->bookmarks_file);
1394 if (!g_file_set_contents (bookmarks_path, contents->str, contents->len, &err))1411 if (!g_file_set_contents (bookmarks_path, contents->str, contents->len, &err))

Subscribers

People subscribed via source and target branches

to all changes: