RESOLVED FIXED99765
[chromium] garden-o-matic should ignore the 10.8 Tests bot 
https://bugs.webkit.org/show_bug.cgi?id=99765
Summary [chromium] garden-o-matic should ignore the 10.8 Tests bot
Dirk Pranke
Reported 2012-10-18 15:34:02 PDT
[chromium] garden-o-matic should ignore the 10.8 Tests bot
Attachments
Patch (2.03 KB, patch)
2012-10-18 15:34 PDT, Dirk Pranke 		abarth: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Dirk Pranke
Comment 1 2012-10-18 15:34:41 PDT
Created attachment 169490 [details] Patch
Adam Barth
Comment 2 2012-10-18 16:00:00 PDT
Comment on attachment 169490 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=169490&action=review > Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/config.js:92 > + return builderName.indexOf('GPU') == -1 && builderName.indexOf('deps') == -1 && builderName.indexOf('ASAN') == -1 && builderName.indexOf('Content Shell') == -1 && builderName.indexOf('Mac10.8 Tests') == -1; I would probably have re-formated this line to make the add/remove diffs easier to read.
Dirk Pranke
Comment 3 2012-10-18 17:19:17 PDT
(In reply to comment #2) > (From update of attachment 169490 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=169490&action=review > > > Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/config.js:92 > > + return builderName.indexOf('GPU') == -1 && builderName.indexOf('deps') == -1 && builderName.indexOf('ASAN') == -1 && builderName.indexOf('Content Shell') == -1 && builderName.indexOf('Mac10.8 Tests') == -1; > > I would probably have re-formated this line to make the add/remove diffs easier to read. ok, will fix.
Dirk Pranke
Comment 4 2012-10-18 17:21:18 PDT
Committed r131827: <http://trac.webkit.org/changeset/131827>
Note You need to log in before you can comment on or make changes to this bug.