On Wed, 12 Jul 2006, Junio C Hamano wrote: > Linus Torvalds <torvalds@xxxxxxxx> writes: > > > > A number of people end up capitalizing the sign-off differently, so you > > have lines like "Signed-Off-By: Xy Zzy <xyzzy@xxxxxxxxxxx>". > > Documentation/SubmittingPatches (the kernel one) does not show > the ugly Camel-Case-With-Hyphen spelling, and I've been wondring > why people do that. Yeah, I actually try to edit it to the "proper" format when I notice it (which is not most of the time, but it's pretty rare to begin with). More commonly, people mistype their own email addresses, and _occasionally_ just mis-type the whole Signed-off-by: line (we've got a few semi-colons instead of colons in the kernel, for example, and some lines that are missing the final '>' in the email etc. So being somewhat forgiving might help, but I think another thing that migth help is a flag to "git-am" to _not_ apply a patch that lacks a previous sign-off. I, for example, don't use the --signoff flag, partly because I want to make sure that I sign of only on patches that already have a sign-off from the previous person when it comes as email (or I add the sign-off only after looking at the patch closely). But if there was a "--error-on-no-signoff" flag, I could use it. Linus - : 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