Title: [239008] trunk/Tools
Revision
239008
Author
aakash_j...@apple.com
Date
2018-12-08 05:02:47 -0800 (Sat, 08 Dec 2018)

Log Message

[ews-build] check-patch-relevance failure should not mark the overall build as failure
https://bugs.webkit.org/show_bug.cgi?id=192510

Reviewed by Lucas Forschler.

* BuildSlaveSupport/ews-build/steps.py:
(CheckPatchRelevance.start):

Modified Paths

Diff

Modified: trunk/Tools/BuildSlaveSupport/ews-build/steps.py (239007 => 239008)


--- trunk/Tools/BuildSlaveSupport/ews-build/steps.py	2018-12-08 07:35:47 UTC (rev 239007)
+++ trunk/Tools/BuildSlaveSupport/ews-build/steps.py	2018-12-08 13:02:47 UTC (rev 239008)
@@ -187,6 +187,8 @@
 
         self._addToLog('stdio', 'This patch does not have relevant changes.')
         self.finished(FAILURE)
+        self.build.results = SKIPPED
+        self.build.buildFinished(['Patch doesn\'t have relevant changes'], SKIPPED)
         return None
 
 

Modified: trunk/Tools/ChangeLog (239007 => 239008)


--- trunk/Tools/ChangeLog	2018-12-08 07:35:47 UTC (rev 239007)
+++ trunk/Tools/ChangeLog	2018-12-08 13:02:47 UTC (rev 239008)
@@ -1,3 +1,13 @@
+2018-12-07  Aakash Jain  <aakash_j...@apple.com>
+
+        [ews-build] check-patch-relevance failure should not mark the overall build as failure
+        https://bugs.webkit.org/show_bug.cgi?id=192510
+
+        Reviewed by Lucas Forschler.
+
+        * BuildSlaveSupport/ews-build/steps.py:
+        (CheckPatchRelevance.start):
+
 2018-12-07  Chris Dumez  <cdu...@apple.com>
 
         ProcessSwap.UseSessionCookiesAfterProcessSwapInPrivateBrowsing API test is failing
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to