Depedency fix-up attempt.
authorCarl Hetherington <cth@carlh.net>
Mon, 3 Jul 2017 00:18:16 +0000 (01:18 +0100)
committerCarl Hetherington <cth@carlh.net>
Mon, 3 Jul 2017 00:18:16 +0000 (01:18 +0100)
cdist

diff --git a/cdist b/cdist
index 45a1573e5b0d9501f7b03e66431287340ec42cf4..0e325147349f7ad1233859052f9c65ab9d0dd2db 100755 (executable)
--- a/cdist
+++ b/cdist
@@ -701,34 +701,25 @@ class Tree(object):
         with TreeDirectory(self):
             return self.cscript[function](self.target, *args)
 
-    def build_dependencies(self, dry_run, options=None):
+    def build_dependencies(self, dry_run):
         if 'dependencies' in self.cscript:
-            if len(inspect.getargspec(self.cscript['dependencies']).args) == 2:
-                deps = self.call('dependencies', options)
-            else:
-                deps = self.call('dependencies')
-
-            to = options
-            if to is None:
-                to = dict()
-
-            for d in deps:
+            for d in self.call('dependencies'):
                 log('Building dependency %s %s of %s' % (d[0], d[1], self.name))
                 dep = globals.trees.get(d[0], d[1], self.target)
 
+                options = dict()
                 # Make the options to pass in from the option_defaults of the thing
                 # we are building and any options specified by the parent.
-
                 if 'option_defaults' in dep.cscript:
                     for k, v in dep.cscript['option_defaults']().items():
-                        to[k] = v
+                        options[k] = v
 
                 if len(d) > 2:
                     for k, v in d[2].items():
-                        to[k] = v
+                        options[k] = v
 
-                dep.build_dependencies(dry_run, to)
-                dep.build(dry_run, to)
+                dep.build_dependencies(dry_run, options)
+                dep.build(dry_run, options)
 
     def build(self, dry_run, options=None):
         if self.built: