prepare-ChangeLog should not list added layout tests in PAL ChangeLog
authordbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 28 Jul 2017 22:31:37 +0000 (22:31 +0000)
committerdbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 28 Jul 2017 22:31:37 +0000 (22:31 +0000)
https://bugs.webkit.org/show_bug.cgi?id=174876

Reviewed by Alex Christensen.

Do not list added layout tests in the PAL ChangeLog as such tests are unlikely to be related
to a PAL change. Unit tests are more appropriate for PAL changes than layout tests because
they provide a more direct way to test the platform abstraction.

As a side effect of this change we no longer list added layout tests in WebCore/platform/gtk/po/ChangeLog.

* Scripts/prepare-ChangeLog:
(generateNewChangeLogs): Only list added layout tests in the WebCore ChangeLog regardless
of the current working directory.

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

Tools/ChangeLog
Tools/Scripts/prepare-ChangeLog

index fccbc19..36480c7 100644 (file)
@@ -1,3 +1,20 @@
+2017-07-28  Daniel Bates  <dabates@apple.com>
+
+        prepare-ChangeLog should not list added layout tests in PAL ChangeLog
+        https://bugs.webkit.org/show_bug.cgi?id=174876
+
+        Reviewed by Alex Christensen.
+
+        Do not list added layout tests in the PAL ChangeLog as such tests are unlikely to be related
+        to a PAL change. Unit tests are more appropriate for PAL changes than layout tests because
+        they provide a more direct way to test the platform abstraction.
+
+        As a side effect of this change we no longer list added layout tests in WebCore/platform/gtk/po/ChangeLog.
+
+        * Scripts/prepare-ChangeLog:
+        (generateNewChangeLogs): Only list added layout tests in the WebCore ChangeLog regardless
+        of the current working directory.
+
 2017-07-28  Jonathan Bedard  <jbedard@apple.com>
 
         Using "auto <function()> -> returnType" breaks prepare-ChangeLog
index b43ad92..e329c59 100755 (executable)
@@ -688,7 +688,7 @@ sub generateNewChangeLogs($$$$$$$$$$$$$$)
         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 (unixPath($prefix) =~ m|/WebCore/$| || @$requiresTests) {
             if (@$addedRegressionTests) {
                 print CHANGE_LOG normalizeLineEndings(testListForChangeLog(sort @$addedRegressionTests), $endl);
             } else {