Merge lp:~jml/launchpad/fix-logger into lp:launchpad
Proposed by
Jonathan Lange
Status: | Merged |
---|---|
Approved by: | Jonathan Lange |
Approved revision: | no longer in the source branch. |
Merged at revision: | 11626 |
Proposed branch: | lp:~jml/launchpad/fix-logger |
Merge into: | lp:launchpad |
Diff against target: |
15 lines (+1/-4) 1 file modified
lib/canonical/launchpad/scripts/logger.py (+1/-4) |
To merge this branch: | bzr merge lp:~jml/launchpad/fix-logger |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jonathan Lange (community) | Approve | ||
Review via email: mp+36532@code.launchpad.net |
Commit message
Simplify FakeLogger, fixing bug caused by spurious string interpolation.
Description of the change
This kills some overly complex custom logger code that turned out to also be buggy.
Basically, if you called FakeLogger methods passing exc_info and there was a '%s' somewhere in your stack, FakeLogger would try to do string interpolation on the stack trace and fail. Now it just uses the built-in Python method to print exceptions to print the exception.
To post a comment you must log in.
Julian did the work. I'm just pushing up & reviewing.