Tweak MultiTestRunner --path argument.
- Accept multiple values instead of embedding separator.
- Make sure this gets used when searching for 'clang' binary.
- Switch makefiles to using --path to stay in sync with cmake.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@77234 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/utils/test/MultiTestRunner.py b/utils/test/MultiTestRunner.py
index 04bd74f..8b48d8d 100755
--- a/utils/test/MultiTestRunner.py
+++ b/utils/test/MultiTestRunner.py
@@ -236,7 +236,7 @@
action="store_true", default=False)
parser.add_option("", "--path", dest="path",
help="Additional paths to add to testing environment",
- action="store", type=str, default=None)
+ action="append", type=str, default=[])
(opts, args) = parser.parse_args()
@@ -246,6 +246,10 @@
parser.error('Support for running with valgrind is '
'temporarily disabled')
+ # FIXME: Move into configuration object.
+ TestRunner.kChildEnv["PATH"] = os.pathsep.join(opts.path +
+ [TestRunner.kChildEnv['PATH']])
+
if opts.clang is None:
opts.clang = TestRunner.inferClang()
if opts.clangcc is None:
@@ -266,9 +270,7 @@
random.shuffle(tests)
if opts.maxTests is not None:
tests = tests[:opts.maxTests]
- if opts.path is not None:
- os.environ["PATH"] = opts.path + ":" + os.environ["PATH"];
-
+
extra = ''
if len(tests) != len(allTests):
extra = ' of %d'%(len(allTests),)