llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-lldb Author: Connie (connieyzhu) <details> <summary>Changes</summary> This patch changes the test that uses the `cat -e` option to `cat -v` so that the test can be run using lit's internal shell. This is an alternative patch to https://github.com/llvm/llvm-project/pull/102061, opting to rewrite the test that uses `cat -e` instead of extending support to the `-e` option. Fixes https://github.com/llvm/llvm-project/issues/102377 --- Full diff: https://github.com/llvm/llvm-project/pull/104878.diff 1 Files Affected: - (modified) lldb/test/Shell/Driver/TestUseColor.test (+1-1) ``````````diff diff --git a/lldb/test/Shell/Driver/TestUseColor.test b/lldb/test/Shell/Driver/TestUseColor.test index 15e5814d8ebadd..dcb9f8c1517306 100644 --- a/lldb/test/Shell/Driver/TestUseColor.test +++ b/lldb/test/Shell/Driver/TestUseColor.test @@ -1,7 +1,7 @@ UNSUPPORTED: system-windows RUN: not %lldb -b -o 'settings set use-color true' -o 'settings show use-color' -o 'bogus' > %t 2>&1 -RUN: cat -e %t | FileCheck %s --check-prefix COLOR +RUN: cat -v %t | FileCheck %s --check-prefix COLOR COLOR: use-color (boolean) = true # The [[ confuses FileCheck so regex match it. COLOR: {{.+}}0;1;31merror: {{.+}}0m'bogus' is not a valid command `````````` </details> https://github.com/llvm/llvm-project/pull/104878 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
