Re: [PATCH] NO_PERL support
- From
Jeff King <peff@peff.net>
- Date
- Apr 3, 2009, 18:37 UTC
- Message-ID
- <20090403183738.GA15926@coredump.intra.peff.net>
- In-Reply-To
- <200904031954.57584.j6t@kdbg.org>
On Fri, Apr 03, 2009 at 07:54:57PM +0200, Johannes Sixt wrote:
> If you base the patch on 'master', you can add this line to test-lib.sh > > test -z "$NO_PERL" && test_set_prereq PERL
Thanks, I'm working up a patch using this now.
> [But actually, if I read you patch correctly, NO_PERL will be set in > t/Makefile only if one runs 'make test' from the main directory. You should > invent some method to detect missing perl (or that NO_PERL was set) if 'make' > is run directly from t/.]
Yes, it should go into GIT-BUILD-OPTIONS which gets sourced by test-lib.sh (and then it not only works with "make test" from "t/", but also with "sh t????-whatever.sh".
-Peff