Hello community,

here is the log from the commit of package deja-dup for openSUSE:Factory 
checked in at 2017-09-17 22:37:34
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/deja-dup (Old)
 and      /work/SRC/openSUSE:Factory/.deja-dup.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "deja-dup"

Sun Sep 17 22:37:34 2017 rev:45 rq:526428 version:36.0

Changes:
--------
--- /work/SRC/openSUSE:Factory/deja-dup/deja-dup.changes        2017-09-14 
21:19:22.327474470 +0200
+++ /work/SRC/openSUSE:Factory/.deja-dup.new/deja-dup.changes   2017-09-17 
22:37:53.304369489 +0200
@@ -1,0 +2,6 @@
+Fri Sep 15 20:17:15 UTC 2017 - zai...@opensuse.org
+
+- Add deja-dup-fix-setting-drive-volume.patch: BackendDrive: Fix
+  setting a drive volume (lp#1717230).
+
+-------------------------------------------------------------------

New:
----
  deja-dup-fix-setting-drive-volume.patch

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ deja-dup.spec ++++++
--- /var/tmp/diff_new_pack.m6CfHz/_old  2017-09-17 22:37:55.048124321 +0200
+++ /var/tmp/diff_new_pack.m6CfHz/_new  2017-09-17 22:37:55.072120948 +0200
@@ -24,6 +24,8 @@
 Group:          Productivity/Archiving/Backup
 Url:            https://launchpad.net/deja-dup
 Source0:        
https://launchpad.net/deja-dup/36/%{version}/+download/%{name}-%{version}.tar.xz
+# PATCH-FIX-UPSTREAM deja-dup-fix-setting-drive-volume.patch lp#1717230 
zai...@opensuse.org -- BackendDrive: Fix setting a drive volume
+Patch0:         deja-dup-fix-setting-drive-volume.patch
 # For make check to work, we need dbus-launch
 BuildRequires:  dbus-1-x11
 BuildRequires:  fdupes
@@ -85,6 +87,7 @@
 
 %prep
 %setup -q
+%patch0 -p1
 
 %build
 export CFLAGS="%{optflags} -Wl,-rpath=%{_libdir}/%{name}"

++++++ deja-dup-fix-setting-drive-volume.patch ++++++
>From 3390fd1b0a906b8bd4d859922844b3d57b7f5fdd Mon Sep 17 00:00:00 2001
From: Michael Terry <m...@mterry.name>
Date: Thu, 14 Sep 2017 11:41:38 -0400
Subject: BackendDrive: Fix setting a drive volume

Use get_uuid() instead of get_identifier(UUID) because it gets the
user-friendly version (i.e. the unencrypted partition, not the outer
LUKS partition).

And actually set the drive UUID when choosing it in the settings.

https://launchpad.net/bugs/1717230

diff --git a/deja-dup/widgets/ConfigLocation.vala 
b/deja-dup/widgets/ConfigLocation.vala
index d51d241..8bd092e 100644
--- a/deja-dup/widgets/ConfigLocation.vala
+++ b/deja-dup/widgets/ConfigLocation.vala
@@ -424,8 +424,7 @@ public class ConfigLocation : ConfigWidget
   {
     if (is_allowed_volume(v))
     {
-      add_volume_full(v.get_identifier(VolumeIdentifier.UUID),
-                      v.get_name(), v.get_icon());
+      add_volume_full(v.get_uuid(), v.get_name(), v.get_icon());
     }
   }
 
@@ -445,8 +444,7 @@ public class ConfigLocation : ConfigWidget
 
   void update_volume(VolumeMonitor monitor, Volume v)
   {
-    update_volume_full(v.get_identifier(VolumeIdentifier.UUID),
-                       v.get_name(), v.get_icon());
+    update_volume_full(v.get_uuid(), v.get_name(), v.get_icon());
   }
 
   void update_volume_full(string uuid, string name, Icon icon)
@@ -460,7 +458,7 @@ public class ConfigLocation : ConfigWidget
 
   void remove_volume(VolumeMonitor monitor, Volume v)
   {
-    remove_volume_full(v.get_identifier(VolumeIdentifier.UUID));
+    remove_volume_full(v.get_uuid());
   }
 
   void remove_volume_full(string uuid)
@@ -622,6 +620,7 @@ public class ConfigLocation : ConfigWidget
 
     // First things first, we must remember that we set a volume
     all_settings[""].set_string(BACKEND_KEY, "drive");
+    all_settings[DRIVE_ROOT].set_string(DRIVE_UUID_KEY, uuid);
 
     var vol = VolumeMonitor.get().get_volume_for_uuid(uuid);
     if (vol == null) {
diff --git a/libdeja/BackendDrive.vala b/libdeja/BackendDrive.vala
index dcfe450..86c752c 100644
--- a/libdeja/BackendDrive.vala
+++ b/libdeja/BackendDrive.vala
@@ -125,7 +125,7 @@ public class BackendDrive : BackendFile
     var icon = volume.get_icon();
 
     // sanity check that these writable settings are for this volume
-    var vol_uuid = volume.get_identifier(VolumeIdentifier.UUID);
+    var vol_uuid = volume.get_uuid();
     var settings_uuid = settings.get_string(DRIVE_UUID_KEY);
     if (vol_uuid != settings_uuid)
       return;
-- 
cgit v0.10.2


Reply via email to