diff options
| author |  Chris Webb <chris@arachsys.com> | 2012-07-09 19:53:26 +0100 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2012-07-09 12:43:58 -0700 |
| commit | d9a93575722d9ea918f090892b06618ddf0d1d65 (patch) | |
| tree | 0df72473b4a15c0fb64dd3a92a1a18dca09b83de /t | |
| parent | Git 1.7.9.7 (diff) | |
| download | tgif-d9a93575722d9ea918f090892b06618ddf0d1d65.tar.xz | |
Allow edit of empty message with commit --amend
"git commit --amend" used on a commit with an empty message fails
unless -m is given, whether or not --allow-empty-message is
specified.
Allow it to proceed to the editor with an empty commit message.
Unless --allow-empty-message is in force, it will still abort later
if an empty message is saved from the editor (this check was
already necessary to prevent a non-empty commit message being edited
to an empty one).
Add a test for --amend --edit of an empty commit message which fails
without this fix, as it's a rare case that won't get frequently
tested otherwise.
Signed-off-by: Chris Webb <chris@arachsys.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't')
| -rwxr-xr-x | t/t7501-commit.sh | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/t/t7501-commit.sh b/t/t7501-commit.sh index 8bb38337a9..19593f8fba 100755 --- a/t/t7501-commit.sh +++ b/t/t7501-commit.sh @@ -124,6 +124,21 @@ test_expect_success '--amend --edit' ' test_cmp expect msg ' +test_expect_success '--amend --edit of empty message' ' + cat >replace <<-\EOF && + #!/bin/sh + echo "amended" >"$1" + EOF + chmod 755 replace && + git commit --allow-empty --allow-empty-message -m "" && + echo more bongo >file && + git add file && + EDITOR=./replace git commit --edit --amend && + git diff-tree -s --format=%s HEAD >msg && + ./replace expect && + test_cmp expect msg +' + test_expect_success '-m --edit' ' echo amended >expect && git commit --allow-empty -m buffer && |