[ews-build] Improve failure summary string for ApplyPatch step
authoraakash_jain@apple.com <aakash_jain@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 20 Mar 2019 18:10:32 +0000 (18:10 +0000)
committeraakash_jain@apple.com <aakash_jain@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 20 Mar 2019 18:10:32 +0000 (18:10 +0000)
https://bugs.webkit.org/show_bug.cgi?id=195995

Reviewed by Alexey Proskuryakov.

* BuildSlaveSupport/ews-build/steps.py:
(ApplyPatch.getResultSummary): Override getResultSummary.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@243217 268f45cc-cd09-0410-ab3c-d52691b4dbfc

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

index 476b5df..b985be4 100644 (file)
@@ -135,6 +135,11 @@ class ApplyPatch(shell.ShellCommand, CompositeStepMixin):
         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'
index 9146997..0d59fb6 100644 (file)
@@ -1,3 +1,13 @@
+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>
 
         AX: Implement support for new meter ARIA role