Changeset 622 for trunk/packages/sipb-xen-remote-server/files/usr

Timestamp:

Jun 21, 2008, 7:33:46 PM (16 years ago)

Author:

price

Message:

not worth defining an exception class for one invocation site

File:

• trunk/packages/sipb-xen-remote-server/files/usr/sbin/sipb-xen-remote-listvms (modified) (2 diffs)

trunk/packages/sipb-xen-remote-server/files/usr/sbin/sipb-xen-remote-listvms

@@ -7 +7 @@
 
 from subprocess import PIPE, Popen
-try:
-    from subprocess import CalledProcessError
-except ImportError:
-    # Python 2.4 doesn't implement CalledProcessError
-    class CalledProcessError(Exception):
-        """This exception is raised when a process run by check_call() returns
-        a non-zero exit status. The exit status will be stored in the
-        returncode attribute."""
-        def __init__(self, returncode, cmd):
-            self.returncode = returncode
-            self.cmd = cmd
-        def __str__(self):
-            return "Command '%s' returned non-zero exit status %d" % (self.cmd, self.returncode)
 import sys
 import yaml
@@ -36 +23 @@
     for p in pipes:
         if p.returncode != 0:
-            raise CalledProcessError(p.returncode, 'remctl')
+            raise RuntimeError("Command '%s' returned non-zero exit status %d"
+                               % ('remctl', p.returncode)) 
     results = [yaml.load(o, yaml.CSafeLoader) for o in outputs]
     results = filter(lambda x: x is not None, results)