Mike Hommey wrote: > 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 ? That leaves GIT_EDITOR set to the new value after the command completes. -brandon -- 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