Re: [PATCH v3 0/8] Hiding refs
- From
Jeff King <peff@peff.net>
- Date
- Mar 18, 2014, 04:18 UTC
- 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:
Show 9 quoted lines
> > 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