[libcxx-commits] [libcxx] [libc++] Simplify how the global stream tests are written (PR #66842)
Louis Dionne via libcxx-commits
libcxx-commits at lists.llvm.org
Thu Oct 5 13:51:56 PDT 2023
ldionne wrote:
> > > In most configs, we do have `bash` set up to be available when running tests on Windows, but I did run into one case in one setup where the tests previously were skipped but now execute and fail due to the missing `echo` executable.
> >
> >
> > Interesting. But aren't we using the Lit builtin `echo` command? I don't think we should ever be trying to call an `echo` executable on the platform?
>
> That was what I thought as well, but I had to dive through a bit of lit to see what it does. It turns out that lit only uses the builtin `echo` if it's not part of a pipeline - probably due to potential blocking, if I understand the comments correctly: https://github.com/llvm/llvm-project/blob/llvmorg-18-init/llvm/utils/lit/lit/TestRunner.py#L742-L748
Ahh, I see. It looks like it's also the target of a TODO. Concretely is this blocking any of your work?
https://github.com/llvm/llvm-project/pull/66842
More information about the libcxx-commits
mailing list