From 1f237a892b2f01424654d2dd3a3750a995afb9f1 Mon Sep 17 00:00:00 2001 From: Jonas Devlieghere Date: Wed, 6 Apr 2022 14:42:10 -0700 Subject: [PATCH 1/2] [lldb] Disable TestPlatformSDK.py Skip TestPlatformSDK.py after 3c2ef811792a656e8dfc82418eb1ad263c2876d2 because it relies on the now disabled remote-macosx platform. --- lldb/test/API/commands/platform/sdk/TestPlatformSDK.py | 1 + 1 file changed, 1 insertion(+) diff --git a/lldb/test/API/commands/platform/sdk/TestPlatformSDK.py b/lldb/test/API/commands/platform/sdk/TestPlatformSDK.py index 8bf950a1bd3c3..cbfedcc68f026 100644 --- a/lldb/test/API/commands/platform/sdk/TestPlatformSDK.py +++ b/lldb/test/API/commands/platform/sdk/TestPlatformSDK.py @@ -39,6 +39,7 @@ def port_not_available(self): return '{} not available'.format(self.PORT) return None + @skipIf @no_debug_info_test @skipUnlessDarwin @expectedFailureIfFn(no_debugserver) From e4b9282901f10ee19148829c7fe0c215767487b8 Mon Sep 17 00:00:00 2001 From: Jonas Devlieghere Date: Mon, 21 Mar 2022 19:18:29 -0700 Subject: [PATCH 2/2] [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 71f60773601d78232d1fcf401d2f28039f28d8cb) --- lldb/test/API/lit.cfg.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/lldb/test/API/lit.cfg.py b/lldb/test/API/lit.cfg.py index 67e892ee13de6..e745766b11046 100644 --- a/lldb/test/API/lit.cfg.py +++ b/lldb/test/API/lit.cfg.py @@ -266,6 +266,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[