Author: Jonas Devlieghere Date: 2022-03-22T11:01:38-07:00 New Revision: f54931865de823fd4c3ab3650d8b8a65900433ff
URL: https://github.com/llvm/llvm-project/commit/f54931865de823fd4c3ab3650d8b8a65900433ff DIFF: https://github.com/llvm/llvm-project/commit/f54931865de823fd4c3ab3650d8b8a65900433ff.diff LOG: [lldb] Set the TERM environment variable for the API tests Avoid "TERM environment variable not set" by either propagating the TERM environment variable or defaulting to vt100. All of our CI is already doing this explicitly through the --env dotest arg, but it's easy to forget when setting up a new job. I don't see any downside in making it the default. Added: Modified: lldb/test/API/lit.cfg.py Removed: ################################################################################ diff --git a/lldb/test/API/lit.cfg.py b/lldb/test/API/lit.cfg.py index 8120c4f716052..48bb5662dc8b5 100644 --- a/lldb/test/API/lit.cfg.py +++ b/lldb/test/API/lit.cfg.py @@ -235,6 +235,9 @@ def delete_module_cache(path): # testFormat: The test format to use to interpret tests. config.test_format = lldbtest.LLDBTest(dotest_cmd) +# Propagate TERM or default to vt100. +config.environment['TERM'] = os.getenv('TERM', default='vt100') + # Propagate FREEBSD_LEGACY_PLUGIN if 'FREEBSD_LEGACY_PLUGIN' in os.environ: config.environment['FREEBSD_LEGACY_PLUGIN'] = os.environ[ _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
