On Thu, Jun 19, 2008 at 12:32:02PM -0500, Brandon Casey wrote: > - test_must_fail GIT_EDITOR="$(pwd)/.git/FAKE_EDITOR" git commit && > + # We intentionally do not use test_must_fail on the next line since the > + # mechanism does not work when setting environment variables inline > + ! GIT_EDITOR="$(pwd)/.git/FAKE_EDITOR" git commit && Doesn't GIT_EDITOR="$(pwd)/.git/FAKE_EDITOR" test_must_fail git commit work ? Mike -- 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