Jonathan Nieder wrote: > --- a/t/test-lib.sh > +++ b/t/test-lib.sh > @@ -338,7 +338,7 @@ test_have_prereq () { > *" $prerequisite "*) > : yes, have it ;; > *) > - ! : nope ;; > + return 1 The ";;" is optional for the last arm in a case statement, but it is probably simpler to include it. Sorry about that. (The only intended effect was only to use “return” that terminates and propagates out of the loop instead of “false” that is overridden by a later “true”.) -- 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