I got it. The verbose is initialised to -1 before. When cmd_commit runs, it changes the value of verbose accordingly to 0 or positive. But when cmd_status runs, it will retain the value -1 and the if clause which accepts all values except 0 will execute. I guess a if statement inside cmd_status which reinitialises it to 0 or positive depending on the situation will solve the problem. On Thu, Mar 24, 2016 at 3:52 PM, Pranit Bauva <pranit.bauva@xxxxxxxxx> wrote: > On Thu, Mar 24, 2016 at 3:34 PM, SZEDER Gábor <szeder@xxxxxxxxxx> wrote: >>> Add commit.verbose configuration variable as a convenience for those >>> who always prefer --verbose. >>> >>> Helped-by: Junio C Hamano <gitster@xxxxxxxxx> >>> Helped-by: Eric Sunshine <sunshine@xxxxxxxxxxxxxx> >>> Signed-off-by: Pranit Bauva <pranit.bauva@xxxxxxxxx> >>> >>> --- >>> The previous version of the patch are: >>> - [v8] $gmane/288820 >>> - [v7] $gmane/288820 >>> - [v6] $gmane/288728 >>> - [v5] $gmane/288728 >>> - [v4] $gmane/288652 >>> - [v3] $gmane/288634 >>> - [v2] $gmane/288569 >>> - [v1] $gmane/287540 >>> >>> Changes with respect to the previous patch: >>> - Compare with -1 as only -1 value is used for unspecified behavior. >>> - Write clean tests >>> --- >>> Documentation/config.txt | 4 ++++ >>> Documentation/git-commit.txt | 3 ++- >>> builtin/commit.c | 13 ++++++++++- >>> t/t7507-commit-verbose.sh | 51 ++++++++++++++++++++++++++++++++++++++++++++ >>> 4 files changed, 69 insertions(+), 2 deletions(-) >> >> Please always run the full test suite before submitting patches to >> make sure that your changes do not inadvertently break something. >> This patch breaks several tests in t7512-status-help.sh, >> t7508-status.sh and t7060-wtstatus.sh. > > Sorry for that. I will make sure I do run the complete test suite. I > currently ran only commit based tests. But now that I think about it > that since status and commit share a lot of things, it might be > possible to break parts of status. I will investigate further as to > what cased this problem though I kind of get a hint that it is because > of verbose being the parent and others consuming it. There are a lot > of tests failing. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html