On 2016-05-13 at 00:40:03 +0200, Edward Z. Yang wrote:
Currently the commit hooks reject commit messages that look like:
Summary: Signed-off-by: Edward Z. Yang
Unfortunately, if I do a one line commit message with -s, Phabricator will automatically reformat my message to have this.
...and that's exactly why I invested time to write that hook in the first place, because Phabricator blatantly violates Git convention regarding the git commit message format. Stock Phabricator insists on its own "structured" Git commit format, thereby breaking almost every Git-tool which expects the usual git-trailer grammar. I've been meaning to fix this (and another related issue of Phabricator using whitespaces in trailer-keys[1] which unsurpisingly breaks git tooling as well but for a different reason, as well as the annoying issue of Signed-off-by lines sometimes confusing Phabricator...*sigh*) in Phabricator for some time but never got to it, but...
2) You should patch Phabricator to stop munging the messages this way. The message formatting is done server-side so as a client I have no way of changing it, you need to fix it. This URL has some guidance on how to do it: https://secure.phabricator.com/T6055
I'd offer to edit the hook myself but there does not seem to be any canonical location where the hooks are versioned.
...I've tried to patch phabricator as suggested in T6055 (and commented there), but it seems I need to tweak Phabricator's commit msg parser as well... [1]: E.g. "Reviewed By:" instead of the better supported "Reviewed-by:", see e.g. https://git.wiki.kernel.org/index.php/CommitMessageConventions