summaryrefslogtreecommitdiff
path: root/builtin/commit.c
diff options
context:
space:
mode:
authorChris Webb <chris@arachsys.com>2012-07-09 18:53:26 (GMT)
committerJunio C Hamano <gitster@pobox.com>2012-07-09 19:43:58 (GMT)
commitd9a93575722d9ea918f090892b06618ddf0d1d65 (patch)
tree0df72473b4a15c0fb64dd3a92a1a18dca09b83de /builtin/commit.c
parentd0f1ea6003d97e63110fa7d50bb07f546a909b6e (diff)
downloadgit-d9a93575722d9ea918f090892b06618ddf0d1d65.zip
git-d9a93575722d9ea918f090892b06618ddf0d1d65.tar.gz
git-d9a93575722d9ea918f090892b06618ddf0d1d65.tar.bz2
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 'builtin/commit.c')
-rw-r--r--builtin/commit.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/builtin/commit.c b/builtin/commit.c
index eae5a29..4b4e7e7 100644
--- a/builtin/commit.c
+++ b/builtin/commit.c
@@ -690,7 +690,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";