All of lore.kernel.org
 help / color / mirror / Atom feed
From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
To: Alan Stern <stern@rowland.harvard.edu>
Cc: "Stefan Richter" <stefanr@s5r6.in-berlin.de>,
	"Julia Lawall" <julia@diku.dk>,
	"Németh Márton" <nm127@freemail.hu>,
	"David Vrabel" <david.vrabel@csr.com>,
	linux-usb@vger.kernel.org, LKML <linux-kernel@vger.kernel.org>,
	cocci@diku.dk
Subject: Re: Changelog quality
Date: Wed, 13 Jan 2010 18:49:42 +0100	[thread overview]
Message-ID: <201001131849.42328.bzolnier@gmail.com> (raw)
In-Reply-To: <Pine.LNX.4.44L0.1001131225160.3518-100000@iolanthe.rowland.org>

On Wednesday 13 January 2010 06:29:07 pm Alan Stern wrote:
> On Wed, 13 Jan 2010, Stefan Richter wrote:
> 
> > If it was by mistake, inclusion of the find-and-replace script into the
> > patch posting *after the --- delimiter* might have increased the chance
> > that a patch reviewer becomes aware of a possible error source
> > (inadequate match patterns...).  So that could be useful during review
> > before commit, but not so much if the change is revisited some time
> > after commit.
> 
> Somewhat tangentially, it's worth mentioning that the comments
> appearing after the "---" delimiter exist only in the original patch
> submissions, not in the final commits.  Hence they are not available to
> anyone reviewing the changes after acceptance.
> 
> It would be nice if there was a way to link automatically a git commit
> to an archived copy of the email message in which it was originally 
> submitted.

AFAIK x86 folks are using "LKML-Reference:" tag with the message ID for
exactly that purpose (just do 'git log arch/x86' to find such commits).

BTW Personally I see nothing wrong with too verbose commit changelogs,
too sparse changelogs are a much bigger annoyance..

--
Bartlomiej Zolnierkiewicz

  parent reply	other threads:[~2010-01-13 17:57 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-01-12  7:49 [PATCH] uwb: make USB device id constant Németh Márton
2010-01-12 16:10 ` David Vrabel
2010-01-12 16:57   ` Németh Márton
2010-01-13 14:59     ` Changelog quality (was Re: [PATCH] uwb: make USB device id constant) Stefan Richter
2010-01-13 15:38       ` Julia Lawall
2010-01-13 17:06         ` Changelog quality Stefan Richter
2010-01-13 17:29           ` Alan Stern
2010-01-13 17:44             ` Greg KH
2010-01-13 18:04               ` Alan Stern
2010-01-13 19:52                 ` Greg KH
2010-01-14  5:24                   ` Németh Márton
2010-01-14  6:05                     ` Julia Lawall
2010-01-14  8:07                       ` Stefan Richter
2010-01-14  8:26                     ` Dmitry Torokhov
2010-01-13 19:19               ` Geert Uytterhoeven
2010-01-13 17:49             ` Bartlomiej Zolnierkiewicz [this message]
2010-01-13 18:22               ` Stefan Richter
2010-01-15  1:03         ` Changelog quality (was Re: [PATCH] uwb: make USB device id constant) Andy Isaacson
2010-01-15  8:13           ` Stefan Richter
2010-01-15  8:24             ` Changelog quality David Miller
2010-01-15  8:50               ` Stefan Richter
2010-01-15  8:54                 ` David Miller
2010-01-15  9:17                   ` Stefan Richter
2010-01-15  9:22                     ` David Miller
2010-01-15  9:43                       ` Julia Lawall
2010-01-15  9:49                         ` Pekka Enberg
2010-01-15 10:05                           ` Julia Lawall
2010-01-15 11:08                             ` Mark Brown
2010-01-15 12:06                               ` Julia Lawall
2010-01-15 12:44                                 ` Pekka Enberg
2010-01-15 13:10                                   ` Julia Lawall
2010-01-15 12:45                                 ` Stefan Richter
2010-01-15 12:52                                   ` Pekka Enberg
2010-01-15 13:39                                 ` Mark Brown
2010-01-15 16:49                                   ` SmPL scripts into build environment? (was: Changelog quality) Németh Márton
2010-01-18 10:58                                     ` SmPL scripts into build environment? Michal Marek
2010-01-18 11:22                                       ` Julia Lawall
2010-01-15 13:28                       ` Changelog quality Stefan Richter

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=201001131849.42328.bzolnier@gmail.com \
    --to=bzolnier@gmail.com \
    --cc=cocci@diku.dk \
    --cc=david.vrabel@csr.com \
    --cc=julia@diku.dk \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=nm127@freemail.hu \
    --cc=stefanr@s5r6.in-berlin.de \
    --cc=stern@rowland.harvard.edu \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.