Browse Source

Merge branch 'cw/amend-commit-without-message'

"commit --amend" used to refuse amending a commit with an empty log
message, with or without "--allow-empty-message".

* cw/amend-commit-without-message:
  Allow edit of empty message with commit --amend
maint
Junio C Hamano 13 years ago
parent
commit
8647b585d8
  1. 2
      builtin/commit.c
  2. 15
      t/t7501-commit.sh

2
builtin/commit.c

@ -640,7 +640,7 @@ static int prepare_to_commit(const char *index_file, const char *prefix, @@ -640,7 +640,7 @@ static int prepare_to_commit(const char *index_file, const char *prefix,
hook_arg1 = "message";
} else if (use_message) {
buffer = strstr(use_message_buffer, "\n\n");
if (!buffer || buffer[2] == '\0')
if (!use_editor && (!buffer || buffer[2] == '\0'))
die(_("commit has empty message"));
strbuf_add(&sb, buffer + 2, strlen(buffer + 2));
hook_arg1 = "commit";

15
t/t7501-commit.sh

@ -138,6 +138,21 @@ test_expect_success '--amend --edit' ' @@ -138,6 +138,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 &&

Loading…
Cancel
Save