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@xxxxxxxxxx" - 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 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html