Skip to content
Permalink
Browse files

[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

llvm-svn: 367123
  • Loading branch information...
mgorny committed Jul 26, 2019
1 parent ffc722a commit 40a10446c080c88176d2c6766736932cd1a48afa
Showing with 1 addition and 1 deletion.
  1. +1 −1 llvm/utils/lit/tests/lit.cfg
@@ -53,7 +53,7 @@ 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",
env="env FILECHECK_OPTS= FILECHECK_DUMP_INPUT_ON_FAILURE=",
lit=os.path.join(lit_path, 'lit.py'))))
config.substitutions.append(('%{python}', '"%s"' % (sys.executable)))

0 comments on commit 40a1044

Please sign in to comment.
You can’t perform that action at this time.