Fix tests which break when using print() and no from __future__ import print_function...
authorWilliam Deegan <bill@baddogconsulting.com>
Mon, 25 Sep 2017 22:19:04 +0000 (15:19 -0700)
committerWilliam Deegan <bill@baddogconsulting.com>
Mon, 25 Sep 2017 22:19:04 +0000 (15:19 -0700)
17 files changed:
test/Deprecated/Options/chdir.py
test/Deprecated/Options/import.py
test/ExecuteInvalidateCache.py
test/SConscript/Return.py
test/SConscript/SConscript.py
test/SConscript/env.py
test/SConstruct.py
test/Scanner/empty-implicit.py
test/Scanner/scan-once.py
test/Variables/chdir.py
test/option--C.py
test/option-f.py
test/overrides.py
test/tool_args.py
test/toolpath/basic.py
test/toolpath/nested/image/SConstruct
test/toolpath/relative_import/image/SConstruct

index ed0aab11b73478509db29fe701ab027344bb7d1f..bfeda1b25c3c251553b6ef1b803c9bf46d181122 100644 (file)
@@ -46,7 +46,7 @@ SConscript_contents = """\
 Import("opts")
 env = Environment()
 opts.Update(env)
-print("VARIABLE =", repr(env['VARIABLE']))
+print("VARIABLE = %s"%repr(env['VARIABLE']))
 """
 
 test.write(['bin', 'opts.cfg'], """\
index b93e6a2614ab0fd356f395aab37f647c9ef97ecc..764a0611a8b3dd92e5ffc59067180f183b0a8dee 100644 (file)
@@ -48,7 +48,7 @@ SConscript_contents = """\
 Import("opts")
 env = Environment()
 opts.Update(env)
-print("VARIABLE =", env.get('VARIABLE'))
+print("VARIABLE = %s"%env.get('VARIABLE'))
 """
 
 test.write(['bin', 'opts.cfg'], """\
index 6d8063e83e9df6dd62dd522136f14e682792147f..acc7701e3f2011502ab6e622a6f4a18e9edfa742 100644 (file)
@@ -40,9 +40,9 @@ subfn = os.path.join('sub', 'foo')
 test.write('SConstruct', """\
 def exists(node):
     if node.exists():
-        print(str(node), "exists")
+        print(str(node)+" exists")
     else:
-        print(str(node), "does not exist")
+        print(str(node)+" does not exist")
 
 Execute(Delete('abc'))
 n1 = File('abc')
index 1f678afc94c841a5b3ab69d747d7c0fc60d270eb..dcbedee2a1bca49a51797378e9df2a9b846512f3 100644 (file)
@@ -39,13 +39,8 @@ x = SConscript('SConscript2')
 y, z = SConscript('SConscript3')
 a4, b4 = SConscript('SConscript4')
 foo, bar = SConscript('SConscript5')
-print("x =", x)
-print("y =", y)
-print("z =", z)
-print("a4 =", a4)
-print("b4 =", b4)
-print("foo =", foo)
-print("bar =", bar)
+for var in ['x','y','z','a4','b4','foo','bar']:
+    print("%s = %s"%(var,globals()[var]))
 """)
 
 test.write('SConscript1', """\
index 2eeb2112d95234ecd2f3f85eea3370de8ddca60b..36288be50a0e3ee336b296146717dbe74b0ee3d6 100644 (file)
@@ -37,7 +37,7 @@ import foo
 
 assert foo.foo == 4
 
-print("SConstruct", os.getcwd())
+print("SConstruct "+ os.getcwd())
 SConscript('SConscript')
 
 x1 = "SConstruct x1"
index a16866c923166237d1c4408dff8e9108cf934ba8..813d7177346c6ef64166ea32999c87776f2b7a57 100644 (file)
@@ -50,14 +50,14 @@ test.write(['sub1', 'SConscript'], """\
 env = Environment()
 env.Import("x")
 print("sub1/SConscript")
-print("x =", x)
+print("x = %s"%x)
 """)
 
 test.write(['sub2', 'SConscript'], """\
 env = Environment()
 env.Import("y")
 print("sub2/SConscript")
-print("y =", y)
+print("y = %s"%y)
 """)
 
 test.write('s1', "\n")
index 9b2807fa496d27d28adc8f1a1d3aec529df48be1..cebdf04c0e08941c9536304854a4bf92da606381 100644 (file)
@@ -41,7 +41,7 @@ wpath = test.workpath()
 
 test.write('sconstruct', """
 import os
-print("sconstruct", os.getcwd())
+print("sconstruct "+os.getcwd())
 """)
 
 test.run(arguments = ".",
@@ -51,7 +51,7 @@ test.run(arguments = ".",
 
 test.write('Sconstruct', """
 import os
-print("Sconstruct", os.getcwd())
+print("Sconstruct "+os.getcwd())
 """)
 
 test.run(arguments = ".",
@@ -60,7 +60,7 @@ test.run(arguments = ".",
 
 test.write('SConstruct', """
 import os
-print("SConstruct", os.getcwd())
+print("SConstruct "+os.getcwd())
 """)
 
 test.run(arguments = ".",
index 45fe9526634c910e51970591e4a52251685ccf65..8d9a47da29128509cd23ecee3a194dc7aca0da84 100644 (file)
@@ -37,7 +37,7 @@ test.write('SConstruct', r"""
 import os.path
 
 def scan(node, env, envkey, arg):
-    print('XScanner: node =', os.path.split(str(node))[1])
+    print('XScanner: node = '+os.path.split(str(node))[1])
     return []
 
 def exists_check(node, env):
index eac1f49aa1c4d7cc7366d207273551adee28ae14..bbe2594c2d5f089075ada8943ef597bb816d441c 100644 (file)
@@ -36,7 +36,7 @@ test.write('SConstruct', r"""
 import os.path
 
 def scan(node, env, envkey, arg):
-    print('XScanner: node =', os.path.split(str(node))[1])
+    print('XScanner: node = '+ os.path.split(str(node))[1])
     return []
 
 def exists_check(node, env):
index 0e9abaa9d3ea978aad02fe3e2f7f120c2b3e8ccf..b5fce2adf6c94c7e55abc74cd82bc61941a1ae79 100644 (file)
@@ -46,7 +46,7 @@ SConscript_contents = """\
 Import("opts")
 env = Environment()
 opts.Update(env)
-print("VARIABLE =", repr(env['VARIABLE']))
+print("VARIABLE = "+repr(env['VARIABLE']))
 """
 
 test.write(['bin', 'opts.cfg'], """\
index 1df3c1a93a281b2db0282d68f4d184564bc4acbe..754d4db1686a1e48550ff9e3886b26a0b3ab9257 100644 (file)
@@ -51,7 +51,7 @@ test.subdir('sub', ['sub', 'dir'])
 
 test.write('SConstruct', """
 import os
-print("SConstruct", os.getcwd())
+print("SConstruct "+os.getcwd())
 """)
 
 test.write(['sub', 'SConstruct'], """
index 9fd1a57e1a6edd049fe79abd2d2e590ced9ee6c4..21afacb329afafa264ab6ed9528540bc26c04168 100644 (file)
@@ -41,12 +41,12 @@ print("SConscript " + os.getcwd())
 
 test.write(subdir_BuildThis, """
 import os
-print("subdir/BuildThis", os.getcwd())
+print("subdir/BuildThis "+ os.getcwd())
 """)
 
 test.write('Build2', """
 import os
-print("Build2", os.getcwd())
+print("Build2 "+ os.getcwd())
 """)
 
 wpath = test.workpath()
index 343cea479dc695148aee90f5503601d82c9a5338..cca8dc7c533313559ae78275d08969b7320583bc 100644 (file)
@@ -35,9 +35,9 @@ _python_ = TestSCons._python_
 test.write('SConstruct', """
 env = Environment(CCFLAGS='-DFOO', LIBS=['a'])
 def build(target, source, env):
-    print("env['CC'] =", env['CC'])
-    print("env['CCFLAGS'] =", env['CCFLAGS'])
-    print("env['LIBS'] =", env['LIBS'])
+    print("env['CC'] = "+env['CC'])
+    print("env['CCFLAGS'] = "+env['CCFLAGS'])
+    print("env['LIBS'] = "+str(env['LIBS']))
 builder = Builder(action=build, CC='buildcc', LIBS='buildlibs')
 env['BUILDERS']['Build'] = builder
 
index b0f69efea003dfa8b8a13ebb137487d6e9308e71..698370539b1354801a68422c34578220bb422f04 100644 (file)
@@ -38,13 +38,13 @@ test = TestSCons.TestSCons()
 test.write('SConstruct', """
 # Test passing kw args to Tool constructor
 env1 = Environment(tools=[Tool('FooTool', toolpath=['.'], kw1='kw1val')])
-print("env1['TOOL_FOO'] =", env1.get('TOOL_FOO'))
-print("env1['kw1'] =", env1.get('kw1'))
+print("env1['TOOL_FOO'] = "+str(env1.get('TOOL_FOO')))
+print("env1['kw1'] = "+env1.get('kw1'))
 
 # Test apply_tools taking a list of (name, kwargs_dict)
 env2 = Environment(tools=[('FooTool', {'kw2':'kw2val'})], toolpath=['.'])
-print("env2['TOOL_FOO'] =", env2.get('TOOL_FOO'))
-print("env2['kw2'] =", env2.get('kw2'))
+print("env2['TOOL_FOO'] = "+str(env2.get('TOOL_FOO')))
+print("env2['kw2'] = "+env2.get('kw2'))
 
 """)
 
index 86dafca8cafab51fae0dcf2af81fec53116a2a83..793497b28674a0452000933133e85446557c47e5 100644 (file)
@@ -35,65 +35,65 @@ def foo(env):
     env['TOOL_FOO'] = 1
 
 env1 = Environment(tools=[foo, 'bar'], toolpath=['tools'])
-print("env1['TOOL_FOO'] =", env1.get('TOOL_FOO'))
-print("env1['TOOL_BAR'] =", env1.get('TOOL_BAR'))
+print("env1['TOOL_FOO'] = %s"%env1.get('TOOL_FOO'))
+print("env1['TOOL_BAR'] = %s"%env1.get('TOOL_BAR'))
 
 # pick a built-in tool with pretty simple behavior
 env2 = Environment(tools=['zip'])
-print("env2['ZIP'] =", env2.get('ZIP'))
-print("env2['TOOL_zip1'] =", env2.get('TOOL_zip1'))
-print("env2['TOOLDIR_zip'] =", env2.get('TOOLDIR_zip'))
+print("env2['ZIP'] = %s"%env2.get('ZIP'))
+print("env2['TOOL_zip1'] = %s"%env2.get('TOOL_zip1'))
+print("env2['TOOLDIR_zip'] = %s"%env2.get('TOOLDIR_zip'))
 
 # Only find tools in current dir, or Scons.Tool.TOOLNAME
 env3 = Environment(tools=['zip'], toolpath=['.'])
-print("env3['ZIP'] =", env3.get('ZIP'))
-print("env3['TOOL_zip1'] =", env3.get('TOOL_zip1'))
-print("env3['TOOLDIR_zip'] =", env3.get('TOOLDIR_zip'))
+print("env3['ZIP'] = %s"%env3.get('ZIP'))
+print("env3['TOOL_zip1'] = %s"%env3.get('TOOL_zip1'))
+print("env3['TOOLDIR_zip'] = %s"%env3.get('TOOLDIR_zip'))
 
 env4 = Environment(tools=['zip'], toolpath=['tools'])
-print("env4['ZIP'] =", env4.get('ZIP'))
-print("env4['TOOL_zip1'] =", env4.get('TOOL_zip1'))
-print("env4['TOOLDIR_zip'] =", env4.get('TOOLDIR_zip'))
+print("env4['ZIP'] = %s"%env4.get('ZIP'))
+print("env4['TOOL_zip1'] = %s"%env4.get('TOOL_zip1'))
+print("env4['TOOLDIR_zip'] = %s"%env4.get('TOOLDIR_zip'))
 
 # Should pick up from tools dir, and then current dir
 env5 = Environment(tools=['zip'], toolpath=['tools', '.'])
-print("env5['ZIP'] =", env5.get('ZIP'))
-print("env5['TOOL_zip1'] =", env5.get('TOOL_zip1'))
-print("env5['TOOLDIR_zip'] =", env5.get('TOOLDIR_zip'))
+print("env5['ZIP'] = %s"%env5.get('ZIP'))
+print("env5['TOOL_zip1'] = %s"%env5.get('TOOL_zip1'))
+print("env5['TOOLDIR_zip'] = %s"%env5.get('TOOLDIR_zip'))
 
 
 # Should pick up from current dir, and then tools dir
 env6 = Environment(tools=['zip'], toolpath=['.', 'tools'])
-print("env6['ZIP'] =", env6.get('ZIP'))
-print("env6['TOOL_zip1'] =", env6.get('TOOL_zip1'))
-print("env6['TOOLDIR_zip'] =", env6.get('TOOLDIR_zip'))
+print("env6['ZIP'] = %s"%env6.get('ZIP'))
+print("env6['TOOL_zip1'] = %s"%env6.get('TOOL_zip1'))
+print("env6['TOOLDIR_zip'] = %s"%env6.get('TOOLDIR_zip'))
 
 env7 = Environment(TOOLPATH="tools", tools=['zip'], toolpath=['$TOOLPATH'])
-print("env7['ZIP'] =", env7.get('ZIP'))
-print("env7['TOOL_zip1'] =", env7.get('TOOL_zip1'))
-print("env7['TOOLDIR_zip'] =", env7.get('TOOLDIR_zip'))
+print("env7['ZIP'] = %s"%env7.get('ZIP'))
+print("env7['TOOL_zip1'] = %s"%env7.get('TOOL_zip1'))
+print("env7['TOOLDIR_zip'] = %s"%env7.get('TOOLDIR_zip'))
 
 env8 = Environment(tools=[])
 env8.Tool('zip', toolpath=['tools'])
-print("env8['ZIP'] =", env8.get('ZIP'))
-print("env8['TOOL_zip1'] =", env8.get('TOOL_zip1'))
-print("env8['TOOLDIR_zip'] =", env8.get('TOOLDIR_zip'))
+print("env8['ZIP'] = %s"%env8.get('ZIP'))
+print("env8['TOOL_zip1'] = %s"%env8.get('TOOL_zip1'))
+print("env8['TOOLDIR_zip'] = %s"%env8.get('TOOLDIR_zip'))
 
 env9 = Environment(tools=[])
 Tool('zip', toolpath=['tools'])(env9)
-print("env9['ZIP'] =", env9.get('ZIP'))
-print("env9['TOOL_zip1'] =", env9.get('TOOL_zip1'))
-print("env9['TOOLDIR_zip'] =", env9.get('TOOLDIR_zip'))
+print("env9['ZIP'] = %s"%env9.get('ZIP'))
+print("env9['TOOL_zip1'] = %s"%env9.get('TOOL_zip1'))
+print("env9['TOOLDIR_zip'] = %s"%env9.get('TOOLDIR_zip'))
 
 env0 = Environment(TOOLPATH='tools', tools=[])
 env0.Tool('zip', toolpath=['$TOOLPATH'])
-print("env0['ZIP'] =", env0.get('ZIP'))
-print("env0['TOOL_zip1'] =", env0.get('TOOL_zip1'))
-print("env0['TOOLDIR_zip'] =", env0.get('TOOLDIR_zip'))
+print("env0['ZIP'] = %s"%env0.get('ZIP'))
+print("env0['TOOL_zip1'] = %s"%env0.get('TOOL_zip1'))
+print("env0['TOOLDIR_zip'] = %s"%env0.get('TOOLDIR_zip'))
 
 base = Environment(tools=[], toolpath=['tools'])
 derived = base.Clone(tools=['bar'])
-print("derived['TOOL_BAR'] =", derived.get('TOOL_BAR'))
+print("derived['TOOL_BAR'] = %s"%derived.get('TOOL_BAR'))
 """)
 
 test.write('zip.py', r"""
index a7c6ceb7d21796cbaa24a26d47e17ce48c90cb7e..1187a7c288f78f33568bf65c33530d47ef0b8702 100644 (file)
@@ -10,26 +10,26 @@ toollist = ['Toolpath_TestTool1',
 \r
 print('Test where tools are located under site_scons/site_tools')\r
 env1 = Environment(tools=toollist)\r
-print("env1['Toolpath_TestTool1'] =", env1.get('Toolpath_TestTool1'))\r
-print("env1['Toolpath_TestTool2'] =", env1.get('Toolpath_TestTool2'))\r
-print("env1['Toolpath_TestTool1_1'] =", env1.get('Toolpath_TestTool1_1'))\r
-print("env1['Toolpath_TestTool1_2'] =", env1.get('Toolpath_TestTool1_2'))\r
-print("env1['Toolpath_TestTool2_1'] =", env1.get('Toolpath_TestTool2_1'))\r
-print("env1['Toolpath_TestTool2_2'] =", env1.get('Toolpath_TestTool2_2'))\r
+print("env1['Toolpath_TestTool1'] = %s"%env1.get('Toolpath_TestTool1'))\r
+print("env1['Toolpath_TestTool2'] = %s"%env1.get('Toolpath_TestTool2'))\r
+print("env1['Toolpath_TestTool1_1'] = %s"%env1.get('Toolpath_TestTool1_1'))\r
+print("env1['Toolpath_TestTool1_2'] = %s"%env1.get('Toolpath_TestTool1_2'))\r
+print("env1['Toolpath_TestTool2_1'] = %s"%env1.get('Toolpath_TestTool2_1'))\r
+print("env1['Toolpath_TestTool2_2'] = %s"%env1.get('Toolpath_TestTool2_2'))\r
 \r
 print('Test where toolpath is set in the env constructor')\r
 env2 = Environment(tools=toollist, toolpath=['Libs/tools_example'])\r
-print("env2['Toolpath_TestTool1'] =", env2.get('Toolpath_TestTool1'))\r
-print("env2['Toolpath_TestTool2'] =", env2.get('Toolpath_TestTool2'))\r
-print("env2['Toolpath_TestTool1_1'] =", env2.get('Toolpath_TestTool1_1'))\r
-print("env2['Toolpath_TestTool1_2'] =", env2.get('Toolpath_TestTool1_2'))\r
-print("env2['Toolpath_TestTool2_1'] =", env2.get('Toolpath_TestTool2_1'))\r
-print("env2['Toolpath_TestTool2_2'] =", env2.get('Toolpath_TestTool2_2'))\r
+print("env2['Toolpath_TestTool1'] = %s"%env2.get('Toolpath_TestTool1'))\r
+print("env2['Toolpath_TestTool2'] = %s"%env2.get('Toolpath_TestTool2'))\r
+print("env2['Toolpath_TestTool1_1'] = %s"%env2.get('Toolpath_TestTool1_1'))\r
+print("env2['Toolpath_TestTool1_2'] = %s"%env2.get('Toolpath_TestTool1_2'))\r
+print("env2['Toolpath_TestTool2_1'] = %s"%env2.get('Toolpath_TestTool2_1'))\r
+print("env2['Toolpath_TestTool2_2'] = %s"%env2.get('Toolpath_TestTool2_2'))\r
 \r
 print('Test a Clone')\r
 base = Environment(tools=[], toolpath=['Libs/tools_example'])\r
 derived = base.Clone(tools=['subdir1.Toolpath_TestTool1_1'])\r
-print("derived['Toolpath_TestTool1_1'] =", derived.get('Toolpath_TestTool1_1'))\r
+print("derived['Toolpath_TestTool1_1'] = %s"%derived.get('Toolpath_TestTool1_1'))\r
 \r
 \r
 print('Test using syspath as the toolpath')\r
@@ -52,18 +52,18 @@ toollist = ['tools_example.Toolpath_TestTool1',
            ]\r
 \r
 env3 = Environment(tools=toollist, toolpath=searchpaths)\r
-print("env3['Toolpath_TestTool1'] =", env3.get('Toolpath_TestTool1'))\r
-print("env3['Toolpath_TestTool2'] =", env3.get('Toolpath_TestTool2'))\r
-print("env3['Toolpath_TestTool1_1'] =", env3.get('Toolpath_TestTool1_1'))\r
-print("env3['Toolpath_TestTool1_2'] =", env3.get('Toolpath_TestTool1_2'))\r
-print("env3['Toolpath_TestTool2_1'] =", env3.get('Toolpath_TestTool2_1'))\r
-print("env3['Toolpath_TestTool2_2'] =", env3.get('Toolpath_TestTool2_2'))\r
+print("env3['Toolpath_TestTool1'] = %s"%env3.get('Toolpath_TestTool1'))\r
+print("env3['Toolpath_TestTool2'] = %s"%env3.get('Toolpath_TestTool2'))\r
+print("env3['Toolpath_TestTool1_1'] = %s"%env3.get('Toolpath_TestTool1_1'))\r
+print("env3['Toolpath_TestTool1_2'] = %s"%env3.get('Toolpath_TestTool1_2'))\r
+print("env3['Toolpath_TestTool2_1'] = %s"%env3.get('Toolpath_TestTool2_1'))\r
+print("env3['Toolpath_TestTool2_2'] = %s"%env3.get('Toolpath_TestTool2_2'))\r
 \r
 \r
 print('Test using PyPackageDir')\r
 toollist = ['Toolpath_TestTool2_1', 'Toolpath_TestTool2_2']\r
 env4 = Environment(tools = toollist, toolpath = [PyPackageDir('tools_example.subdir1.subdir2')])\r
-print("env4['Toolpath_TestTool2_1'] =", env4.get('Toolpath_TestTool2_1'))\r
-print("env4['Toolpath_TestTool2_2'] =", env4.get('Toolpath_TestTool2_2'))\r
+print("env4['Toolpath_TestTool2_1'] = %s"%env4.get('Toolpath_TestTool2_1'))\r
+print("env4['Toolpath_TestTool2_2'] = %s"%env4.get('Toolpath_TestTool2_2'))\r
 \r
 sys.path = oldsyspath\r
index 6156929b52ca11ee4a4834d20ce797c41506de9c..c9b28bb1de121ad70190b329f119642950b49b58 100644 (file)
@@ -1,10 +1,10 @@
 env = Environment(tools=['TestTool1', 'TestTool1.TestTool1_2'], toolpath=['tools'])\r
 \r
 # Test a relative import within the root of the tools directory\r
-print("env['TestTool1'] =", env.get('TestTool1'))\r
-print("env['TestTool1_1'] =", env.get('TestTool1_1'))\r
+print("env['TestTool1'] = %s"%env.get('TestTool1'))\r
+print("env['TestTool1_1'] = %s"%env.get('TestTool1_1'))\r
 \r
 # Test a relative import within a sub dir\r
-print("env['TestTool1_2'] =", env.get('TestTool1_2'))\r
-print("env['TestTool1_2_1'] =", env.get('TestTool1_2_1'))\r
-print("env['TestTool1_2_2'] =", env.get('TestTool1_2_2'))\r
+print("env['TestTool1_2'] = %s"%env.get('TestTool1_2'))\r
+print("env['TestTool1_2_1'] = %s"%env.get('TestTool1_2_1'))\r
+print("env['TestTool1_2_2'] = %s"%env.get('TestTool1_2_2'))\r