Changeset 92055 in webkit

Timestamp:

Jul 30, 2011 11:25:09 AM (13 years ago)

Author:

abarth@webkit.org

Message:

garden-o-matic should look for the "patch by" line in revisions ostensibly authored by the commit-queue
​https://bugs.webkit.org/show_bug.cgi?id=65231

Reviewed by Dimitri Glazkov.

The human readable name is easier to understand, especially when the patch is landed by the commit-queue.

• BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/Trac.js:

Location:

trunk/Tools

Files:

• BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/Trac.js (modified) (2 diffs)
• ChangeLog (modified) (1 diff)

trunk/Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/Trac.js

@@ -39 +39 @@
 {
     var regexp = /Reviewed by ([^.]+)/;
+    return findUsingRegExp(message, regexp);
+}
+
+function findAuthor(message)
+{
+    var regexp = /Patch by ([^<]+) </;
     return findUsingRegExp(message, regexp);
 }
@@ -97 +103 @@
             'title': title,
             'summary': findSummary(message),
-            'author': author,
+            'author': findAuthor(message) || author,
             'reviewer': findReviewer(message),
             'bugID': findBugID(message),

trunk/Tools/ChangeLog

@@ +1 @@
+2011-07-30  Adam Barth  <abarth@webkit.org>
+
+        garden-o-matic should look for the "patch by" line in revisions ostensibly authored by the commit-queue
+        https://bugs.webkit.org/show_bug.cgi?id=65231
+
+        Reviewed by Dimitri Glazkov.
+
+        The human readable name is easier to understand, especially when the patch is landed by the commit-queue.
+
+        * BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/Trac.js:
+
 2011-07-30  Adam Barth  <abarth@webkit.org>