Re: [PATCH 2/3] builtin/fetch: parse recurse-submodules-default at default options parsing

[Stefan Beller <sbeller@xxxxxxxxxx>]

On Fri, Jun 23, 2017 at 5:51 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote:

>>>>       if (recurse_submodules != RECURSE_SUBMODULES_OFF) {
>>>> -             if (recurse_submodules_default) {
>>>> -                     int arg = parse_fetch_recurse_submodules_arg("--recurse-submodules-default", recurse_submodules_default);
>>>> -                     set_config_fetch_recurse_submodules(arg);
>>>> -             }
>>>> +             if (recurse_submodules_default != RECURSE_SUBMODULES_DEFAULT)
>>>> +                     set_config_fetch_recurse_submodules(recurse_submodules_default);
>>>


>
> I am not talking about the outer "if" condition.

I agree with your analysis, my answer was evasive.
I'll dig into the details why we do not set the default by default.