Re: git-format-patch should include a checksum

[Linus Torvalds <torvalds@linux-foundation.org>]

On Wed, 27 Jan 2010, Sverre Rabbelier wrote:
> 
> I would imagine that the checksum is taken over just the actual commit
> message, perhaps author information, and use the patch-id for the
> patch itself, that way any comments after triple dash would be ignored, right?

That wouldn't work either. People can, should, and do add extra things to 
the message before applying it.

Examples of things I tend to add/change in the commit message:
 - add ack's from people in the same thread
 - add "Cc: stable@kernel.org" 
 - re-flow paragraphs when somebody uses a mailer that makes a mess of it.
 - occasionally fix spelling and grammar

so if there is some checksum that screws that up and requires me to then 
use a "--force" flag to apply it, that would be a bad thing.

I also do edit patches manually too. Having lived with people sending me 
patches for the last almost twenty years, I can edit patches in my sleep. 
Doing things like renaming new variables etc by search-and-replace on the 
patch may not be something I do _often_, but it happens.

In short, it might make sense to have some anti-corruption logic, but I 
suspect it needs a lot of thought. 

		Linus