Log Message
[ews-build] Improve failure summary string for ApplyPatch step https://bugs.webkit.org/show_bug.cgi?id=195995
Reviewed by Alexey Proskuryakov. * BuildSlaveSupport/ews-build/steps.py: (ApplyPatch.getResultSummary): Override getResultSummary.
Modified Paths
Diff
Modified: trunk/Tools/BuildSlaveSupport/ews-build/steps.py (243216 => 243217)
--- trunk/Tools/BuildSlaveSupport/ews-build/steps.py 2019-03-20 18:03:39 UTC (rev 243216)
+++ trunk/Tools/BuildSlaveSupport/ews-build/steps.py 2019-03-20 18:10:32 UTC (rev 243217)
@@ -135,7 +135,12 @@
d.addCallback(lambda _: self.downloadFileContentToWorker('.buildbot-patched', 'patched\n'))
d.addCallback(lambda res: shell.ShellCommand.start(self))
+ def getResultSummary(self):
+ if self.results != SUCCESS:
+ return {u'step': u'Patch does not apply'}
+ return super(ApplyPatch, self).getResultSummary()
+
class CheckPatchRelevance(buildstep.BuildStep):
name = 'check-patch-relevance'
description = ['check-patch-relevance running']
Modified: trunk/Tools/ChangeLog (243216 => 243217)
--- trunk/Tools/ChangeLog 2019-03-20 18:03:39 UTC (rev 243216)
+++ trunk/Tools/ChangeLog 2019-03-20 18:10:32 UTC (rev 243217)
@@ -1,3 +1,13 @@
+2019-03-20 Aakash Jain <aakash_j...@apple.com>
+
+ [ews-build] Improve failure summary string for ApplyPatch step
+ https://bugs.webkit.org/show_bug.cgi?id=195995
+
+ Reviewed by Alexey Proskuryakov.
+
+ * BuildSlaveSupport/ews-build/steps.py:
+ (ApplyPatch.getResultSummary): Override getResultSummary.
+
2019-03-20 Joanmarie Diggs <jdi...@igalia.com>
AX: Implement support for new meter ARIA role
_______________________________________________ webkit-changes mailing list webkit-changes@lists.webkit.org https://lists.webkit.org/mailman/listinfo/webkit-changes