Merge lp:~mterry/duplicity/partial-encryption into lp:duplicity/0.6

Proposed by Michael Terry
Status: Merged
Merged at revision: 784
Proposed branch: lp:~mterry/duplicity/partial-encryption
Merge into: lp:duplicity/0.6
Diff against target: 15 lines (+5/-0)
1 file modified
duplicity/collections.py (+5/-0)
To merge this branch: bzr merge lp:~mterry/duplicity/partial-encryption
Reviewer Review Type Date Requested Status
duplicity-team Pending
Review via email: mp+73566@code.launchpad.net

Description of the change

See bug 838264 for background info.

This branch allows upgrading a partial chain's info if it turns out that its files are really encrypted. I'm not 100% it's safe, but it seems reasonable. A more experience eye would be helpful.

It only allows upgrading encryption status of partial chains to True. Otherwise, if the encryption states don't match, it rejects the filename (which is new behavior).

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'duplicity/collections.py'
--- duplicity/collections.py 2011-08-21 00:00:56 +0000
+++ duplicity/collections.py 2011-08-31 17:49:27 +0000
@@ -85,6 +85,11 @@
85 if (pr.start_time != self.start_time or85 if (pr.start_time != self.start_time or
86 pr.end_time != self.end_time):86 pr.end_time != self.end_time):
87 return False87 return False
88 if pr.encrypted != self.encrypted:
89 if self.partial and pr.encrypted:
90 self.encrypted = pr.encrypted
91 else:
92 return False
8893
89 if pr.manifest:94 if pr.manifest:
90 self.set_manifest(filename)95 self.set_manifest(filename)

Subscribers

People subscribed via source and target branches

to all changes: