Skip to content

Commit e4b9282

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. (cherry picked from commit 71f6077)
1 parent 1f237a8 commit e4b9282

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
@@ -266,6 +266,9 @@ def delete_module_cache(path):
266266
# testFormat: The test format to use to interpret tests.
267267
config.test_format = lldbtest.LLDBTest(dotest_cmd)
268268

269+
# Propagate TERM or default to vt100.
270+
config.environment['TERM'] = os.getenv('TERM', default='vt100')
271+
269272
# Propagate FREEBSD_LEGACY_PLUGIN
270273
if 'FREEBSD_LEGACY_PLUGIN' in os.environ:
271274
config.environment['FREEBSD_LEGACY_PLUGIN'] = os.environ[

0 commit comments

Comments
 (0)