From: Philippe Blain <levraiphilippeblain@gmail.com>
Date: Fri, 16 Jul 2021 01:28:16 GMT
Subject: Re: [PATCH] parse-options: don't complete option aliases by default
Message-ID: <f85969ca-48e0-b2f1-6a42-7e4cc41a3639@gmail.com>
In-Reply-To: <xmqqy2a7s8vh.fsf@gitster.g>

Hi Junio,

Le 2021-07-15 à 14:57, Junio C Hamano a écrit :
> Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:
> 
>> I'm a bit biased here since I like --recursive better, but let's not
>> drag that whole argument up again. I don't find the argument in
>> bb62e0a99fc (clone: teach --recurse-submodules to optionally take a
>> pathspec, 2017-03-17) convincing enough to have moved such a prominent
>> use-case to a longer option name.
> 
> I do agree that it is a separate topic to pick one or the other as
> the primary.  I think this change means that the users no longer
> need to stop after hitting a tab:
> 
>      clone --recurs<TAB>
> 
> and instead get to "clone --recurse-submodules" right away, which is
> a positigve change.  

This is indeed the goal, I'll update the commit message to make that
clearer.

Of course, if the separate topic swaps which is
> the canonical and which is the alias, the same user would get the
> new canonical one right away from the same state.
> 
> So it does look like a good change to me.
> 

Thanks,
Philippe.