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

Michal Gorny via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 26 08:39:05 PDT 2019


Author: mgorny
Date: Fri Jul 26 08:39:05 2019
New Revision: 367123

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

Set environment variables to empty values rather than attempting
to unset them via 'env -u', in order to fix NetBSD test regression
caused by r366980.  POSIX does not guarantee that env(1) supports '-u'
option, and indeed NetBSD env(1) does not support it.

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

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=367123&r1=367122&r2=367123&view=diff
==============================================================================
--- llvm/trunk/utils/lit/tests/lit.cfg (original)
+++ llvm/trunk/utils/lit/tests/lit.cfg Fri Jul 26 08:39:05 2019
@@ -53,7 +53,7 @@ config.substitutions.append(('%{inputs}'
     config.test_source_root, 'Inputs')))
 config.substitutions.append(('%{lit}',
     "{env} %{{python}} {lit}".format(
-        env="env -u FILECHECK_OPTS -u FILECHECK_DUMP_INPUT_ON_FAILURE",
+        env="env FILECHECK_OPTS= 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