Antoine Pelisse <apelisse@xxxxxxxxx> 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).
> +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
'
--
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