Make the (s) and variables upper-case.
authorstevenknight <stevenknight@fdb21ef1-2011-0410-befe-b5e4ea1792b1>
Wed, 31 Oct 2001 03:47:18 +0000 (03:47 +0000)
committerstevenknight <stevenknight@fdb21ef1-2011-0410-befe-b5e4ea1792b1>
Wed, 31 Oct 2001 03:47:18 +0000 (03:47 +0000)
git-svn-id: http://scons.tigris.org/svn/scons/trunk@113 fdb21ef1-2011-0410-befe-b5e4ea1792b1

20 files changed:
src/engine/SCons/Builder.py
src/engine/SCons/BuilderTests.py
src/engine/SCons/Defaults.py
src/engine/SCons/UtilTests.py
test/Command.py
test/Default.py
test/Depends.py
test/ENV.py
test/Library.py
test/builderrors.py
test/multiline.py
test/option--.py
test/option-c.py
test/option-i.py
test/option-j.py
test/option-k.py
test/option-n.py
test/option-s.py
test/subdir.py
test/up-to-date.py

index 7febeb2538f414f6bd1adb17dd360040840350a6..4023530931c4693fef1f4a955fc9818f096587b8 100644 (file)
@@ -312,13 +312,13 @@ class CommandAction(ActionBase):
            t = kw['target']
            if type(t) is type(""):
                t = [t]
-           loc['targets'] = PathList(map(os.path.normpath, t))
-           loc['target'] = loc['targets'][0]
+           loc['TARGETS'] = PathList(map(os.path.normpath, t))
+           loc['TARGET'] = loc['TARGETS'][0]
        if kw.has_key('source'):
            s = kw['source']
            if type(s) is type(""):
                s = [s]
-            loc['sources'] = PathList(map(os.path.normpath, s))
+            loc['SOURCES'] = PathList(map(os.path.normpath, s))
 
        glob = {}
        if kw.has_key('env'):
index 6c9c67896215dd2d68ee07c015f546d343807ce2..e24784d38c35cce7b47dd66b7c11aaf4b878d135 100644 (file)
@@ -121,7 +121,7 @@ class BuilderTestCase(unittest.TestCase):
        c = test.read(outfile, 'r')
        assert c == "act.py: xyzzy\n", c
 
-       cmd2 = r'%s %s %s $target' % (python, act_py, outfile)
+       cmd2 = r'%s %s %s $TARGET' % (python, act_py, outfile)
 
        builder = SCons.Builder.Builder(action = cmd2)
        r = builder.execute(target = 'foo')
@@ -129,7 +129,7 @@ class BuilderTestCase(unittest.TestCase):
        c = test.read(outfile, 'r')
        assert c == "act.py: foo\n", c
 
-       cmd3 = r'%s %s %s ${targets}' % (python, act_py, outfile)
+       cmd3 = r'%s %s %s ${TARGETS}' % (python, act_py, outfile)
 
        builder = SCons.Builder.Builder(action = cmd3)
        r = builder.execute(target = ['aaa', 'bbb'])
@@ -137,7 +137,7 @@ class BuilderTestCase(unittest.TestCase):
        c = test.read(outfile, 'r')
        assert c == "act.py: aaa bbb\n", c
 
-       cmd4 = r'%s %s %s $sources' % (python, act_py, outfile)
+       cmd4 = r'%s %s %s $SOURCES' % (python, act_py, outfile)
 
        builder = SCons.Builder.Builder(action = cmd4)
        r = builder.execute(source = ['one', 'two'])
@@ -145,7 +145,7 @@ class BuilderTestCase(unittest.TestCase):
        c = test.read(outfile, 'r')
        assert c == "act.py: one two\n", c
 
-       cmd4 = r'%s %s %s ${sources[:2]}' % (python, act_py, outfile)
+       cmd4 = r'%s %s %s ${SOURCES[:2]}' % (python, act_py, outfile)
 
        builder = SCons.Builder.Builder(action = cmd4)
        r = builder.execute(source = ['three', 'four', 'five'])
@@ -153,7 +153,7 @@ class BuilderTestCase(unittest.TestCase):
        c = test.read(outfile, 'r')
        assert c == "act.py: three four\n", c
 
-       cmd5 = r'%s %s %s $target XYZZY' % (python, act_py, outfile)
+       cmd5 = r'%s %s %s $TARGET XYZZY' % (python, act_py, outfile)
 
        builder = SCons.Builder.Builder(action = cmd5)
        r = builder.execute(target = 'out5', env = {'ENV' : {'XYZZY' : 'xyzzy'}})
index 1c5fda53b8e3076460b2ef45bc29c2305c8c03ff..9008026250eb58e41ebf4a7c0001da4e8f4f2455 100644 (file)
@@ -71,16 +71,16 @@ if os.name == 'posix':
     ConstructionEnvironment = {
         'CC'         : 'cc',
         'CCFLAGS'    : '',
-        'CCCOM'      : '$CC $CCFLAGS -c -o $target $sources',
+        'CCCOM'      : '$CC $CCFLAGS -c -o $TARGET $SOURCES',
         'CXX'        : 'c++',
         'CXXFLAGS'   : '$CCFLAGS',
-        'CXXCOM'     : '$CXX $CXXFLAGS -c -o $target $sources',
+        'CXXCOM'     : '$CXX $CXXFLAGS -c -o $TARGET $SOURCES',
         'LINK'       : '$CXX',
         'LINKFLAGS'  : '',
-        'LINKCOM'    : '$LINK $LINKFLAGS -o $target $sources',
+        'LINKCOM'    : '$LINK $LINKFLAGS -o $TARGET $SOURCES',
         'AR'         : 'ar',
         'ARFLAGS'    : 'r',
-        'ARCOM'      : '$AR $ARFLAGS $target $sources\nranlib $target',
+        'ARCOM'      : '$AR $ARFLAGS $TARGET $SOURCES\nranlib $TARGET',
         'BUILDERS'   : [Object, Program, Library],
         'OBJPREFIX'  : '',
         'OBJSUFFIX'  : '.o',
@@ -96,16 +96,16 @@ elif os.name == 'nt':
     ConstructionEnvironment = {
         'CC'         : 'cl',
         'CCFLAGS'    : '/nologo',
-        'CCCOM'      : '$CC $CCFLAGS /c $sources /Fo$target',
+        'CCCOM'      : '$CC $CCFLAGS /c $SOURCES /Fo$TARGET',
         'CXX'        : '$CC',
         'CXXFLAGS'   : '$CCFLAGS',
-        'CXXCOM'     : '$CXX $CXXFLAGS /c $sources /Fo$target',
+        'CXXCOM'     : '$CXX $CXXFLAGS /c $SOURCES /Fo$TARGET',
         'LINK'       : 'link',
         'LINKFLAGS'  : '',
-        'LINKCOM'    : '$LINK $LINKFLAGS /out:$target $sources',
+        'LINKCOM'    : '$LINK $LINKFLAGS /out:$TARGET $SOURCES',
         'AR'         : 'lib',
         'ARFLAGS'    : '/nologo',
-        'ARCOM'      : '$AR $ARFLAGS /out:$target $sources',
+        'ARCOM'      : '$AR $ARFLAGS /out:$TARGET $SOURCES',
         'BUILDERS'   : [Object, Program, Library],
         'OBJPREFIX'  : '',
         'OBJSUFFIX'  : '.obj',
index f8829da95d87988d31bff220af5d539bd4e4beae..59ce3444e84206382e02c7490a4b389831644197 100644 (file)
@@ -77,11 +77,11 @@ class UtilTestCase(unittest.TestCase):
     def test_subst(self):
        """Test the subst function."""
        loc = {}
-        loc['targets'] = PathList(map(os.path.normpath, [ "./foo/bar.exe",
+        loc['TARGETS'] = PathList(map(os.path.normpath, [ "./foo/bar.exe",
                                                           "/bar/baz.obj",
                                                           "../foo/baz.obj" ]))
-       loc['target'] = loc['targets'][0]
-        loc['sources'] = PathList(map(os.path.normpath, [ "./foo/blah.cpp",
+        loc['TARGET'] = loc['TARGETS'][0]
+        loc['SOURCES'] = PathList(map(os.path.normpath, [ "./foo/blah.cpp",
                                                           "/bar/ack.cpp",
                                                           "../foo/ack.c" ]))
         loc['xxx'] = None
@@ -94,34 +94,34 @@ class UtilTestCase(unittest.TestCase):
                 return string.replace(str, '/', os.sep)
 
 
-        newcom = scons_subst("test $targets $sources", loc, {})
+        newcom = scons_subst("test $TARGETS $SOURCES", loc, {})
         assert newcom == cvt("test foo/bar.exe /bar/baz.obj ../foo/baz.obj foo/blah.cpp /bar/ack.cpp ../foo/ack.c")
 
-        newcom = scons_subst("test ${targets[:]} ${sources[0]}", loc, {})
+        newcom = scons_subst("test ${TARGETS[:]} ${SOURCES[0]}", loc, {})
         assert newcom == cvt("test foo/bar.exe /bar/baz.obj ../foo/baz.obj foo/blah.cpp")
 
-        newcom = scons_subst("test ${targets[1:]}v", loc, {})
+        newcom = scons_subst("test ${TARGETS[1:]}v", loc, {})
         assert newcom == cvt("test /bar/baz.obj ../foo/baz.objv")
 
-        newcom = scons_subst("test $target", loc, {})
+        newcom = scons_subst("test $TARGET", loc, {})
         assert newcom == cvt("test foo/bar.exe")
 
-        newcom = scons_subst("test $target$source[0]", loc, {})
+        newcom = scons_subst("test $TARGET$SOURCE[0]", loc, {})
         assert newcom == cvt("test foo/bar.exe[0]")
 
-        newcom = scons_subst("test ${target.file}", loc, {})
+        newcom = scons_subst("test ${TARGET.file}", loc, {})
         assert newcom == cvt("test bar.exe")
 
-        newcom = scons_subst("test ${target.filebase}", loc, {})
+        newcom = scons_subst("test ${TARGET.filebase}", loc, {})
         assert newcom == cvt("test bar")
 
-        newcom = scons_subst("test ${target.suffix}", loc, {})
+        newcom = scons_subst("test ${TARGET.suffix}", loc, {})
         assert newcom == cvt("test .exe")
 
-        newcom = scons_subst("test ${target.base}", loc, {})
+        newcom = scons_subst("test ${TARGET.base}", loc, {})
         assert newcom == cvt("test foo/bar")
 
-        newcom = scons_subst("test ${target.dir}", loc, {})
+        newcom = scons_subst("test ${TARGET.dir}", loc, {})
         assert newcom == cvt("test foo")
 
         newcom = scons_subst("test $xxx", loc, {})
index e9d74528472b513761cc82dc0ed44da2d80e353a..2b704fc1363141badaabd42486c01e0fbce8e0a8 100644 (file)
@@ -42,12 +42,12 @@ file.close()
 test.write('SConstruct', """
 env = Environment()
 env.Command(target = 'f1.out', source = 'f1.in',
-            action = r'%s build.py $target $sources')
+            action = r'%s build.py $TARGET $SOURCES')
 env.Command(target = 'f2.out', source = 'f2.in',
-            action = r'%s' + " build.py temp2 $sources\\n" + r'%s' + " build.py $target temp2")
+            action = r'%s' + " build.py temp2 $SOURCES\\n" + r'%s' + " build.py $TARGET temp2")
 env.Command(target = 'f3.out', source = 'f3.in',
-            action = [r'%s build.py temp3 $sources',
-                      r'%s build.py $target temp3'])
+            action = [r'%s build.py temp3 $SOURCES',
+                      r'%s build.py $TARGET temp3'])
 # Eventually, add ability to do execute Python code.
 """ % (python, python, python, python, python))
 
index 9155660d09dc41819937f7e4bd2acca93b1fb497..93e6be60015c94fca4acb7f845e4b693db97ca93 100644 (file)
@@ -43,7 +43,7 @@ file.close()
 """)
 
 test.write(['one', 'SConstruct'], """
-B = Builder(name = 'B', action = r'%s ../build.py $target $sources')
+B = Builder(name = 'B', action = r'%s ../build.py $TARGET $SOURCES')
 env = Environment(BUILDERS = [B])
 env.B(target = 'foo.out', source = 'foo.in')
 env.B(target = 'bar.out', source = 'bar.in')
@@ -51,7 +51,7 @@ Default('foo.out')
 """ % python)
 
 test.write(['two', 'SConstruct'], """
-B = Builder(name = 'B', action = r'%s ../build.py $target $sources')
+B = Builder(name = 'B', action = r'%s ../build.py $TARGET $SOURCES')
 env = Environment(BUILDERS = [B])
 env.B(target = 'foo.out', source = 'foo.in')
 env.B(target = 'bar.out', source = 'bar.in')
@@ -59,7 +59,7 @@ Default('foo.out', 'bar.out')
 """ % python)
 
 test.write(['three', 'SConstruct'], """
-B = Builder(name = 'B', action = r'%s ../build.py $target $sources')
+B = Builder(name = 'B', action = r'%s ../build.py $TARGET $SOURCES')
 env = Environment(BUILDERS = [B])
 env.B(target = 'foo.out', source = 'foo.in')
 env.B(target = 'bar.out', source = 'bar.in')
index 5fb04793241cfedb354680526086be4362cb916d..822573a5f795e5ce81784c2487a12453f6ed4cce 100644 (file)
@@ -43,9 +43,9 @@ file.close()
 
 test.write('SConstruct', """
 Foo = Builder(name = "Foo",
-              action = "%s build.py $target $sources subdir/foo.dep")
+              action = "%s build.py $TARGET $SOURCES subdir/foo.dep")
 Bar = Builder(name = "Bar",
-              action = "%s build.py $target $sources subdir/bar.dep")
+              action = "%s build.py $TARGET $SOURCES subdir/bar.dep")
 env = Environment(BUILDERS = [Foo, Bar])
 env.Depends(target = ['f1.out', 'f2.out'], dependency = 'subdir/foo.dep')
 env.Depends(target = 'f3.out', dependency = 'subdir/bar.dep')
index 1ff699d49a245fa1cc9410d5c2d5233794531ef2..50032df4faec7c56bac96807b32469ad6c9964d0 100644 (file)
@@ -40,7 +40,7 @@ test.write('SConstruct', """
 import os
 bin1_path = r'%s' + os.pathsep + os.environ['PATH']
 bin2_path = r'%s' + os.pathsep + os.environ['PATH']
-Bld = Builder(name = 'Bld', action = "build.py $target $sources")
+Bld = Builder(name = 'Bld', action = "build.py $TARGET $SOURCES")
 bin1 = Environment(ENV = {'PATH' : bin1_path}, BUILDERS = [Bld])
 bin2 = Environment(ENV = {'PATH' : bin2_path}, BUILDERS = [Bld])
 bin1.Bld(target = 'bin1.out', source = 'input')
index 7e832cabc10bf322e8b64f6f5fca6aedd019ee11..3c50f3dcb5dd8f1a21af51dae7cfb7840d8f2fcc 100644 (file)
@@ -31,7 +31,7 @@ test = TestSCons.TestSCons()
 #XXX Need to switch TestBld to Program() when LIBS variable is working.
 test.write('SConstruct', """
 TestBld = Builder(name='TestBld',
-                  action='cc -o $target $sources -L./ -lfoo1 -lfoo2 -lfoo3')
+                  action='cc -o $TARGET $SOURCES -L./ -lfoo1 -lfoo2 -lfoo3')
 env = Environment(BUILDERS=[ TestBld, Library ])
 env.Library(target = 'foo1', source = 'f1.c')
 env.Library(target = 'foo2', source = 'f2a.c f2b.c f2c.c')
index 769215a9f0135b60b6c6e1e62cb6347abf7578a1..c76568e9fe1dc092d6f5b15a0611a6257b415c1e 100644 (file)
@@ -46,8 +46,8 @@ sys.exit(exitval)
 """)
 
 test.write(['one', 'SConstruct'], """
-B0 = Builder(name = 'B0', action = r'%s ../build.py 0 $target $sources')
-B1 = Builder(name = 'B1', action = r'%s ../build.py 1 $target $sources')
+B0 = Builder(name = 'B0', action = r'%s ../build.py 0 $TARGET $SOURCES')
+B1 = Builder(name = 'B1', action = r'%s ../build.py 1 $TARGET $SOURCES')
 env = Environment(BUILDERS = [B0, B1])
 env.B1(target = 'f1.out', source = 'f1.in')
 env.B0(target = 'f2.out', source = 'f2.in')
@@ -66,8 +66,8 @@ test.fail_test(os.path.exists(test.workpath('f2.out')))
 test.fail_test(os.path.exists(test.workpath('f3.out')))
 
 test.write(['two', 'SConstruct'], """
-B0 = Builder(name = 'B0', action = r'%s ../build.py 0 $target $sources')
-B1 = Builder(name = 'B1', action = r'%s ../build.py 1 $target $sources')
+B0 = Builder(name = 'B0', action = r'%s ../build.py 0 $TARGET $SOURCES')
+B1 = Builder(name = 'B1', action = r'%s ../build.py 1 $TARGET $SOURCES')
 env = Environment(BUILDERS = [B0, B1])
 env.B0(target = 'f1.out', source = 'f1.in')
 env.B1(target = 'f2.out', source = 'f2.in')
@@ -86,8 +86,8 @@ test.fail_test(os.path.exists(test.workpath('f2.out')))
 test.fail_test(os.path.exists(test.workpath('f3.out')))
 
 test.write(['three', 'SConstruct'], """
-B0 = Builder(name = 'B0', action = r'%s ../build.py 0 $target $sources')
-B1 = Builder(name = 'B1', action = r'%s ../build.py 1 $target $sources')
+B0 = Builder(name = 'B0', action = r'%s ../build.py 0 $TARGET $SOURCES')
+B1 = Builder(name = 'B1', action = r'%s ../build.py 1 $TARGET $SOURCES')
 env = Environment(BUILDERS = [B0, B1])
 env.B0(target = 'f1.out', source = 'f1.in')
 env.B0(target = 'f2.out', source = 'f2.in')
index 7560d5594ddf61ce43dd7cd8d263f57ca9feba79..9b7ee616681f591593fd0d8e5c3dfa7a85e5b137 100644 (file)
@@ -42,9 +42,9 @@ sys.exit(0)
 """)
 
 test.write('SConstruct', """
-B1 = Builder(name = 'B1', action = [r'%s build.py .temp $sources',
-                                    r'%s build.py $targets .temp'])
-B2 = Builder(name = 'B2', action = r'%s' + " build.py .temp $sources\\n" + r'%s' + " build.py $targets .temp")
+B1 = Builder(name = 'B1', action = [r'%s build.py .temp $SOURCES',
+                                    r'%s build.py $TARGETS .temp'])
+B2 = Builder(name = 'B2', action = r'%s' + " build.py .temp $SOURCES\\n" + r'%s' + " build.py $TARGETS .temp")
 env = Environment(BUILDERS = [B1, B2])
 env.B1(target = 'foo1.out', source = 'foo1.in')
 env.B2(target = 'foo2.out', source = 'foo2.in')
index 5370488bbf6fa89588d5de208c7e6a691edf5fb4..ada5f58eb67b08b52458a7e64ee16daa140cc569 100644 (file)
@@ -42,7 +42,7 @@ file.close()
 
 test.write('SConstruct', """
 MyBuild = Builder(name = "MyBuild",
-                  action = r'%s build.py $targets')
+                  action = r'%s build.py $TARGETS')
 env = Environment(BUILDERS = [MyBuild])
 env.MyBuild(target = '-f1.out', source = 'f1.in')
 env.MyBuild(target = '-f2.out', source = 'f2.in')
index b188d537dc37b336a5909d08498eeaa040199476..105db49523b48a8cddc1cf4072baad1ba0fda765 100644 (file)
@@ -41,7 +41,7 @@ file.close()
 """)
 
 test.write('SConstruct', """
-B = Builder(name = 'B', action = r'%s build.py $targets $sources')
+B = Builder(name = 'B', action = r'%s build.py $TARGETS $SOURCES')
 env = Environment(BUILDERS = [B])
 env.B(target = 'foo1.out', source = 'foo1.in')
 env.B(target = 'foo2.out', source = 'foo2.in')
index 89915504cd99a80042414d6890318e2a92db0bf6..d572be6a923a6fdd9855109c9248ea2274848ffc 100644 (file)
@@ -46,8 +46,8 @@ sys.exit(1)
 """)
 
 test.write('SConstruct', """
-Succeed = Builder(name = "Succeed", action = r'%s succeed.py $targets')
-Fail = Builder(name = "Fail", action = r'%s fail.py $targets')
+Succeed = Builder(name = "Succeed", action = r'%s succeed.py $TARGETS')
+Fail = Builder(name = "Fail", action = r'%s fail.py $TARGETS')
 env = Environment(BUILDERS = [Succeed, Fail])
 env.Fail(target = 'aaa.1', source = 'aaa.in')
 env.Succeed(target = 'aaa.out', source = 'aaa.1')
index 3eb0b12d95a7216f5e3e11ebdb4316937ae16ebf..168f2b51d42bbaa5ecc31fd9c1daca6085ebe8df 100644 (file)
@@ -53,7 +53,7 @@ file.close()
 
 test.write('SConstruct', """
 MyBuild = Builder(name = "MyBuild",
-                  action = r'%s build.py $targets')
+                  action = r'%s build.py $TARGETS')
 env = Environment(BUILDERS = [MyBuild])
 env.MyBuild(target = 'f1', source = 'f1.in')
 env.MyBuild(target = 'f2', source = 'f2.in')
index 6b9e89abddb7d9c5b205891e1dd9b4a4b7175191..83e4020e031848de7b6b8be9053595a9e64b8ba5 100644 (file)
@@ -46,8 +46,8 @@ sys.exit(1)
 """)
 
 test.write('SConstruct', """
-Succeed = Builder(name = "Succeed", action = r'%s succeed.py $targets')
-Fail = Builder(name = "Fail", action = r'%s fail.py $targets')
+Succeed = Builder(name = "Succeed", action = r'%s succeed.py $TARGETS')
+Fail = Builder(name = "Fail", action = r'%s fail.py $TARGETS')
 env = Environment(BUILDERS = [Succeed, Fail])
 env.Fail(target = 'aaa.1', source = 'aaa.in')
 env.Succeed(target = 'aaa.out', source = 'aaa.1')
index 0a5a225ca4323e3a9802288f4ebacb3c8bc84268..23c6bd7e2c2863a148cedf55d1c72363c86b44a6 100644 (file)
@@ -42,7 +42,7 @@ file.close()
 
 test.write('SConstruct', """
 MyBuild = Builder(name = "MyBuild",
-                  action = r'%s build.py $targets')
+                  action = r'%s build.py $TARGETS')
 env = Environment(BUILDERS = [MyBuild])
 env.MyBuild(target = 'f1.out', source = 'f1.in')
 env.MyBuild(target = 'f2.out', source = 'f2.in')
index 666060ee5d34cdad3dc84d63f660a0469f9263f0..b34a02ee51d3738f6ef8d8f51a0c9dce0c5eef92 100644 (file)
@@ -42,7 +42,7 @@ file.close()
 
 test.write('SConstruct', """
 MyBuild = Builder(name = "MyBuild",
-                  action = r'%s build.py $target')
+                  action = r'%s build.py $TARGET')
 env = Environment(BUILDERS = [MyBuild])
 env.MyBuild(target = 'f1.out', source = 'f1.in')
 env.MyBuild(target = 'f2.out', source = 'f2.in')
index 7071b54a57b544becf9290ddac0de53b6703f0bc..21611959b0c546b2176a39d6a7c90fb10784952f 100644 (file)
@@ -40,7 +40,7 @@ file.close()
 """)
 
 test.write('SConstruct', """
-B = Builder(name = "B", action = "python build.py $targets $sources")
+B = Builder(name = "B", action = "python build.py $TARGETS $SOURCES")
 env = Environment(BUILDERS = [B])
 env.B(target = 'subdir/f1.out', source = 'subdir/f1.in')
 env.B(target = 'subdir/f2.out', source = 'subdir/f2.in')
index dd7d86af060b31b20d9a093698b9439c5d5a24bd..7620409f6e208b20fc06dd7b42bd32043005e06c 100644 (file)
@@ -42,7 +42,7 @@ file.close()
 """)
 
 test.write('SConstruct', """
-B = Builder(name = "B", action = r'%s build.py $targets $sources')
+B = Builder(name = "B", action = r'%s build.py $TARGETS $SOURCES')
 env = Environment(BUILDERS = [B])
 env.B(target = 'f1.out', source = 'f1.in')
 env.B(target = 'f2.out', source = 'f2.in')