PATCH: Don't warn if bash is missing

[Reid Kleckner]

I don't have Michael's original patch yet (maybe llvm-commits is slow?),
but I agree with the concept. We don't need a warning for this.

On Thu, Jan 29, 2015 at 10:45 AM, Aaron Ballman <aaron at aaronballman.com>
wrote:

> On Thu, Jan 29, 2015 at 1:19 PM, Edwards, Michael
> <Michael_Edwards at playstation.sony.com> wrote:
> > Hi,
> >
> >
> > It looks like bash will always be available on the supported platforms
> > except Windows.  In Windows we can run the tests without bash.  I'm
> hoping
> > we can remove this warning as it is causing my Buildbots to remain yellow
> > all the time.  I'm not sure if it is needed anywhere else?
>
> I would love to see that warning go away, and this patch LGTM, but you
> should wait for someone more Lit-savvy to agree before committing.
>
> I'm not certain if this affects your bots or not, but when I test
> locally, I usually wind up with:
>
> 60>  lit.py: LitConfig.py:94: note: Unable to find 'bash.exe'.
> 60>  lit.py: lit.cfg:195: note: using clang:
> 'E:/llvm/2013/Debug/bin/clang.EXE'
> 60>  lit.py: lit.cfg:333: note: Did not find clang-interpreter in
> E:/llvm/2013/Debug/bin;E:/llvm/2013/Debug/bin
> 60>  lit.py: LitConfig.py:94: note: Unable to find 'bash.exe'.
> 60>  lit.py: lit.cfg:271: note: Did not find llvm-go in
> E:/llvm/2013/Debug/bin
>
> Do you see the other notes as well?
>
> ~Aaron
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150129/cceca987/attachment.html>