Revert r366980: "[lit] Protect full test suite from FILECHECK_OPTS"
authorJoel E. Denny <jdenny.ornl@gmail.com>
Thu, 1 Aug 2019 22:26:51 +0000 (22:26 +0000)
committerJoel E. Denny <jdenny.ornl@gmail.com>
Thu, 1 Aug 2019 22:26:51 +0000 (22:26 +0000)
Windows bots are broken.  See recent D65335 and D65156 comments.

llvm-svn: 367627

llvm/utils/lit/tests/lit.cfg
llvm/utils/lit/tests/shtest-run-at-line.py

index 2830956..ebdcb50 100644 (file)
@@ -44,17 +44,10 @@ for attribute in ('llvm_tools_dir', 'lit_tools_dir'):
     if directory:
         llvm_config.with_environment('PATH', directory, append_path=True)
 
-# This test suite calls %{lit} to test lit's behavior for the sample test
-# suites in %{inputs}.  This test suite's results are then determined in part
-# by %{lit}'s textual output, which includes the output of FileCheck calls
-# within %{inputs}'s test suites.  Thus, %{lit} clears environment variables
-# that can affect FileCheck's output.
 config.substitutions.append(('%{inputs}', os.path.join(
     config.test_source_root, 'Inputs')))
-config.substitutions.append(('%{lit}',
-    "{env} %{{python}} {lit}".format(
-        env="env -u FILECHECK_OPTS -u FILECHECK_DUMP_INPUT_ON_FAILURE",
-        lit=os.path.join(lit_path, 'lit.py'))))
+config.substitutions.append(('%{lit}', "%%{python} %s" % (
+    os.path.join(lit_path, 'lit.py'),)))
 config.substitutions.append(('%{python}', '"%s"' % (sys.executable)))
 
 # Enable coverage.py reporting, assuming the coverage module has been installed
index cd0e081..7e5d53b 100644 (file)
@@ -1,7 +1,7 @@
 # Check that -vv makes the line number of the failing RUN command clear.
 # (-v is actually sufficient in the case of the internal shell.)
 #
-# RUN: not %{lit} -j 1 -vv %{inputs}/shtest-run-at-line > %t.out
+# RUN: env -u FILECHECK_OPTS not %{lit} -j 1 -vv %{inputs}/shtest-run-at-line > %t.out
 # RUN: FileCheck --input-file %t.out %s
 #
 # END.