[PATCH] Makefile: fix NO_PERL bug with gitweb

When the user has defined NO_PERL, we want to skip building gitweb entirely. However, the conditional to add gitweb/gitweb.cgi to OTHER_PROGRAMS was evaluated before we actually parsed the user's config.mak. This meant that "make NO_PERL=NoThanks" worked fine, but putting "NO_PERL=NoThanks" into your config.mak broke the build (it wanted gitweb.cgi to satisfy "all", but the rule to build it was conditionally ignored, so it complained).

Signed-off-by: Jeff King <peff@peff.net>
---
Robin, I believe your original patch has this problem, too. I'm assuming
it never came up because the Gentoo build specifies the options on the
make command-line instead of through config.mak (which is also why my
earlier testing didn't catch it). So you may or may not want to modify
your patch.

I moved the offending code close to the gitweb.cgi rule definition (it was originally near the OTHER_PROGRAMS definition). I tried to find a spot that would make some sense, rather than just plopping it randomly somewhere after config.mak has been read. But if something else would make more sense, Junio, feel free to tweak the patch.

 Makefile |    4 +---
 1 files changed, 1 insertions(+), 3 deletions(-)