git-add-reviewer: Do not put the bug title and its URL in the same line in the commit...
authorrakuco@webkit.org <rakuco@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 4 May 2012 03:25:07 +0000 (03:25 +0000)
committerrakuco@webkit.org <rakuco@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 4 May 2012 03:25:07 +0000 (03:25 +0000)
https://bugs.webkit.org/show_bug.cgi?id=85570

Reviewed by Adam Roben.

The standard format of commit messages and ChangeLog entries has a
short summary in the first line, followed by the bug URL in
Bugzilla. Passing "%s" in the pretty format line to git-rev-list
merges these two lines into a single one separated by a space
character, which is not what we want.

Since "%s%n%n%b" is in practice "%B" (raw message), use it
instead, as the first two lines do not end up being merged
together.

* Scripts/git-add-reviewer:
(writeCommitMessageToFile):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@116064 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/Scripts/git-add-reviewer

index b8b2f5a..05691a7 100644 (file)
@@ -1,3 +1,23 @@
+2012-05-03  Raphael Kubo da Costa  <rakuco@webkit.org>
+
+        git-add-reviewer: Do not put the bug title and its URL in the same line in the commit message.
+        https://bugs.webkit.org/show_bug.cgi?id=85570
+
+        Reviewed by Adam Roben.
+
+        The standard format of commit messages and ChangeLog entries has a
+        short summary in the first line, followed by the bug URL in
+        Bugzilla. Passing "%s" in the pretty format line to git-rev-list
+        merges these two lines into a single one separated by a space
+        character, which is not what we want.
+
+        Since "%s%n%n%b" is in practice "%B" (raw message), use it
+        instead, as the first two lines do not end up being merged
+        together.
+
+        * Scripts/git-add-reviewer:
+        (writeCommitMessageToFile):
+
 2012-05-03  Ojan Vafai  <ojan@chromium.org>
 
         Improve UI for garden-o-matic examine view when there are fewer than 5 tests.
index 4b977a8..16bb06b 100755 (executable)
@@ -346,7 +346,7 @@ sub writeCommitMessageToFile($)
     my ($file) = @_;
 
     open FILE, ">", $file or return fail("Couldn't open $file.");
-    open MESSAGE, "-|", qw(git rev-list --max-count=1 --pretty=format:%s%n%n%b HEAD) or return fail("Error running git rev-list.");
+    open MESSAGE, "-|", qw(git rev-list --max-count=1 --pretty=format:%B HEAD) or return fail("Error running git rev-list.");
     my $commitLine = <MESSAGE>;
     foreach my $line (<MESSAGE>) {
         print FILE $line;