[LLVMdev] Testing requirements amendment

Eli Friedman eli.friedman at gmail.com
Mon Dec 19 18:12:53 PST 2011


On Mon, Dec 19, 2011 at 5:57 PM, David Fang <fang at csl.cornell.edu> wrote:
> Hi,
>
> I havea minor suggestion for http://llvm.org/docs/TestingGuide.html:
>
> Under 'Requirements', might be worth mentioning that running the basic
> llvm/clang regression tests (check-all) requires a version of bash that
> supports pipefail, i.e. bash 3.0+.  On older system where /bin/sh is too
> old, one simply needs to hack LitConfig.py to point to
> /somewhere/else/bash that meets this requirement.
> (This method has been tested on powerpc-darwin8.)

I would prefer a patch to make lit fail in an understandable way if it
detects an incompatible version of bash; if someone sees a test
failing, the version of bash isn't going to be high on the list of
things to check.

-Eli




More information about the llvm-dev mailing list