[llvm] 30d77fb - [lit] Apply aa71680f2948's fix to an additional test

Joel E. Denny via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 19 15:41:28 PDT 2023


Author: Joel E. Denny
Date: 2023-09-19T18:40:50-04:00
New Revision: 30d77fb80857e645b300c8f59cad9414d090e083

URL: https://github.com/llvm/llvm-project/commit/30d77fb80857e645b300c8f59cad9414d090e083
DIFF: https://github.com/llvm/llvm-project/commit/30d77fb80857e645b300c8f59cad9414d090e083.diff

LOG: [lit] Apply aa71680f2948's fix to an additional test

Seen at
<https://lab.llvm.org/buildbot/#/builders/216/builds/27538/steps/7/logs/FAIL__lit___shtest-external-shell-kill_py>.

Added: 
    

Modified: 
    llvm/utils/lit/tests/lit.cfg
    llvm/utils/lit/tests/shtest-external-shell-kill.py
    llvm/utils/lit/tests/shtest-run-at-line.py

Removed: 
    


################################################################################
diff  --git a/llvm/utils/lit/tests/lit.cfg b/llvm/utils/lit/tests/lit.cfg
index 438fdbd93c39812..1382f7ef4ab00e5 100644
--- a/llvm/utils/lit/tests/lit.cfg
+++ b/llvm/utils/lit/tests/lit.cfg
@@ -73,6 +73,16 @@ config.substitutions.append(
 )
 config.substitutions.append(("%{python}", '"%s"' % (sys.executable)))
 
+# This diagnostic sometimes appears in windows when using bash as an external
+# shell.  Ignore it in lit's output where we need to strictly check only the
+# relevant output.
+config.substitutions.append(
+    (
+        "%{filter-lit}",
+        "grep -v 'bash.exe: warning: could not find /tmp, please create!'",
+    )
+)
+
 # Enable coverage.py reporting, assuming the coverage module has been installed
 # and sitecustomize.py in the virtualenv has been modified appropriately.
 if lit_config.params.get("check-coverage", None):

diff  --git a/llvm/utils/lit/tests/shtest-external-shell-kill.py b/llvm/utils/lit/tests/shtest-external-shell-kill.py
index 73f8d8601af620b..2de9766aa2fa2fa 100644
--- a/llvm/utils/lit/tests/shtest-external-shell-kill.py
+++ b/llvm/utils/lit/tests/shtest-external-shell-kill.py
@@ -20,7 +20,7 @@
 # The last FileCheck directive below checks that the debugging commands for the
 # above RUN line are not killed and do execute at the right time.
 
-# RUN: %{lit} -a %{inputs}/shtest-external-shell-kill | FileCheck %s
+# RUN: %{lit} -a %{inputs}/shtest-external-shell-kill | %{filter-lit} | FileCheck %s
 # END.
 
 #       CHECK: Command Output (stdout):

diff  --git a/llvm/utils/lit/tests/shtest-run-at-line.py b/llvm/utils/lit/tests/shtest-run-at-line.py
index a1bdb039805ad4f..18086f6fa10d650 100644
--- a/llvm/utils/lit/tests/shtest-run-at-line.py
+++ b/llvm/utils/lit/tests/shtest-run-at-line.py
@@ -1,15 +1,9 @@
 # Check that -a/-v/-vv makes the line number of the failing RUN command clear.
 
 
-# This diagnostic sometimes appears in windows when using bash as an external
-# shell.  Ignore it so we can strictly check the relevant output.
-#
-# DEFINE: %{filter} = \
-# DEFINE:   grep -v 'bash.exe: warning: could not find /tmp, please create!'
-
-# RUN: not %{lit} -a %{inputs}/shtest-run-at-line | %{filter} | FileCheck %s
-# RUN: not %{lit} -v %{inputs}/shtest-run-at-line | %{filter} | FileCheck %s
-# RUN: not %{lit} -vv %{inputs}/shtest-run-at-line | %{filter} | FileCheck %s
+# RUN: not %{lit} -a %{inputs}/shtest-run-at-line | %{filter-lit} | FileCheck %s
+# RUN: not %{lit} -v %{inputs}/shtest-run-at-line | %{filter-lit} | FileCheck %s
+# RUN: not %{lit} -vv %{inputs}/shtest-run-at-line | %{filter-lit} | FileCheck %s
 # END.
 
 


        


More information about the llvm-commits mailing list