On Wed, Mar 15, 2017 at 11:29 AM, Junio C Hamano <gitster@xxxxxxxxx> wrote:
> Stefan Beller <sbeller@xxxxxxxxxx> writes:
>
>> As 'var' contains the whole value we get error messages that repeat
>> the section and key currently:
>>
>> warning: Invalid parameter 'true' for config option 'submodule.submodule.plugins/hooks.ignore.ignore'
>>
>> Fix this by only giving the section name in the warning.
>>
>> Signed-off-by: Stefan Beller <sbeller@xxxxxxxxxx>
>> ---
>> submodule-config.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/submodule-config.c b/submodule-config.c
>> index 93453909cf..bb069bc097 100644
>> --- a/submodule-config.c
>> +++ b/submodule-config.c
>> @@ -333,7 +333,7 @@ static int parse_config(const char *var, const char *value, void *data)
>> strcmp(value, "all") &&
>> strcmp(value, "none"))
>> warning("Invalid parameter '%s' for config option "
>> - "'submodule.%s.ignore'", value, var);
>> + "'submodule.%s.ignore'", value, name.buf);
>
> Obviously correct. Nobody seems to have complained about this since
> it was first written in 2.6 days, which is a bit sad, though.
>
> Thanks.
https://trends.google.com/trends/explore?date=all&q=submodule,%2Fm%2F05vqwg
(Comparing "Submodule" (search term) with "Git" (System Software) over 10 years)
#TiL
* submodule searches are less than 1% of Git searches,
so not a lot of users compared to Git itself.
* submodules are only used in larger countries unlike Git,
which is used in a lot more countries (well, that may be
displaying issues)
* submodules are a German thing (relatively speaking)
followed by Russia, US, Canada, Australia, Japan.
:-)