On Sat, 28 Jul 2007 06:38:19 -0700 Junio C Hamano <gitster@xxxxxxxxx> wrote: > Actually the patched behaviour actively encourages a bad (not in > the sense that those oneline tools will not work well, but in > the sense that these messages are reader unfriendly) practice; I > do not think what the patch did deserves to be called "fixed". > And that is one of the reasons, other than that we are in -rc > freeze that we do not add anything but unarguable fixes, that I > am not considering the patch for inclusion right now. > First, i didn't read the patch and i have no stake at all in non-conforming commit messages; i always follow the convention. Having said that, the current behavior of Git crosses the line from advocating the common commit message format into the realm of not-working properly with non-conforming commit messages. I would argue that you shouldn't try to have it both ways. Either Git supports non-conforming message formats, in which case the current behavior seems buggy. Or Git does not support commit messages that deviate from the standard, in which case the documentation should be updated to state so bluntly. If handling email-mangled commit messages means that non-conforming formats can no longer be fully supported (while discouraged) then perhaps the time has come to explicitly state that people should not expect Git to handle anything but the 1+empty+description commit message format. Sean - 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
