Re: git-applymbox broken?

[Linus Torvalds <torvalds@osdl.org>]

On Mon, 12 Jun 2006, Eric W. Biederman wrote:
> 
> Below is an example of the kind of patch that inspired me to relax the
> rules on parsing in body headers (this comes from Andi Kleen quilt tree).

And this is wrong.

We should _not_ accept crappy patches, and then start guessing at what the 
person meant.

>From the very beginning of git, I tried to make it extremely clear that 
there is never any guessing going on. We don't use "heuristics" except as 
a pure optimization: ie a heuristic can have a _performance_ impact, but 
it must never EVER have semantic impact.

SCM's are not about guessing. They are about saving the _exact_ state that 
the user asked for. No "let's try to be nice", no gray areas.

If the new git-applymbox just takes random lines from the body of the 
email, and decides that they may be authorship information, then that is a 
BUG. The "From: " line in the middle of an email may well be about 
somebody having _discovered_ the bug, and we're quoting him as part of the 
explanation. It does NOT mean that it's about authorship.

So we should ONLY check for "From:" (and perhaps "Subject:" and "Date:") 
at the very top of the email body. NOWHERE ELSE.

The fact that somebody has a crappy quilt tree, and the fact that quilt is 
very much a "anything goes" kind of laissez faire system does not mean, 
and should NEVER mean that git becomes the same kind of mess of "let's do 
a best effort and try to guess what somebody means" kind of thing.

I check and edit my emails before I apply them, and I try to teach the 
people who send them manners and what the rules are. THAT is the way to 
handle this, not by having the tool itself become unreliable and random

		Linus