Skip to content

Commit 71f6077

Browse files
committed
[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.
1 parent 60ff1e4 commit 71f6077

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

lldb/test/API/lit.cfg.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -254,6 +254,9 @@ def delete_module_cache(path):
254254
# testFormat: The test format to use to interpret tests.
255255
config.test_format = lldbtest.LLDBTest(dotest_cmd)
256256

257+
# Propagate TERM or default to vt100.
258+
config.environment['TERM'] = os.getenv('TERM', default='vt100')
259+
257260
# Propagate FREEBSD_LEGACY_PLUGIN
258261
if 'FREEBSD_LEGACY_PLUGIN' in os.environ:
259262
config.environment['FREEBSD_LEGACY_PLUGIN'] = os.environ[

0 commit comments

Comments
 (0)