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 /builtin | |
| parent | d0f1ea6003d97e63110fa7d50bb07f546a909b6e (diff) | |
| download | git-d9a93575722d9ea918f090892b06618ddf0d1d65.tar.gz git-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 'builtin')
| -rw-r--r-- | builtin/commit.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/builtin/commit.c b/builtin/commit.c index eae5a29ae..4b4e7e797 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"; |