Add High Sierra bots to buildbot and dashboard
Created attachment 321715 [details] Patch
Comment on attachment 321715 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=321715&action=review > Tools/BuildSlaveSupport/build.webkit.org-config/config.json:170 > + "triggers": ["sierra-release-tests-wk1", "sierra-release-tests-wk2", "sierra-leaks"], I think that "sierra-leaks" needs to be removed too. > Tools/BuildSlaveSupport/build.webkit.org-config/config.json:411 > "builderNames": ["Apple Sierra Release (Build)", "Apple Sierra Debug (Build)", "Apple Sierra Release (32-bit Build)", "Apple Sierra LLINT CLoop (BuildAndTest)", "Apple Sierra 32-bit JSC (BuildAndTest)"] I think that some of these need to be removed (JSC, 32-bit).
(In reply to Alexey Proskuryakov from comment #2) > Comment on attachment 321715 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=321715&action=review > > > Tools/BuildSlaveSupport/build.webkit.org-config/config.json:170 > > + "triggers": ["sierra-release-tests-wk1", "sierra-release-tests-wk2", "sierra-leaks"], > > I think that "sierra-leaks" needs to be removed too. Thanks, I will remove this. > > > Tools/BuildSlaveSupport/build.webkit.org-config/config.json:411 > > "builderNames": ["Apple Sierra Release (Build)", "Apple Sierra Debug (Build)", "Apple Sierra Release (32-bit Build)", "Apple Sierra LLINT CLoop (BuildAndTest)", "Apple Sierra 32-bit JSC (BuildAndTest)"] > > I think that some of these need to be removed (JSC, 32-bit). Ditto. Unfortunately unit tests didn't catch these errors. I'll have to file a bug for that. Thanks for the review!
Created attachment 321734 [details] Patch for landing
Comment on attachment 321734 [details] Patch for landing Clearing flags on attachment: 321734 Committed r222469: <http://trac.webkit.org/changeset/222469>
All reviewed patches have been landed. Closing bug.
<rdar://problem/34692989>