From 027499a82434ea7a784b2e5a0227540f00fbc307 Mon Sep 17 00:00:00 2001 From: Med Ismail Bennani Date: Thu, 19 May 2022 14:47:04 -0700 Subject: [PATCH] [lldb/test] Fix PExpect.launch issue when disabling color support This patch should fix a bug in PExpect.launch that happened when color support is not enabled. In that case, we need to add the `--no-use-colors` flag to lldb's launch argument list. However, previously, each character to the string was appended separately to the `args` list. This patch solves that by adding the whole string to the list. This should fix the TestIOHandlerResize failure on GreenDragon. Differential Revision: https://reviews.llvm.org/D126021 Signed-off-by: Med Ismail Bennani --- lldb/packages/Python/lldbsuite/test/lldbpexpect.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lldb/packages/Python/lldbsuite/test/lldbpexpect.py b/lldb/packages/Python/lldbsuite/test/lldbpexpect.py index 1083705a2011..22a30c5d22c4 100644 --- a/lldb/packages/Python/lldbsuite/test/lldbpexpect.py +++ b/lldb/packages/Python/lldbsuite/test/lldbpexpect.py @@ -34,7 +34,7 @@ class PExpectTest(TestBase): args += run_under args += [lldbtest_config.lldbExec, '--no-lldbinit'] if not use_colors: - args += '--no-use-colors' + args.append('--no-use-colors') for cmd in self.setUpCommands(): if "use-color false" in cmd and use_colors: continue