"Philip Oakley" <philipoakley@xxxxxxx> writes: > From: "Junio C Hamano" <gitster@xxxxxxxxx> >> >> I am not sure what this patch is trying to achieve. > > It was probably a bit of 'don't mess with working code', given that > I'd used the NO_PERL solution, rather 'document the issue'.... Then this part applies, I think. >> If you are not touching what this Makefile actually does, then I >> would imagine that you are running the "buildsystems code" that >> actually drives 'make -n' with some "available solution" (perhaps >> you are running 'make -n NO_PERL=NoThanks' or something like that)? >> Then instead of a command like this that nobody would read in this >> file, the same command can instead go there to explain what the >> "workaround" (e.g. unusual-looking 'make -n NO_PERL=NoThanks') is >> doing? Thanks. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html