public inbox for gcc@gcc.gnu.org
 help / color / mirror / Atom feed
From: Michael Matz <matz@suse.de>
To: "Richard Earnshaw (lists)" <Richard.Earnshaw@arm.com>
Cc: Segher Boessenkool <segher@kernel.crashing.org>,
	    Alexander Monakov <amonakov@ispras.ru>,
	    Gerald Pfeifer <gerald@pfeifer.com>,
	gcc-patches@gcc.gnu.org,     GCC Development <gcc@gcc.gnu.org>
Subject: Re: [PATCH, v3] wwwdocs: e-mail subject lines for contributions
Date: Mon, 03 Feb 2020 17:31:00 -0000	[thread overview]
Message-ID: <alpine.LSU.2.21.2002031657430.12763@wotan.suse.de> (raw)
In-Reply-To: <a96e3375-4824-de4a-93ee-7667902449bf@arm.com>

Hello,

On Mon, 3 Feb 2020, Richard Earnshaw (lists) wrote:

> Where does your '50 chars' limit come from?  It's not in the glibc text, 
> and it's not in the linux kernel text either.  AFAICT this is your 
> invention and you seem to be the only person proposing it.

Nope, it's fairly common, so much so that it's included in the "commonly 
accepted rules" that googling for "git subject lines" gives you (as a 
snippet glimpse from some website), and that vim changes color when 
spilling over 50 chars.  I actually thought it was universal and obvious 
until this thread (which is why I admittedly only did the above google 
right before writing this mail).  For the rationale: 'git log --oneline' 
with hash and author or date should fit the usual 72 char limit.  (An 
11-character hash plus space alone would come out as 60 chars for the 
subject)

That's also the reason why some people (certainly me) are nervous about or 
dislike all the "tags" in the subject line.  E.g. what essential 
information (and subjects are for essential info, right?) is "[committed]" 
(or, even worse "[patch]") supposed to transport?  If the rest of the 
subject doesn't interest me, I don't care if something was committed or 
not; if it _does_ interest me, then I'm going to look at the mail/patch 
either way, if committed or not; at which point the info if the author 
required review or has already committed it could be gives in the body as 
well.  Similar for some other metainfo tags.  (The "subsystem:" is good, 
though).

And if we must have these tags, then why not at least short ones?  Why 
isn't "[cmt]" or something enough?  There will be very few tags, so they 
become mnemonic pretty much immediately.  What becomes clearer when 
writing "[patch v2 1/13]" in comparison to "[v2 1/13]"?


Ciao,
Michael.

  parent reply	other threads:[~2020-02-03 17:31 UTC|newest]

Thread overview: 49+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <c3928f40-2d71-fb5b-f2e0-3878ac88a2b7@arm.com>
     [not found] ` <alpine.LSU.2.21.2001191425160.739@anthias.pfeifer.com>
     [not found]   ` <353faf3e-bf43-eb4d-542d-45a53dce77b2@arm.com>
2020-01-21 15:40     ` [PATCH, v2] " Richard Earnshaw (lists)
2020-01-21 16:03       ` Jakub Jelinek
2020-01-21 16:14         ` Richard Earnshaw (lists)
2020-01-21 16:38           ` Jakub Jelinek
2020-01-21 16:43             ` Richard Earnshaw (lists)
2020-01-21 19:27               ` Jason Merrill
2020-01-22  3:46                 ` Richard Earnshaw (lists)
2020-01-22 10:00                   ` Jakub Jelinek
2020-01-22 13:50                     ` Richard Earnshaw (lists)
2020-01-22 13:54                       ` Jakub Jelinek
2020-01-22 17:41                   ` Richard Sandiford
2020-01-22 17:45                     ` Marek Polacek
2020-01-22 17:50                       ` Richard Earnshaw (lists)
2020-01-22  9:07       ` Segher Boessenkool
2020-01-22 16:05         ` Richard Earnshaw (lists)
2020-01-22 16:37           ` Segher Boessenkool
2020-01-22 18:49       ` [PATCH, v3] " Richard Earnshaw (lists)
2020-02-03 11:41         ` Richard Earnshaw (lists)
2020-02-03 11:54           ` Alexander Monakov
2020-02-03 11:59             ` Richard Earnshaw (lists)
2020-02-03 12:51               ` Alexander Monakov
2020-02-03 14:11                 ` Jason Merrill
2020-02-03 15:13                   ` Richard Earnshaw (lists)
2020-02-03 15:15                     ` Richard Earnshaw (lists)
2020-02-03 13:54             ` Segher Boessenkool
2020-02-03 14:00               ` Richard Earnshaw (lists)
2020-02-03 14:13                 ` Jonathan Wakely
2020-02-03 15:05                   ` Richard Earnshaw (lists)
2020-02-03 16:12                     ` Andrew Clayton
2020-02-03 17:17                 ` Segher Boessenkool
2020-02-03 17:19                 ` Segher Boessenkool
2020-02-03 17:31                 ` Michael Matz [this message]
2020-02-03 17:36                   ` Richard Earnshaw (lists)
2020-02-03 17:49                     ` Michael Matz
2020-02-03 17:54                       ` Richard Earnshaw (lists)
2020-02-03 18:20                         ` Michael Matz
2020-02-03 19:48                           ` Segher Boessenkool
2020-02-03 17:54                       ` Jakub Jelinek
2020-02-03 18:09                         ` Michael Matz
2020-02-04 10:41                           ` Andrew Stubbs
2020-02-03 18:57                         ` Segher Boessenkool
2020-02-03 18:03                       ` Joseph Myers
2020-02-03 17:34                 ` Segher Boessenkool
2020-02-27 13:38           ` Nathan Sidwell
2020-03-02 13:01             ` Richard Earnshaw (lists)
2020-03-02 13:35               ` Segher Boessenkool
2020-03-02 14:31               ` Nathan Sidwell
2020-03-02 14:41                 ` Jonathan Wakely
2020-03-02 15:22                   ` Richard Earnshaw (lists)

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=alpine.LSU.2.21.2002031657430.12763@wotan.suse.de \
    --to=matz@suse.de \
    --cc=Richard.Earnshaw@arm.com \
    --cc=amonakov@ispras.ru \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=gcc@gcc.gnu.org \
    --cc=gerald@pfeifer.com \
    --cc=segher@kernel.crashing.org \
    /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).