This is the mail archive of the
mailing list for the GCC project.
Re: [PATCH, v2] wwwdocs: e-mail subject lines for contributions
- From: Jakub Jelinek <jakub at redhat dot com>
- To: "Richard Earnshaw (lists)" <Richard dot Earnshaw at arm dot com>
- Cc: Gerald Pfeifer <gerald at pfeifer dot com>, gcc-patches at gcc dot gnu dot org, GCC Development <gcc at gcc dot gnu dot org>
- Date: Tue, 21 Jan 2020 16:04:40 +0100
- Subject: Re: [PATCH, v2] wwwdocs: e-mail subject lines for contributions
- References: <firstname.lastname@example.org> <alpine.LSU.email@example.com> <firstname.lastname@example.org> <email@example.com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Tue, Jan 21, 2020 at 02:52:00PM +0000, Richard Earnshaw (lists) wrote:
> [updated, following some comments from Gerald, main differences are
> slight tweaks to the html markup and changing "email" to "e-mail"]
> This patch proposes some new (additional) rules for email subject lines
> when contributing to GCC. The goal is to make sure that, as far as
> possible, the subject for a patch will form a good summary when the
> message is committed to the repository if applied with 'git am'. Where
> possible, I've tried to align these rules with those already in
> use for glibc, so that the differences are minimal and only where
> Some things that differ from existing practice (at least by some people)
> - Use '<topic>:' rather than '[<topic>]'
> - This is more git friendly and works with 'git am'.
> - Put bug numbers at the end of the line rather than the beginning.
> - The bug number is useful, but not as useful as the brief summary.
> Also, use the shortened form, as the topic part is more usefully
> conveyed in the proper topic field (see above).
Some examples would be useful I'd say, e.g. it is unclear in what way you
want the PR number to be appended, shall it be
something: whatever words describe it PR12345
something: whatever words describe it (PR12345)
something: whatever words describe it: PR12345
something: whatever words describe it [PR12345]
or something else?
Also, it would be nice to stress that the PR long form should be in the
ChangeLog and somewhere on the later lines of the commit message, I don't
think we pick up the shorter form from the first line when it short form (I
could be wrong, but e.g.
commit didn't make it into bugzilla).