Linus Torvalds <torvalds@xxxxxxxx> writes: > Yes. However, I think the sign-off detection is a bit broken (quite > independently of your patch). > > A number of people end up capitalizing the sign-off differently, so you > have lines like "Signed-Off-By: Xy Zzy <xyzzy@xxxxxxxxxxx>". > > Also, at least for the kernel, we often have alternative formats, like > > Acked-by: Elliot Xavier Ample <example@xxxxxxxxx> > > and for that case, adding the extra newline is actually bad. > > So I would suggest a totally different approach: instead of using > "strstr(comments, signed_off_by)", it would probably be much better to > just look for the last non-empty line, and see if it matches the format > > "^[nonspace]*: .*@.*$" Documentation/SubmittingPatches (the kernel one) does not show the ugly Camel-Case-With-Hyphen spelling, and I've been wondring why people do that. A hidden agenda by me was to migrate people away from that practice, but that is an independent issue ;-). I like your "detect lines that looks like a RFC2822 header that has some e-mail address" approach quite a lot. - : 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