Merge lp:~abentley/lp-dev-utils/ec2-errors into lp:lp-dev-utils

Proposed by Aaron Bentley
Status: Merged
Approved by: j.c.sackett
Approved revision: 120
Merged at revision: 120
Proposed branch: lp:~abentley/lp-dev-utils/ec2-errors
Merge into: lp:lp-dev-utils
Diff against target: 21 lines (+4/-2)
1 file modified
ec2 (+4/-2)
To merge this branch: bzr merge lp:~abentley/lp-dev-utils/ec2-errors
Reviewer Review Type Date Requested Status
j.c.sackett (community) Approve
Review via email: mp+116891@code.launchpad.net

Commit message

Better handling of ec2 user errors.

Description of the change

This just fixes the ec2 error handling so that user errors don't generate tracebacks.

To post a comment you must log in.
Revision history for this message
j.c.sackett (jcsackett) wrote :

Looks good. Thanks.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'ec2'
--- ec2 2012-02-24 20:10:13 +0000
+++ ec2 2012-07-26 15:22:23 +0000
@@ -14,7 +14,7 @@
14import sys14import sys
1515
16import bzrlib16import bzrlib
17from bzrlib.errors import BzrCommandError17from bzrlib.errors import BzrError
1818
19from ec2test import builtins19from ec2test import builtins
20from ec2test.controller import (20from ec2test.controller import (
@@ -46,7 +46,9 @@
46 args = ['help']46 args = ['help']
47 try:47 try:
48 controller.run(args)48 controller.run(args)
49 except BzrCommandError, e:49 except BzrError as e:
50 if e.internal_error:
51 raise
50 sys.exit('ec2: ERROR: ' + str(e))52 sys.exit('ec2: ERROR: ' + str(e))
5153
5254

Subscribers

People subscribed via source and target branches