From: Jeff King <peff@peff.net>
Date: Tue, 18 Mar 2014 04:18:55 GMT
Subject: Re: [PATCH v3 0/8] Hiding refs
Message-ID: <20140318041855.GB7252@sigill.intra.peff.net>
In-Reply-To: <CACsJy8A1=U2=TGoKyo5mo1fLW+hBR1psn1J6S0=391fei2JULw@mail.gmail.com>

On Sat, Mar 15, 2014 at 08:23:08AM +0700, Duy Nguyen wrote:

> > The default would start at false, and people who know their server is
> > very up-to-date can turn it on. And then when many server
> > implementations support it, flip the default to auto. And either leave
> > it there forever, or eventually just set it to "true" and drop "auto"
> > entirely as a code cleanup.
> 
> I would add that upload-pack also advertises about the availability of
> upload-pack2 and the client may set the remote.*.useUploadPack2 to
> either yes or auto so next time upload-pack2 will be used.

Good idea. If our auto probe is "try 1, learn to upgrade to 2 for next
time", we do not have to be so conservative about flipping it on (as
compared to my "try 2, fall back to 1").

-Peff