Re: Fw: [PATCH 31/49] PCI: PCI/Cardbus cards hidden, needs pci=assign-busses to fix

From: Andrew Morton <akpm@osdl.org>
Date: Mar 24, 2006, 01:51 UTC
Message-ID: <20060323175126.7ff71032.akpm@osdl.org>
In-Reply-To: <7vbqvw3a62.fsf@assigned-by-dhcp.cox.net>

Junio C Hamano <junkio@cox.net> wrote:

Show 10 quoted lines

>
> The second best would be to add the duplicated From: to name the
> author (who is _not_ you) to the top of the body of the message.
> I do not particularly like that format myself, though.  Sender:
> header was invented to send an e-mail authored by somebody other
> than the sender of the message at the mail transport level, long
> before Documentation/SubmittingPatches were written and git was
> invented, and somehow I think that is a more kosher way to
> handle that than the "extra From: at the beginning of the
> message" clutch recommended in SubmittingPatches document.

The email I received from Greg had no Sender: header at all. I could find no indication of who authored the patch in that email.

The convention of adding the From: to the top of the body of the changelog is explicit and simple - I think it's a reasonable thing to do.

We wouldn't want to attempt to mix this concept up with email envelopes or email headers or anything like that. The authorship is an attribute of the patch, and has nothing to do with how it was transported, stored or anything like that.