> + if (!strcmp(k, "commit.verbose")){ > v3 did this line correctly but you somehow lost the SP between > "){". What happened? I will include the SP between )) and { . > Don't you need a test that status is not broken when the variable is > set? I will include the test for status too. But I am a bit confused where should I place them? This patch triggers the possibility of a breakage in status related to verbose but other tests related to status are in different files. Could you tell me the filename where I should place these test? -- 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