[libcxx-commits] [PATCH] D101779: [libcxx] Update docs regarding the need for bash/posix tools for tests on Windows. NFC.

[Martin Storsjö via Phabricator via libcxx-commits]

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

After 39bbfb77264a4a7a216921c2b70a30ba0f27eb56 <https://reviews.llvm.org/rG39bbfb77264a4a7a216921c2b70a30ba0f27eb56>, bash is no longer
a hard requirement.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D101779

Files:
  libcxx/docs/BuildingLibcxx.rst


Index: libcxx/docs/BuildingLibcxx.rst
===================================================================
--- libcxx/docs/BuildingLibcxx.rst
+++ libcxx/docs/BuildingLibcxx.rst
@@ -98,7 +98,10 @@
 ~~~~~~~~~~~~~~~~~~~~
 
 Building with ninja is required for development to enable tests.
-Running the tests also requires a Bash shell and Python to be available.
+A couple of tests require Bash to be available, and a couple dozens
+of tests require other posix tools (cp, grep and similar - LLVM's tests
+require the same). Without those tools the vast majority of tests
+can still be ran successfully.
 
 If Git for Windows is available, that can be used to provide the bash
 shell by adding the right bin directory to the path, e.g.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D101779.342505.patch
Type: text/x-patch
Size: 732 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20210503/13952427/attachment.bin>