diff options
author | Steven Knight <knight@baldmt.com> | 2008-09-11 01:28:15 (GMT) |
---|---|---|
committer | Steven Knight <knight@baldmt.com> | 2008-09-11 01:28:15 (GMT) |
commit | 4519bfd9be6fcf55732666e7c541faea0a8f1533 (patch) | |
tree | 12b302fe4ee7d4dcf422600af9d4b990eee2a19b /test | |
parent | fea8478717f8cce00ed0c0aa7d9ad79484bbd57c (diff) | |
download | SCons-4519bfd9be6fcf55732666e7c541faea0a8f1533.zip SCons-4519bfd9be6fcf55732666e7c541faea0a8f1533.tar.gz SCons-4519bfd9be6fcf55732666e7c541faea0a8f1533.tar.bz2 |
Issue 1681: Fix the ability of add_src_builder() to handle arbitrary
builder objects by eliminating unintentional re-use of a class
list for the src_builder attribute.
Diffstat (limited to 'test')
-rw-r--r-- | test/Builder/add_src_builder.py | 65 |
1 files changed, 65 insertions, 0 deletions
diff --git a/test/Builder/add_src_builder.py b/test/Builder/add_src_builder.py new file mode 100644 index 0000000..f67acc0 --- /dev/null +++ b/test/Builder/add_src_builder.py @@ -0,0 +1,65 @@ +#!/usr/bin/env python +# +# __COPYRIGHT__ +# +# 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__" + +""" +Verify that we can call add_src_builder() to add a builder to +another on the fly. + +This used to trigger infinite recursion (issue 1681) because the +same src_builder list object was being re-used between all Builder +objects that weren't initialized with a separate src_builder. +""" + +import TestSCons + +test = TestSCons.TestSCons() + +test.write('SConstruct', """\ +copy_out = Builder(action = Copy('$TARGET', '$SOURCE'), + suffix = '.out', + src_suffix = '.mid') + +copy_mid = Builder(action = Copy('$TARGET', '$SOURCE'), + suffix = '.mid', \ + src_suffix = '.in') + +env = Environment() +env['BUILDERS']['CopyOut'] = copy_out +env['BUILDERS']['CopyMid'] = copy_mid + +copy_out.add_src_builder(copy_mid) + +env.CopyOut('file1.out', 'file1.in') +""") + +test.write('file1.in', "file1.in\n") + +test.run() + +test.must_match('file1.mid', "file1.in\n") +test.must_match('file1.out', "file1.in\n") + +test.pass_test() |