X-Git-Url: http://xvm.mit.edu/gitweb/invirt/packages/invirt-base.git/blobdiff_plain/d0bf0e362df11c1bf74fce3bc80ecd6d95ec8b41..efe0510028323cd64653d5b7195e8c4d4ca1455e:/python/invirt/common.py

diff --git a/python/invirt/common.py b/python/invirt/common.py
index 603b7f3..57fea45 100644
--- a/python/invirt/common.py
+++ b/python/invirt/common.py
@@ -8,30 +8,30 @@ import subprocess
 class InvirtConfigError(AttributeError):
     pass
 
-class struct(object):
+class struct(dict):
     'A simple namespace object.'
-    def __init__(self, d = {}, __prefix = None, **kwargs):
-        'd is the dictionary or the items-iterable to update my __dict__ with.'
-        dct = {}
-        dct.update(d)
-        dct.update(kwargs)
-        self.__dict__.update(dct)
-        self.__keys = set(dct)
+    def __init__(self, d = {}, __prefix = None, __default=None, **kwargs):
+        super(struct, self).__init__(d)
         self.__prefix = __prefix
+        self.__default = __default
+        self.update(kwargs)
     def __getattr__(self, key):
-        # XX ideally these would point a frame higher on the stack.
-        prefix = self.__prefix
-        if prefix is not None:
-            raise InvirtConfigError('missing configuration variable %s%s'
-                                    % (prefix, key))
-        else:
-            raise AttributeError("anonymous struct has no member '%s'"
-                                 % (key,))
-    def __iter__(self):
-        for i in self.__keys:
-            yield i
+        try:
+            return self[key]
+        except KeyError:
+            if self.__default is None:
+                # XX ideally these would point a frame higher on the stack.
+                prefix = self.__prefix
+                if prefix is not None:
+                    raise InvirtConfigError('missing configuration variable '
+                                            '%s%s' % (prefix, key))
+                else:
+                    raise AttributeError("anonymous struct has no member '%s'"
+                                         % (key,))
+            else:
+                return struct({}, '', self.__default)
 
-def dicts2struct(x, prefix = None):
+def dicts2struct(x, prefix = None, default = None):
     """
     Given a tree of lists/dicts, perform a deep traversal to transform all the
     dicts to structs.
@@ -41,13 +41,15 @@ def dicts2struct(x, prefix = None):
     else:
         def newprefix(k): return prefix
     if type(x) == dict:
-        return struct(((k, dicts2struct(v, newprefix(k)))
+        return struct(((k, dicts2struct(v, newprefix(k), default))
                        for k,v in x.iteritems()),
-                      prefix)
+                      prefix,
+                      default)
     elif type(x) == list:
-        return [dicts2struct(v, newprefix(i)) for i, v in enumerate(x)]
+        return [dicts2struct(v, newprefix(i), default)
+                for i, v in enumerate(x)]
     elif x is None:
-        return struct({}, prefix)
+        return struct({}, prefix, default)
     else:
         return x
 
@@ -80,11 +82,12 @@ def captureOutput(popen_args, stdin_str=None, *args, **kwargs):
     if 'stdout' not in kwargs:
         kwargs['stdout'] = subprocess.PIPE
     if 'stderr' not in kwargs:
-        kwargs['stderr'] = subprocess.STDOUT
+        kwargs['stderr'] = subprocess.PIPE
     p = subprocess.Popen(popen_args, *args, **kwargs)
-    out, _ = p.communicate(stdin_str)
+    out, err = p.communicate(stdin_str)
     if p.returncode:
-        raise subprocess.CalledProcessError(p.returncode, popen_args, out)
+        raise subprocess.CalledProcessError(p.returncode, '%s, stdout: %s, stderr: %s' %
+                                            (popen_args, out, err))
     return out
 
 #