[Lldb-commits] [lldb] 6558e04 - [lldb][test] Change unsupported cat -e to cat -v to work with lit internal shell (#104878)

via lldb-commits lldb-commits at lists.llvm.org
Tue Aug 20 11:54:21 PDT 2024


Author: Connie Zhu
Date: 2024-08-20T11:54:17-07:00
New Revision: 6558e04f02858bff61c01c9ec0900849fc8532f5

URL: https://github.com/llvm/llvm-project/commit/6558e04f02858bff61c01c9ec0900849fc8532f5
DIFF: https://github.com/llvm/llvm-project/commit/6558e04f02858bff61c01c9ec0900849fc8532f5.diff

LOG: [lldb][test] Change unsupported cat -e to cat -v to work with lit internal shell (#104878)

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. For `cat`, the `-v`
option prints non-printing characters in ^ and M- notation, while the
`-e` option adds `$` to the end of lines in addition to printing
non-printing characters in ^ and M- notation. 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

Added: 
    

Modified: 
    lldb/test/Shell/Driver/TestUseColor.test

Removed: 
    


################################################################################
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


        


More information about the lldb-commits mailing list