Make prepare-ChangeLog -g <commit> generate a more standard ChangeLog entry.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 12 Jul 2017 17:16:48 +0000 (17:16 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 12 Jul 2017 17:16:48 +0000 (17:16 +0000)
https://bugs.webkit.org/show_bug.cgi?id=174197

Patch by Emilio Cobos Álvarez <ecobos@igalia.com> on 2017-07-12
Reviewed by Darin Adler.

This bit me the first time I tried to submit a patch to WebKit, and still I have
to manually edit the ChangeLog every time I use the script.

This generates a more convenient and standard ChangeLog entry.

* Scripts/prepare-ChangeLog:
(generateNewChangeLogs):

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

Tools/ChangeLog
Tools/Scripts/prepare-ChangeLog

index 4c5ccfb..8a39fcc 100644 (file)
@@ -1,3 +1,18 @@
+2017-07-12  Emilio Cobos Álvarez  <ecobos@igalia.com>
+
+        Make prepare-ChangeLog -g <commit> generate a more standard ChangeLog entry.
+        https://bugs.webkit.org/show_bug.cgi?id=174197
+
+        Reviewed by Darin Adler.
+
+        This bit me the first time I tried to submit a patch to WebKit, and still I have
+        to manually edit the ChangeLog every time I use the script.
+
+        This generates a more convenient and standard ChangeLog entry.
+
+        * Scripts/prepare-ChangeLog:
+        (generateNewChangeLogs):
+
 2017-07-12  Matt Lewis  <jlewis3@apple.com>
 
         Unreviewed, rolling out r219401.
index bee7778..0f44f20 100755 (executable)
@@ -673,15 +673,20 @@ sub generateNewChangeLogs($$$$$$$$$$$$$$)
         my ($reviewer, $description) = reviewerAndDescriptionForGitCommit($gitCommit, $gitReviewer) if $gitCommit;
         $reviewer = "NOBODY (OO" . "PS!)" if !$reviewer;
 
-        print CHANGE_LOG normalizeLineEndings($description . "\n", $endl) if $description;
+        ($bugDescription, $description) =
+            ($description =~ /^(?:\s*(.*)\n)?(?:\s*\n)*((?:\n|.)*)/)
+            if !$bugDescription && $description;
+
+        $bugDescription = "Need a short description (OOPS!)." unless $bugDescription;
+        $bugURL = "Need the bug URL (OOPS!)." unless $bugURL;
 
-        $bugDescription = "Need a short description (OOPS!).\n        Need the bug URL (OOPS!)." unless $bugDescription;
         print CHANGE_LOG normalizeLineEndings("        $bugDescription\n", $endl) if $bugDescription;
         print CHANGE_LOG normalizeLineEndings("        $bugURL\n", $endl) if $bugURL;
         print CHANGE_LOG normalizeLineEndings("        $bugRadarURL\n", $endl) if $bugRadarURL;
         print CHANGE_LOG normalizeLineEndings("\n", $endl);
 
         print CHANGE_LOG normalizeLineEndings("        Reviewed by $reviewer.\n\n", $endl);
+        print CHANGE_LOG normalizeLineEndings($description . "\n", $endl) if $description;
 
         if ($prefix =~ m/WebCore/ || `pwd` =~ m/WebCore/ || @$requiresTests) {
             if (@$addedRegressionTests) {