Junio C Hamano, Wed, Sep 16, 2020 06:17:41 +0200: > Jeff King <peff@xxxxxxxx> writes: > > > On Tue, Sep 15, 2020 at 01:54:07PM +0200, Alex Riesen wrote: > > > >> The options indicate user intent for the whole fetch operation, and > >> ignoring them in sub-fetches is quite unexpected when, for instance, > >> it is intended to limit all of the communication to a specific transport > >> protocol for some reason. > >> > >> Signed-off-by: Alex Riesen <alexander.riesen@xxxxxxxxxxx> > >> --- > > > > Regardless of whether we move forward with the parse-options flag or > > config discussed in the other thread, I think this is an obvious > > improvement that we should take in the meantime. > > Yes. Others can wait. ipversion configuration variable is probably > easier to sell; ... Is its choice of namespace (transfer.) alright, too? > ... parse_options thing deserves a longer and deeper > thought as it will affect the API future codebase would rely on. Of course. Regards, Alex
