Incorporated changes from Duy Nguyen and Jeremiah Mahler.

Jeremiah, I didn't make the changes about `<<-EOF` or `test_expect_success`
because I'm guessing that keeping the local style of the code intact is more
important than using those. Do you think it makes sense to refactor the rest of
the test file (t/ to use those? I could also change the
other `git config` calls to use `test_config`.

Duy, you were right about `-V`. Do you know of a simple way to add that
shortened flag? `OPT_BOOL('v', "verbose", ...)` gives me `-v`, `--verbose`, and
`--no-verbose`, but no `-V` as a shortened form of `--no-verbose`.

commit 1a49356b87c9028e68e731f34790c11a3075f736
Author: Caleb Thompson <>
Date:   Fri May 23 11:47:44 2014 -0500

    commit: support commit.verbose and --no-verbose

    Add a new configuration variable commit.verbose to implicitly pass
    `--verbose` to `git-commit`. Add `--no-verbose` to commit to negate that

    Signed-off-by: Caleb Thompson <>
    Reviewed-by: Duy Nguyen <>
    Reviewed-by: Jeremiah Mahler <>

diff --git a/Documentation/config.txt b/Documentation/config.txt
index 1932e9b..a245928 100644
--- a/Documentation/config.txt
+++ b/Documentation/config.txt
@@ -1009,6 +1009,11 @@ commit.template::
        "`~/`" is expanded to the value of `$HOME` and "`~user/`" to the
        specified user's home directory.

+       A boolean to enable/disable inclusion of diff information in the
+       commit message template when using an editor to prepare the commit
+       message.  Defaults to false.
        Specify an external helper to be called when a username or
        password credential is needed; the helper may consult external
diff --git a/Documentation/git-commit.txt b/Documentation/git-commit.txt
index 0bbc8f5..d7b50e2 100644
--- a/Documentation/git-commit.txt
+++ b/Documentation/git-commit.txt
@@ -282,7 +282,13 @@ configuration variable documented in linkgit:git-config[1].
        Show unified diff between the HEAD commit and what
        would be committed at the bottom of the commit message
        template.  Note that this diff output doesn't have its
-       lines prefixed with '#'.
+       lines prefixed with '#'.  The `commit.verbose` configuration
+       variable can be set to true to implicitly send this option.
+       Do not show the unified diff  at the bottom of the commit message
+       template.  This is the default behavior, but can be used to override
+       the`commit.verbose` configuration variable.

diff --git a/builtin/commit.c b/builtin/commit.c
index 9cfef6c..7978d7f 100644
--- a/builtin/commit.c
+++ b/builtin/commit.c
@@ -1417,6 +1417,10 @@ static int git_commit_config(const char *k, const char 
*v, void *cb)
                sign_commit = git_config_bool(k, v) ? "" : NULL;
                return 0;
+       if (!strcmp(k, "commit.verbose")) {
+               verbose = git_config_bool(k, v);
+               return 0;
+       }

        status = git_gpg_config(k, v, NULL);
        if (status)
@@ -1484,7 +1488,7 @@ int cmd_commit(int argc, const char **argv, const char 
        static struct wt_status s;
        static struct option builtin_commit_options[] = {
                OPT__QUIET(&quiet, N_("suppress summary after successful 
-               OPT__VERBOSE(&verbose, N_("show diff in commit message 
+               OPT_BOOL('v', "verbose", &verbose, N_("show diff in commit 
message template")),
                OPT_GROUP(N_("Commit message options")),
                OPT_FILENAME('F', "file", &logfile, N_("read message from 
diff --git a/contrib/completion/git-completion.bash 
index 2c59a76..b8f4b94 100644
--- a/contrib/completion/git-completion.bash
+++ b/contrib/completion/git-completion.bash
@@ -1976,6 +1976,7 @@ _git_config ()
+               commit.verbose
diff --git a/t/ b/t/
index 2ddf28c..bea5d88 100755
--- a/t/
+++ b/t/
@@ -10,6 +10,12 @@ EOF
 chmod +x check-for-diff
 test_set_editor "$PWD/check-for-diff"
+cat >check-for-no-diff <<EOF
+exec grep -v '^diff --git' "\$1"
+chmod +x check-for-no-diff
 cat >message <<'EOF'

@@ -48,6 +54,21 @@ test_expect_success 'verbose diff is stripped out 
(mnemonicprefix)' '
        check_message message

+test_expect_success 'commit shows verbose diff with set commit.verbose' '
+       echo morecontent >file &&
+       git add file &&
+       test_config commit.verbose true &&
+       check_message message
+test_expect_success 'commit does not show verbose diff with --no-verbose' '
+       echo morecontent >file &&
+       git add file &&
+       test_config commit.verbose true &&
+       test_set_editor "$PWD/check-for-no-diff" &&
+       git commit --amend --no-verbose
 cat >diff <<'EOF'
 This is an example commit message that contains a diff.

Attachment: pgpJKJgq4upHb.pgp
Description: PGP signature

Reply via email to