Changeset 2095 for trunk/packages/invirt-base/python/invirt/remctl.py

Timestamp:

Feb 5, 2009, 3:19:30 AM (15 years ago)

Author:

broder

Message:

Get rid of confusing err=True option to invirt.remctl.remctl.

File:

• trunk/packages/invirt-base/python/invirt/remctl.py (modified) (1 diff)

trunk/packages/invirt-base/python/invirt/remctl.py

@@ -37 +37 @@
                          stderr=subprocess.PIPE)
     v = p.wait()
-    if kws.get('err'):
-        return p.stdout.read(), p.stderr.read()
     if v:
-        print >> sys.stderr, 'Error', v, 'on remctl', args, ':'
-        print >> sys.stderr, p.stderr.read()
-        raise CodeError('ERROR on remctl')
+        raise CodeError('ERROR on remctl %s:\n%s' % (args, p.stderr.read()), v)
     return p.stdout.read()