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

[Martin Storsjö via Phabricator via libcxx-commits]

mstorsjo created this revision.
mstorsjo added a reviewer: ldionne.
Herald added a project: All.
mstorsjo requested review of this revision.
Herald added a project: libc++.
Herald added a reviewer: libc++.

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


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D131446

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


Index: libcxx/test/libcxx/selftest/remote-substitutions.sh.cpp
===================================================================
--- libcxx/test/libcxx/selftest/remote-substitutions.sh.cpp
+++ 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'


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D131446.450962.patch
Type: text/x-patch
Size: 564 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20220808/46256c3a/attachment.bin>