This is an archive of the discontinued LLVM Phabricator instance.

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

Authored by mstorsjo on Aug 8 2022, 2:50 PM.

Details

Summary

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.

Diff Detail

Event Timeline

mstorsjo created this revision.Aug 8 2022, 2:50 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 8 2022, 2:50 PM
mstorsjo requested review of this revision.Aug 8 2022, 2:50 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 8 2022, 2:50 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
philnik accepted this revision.Aug 18 2022, 5:19 AM
This revision is now accepted and ready to land.Aug 18 2022, 5:19 AM
ldionne accepted this revision.Aug 18 2022, 1:29 PM