Jeff King <peff@xxxxxxxx> writes: >> Mostly sending this to get the discussion started about what changes >> we want before flipping the default. > > I can't actually think of any changes we'd want to make. AFAIK aside > from the negotiation problem, v2 is good to go. When we flipped it off > by default for 2.27 out of caution, I had hoped we would flip it back on > for the 2.28 cycle to get more exposure. If there were any changes we already can think of before going public with v2 at this moment, it makes it definitely way too late to propose making it the default again. I do not think of any changes either, so I'd say it is a good sign ;-) > And that has been the only bug people have reported for 2.26. That > implies to me that: > > - we won't get significantly more information by leaving v2-as-default > in "next" or even "master" before it actually hits a release > > - there probably aren't other major problems lurking, given that > people clearly upgraded to 2.26, found the negotiation problem, but > never reported any other issues I've already said elsewhere that it is way too late to propose this flipping back the default for this cycle but it was mainly out of principle. I do agree with you and Jonathan that we won't see any further breakages in the v2 code until we expose more users to it by making it the default in a released version. I am afraid that "there probably aren't other" may be overly optimistic, as the bug in 2.26 crippled the negotiation logic and forced it to punt, which was so severe that it would have hidden any other bugs in the negotiation logic. If there is another bug in v2 negotiation logic that makes the sender to omit objects that should be sent, it would not have been observed in 2.26 because the effect of the more severe bug was to cripple the negotiation logic itself and to make it punt, sending more objects all the way down the history. Now, with that larger bug fixed post 2.26, we can start to see if there are other bugs hidden by it. In any case, we've learned in 2.26 that it is unlikely that such bugs would be uncovered until v2 is made the default again in a released version to be used by more users. So, let's flip the default in -rc0; we can revert if we see something funny in 2.28.1 in the worst case. Thanks.
