Support using Dirs as sources for builds. (Charles Crain)

4 files changed, 60 insertions, 8 deletions

diff --git a/src/CHANGES.txt b/src/CHANGES.txt
index 5a7f01c..a5d2b3d 100644
--- a/src/CHANGES.txt
+++ b/src/CHANGES.txt
@@ -31,6 +31,9 @@ RELEASE X.XX - XXX
 
   - Fix some Python 2.2 specific things in various tool modules.
 
+  - Support directories as build sources, so that a rebuild of a target
+    can be triggered if anything underneath the directory changes.
+
   From Christian Engel:
 
   - Support more flexible inclusion of separate C and C++ compilers.
diff --git a/src/engine/SCons/Node/FS.py b/src/engine/SCons/Node/FS.py
index 0abc21b..d93a6a4 100644
--- a/src/engine/SCons/Node/FS.py
+++ b/src/engine/SCons/Node/FS.py
@@ -41,6 +41,7 @@ import os.path
 import shutil
 import stat
 import string
+import cStringIO
 
 import SCons.Action
 import SCons.Errors
@@ -902,6 +903,17 @@ class FS:
             message = "building associated BuildDir targets: %s" % string.join(map(str, targets))
         return targets, message
 
+class DummyExecutor:
+    """Dummy executor class returned by Dir nodes to bamboozle SCons
+    into thinking we are an actual derived node, where our sources are
+    our directory entries."""
+    def get_raw_contents(self):
+        return ''
+    def get_contents(self):
+        return ''
+    def get_timestamp(self):
+        return None
+
 class Dir(Base):
     """A class for directories in a file system.
     """
@@ -931,6 +943,7 @@ class Dir(Base):
         self.entries['..'] = self.dir
         self.cwd = self
         self.builder = 1
+        self.searched = 0
         self._sconsign = None
         self.build_dirs = []
 
@@ -1024,6 +1037,19 @@ class Dir(Base):
                              self.all_children(scan))
 
     def all_children(self, scan=1):
+        # Before we traverse our children, make sure we have created Nodes
+        # for any files that this directory contains.  We need to do this
+        # so any change in a file in this directory will cause it to
+        # be out of date.
+        if not self.searched:
+            try:
+                for filename in os.listdir(self.abspath):
+                    if filename != '.sconsign':
+                        self.Entry(filename)
+            except OSError:
+                # Directory does not exist.  No big deal
+                pass
+            self.searched = 1
         keys = filter(lambda k: k != '.' and k != '..', self.entries.keys())
         kids = map(lambda x, s=self: s.entries[x], keys)
         def c(one, two):
@@ -1052,10 +1078,6 @@ class Dir(Base):
         """A directory does not get scanned."""
         return None
 
-    def calc_signature(self, calc):
-        """A directory has no signature."""
-        return None
-
     def set_bsig(self, bsig):
         """A directory has no signature."""
         bsig = None
@@ -1065,9 +1087,12 @@ class Dir(Base):
         csig = None
 
     def get_contents(self):
-        """Return a fixed "contents" value of a directory."""
-        return ''
-
+        """Return aggregate contents of all our children."""
+        contents = cStringIO.StringIO()
+        for kid in self.children(None):
+            contents.write(kid.get_contents())
+        return contents.getvalue()
+    
     def prepare(self):
         pass
 
@@ -1111,6 +1136,24 @@ class Dir(Base):
             return self.srcdir
         return Base.srcnode(self)
 
+    def get_executor(self, create=1):
+        """Fetch the action executor for this node.  Create one if
+        there isn't already one, and requested to do so."""
+        try:
+            executor = self.executor
+        except AttributeError:
+            executor = DummyExecutor()
+            self.executor = executor
+        return executor
+
+    def get_timestamp(self):
+        """Return the latest timestamp from among our children"""
+        stamp = None
+        for kid in self.children(None):
+            if kid.get_timestamp() > stamp:
+                stamp = kid.get_timestamp()
+        return stamp
+
 class File(Base):
     """A class for files in a file system.
     """
diff --git a/src/engine/SCons/Node/FSTests.py b/src/engine/SCons/Node/FSTests.py
index ded5e9a..53e2013 100644
--- a/src/engine/SCons/Node/FSTests.py
+++ b/src/engine/SCons/Node/FSTests.py
@@ -1146,13 +1146,15 @@ class EntryTestCase(unittest.TestCase):
                 self.val = val
             def csig(self, node, cache):
                 return self.val
+            def bsig(self, node, cache):
+                return self.val + 222
         test.subdir('e5d')
         test.write('e5f', "e5f\n")
 
         e5d = fs.Entry('e5d')
         sig = e5d.calc_signature(MyCalc(555))
         assert e5d.__class__ is SCons.Node.FS.Dir, e5d.__class__
-        assert sig is None, sig
+        assert sig == 777, sig
 
         e5f = fs.Entry('e5f')
         sig = e5f.calc_signature(MyCalc(666))
diff --git a/src/engine/SCons/Node/__init__.py b/src/engine/SCons/Node/__init__.py
index dbcc4f0..1eb7f29 100644
--- a/src/engine/SCons/Node/__init__.py
+++ b/src/engine/SCons/Node/__init__.py
@@ -411,6 +411,8 @@ class Node:
         self.env = env
 
     def calculator(self):
+        import SCons.Defaults
+        
         env = self.env or SCons.Defaults.DefaultEnvironment()
         return env.get_calculator()
 
@@ -426,6 +428,8 @@ class Node:
             return self._calculated_sig
         except AttributeError:
             if self.is_derived():
+                import SCons.Defaults
+                
                 env = self.env or SCons.Defaults.DefaultEnvironment()
                 if env.use_build_signature():
                     sig = self.rfile().calc_bsig(calc, self)