[libcxx-commits] [libcxx] 8437c8f - [libcxx] [test] Mark the libcxx/selftest/remote-substitutions.sh.cpp test as requiring bash in the executor

[Martin Storsjö via libcxx-commits]

Author: Martin Storsjö
Date: 2022-08-19T11:42:56+03:00
New Revision: 8437c8ff422fc9676e5de75aa9483d9e24691074

URL: https://github.com/llvm/llvm-project/commit/8437c8ff422fc9676e5de75aa9483d9e24691074
DIFF: https://github.com/llvm/llvm-project/commit/8437c8ff422fc9676e5de75aa9483d9e24691074.diff

LOG: [libcxx] [test] Mark the libcxx/selftest/remote-substitutions.sh.cpp test as requiring bash in the executor

It's unclear to me why this wasn't tagged this way already in
87fe0709d4ad7c3128f998f14c66245ebb423e4f / D114612 where the
feature flag executor-has-no-bash was added, as this test did exist
in its current form already at that time.

Differential Revision: https://reviews.llvm.org/D131446

Added: 
    

Modified: 
    libcxx/test/libcxx/selftest/remote-substitutions.sh.cpp

Removed: 
    


################################################################################
diff  --git a/libcxx/test/libcxx/selftest/remote-substitutions.sh.cpp b/libcxx/test/libcxx/selftest/remote-substitutions.sh.cpp
index dfd4f4be48888..da25df993123f 100644
--- a/libcxx/test/libcxx/selftest/remote-substitutions.sh.cpp
+++ b/libcxx/test/libcxx/selftest/remote-substitutions.sh.cpp
@@ -15,6 +15,7 @@
 // We also check that the path of test-executables is replaced whether they
 // appear first in the command-line or not.
 
+// UNSUPPORTED: executor-has-no-bash
 // RUN: %{cxx} %s %{flags} %{compile_flags} %{link_flags} -o %t.exe
 // RUN: %{exec} %t.exe 0
 // RUN: %{exec} bash -c '! %t.exe 1'