Author: aconway Date: Wed Nov 18 19:19:49 2009 New Revision: 881888 URL: http://svn.apache.org/viewvc?rev=881888&view=rev Log: Improved exception handling in brokertest.Broker.
Don't mask the original exception unless it's a ValueError. Modified: qpid/trunk/qpid/python/qpid/brokertest.py Modified: qpid/trunk/qpid/python/qpid/brokertest.py URL: http://svn.apache.org/viewvc/qpid/trunk/qpid/python/qpid/brokertest.py?rev=881888&r1=881887&r2=881888&view=diff ============================================================================== --- qpid/trunk/qpid/python/qpid/brokertest.py (original) +++ qpid/trunk/qpid/python/qpid/brokertest.py Wed Nov 18 19:19:49 2009 @@ -149,7 +149,7 @@ Popen.__init__(self, cmd, expect) try: self.port = int(self.stdout.readline()) - except Exception, e: + except ValueError, e: raise Exception("Failed to start broker %s (%s)" % (self.name, self.pname)) test.cleanup_stop(self) self.host = "localhost" # Placeholder for remote brokers. --------------------------------------------------------------------- Apache Qpid - AMQP Messaging Implementation Project: http://qpid.apache.org Use/Interact: mailto:commits-subscr...@qpid.apache.org