Git development
 help / color / mirror / Atom feed
From: Junio C Hamano <gitster@pobox.com>
To: Jeff King <peff@peff.net>
Cc: Jonathan Nieder <jrnieder@gmail.com>, git@vger.kernel.org
Subject: Re: [RFC] should these two topics graduate to 'master' soon?
Date: Thu, 30 Mar 2017 12:18:59 -0700	[thread overview]
Message-ID: <xmqq60iq4lp8.fsf@gitster.mtv.corp.google.com> (raw)
In-Reply-To: <20170328191911.22wxjbukv47jflgl@sigill.intra.peff.net> (Jeff King's message of "Tue, 28 Mar 2017 15:19:11 -0400")

Jeff King <peff@peff.net> writes:

> On Tue, Mar 28, 2017 at 11:51:49AM -0700, Jonathan Nieder wrote:
>
>> > * jc/merge-drop-old-syntax (2015-04-29) 1 commit
>> >
>> >   This topic stops "git merge <message> HEAD <commit>" syntax that
>> >   has been deprecated since October 2007 (and we have issued a
>> >   warning message since around v2.5.0 when the ancient syntax was
>> >   used).
>> >
>> > * jk/no-looking-at-dotgit-outside-repo-final (2016-10-26) 1 commit
>> >
>> >   This is the endgame of the topic to avoid blindly falling back to
>> >   ".git" when the setup sequence said we are _not_ in Git repository.
>> >   A corner case that happens to work right now may be broken by a call
>> >   to die("BUG").
>> >
>> > I am leaning toward including the former in the upcoming release,
>> > whose -rc0 is tentatively scheduled to happen on Apr 20th.  I think
>> > the rest of the system is also ready for the latter (back when we
>> > merged it to 'next' and started cooking, there were still a few
>> > codepaths that triggered its die(), which have been fixed).
>> >
>> > Opinions?
>> 
>> Google has been running with both of these for a while.  Any problems
>> we ran into were already reported and fixed.  I would be all for
>> including them in the next release.
>
> Thanks, I was wondering how much exposure the latter got. It might be a
> good idea to merge it to "master" early in the post-2.13 cycle to get a
> little more exposure (since the point of it is really to flush out
> unusual cases, the more people run it before we make a release the
> better). But I'm also OK if it's merged to master this cycle, as long as
> it's soon-ish. It's much better to flush out problems in pre-release
> master than in a released version.

OK, let's do both during this cycle, before I take a short break at
the beginning of next month.

Thanks.

  reply	other threads:[~2017-03-30 19:19 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-28 18:35 [RFC] should these two topics graduate to 'master' soon? Junio C Hamano
2017-03-28 18:51 ` Jonathan Nieder
2017-03-28 19:19   ` Jeff King
2017-03-30 19:18     ` Junio C Hamano [this message]
2017-03-28 18:57 ` Stefan Beller
2017-03-29 12:55 ` Jon Loeliger

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=xmqq60iq4lp8.fsf@gitster.mtv.corp.google.com \
    --to=gitster@pobox.com \
    --cc=git@vger.kernel.org \
    --cc=jrnieder@gmail.com \
    --cc=peff@peff.net \
    /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