Merge lp:~mgill25/mailman/branch2 into lp:mailman

Proposed by Manish Gill
Status: Rejected
Rejected by: Barry Warsaw
Proposed branch: lp:~mgill25/mailman/branch2
Merge into: lp:mailman
Diff against target: 16 lines (+3/-2)
1 file modified
src/mailman/core/switchboard.py (+3/-2)
To merge this branch: bzr merge lp:~mgill25/mailman/branch2
Reviewer Review Type Date Requested Status
Mailman Coders Pending
Review via email: mp+158359@code.launchpad.net

Description of the change

To post a comment you must log in.

Unmerged revisions

7211. By Manish Gill <email address hidden>

#1165589 fix: Log exception message for switchboard.finish()

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/mailman/core/switchboard.py'
2--- src/mailman/core/switchboard.py 2013-01-20 21:23:09 +0000
3+++ src/mailman/core/switchboard.py 2013-04-11 12:56:26 +0000
4@@ -183,9 +183,10 @@
5 os.rename(bakfile, psvfile)
6 else:
7 os.unlink(bakfile)
8- except EnvironmentError:
9+ except EnvironmentError as error:
10 elog.exception(
11- 'Failed to unlink/preserve backup file: %s', bakfile)
12+ 'Unlink/Preserve backup file exception: %s\n'
13+ 'Failed to unlink/preserve backup file: %s', error, bakfile)
14
15 @property
16 def files(self):