[Lldb-commits] [lldb] f549318 - [lldb] Set the TERM environment variable for the API tests

[Jonas Devlieghere via lldb-commits]

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[