Attachment 371315 Details for Bug 198218 – Patch

[patch] Patch

bug-198218-20190604152013.patch (text/plain), 1.69 KB, created by Aakash Jain on 2019-06-04 12:20:14 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Aakash Jain

Created: 2019-06-04 12:20:14 PDT

Size: 1.69 KB

patch

obsolete

>Index: Tools/ChangeLog
>===================================================================
>--- Tools/ChangeLog	(revision 246076)
>+++ Tools/ChangeLog	(working copy)
>@@ -1,3 +1,15 @@
>+2019-06-04  Aakash Jain  <aakash_jain@apple.com>
>+
>+        [ews-build] Do not display unnecessary steps in the Buildbot build page UI
>+        https://bugs.webkit.org/show_bug.cgi?id=198218
>+        <rdar://problem/51104544>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * BuildSlaveSupport/ews-build/steps.py:
>+        (ApplyPatch.hideStepIf):
>+        (ValidatePatch.start):
>+
> 2019-06-04  Takashi Komori  <Takashi.Komori@sony.com>
> 
>         [WinCairo] Implement cpu and memory measuring functions.
>Index: Tools/BuildSlaveSupport/ews-build/steps.py
>===================================================================
>--- Tools/BuildSlaveSupport/ews-build/steps.py	(revision 246076)
>+++ Tools/BuildSlaveSupport/ews-build/steps.py	(working copy)
>@@ -124,6 +124,9 @@ class ApplyPatch(shell.ShellCommand, Com
>     haltOnFailure = True
>     command = ['Tools/Scripts/svn-apply', '--force', '.buildbot-diff']
> 
>+    def hideStepIf(self, results, step):
>+        return results == SUCCESS and self.getProperty('validated', '') == False
>+
>     def _get_patch(self):
>         sourcestamp = self.build.getSourceStamp(self.getProperty('codebase', ''))
>         if not sourcestamp or not sourcestamp.patch:
>@@ -368,6 +371,7 @@ class ValidatePatch(buildstep.BuildStep)
> 
>         if obsolete == -1 or review_denied == -1 or bug_closed == -1:
>             self.finished(WARNINGS)
>+            self.setProperty('validated', False)
>             return None
> 
>         self._addToLog('stdio', 'Bug is open.\nPatch is not obsolete.\nPatch is not marked r-.\n')

Actions: View | Formatted Diff | Diff

Attachments on bug 198218: 370570 | 370574 | 371315 | 371329