Title: [243217] trunk/Tools
Revision
243217
Author
aakash_j...@apple.com
Date
2019-03-20 11:10:32 -0700 (Wed, 20 Mar 2019)

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

Reply via email to