| Summary: | Fine tuning the ARM EFL bots | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Gabor Rapcsanyi <rgabor> | ||||
| Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> | ||||
| Status: | RESOLVED FIXED | ||||||
| Severity: | Normal | CC: | commit-queue, lforschler, ossy | ||||
| Priority: | P2 | ||||||
| Version: | 528+ (Nightly build) | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Attachments: |
|
||||||
|
Description
Gabor Rapcsanyi
2014-02-04 01:59:41 PST
Created attachment 223089 [details]
proposed patch
Comment on attachment 223089 [details]
proposed patch
r=me.
Lucas, could you possibly restart the master after the landing?
Comment on attachment 223089 [details] proposed patch Clearing flags on attachment: 223089 Committed r163372: <http://trac.webkit.org/changeset/163372> All reviewed patches have been landed. Closing bug. master restarted. Comment on attachment 223089 [details] proposed patch View in context: https://bugs.webkit.org/attachment.cgi?id=223089&action=review > Tools/BuildSlaveSupport/build.webkit.org-config/master.cfg:139 > + def evaluateCommand(self, cmd): > + if cmd.rc != 0: > + return WARNINGS I think I got what caused the exception. You should return with SUCCESS after it, similar to other evaluateCommands in master.cfg (In reply to comment #6) > (From update of attachment 223089 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=223089&action=review > > > Tools/BuildSlaveSupport/build.webkit.org-config/master.cfg:139 > > + def evaluateCommand(self, cmd): > > + if cmd.rc != 0: > > + return WARNINGS > > I think I got what caused the exception. You > should return with SUCCESS after it, similar > to other evaluateCommands in master.cfg Yes sure, the fix landed at: https://trac.webkit.org/r163441 I've restarted the webkit build master for you. |