diff options
author | David Greene <greened@obbligato.org> | 2019-01-07 16:24:37 +0000 |
---|---|---|
committer | David Greene <greened@obbligato.org> | 2019-01-07 16:24:37 +0000 |
commit | 6f1394ff86e779ed959d7d73dd47fafef589900a (patch) | |
tree | 24c4a395227395615ac10b9ab7aaf49ed0b17437 /llvm/utils/lit/tests | |
parent | 4d4213f69449f1f6c1a1890746b330727b57cad9 (diff) | |
download | bcm5719-llvm-6f1394ff86e779ed959d7d73dd47fafef589900a.tar.gz bcm5719-llvm-6f1394ff86e779ed959d7d73dd47fafef589900a.zip |
[lit] Respect PYTHONPATH
If a user has PYTHONPATH set in the environment, append new entries to
it rather than blindly setting PYTHONPATH to a fixed string. This
allows tests to, for example, find psutil if it is in
PYTHONPATH. Without this change, lit will detect psutil but then
various tests will fail because PYTHONPATH has been overwritten and
psutil cannot be found.
llvm-svn: 350536
Diffstat (limited to 'llvm/utils/lit/tests')
-rw-r--r-- | llvm/utils/lit/tests/Inputs/shtest-timeout/lit.cfg | 10 | ||||
-rw-r--r-- | llvm/utils/lit/tests/lit.cfg | 10 |
2 files changed, 18 insertions, 2 deletions
diff --git a/llvm/utils/lit/tests/Inputs/shtest-timeout/lit.cfg b/llvm/utils/lit/tests/Inputs/shtest-timeout/lit.cfg index 96bf18170a8..6256f5a9911 100644 --- a/llvm/utils/lit/tests/Inputs/shtest-timeout/lit.cfg +++ b/llvm/utils/lit/tests/Inputs/shtest-timeout/lit.cfg @@ -28,5 +28,13 @@ config.test_source_root = os.path.dirname(__file__) config.test_exec_root = config.test_source_root config.target_triple = '(unused)' src_root = os.path.join(config.test_source_root, '..') -config.environment['PYTHONPATH'] = src_root + +pythonpath_list = [src_root] +# Ensure the user's PYTHONPATH is included. +if 'PYTHONPATH' in os.environ: + pythonpath_list.append(os.environ['PYTHONPATH']) +if 'PYTHONPATH' in config.environment: + pythonpath_list.append(config.environment['PYTHONPATH']) +config.environment['PYTHONPATH'] = os.pathsep.join(pythonpath_list) + config.substitutions.append(('%{python}', '"%s"' % (sys.executable))) diff --git a/llvm/utils/lit/tests/lit.cfg b/llvm/utils/lit/tests/lit.cfg index 01a3431b435..2af93d6314a 100644 --- a/llvm/utils/lit/tests/lit.cfg +++ b/llvm/utils/lit/tests/lit.cfg @@ -34,7 +34,15 @@ if llvm_src_root != None: else: lit_path = src_root -config.environment['PYTHONPATH'] = lit_path # Required because some tests import the lit module +pythonpath_list = [lit_path] # Required because some tests import the lit module + +# Ensure the user's PYTHONPATH is included. +if 'PYTHONPATH' in os.environ: + pythonpath_list.append(os.environ['PYTHONPATH']) +if 'PYTHONPATH' in config.environment: + pythonpath_list.append(config.environment['PYTHONPATH']) +config.environment['PYTHONPATH'] = os.pathsep.join(pythonpath_list) + config.substitutions.append(('%{src_root}', src_root)) config.substitutions.append(('%{inputs}', os.path.join( src_root, 'tests', 'Inputs'))) |