Messages in this thread | | | Date | Sat, 28 Feb 2009 20:25:07 +0100 | From | Stefan Richter <> | Subject | Re: [PATCH] checkpatch: Warn on empty commit log bodies |
| |
Mark Brown wrote: [some people disregard the patch title in the Subject header] > I believe the issue is the UI of a MUA - the subject line of a mail is > normally presented separately to the body and isn't always as > immediately prominent as the body so it's harder work to look at it.
That's not universally true for all MUA; and in affected MUAs only if mails are read one-after-another, not if mails are read by looking at the list of messages in a mailbox first, then going to the interesting messages. So if there are people who disregard the patch title for this reason, then I dare to say it is because their view is limited by their particular MUA and their individual mail reading preferences. Their preferences still don't make the convention go away that the Subject is the title. (And that the title should be a quick intro into what the patch is about.)
> This is sensible for e-mail since the general style is that the subject > line shouldn't be required in order to comprehend what the message is > about
No. The subject is the primary means to establish context for the message (along with the mailinglist topic). We ask people to post with a good subject.
And since the subject already established context, there is no need to repeat its information in the e-mail body.
> The connection with git is that it doesn't really draw a similar > distinction so the issue isn't as immediately obvious when you're > working within it.
It isn't just a non-obvious issue with git, it is *no issue* with git in the first place. It also is no issue with other patch importing tools like quilt. All those tools either explicitly or implicitly support the notion that the RFC 2822 Subject header contains the patch title. They support it because the *people* who use those tools support it.
Since "Subject = title = beginning of changelog" is the long established norm and since the other patch handling tools (and people who handle patches) support this norm, checkpatch should follow this convention as well and count a non-empty RFC 2822 Subject header as one non-empty changelog line. -- Stefan Richter -=====-==--= --=- ===-- http://arcgraph.de/sr/
| |