Changeset 243217 in webkit

Timestamp:

Mar 20, 2019, 11:10:32 AM (7 years ago)

Author:

aakash_jain@apple.com

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.

Location:

trunk/Tools

Files:

• BuildSlaveSupport/ews-build/steps.py (modified) (1 diff)
• ChangeLog (modified) (1 diff)

trunk/Tools/BuildSlaveSupport/ews-build/steps.py

@@ -136 +136 @@
         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):

trunk/Tools/ChangeLog

@@ +1 @@
+2019-03-20  Aakash Jain  <aakash_jain@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  <jdiggs@igalia.com>