git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jonathan Nieder <jrnieder@gmail.com>
To: Junio C Hamano <gitster@pobox.com>
Cc: Johannes Schindelin <Johannes.Schindelin@gmx.de>, git@vger.kernel.org
Subject: Re: jn/experimental-opts-into-proto-v2, was Re: What's cooking in git.git (May 2020, #09; Tue, 26)
Date: Wed, 27 May 2020 16:58:08 -0700	[thread overview]
Message-ID: <20200527235808.GD56118@google.com> (raw)
In-Reply-To: <xmqq1rn5kpzi.fsf@gitster.c.googlers.com>

Hi,

Junio C Hamano wrote:
> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
>> On Tue, 26 May 2020, Junio C Hamano wrote:

>>> * jn/experimental-opts-into-proto-v2 (2020-05-21) 1 commit
>>>   (merged to 'next' on 2020-05-24 at 53cd664dfe)
>>>  + config: let feature.experimental imply protocol.version=2
>>>
>>>  "feature.experimental" configuration variable is to let volunteers
>>>  easily opt into a set of newer features, which use of the v2
>>>  transport protocol is now a part of.
>>>
>>>  Will cook in 'next'.
>>
>> Given that the `feature.experimental` option is supposed to increase
>> testing in the wild, sort of `next` but for users rather than developers
>> who build their own Git, could we maybe integrate this into v2.27.0,
>> still?

Yes, that was my hope when writing it.

> I have been wondering about the same thing, and if it were not using
> its own custom way to read the configuration, it would have been a
> non-brainer to merge it down before the release.

Hm, do you have more detail about this?  git_config_get_bool feels
very standard and non-custom.

Do you mean that you would like it to go in repo-settings.c?

>                                                   But this late in
> the cycle, I'd rather not.

Sure, I can understand.

Would this be something to put on "master" soon after the release?

Dscho, do you think it would be useful for downstreams such as Debian
to apply it?  (For Debian in particular, it's not hard to get "next"
from experimental, so maybe not.)

Thanks,
Jonathan

  reply	other threads:[~2020-05-27 23:58 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-05-26 18:47 What's cooking in git.git (May 2020, #09; Tue, 26) Junio C Hamano
2020-05-27  3:27 ` jn/experimental-opts-into-proto-v2, was " Johannes Schindelin
2020-05-27 21:49   ` Junio C Hamano
2020-05-27 23:58     ` Jonathan Nieder [this message]
2020-05-28  0:43       ` Junio C Hamano
2020-05-27 20:05         ` Johannes Schindelin
2020-05-28 14:44           ` Junio C Hamano

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200527235808.GD56118@google.com \
    --to=jrnieder@gmail.com \
    --cc=Johannes.Schindelin@gmx.de \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).