Storm

Merge lp://staging/~cjwatson/storm/psycopg2-missing-diag into lp://staging/storm

  1. psycopg2-missing-diag
  2. Merge into trunk
Proposed by Colin Watson
Status: Merged
Merged at revision: 520
Proposed branch: lp://staging/~cjwatson/storm/psycopg2-missing-diag
Merge into: lp://staging/storm
Diff against target: 16 lines (+5/-1)
1 file modified
storm/databases/postgres.py (+5/-1)
To merge this branch: bzr merge lp://staging/~cjwatson/storm/psycopg2-missing-diag
Reviewer Review Type Date Requested Status
Kristian Glass (community) Approve
Storm Developers Pending
Review via email: mp+371169@code.staging.launchpad.net

Commit message

Fix error wrapping with psycopg2 < 2.5.

Description of the change

psycopg2.Error.diag was only added in 2.5, so don't try to wrap it if it doesn't exist.

Bug introduced in https://code.launchpad.net/~cjwatson/storm/refactor-exception-wrapping/+merge/369319.

To post a comment you must log in.
Revision history for this message
Kristian Glass (doismellburning) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to status/vote changes: