[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:02:53 PDT 2024


https://github.com/connieyzhu created https://github.com/llvm/llvm-project/pull/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. 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

>From 67388155c20144bb253841eb6567568f0ef63577 Mon Sep 17 00:00:00 2001
From: Connie Zhu <connieyzhu at google.com>
Date: Mon, 19 Aug 2024 22:55:34 +0000
Subject: [PATCH] [llvm-lit][test] Change unsupported cat -e to cat -v to work
 with lit internal shell.

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.
---
 lldb/test/Shell/Driver/TestUseColor.test | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

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