[llvm] r367626 - Revert r367123: "[llvm] [lit/tests] Replace 'env -u' with more portable construct"

Joel E. Denny via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 1 15:26:37 PDT 2019


Author: jdenny
Date: Thu Aug  1 15:26:37 2019
New Revision: 367626

URL: http://llvm.org/viewvc/llvm-project?rev=367626&view=rev
Log:
Revert r367123: "[llvm] [lit/tests] Replace 'env -u' with more portable construct"

Must be reverted in order to revert r366980, which breaks windows
bots.  See recent D65335 and D65156 comments.

Modified:
    llvm/trunk/utils/lit/tests/lit.cfg

Modified: llvm/trunk/utils/lit/tests/lit.cfg
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/lit/tests/lit.cfg?rev=367626&r1=367625&r2=367626&view=diff
==============================================================================
--- llvm/trunk/utils/lit/tests/lit.cfg (original)
+++ llvm/trunk/utils/lit/tests/lit.cfg Thu Aug  1 15:26:37 2019
@@ -53,7 +53,7 @@ config.substitutions.append(('%{inputs}'
     config.test_source_root, 'Inputs')))
 config.substitutions.append(('%{lit}',
     "{env} %{{python}} {lit}".format(
-        env="env FILECHECK_OPTS= FILECHECK_DUMP_INPUT_ON_FAILURE=",
+        env="env -u FILECHECK_OPTS -u FILECHECK_DUMP_INPUT_ON_FAILURE",
         lit=os.path.join(lit_path, 'lit.py'))))
 config.substitutions.append(('%{python}', '"%s"' % (sys.executable)))
 




More information about the llvm-commits mailing list