Re: [PATCH] merge: Honor prepare-commit-msg return code
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jan 2, 2013, 20:40 UTC
- Message-ID
- <7v623f2uam.fsf@alter.siamese.dyndns.org>
- In-Reply-To
- <1357152170-5511-1-git-send-email-apelisse@gmail.com>
Antoine Pelisse <apelisse@gmail.com> writes:
> prepare-commit-msg hook is run when committing to prepare the log > message. If the exit-status is non-zero, the commit should be aborted.
I was scratching my head why you CC'ed Jay, until I dug up 65969d4 (merge: honor prepare-commit-msg hook, 2011-02-14).
Show 6 quoted lines
> +test_expect_success 'with failing hook (merge)' '
> +
> + git checkout -B other HEAD@{1} &&
> + echo "more" >> file &&
> + git add file &&
> + chmod -x $HOOK &&I have a feeling that this will break folks without POSIXPERM prerequisite.
How about doing it this way instead? This old test script seems to want a lot of style clean-ups, but I refrained from doing any in this fixlet.
Thanks.
t/t7505-prepare-commit-msg-hook.sh | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/t/t7505-prepare-commit-msg-hook.sh b/t/t7505-prepare-commit-msg-hook.sh index bc497bc..3573751 100755 --- a/t/t7505-prepare-commit-msg-hook.sh +++ b/t/t7505-prepare-commit-msg-hook.sh @@ -172,11 +172,12 @@ test_expect_success 'with failing hook (merge)' ' git checkout -B other HEAD@{1} && echo "more" >> file && git add file && - chmod -x $HOOK && + rm -f "$HOOK" && git commit -m other && - chmod +x $HOOK && + write_script "$HOOK" <<-EOF + exit 1 + EOF git checkout - && - head=`git rev-parse HEAD` && test_must_fail git merge other '