Junio C Hamano wrote:
>> Junio C Hamano wrote:
>>> Yeah, but why change it so much? Wouldn't writing
>>> "$GIT_BUILD_DIR/git" >/dev/null
>>> if test $? != 1
>>> : You haven't built git!
>>> just like the original in 0000 be sufficient??
>> Because that emits an ugly
>> ./test-lib.sh: 54: /home/artagnon/src/git/t/../git: not found
> Don't you deserve it? ;-)
> The full message would read
> ./test-lib.sh: 54: /home/artagnon/src/git/t/../git: not found
> error: you do not seem to have built git yet.
> which looks perfectly sensible to me. It makes it clear where on
> the filesystem the test script expects your "git", which is an added
Fair enough- use your version then. Let me know if you want me to
submit a revised patch.
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html