[Lldb-commits] [lldb] [llvm-lit][test] Change unsupported cat -e to cat -v to work with lit internal shell (PR #104878)
via lldb-commits
lldb-commits at lists.llvm.org
Mon Aug 19 16:03:21 PDT 2024
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
More information about the lldb-commits
mailing list