diff options
author | Mika Hamalainen <mika.hamalainen@accenture.com> | 2011-05-27 13:12:05 +0300 |
---|---|---|
committer | Mika Hamalainen <mika.hamalainen@accenture.com> | 2011-08-03 15:00:47 +0300 |
commit | b1934ee5b0bb61543e9cc9e3e69e9520c869fe35 (patch) | |
tree | fb1291174f6974d768c8663e544017ed9f752bc2 | |
parent | be85b877bba393c4826e6e5e8838e4690a48e8b7 (diff) |
[PATCH] don't look for footers in the first two lines of a commit message
Looking into the first line will make "tagged" subjects ("foo: bar")
throw off the footer logic. Looking into the second line makes no
sense whatsoever.
Change-Id: I0666d2354215207dd0f80f7976dac0ab6d198219
-rw-r--r-- | gerrit-server/src/main/resources/com/google/gerrit/server/tools/root/hooks/commit-msg | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gerrit-server/src/main/resources/com/google/gerrit/server/tools/root/hooks/commit-msg b/gerrit-server/src/main/resources/com/google/gerrit/server/tools/root/hooks/commit-msg index f83d6d805f..106bd97ffa 100644 --- a/gerrit-server/src/main/resources/com/google/gerrit/server/tools/root/hooks/commit-msg +++ b/gerrit-server/src/main/resources/com/google/gerrit/server/tools/root/hooks/commit-msg @@ -56,7 +56,7 @@ add_ChangeId() { @message = split /\n/; $haveFooter = 0; $startFooter = @message; - for($line = @message - 1; $line >= 0; $line--) { + for($line = @message - 1; $line >= 2; $line--) { $_ = $message[$line]; if (/^[a-zA-Z0-9-]+:/ && !m,^[a-z0-9-]+://,) { |