[libcxx][lit] Fix running testsuite with python2.7 after 9020d28688492c437abb648b6ab6...
authorAlex Richardson <Alexander.Richardson@cl.cam.ac.uk>
Mon, 27 Jul 2020 09:15:17 +0000 (10:15 +0100)
committerAlex Richardson <Alexander.Richardson@cl.cam.ac.uk>
Mon, 27 Jul 2020 09:15:18 +0000 (10:15 +0100)
Python 2.7 fails with TypeError: unsupported operand type(s) for +: 'NoneType' and 'str'
if you pass None as the prefix argument to NamedTemporaryFile.

Reviewed By: ldionne, bjope, #libc

Differential Revision: https://reviews.llvm.org/D84595

libcxx/utils/libcxx/test/dsl.py

index cd500e1..95d23df 100644 (file)
@@ -52,7 +52,7 @@ def _executeScriptInternal(test, commands):
     res = ('', '', 127, None)
   return res
 
-def _makeConfigTest(config, testPrefix=None):
+def _makeConfigTest(config, testPrefix=''):
   sourceRoot = os.path.join(config.test_exec_root, '__config_src__')
   execRoot = os.path.join(config.test_exec_root, '__config_exec__')
   suite = lit.Test.TestSuite('__config__', sourceRoot, execRoot, config)
@@ -83,7 +83,7 @@ def sourceBuilds(config, source):
     _executeScriptInternal(test, ['rm %t.exe'])
     return exitCode == 0
 
-def programOutput(config, program, args=[], testPrefix=None):
+def programOutput(config, program, args=[], testPrefix=''):
   """
   Compiles a program for the test target, run it on the test target and return
   the output.