Add the Ignore() method.
authorstevenknight <stevenknight@fdb21ef1-2011-0410-befe-b5e4ea1792b1>
Thu, 24 Jan 2002 06:29:31 +0000 (06:29 +0000)
committerstevenknight <stevenknight@fdb21ef1-2011-0410-befe-b5e4ea1792b1>
Thu, 24 Jan 2002 06:29:31 +0000 (06:29 +0000)
git-svn-id: http://scons.tigris.org/svn/scons/trunk@220 fdb21ef1-2011-0410-befe-b5e4ea1792b1

doc/man/scons.1
src/CHANGES.txt
src/engine/SCons/Environment.py
src/engine/SCons/EnvironmentTests.py
src/engine/SCons/Node/NodeTests.py
src/engine/SCons/Node/__init__.py
test/Ignore.py [new file with mode: 0644]

index dfb96c7d9d1ce0fec5f4b9bbb65e3e3fad48904a..9a2e9a0fb8473935d38747f68384554ff0c839a9 100644 (file)
@@ -647,6 +647,17 @@ dict = env.Dictionary()
 cc_dict = env.Dictionary('CC', 'CCFLAGS', 'CCCOM')
 .EE
 
+.TP
+.RI Ignore( target ", " dependency )
+The specified dependency file(s)
+will be ignored when deciding if
+the target file(s) need to be rebuilt.
+
+.ES
+env.Ignore('foo', 'foo.c')
+env.Ignore('bar', ['bar1.h', 'bar2.h'])
+.EE
+
 .TP
 .RI Install( dir ", " source )
 Installs one or more files in a destination directory.
index ea9bf431f7bec62b748e18d931aa51075f541dff..a24671106a8a303bf67d31947ff1d7ee8980d88e 100644 (file)
@@ -19,6 +19,8 @@ RELEASE 0.04 -
 
   - Allow LIBS and LIBPATH to be strings, not just arrays.
 
+  - Print a traceback if a Python-function builder throws an exception.
+
   From Steven Knight:
 
   - Fix using a directory as a Default(), and allow Default() to
@@ -46,6 +48,8 @@ RELEASE 0.04 -
     surround $_INCDIRS and $_LIBDIRS so we don't rebuild in response
     to changes to -I or -L options.
 
+  - Add the Ignore() method to ignore dependencies.
+
   From Steve Leblanc:
 
   - Add var=value command-line arguments.
index 612ae9ac9320f8c972f68cb4605ca4b86edbc0c3..3c1ee430a4917681f62abb1716c44f4d76d8aa4d 100644 (file)
@@ -174,6 +174,17 @@ class Environment:
            tlist = tlist[0]
        return tlist
 
+    def Ignore(self, target, dependency):
+        """Ignore a dependency."""
+        tlist = SCons.Util.scons_str2nodes(target)
+        dlist = SCons.Util.scons_str2nodes(dependency)
+        for t in tlist:
+            t.add_ignore(dlist)
+
+        if len(tlist) == 1:
+            tlist = tlist[0]
+        return tlist
+
     def Dictionary(self, *args):
        if not args:
            return self._dict
index a65cc33e4d4106f7937c6eace0891d72d75f49f3..59f3c78b7be41d5275ab3de05d99a1bb41a45f8f 100644 (file)
@@ -258,6 +258,17 @@ class EnvironmentTestCase(unittest.TestCase):
        assert d.__class__.__name__ == 'File'
        assert d.path == 'Environment.py'
 
+    def test_Ignore(self):
+        """Test the explicit Ignore method."""
+        env = Environment()
+        t = env.Ignore(target='targ.py', dependency='dep.py')
+        assert t.__class__.__name__ == 'File'
+        assert t.path == 'targ.py'
+        assert len(t.ignore) == 1
+        i = t.ignore[0]
+        assert i.__class__.__name__ == 'File'
+        assert i.path == 'dep.py'
+
     def test_Command(self):
         """Test the Command() method."""
         env = Environment()
index 10507679a86489e5535e6a66dd563be78af0e828..aa9dccb1c24ef331410cf8791d31557990457e2a 100644 (file)
@@ -312,6 +312,38 @@ class NodeTestCase(unittest.TestCase):
         assert node.implicit[3] == [two, three]
         assert node.implicit[4] == [three, four, one]
 
+    def test_add_ignore(self):
+        """Test adding files whose dependencies should be ignored.
+        """
+        node = SCons.Node.Node()
+        assert node.ignore == []
+
+        zero = SCons.Node.Node()
+        try:
+            node.add_ignore(zero)
+        except TypeError:
+            pass
+        else:
+            assert 0
+
+        one = SCons.Node.Node()
+        two = SCons.Node.Node()
+        three = SCons.Node.Node()
+        four = SCons.Node.Node()
+
+        node.add_ignore([one])
+        assert node.ignore == [one]
+        node.add_ignore([two, three])
+        assert node.ignore == [one, two, three]
+        node.add_ignore([three, four, one])
+        assert node.ignore == [one, two, three, four]
+
+        assert zero.get_parents() == []
+        assert one.get_parents() == [node]
+        assert two.get_parents() == [node]
+        assert three.get_parents() == [node]
+        assert four.get_parents() == [node]
+
     def test_scan(self):
         """Test Scanner functionality"""
         class DummyScanner:
index c6418951d1ed5689731883d03757c757ca07d97c..03705f7127a48c4d1b5ef45fd9bcd2a5ba45b0e4 100644 (file)
@@ -59,6 +59,7 @@ class Node:
         self.sources = []       # source files used to build node
         self.depends = []       # explicit dependencies (from Depends)
         self.implicit = {}     # implicit (scanned) dependencies
+        self.ignore = []       # dependencies to ignore
         self.parents = []
         self.wkids = None       # Kids yet to walk, when it's an array
        self.builder = None
@@ -167,6 +168,10 @@ class Node:
        """Adds dependencies. The depend argument must be a list."""
         self._add_child(self.depends, depend)
 
+    def add_ignore(self, depend):
+        """Adds dependencies to ignore. The depend argument must be a list."""
+        self._add_child(self.ignore, depend)
+
     def add_source(self, source):
        """Adds sources. The source argument must be a list."""
         self._add_child(self.sources, source)
@@ -201,9 +206,10 @@ class Node:
 
     def children(self):
         #XXX Need to remove duplicates from this
-        return self.sources \
-               + self.depends \
-               + reduce(lambda x, y: x + y, self.implicit.values(), [])
+        return filter(lambda x, i=self.ignore: x not in i,
+                      self.sources \
+                      + self.depends \
+                      + reduce(lambda x, y: x + y, self.implicit.values(), []))
 
     def get_parents(self):
         return self.parents
diff --git a/test/Ignore.py b/test/Ignore.py
new file mode 100644 (file)
index 0000000..32e6637
--- /dev/null
@@ -0,0 +1,96 @@
+#!/usr/bin/env python
+#
+# Copyright (c) 2001 Steven Knight
+#
+# Permission is hereby granted, free of charge, to any person obtaining
+# a copy of this software and associated documentation files (the
+# "Software"), to deal in the Software without restriction, including
+# without limitation the rights to use, copy, modify, merge, publish,
+# distribute, sublicense, and/or sell copies of the Software, and to
+# permit persons to whom the Software is furnished to do so, subject to
+# the following conditions:
+#
+# The above copyright notice and this permission notice shall be included
+# in all copies or substantial portions of the Software.
+#
+# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY
+# KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE
+# WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+# NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+# LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+# OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+# WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+#
+
+__revision__ = "__FILE__ __REVISION__ __DATE__ __DEVELOPER__"
+
+import sys
+import TestSCons
+
+python = sys.executable
+
+test = TestSCons.TestSCons()
+
+test.subdir('subdir')
+
+test.write('build.py', r"""
+import sys
+contents = open(sys.argv[2], 'rb').read() + open(sys.argv[3], 'rb').read()
+file = open(sys.argv[1], 'wb')
+for arg in sys.argv[2:]:
+    file.write(open(arg, 'rb').read())
+file.close()
+""")
+
+test.write('SConstruct', """
+Foo = Builder(name = "Foo",
+              action = r"%s build.py $TARGET $SOURCES")
+Bar = Builder(name = "Bar",
+              action = r"%s build.py $TARGET $SOURCES")
+env = Environment(BUILDERS = [Foo, Bar])
+env.Foo(target = 'f1.out', source = ['f1a.in', 'f1b.in'])
+env.Ignore(target = 'f1.out', dependency = 'f1b.in')
+SConscript('subdir/SConscript', "env")
+""" % (python, python))
+
+test.write(['subdir', 'SConscript'], """
+Import("env")
+env.Bar(target = 'f2.out', source = ['f2a.in', 'f2b.in'])
+env.Ignore('f2.out', 'f2a.in')
+""")
+
+test.write('f1a.in', "f1a.in\n")
+test.write('f1b.in', "f1b.in\n")
+
+test.write(['subdir', 'f2a.in'], "subdir/f2a.in\n")
+test.write(['subdir', 'f2b.in'], "subdir/f2b.in\n")
+
+test.run(arguments = '.')
+
+test.fail_test(test.read('f1.out') != "f1a.in\nf1b.in\n")
+test.fail_test(test.read(['subdir', 'f2.out']) !=
+               "subdir/f2a.in\nsubdir/f2b.in\n")
+
+test.up_to_date(arguments = '.')
+
+test.write('f1b.in', "f1b.in 2\n")
+test.write(['subdir', 'f2a.in'], "subdir/f2a.in 2\n")
+
+test.up_to_date(arguments = '.')
+
+test.fail_test(test.read('f1.out') != "f1a.in\nf1b.in\n")
+test.fail_test(test.read(['subdir', 'f2.out']) !=
+               "subdir/f2a.in\nsubdir/f2b.in\n")
+
+test.write('f1a.in', "f1a.in 2\n")
+test.write(['subdir', 'f2b.in'], "subdir/f2b.in 2\n")
+
+test.run(arguments = '.')
+
+test.fail_test(test.read('f1.out') != "f1a.in 2\nf1b.in 2\n")
+test.fail_test(test.read(['subdir', 'f2.out']) !=
+               "subdir/f2a.in 2\nsubdir/f2b.in 2\n")
+
+test.up_to_date(arguments = '.')
+
+test.pass_test()