Merge lp:~mterry/duplicity/797758 into lp:duplicity/0.6
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | 749 | ||||
Proposed branch: | lp:~mterry/duplicity/797758 | ||||
Merge into: | lp:duplicity/0.6 | ||||
Diff against target: |
374 lines (+37/-37) 17 files modified
duplicity-bin (+2/-2) duplicity/backend.py (+9/-9) duplicity/backends/ftpbackend.py (+1/-1) duplicity/backends/ftpsbackend.py (+1/-1) duplicity/backends/hsibackend.py (+1/-1) duplicity/backends/imapbackend.py (+1/-1) duplicity/backends/localbackend.py (+1/-1) duplicity/collections.py (+2/-2) duplicity/commandline.py (+3/-3) duplicity/dup_temp.py (+1/-1) duplicity/gpg.py (+6/-6) duplicity/manifest.py (+1/-1) duplicity/pexpect.py (+3/-3) duplicity/selection.py (+1/-1) duplicity/tempdir.py (+2/-2) testing/backendtest.py (+1/-1) testing/config.py.tmpl (+1/-1) |
||||
To merge this branch: | bzr merge lp:~mterry/duplicity/797758 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
duplicity-team | Pending | ||
Review via email: mp+64942@code.launchpad.net |
Description of the change
From bug 797758:
1) Duplicity calls self.last_
2) Which executes the following:
try:
self.
except:
log.
pass
3) If the backend.delete function encounters a FatalError, sys.exit will be called, which should close duplicity.
4) But the way that except clause is written, it won't. Blanket "except:" clauses will catch even SystemExit exceptions. See http://
The fix is to use "except Exception:" instead. This branch does that for all except: lines. I assume that is correct, that we never intentionally want to block SystemExit or KeyboardInterrupt?