Stefan Beller <[email protected]> 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 <[email protected]> > --- > 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.
