public inbox for gcc@gcc.gnu.org
 help / color / mirror / Atom feed
From: Segher Boessenkool <segher@kernel.crashing.org>
To: Jeff Law <law@redhat.com>
Cc: Richard Sandiford <richard.sandiford@arm.com>,
	       "H.J. Lu" <hjl.tools@gmail.com>,
	paul@mad-scientist.net,        GCC Development <gcc@gcc.gnu.org>
Subject: Re: Git ChangeLog policy for GCC Testsuite inquiry
Date: Thu, 06 Feb 2020 13:51:00 -0000	[thread overview]
Message-ID: <20200206135101.GY22482@gate.crashing.org> (raw)
In-Reply-To: <7743417118d2067cf0e9e1b840564f9f2d5e72dc.camel@redhat.com>

On Wed, Feb 05, 2020 at 06:43:54PM -0700, Jeff Law wrote:
> On Wed, 2020-02-05 at 15:18 -0600, Segher Boessenkool wrote:
> > As a reviewer, the changelog is priceless still.  We shouldn't drop the
> > changelog before people write *good* commit messages (and we are still
> > quite far from that goal).
> I believe the current proposal is not to switch immediately, but to do
> so after gcc-10 is released.  Feel free to suggest improvements to
> ChangeLogs or summaries in the mean time to get folks to start
> rethinking what they write in 'em.

I think I will be quite busy until the end of stage 4 already, but sure,
of course I will not shut up.

> And FWIW, we're talking about the ChangeLog *file* here.

I don't think we were.  I have absolutely nothing against only putting
the changelog entries at the end of the commit message and extracting
that later, of course (although it is almost trivial to copy it to the
changelog files manually, I have the same workflow for it as when we
still committed with SVN, just "git am" instead of more clunky stuff,
I suspect it will streamline things, esp. for new people).

> If folks
> continued writing the same log messages and put them into git, I
> personally think that's sufficient to transition away from having a
> ChangeLog file in the source tree.

Sure.  Just needs a flag day, always painful, but it will help.

> I don't want to make perfect the enemy of the very very good here and
> moving away from a ChangeLog file in the source tree is, IMHO, very
> very good.

If you rebase changelog files, then yes, it's a bloody pain ;-)


Segher

  reply	other threads:[~2020-02-06 13:51 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CAGWvnyktjKVp-T9jrGUt_rmzgaoS0Z5N1OPP9oaOvk0Xehrb_Q@mail.gmail.com>
     [not found] ` <28d80650-26ec-04eb-65af-76151da4e411@redhat.com>
     [not found]   ` <CAGWvny=RM5cPi9VoJOKMoPNvgcybavT0CCcKSowxRNZwfrBqNw@mail.gmail.com>
     [not found]     ` <CADzB+2nLasW5mdFBWFd7xxXRLx8+UMnfXTwOYzU_x4c8=WcQRQ@mail.gmail.com>
2020-01-24 21:36       ` David Edelsohn
2020-01-24 21:45         ` Jeff Law
2020-01-24 22:38           ` Eric Botcazou
2020-01-24 22:39             ` Florian Weimer
2020-01-24 22:56             ` Jeff Law
2020-01-25  9:31               ` Jakub Jelinek
2020-01-25 10:53                 ` Paul Smith
2020-01-25 14:07                   ` H.J. Lu
2020-02-03 18:55                     ` Richard Sandiford
2020-02-03 20:24                       ` Jeff Law
2020-02-05 21:18                         ` Segher Boessenkool
2020-02-06  8:51                           ` Richard Sandiford
2020-02-06 13:37                             ` Segher Boessenkool
2020-02-06 13:01                           ` Jeff Law
2020-02-06 13:51                             ` Segher Boessenkool [this message]
2020-02-06 14:01                               ` Richard Biener
2020-02-06 14:40                                 ` Jonathan Wakely
2020-02-07 18:37                                   ` Tom Tromey
2020-02-07 21:41                                     ` Jason Merrill
2020-02-07 22:34                                       ` Tom Tromey
2020-02-08 16:50                                         ` Segher Boessenkool
2020-02-08 23:55                                           ` Andrew Pinski
2020-02-09 10:08                                             ` Segher Boessenkool
2020-02-10 17:51                                           ` Matthew Malcomson
2020-02-11  0:37                                             ` Segher Boessenkool
2020-02-06 22:25                                 ` Segher Boessenkool
2020-02-07  9:20                                   ` Richard Biener
2020-02-07 10:08                                     ` Jonathan Wakely
2020-02-07 23:17                                       ` Alan Modra
2020-02-08 19:58                                         ` Segher Boessenkool
2020-02-09 10:46                                           ` Jonathan Wakely
2020-02-09 17:49                                             ` Segher Boessenkool
2020-02-07 13:48                                     ` Segher Boessenkool
2020-02-07 13:56                                       ` Richard Earnshaw (lists)
2020-02-07 15:33                                         ` Segher Boessenkool
2020-02-07 15:43                                           ` Richard Earnshaw (lists)
2020-02-07 16:00                                             ` Segher Boessenkool
2020-02-10 13:09                                             ` Richard Biener
2020-02-10 17:27                                   ` Hans-Peter Nilsson
2020-02-06 14:56                             ` Jakub Jelinek
2020-02-06 16:18                               ` Segher Boessenkool
2020-02-06 16:25                                 ` Jakub Jelinek
2020-02-06 18:58                                 ` Jason Merrill
2020-02-06 23:10                                   ` Segher Boessenkool
2020-01-25 22:40               ` Nathan Sidwell
2020-01-26 14:59                 ` Jeff Law
2020-02-03 10:15                   ` Richard Earnshaw (lists)
2020-02-03 13:01                     ` Nathan Sidwell

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=20200206135101.GY22482@gate.crashing.org \
    --to=segher@kernel.crashing.org \
    --cc=gcc@gcc.gnu.org \
    --cc=hjl.tools@gmail.com \
    --cc=law@redhat.com \
    --cc=paul@mad-scientist.net \
    --cc=richard.sandiford@arm.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 read-only IMAP folder(s) and NNTP newsgroup(s).