Title: [240107] trunk/Tools
Revision
240107
Author
aakash_j...@apple.com
Date
2019-01-17 05:52:45 -0800 (Thu, 17 Jan 2019)

Log Message

[ews-build] unapply-patch step should not update source
https://bugs.webkit.org/show_bug.cgi?id=193234

Reviewed by Lucas Forschler.

* BuildSlaveSupport/ews-build/steps.py:
(CleanWorkingDirectory): Use clean-webkit script to clean the working directory.
(UnApplyPatchIfRequired): Use CleanWorkingDirectory as base class.

Modified Paths

Diff

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


--- trunk/Tools/BuildSlaveSupport/ews-build/steps.py	2019-01-17 08:10:53 UTC (rev 240106)
+++ trunk/Tools/BuildSlaveSupport/ews-build/steps.py	2019-01-17 13:52:45 UTC (rev 240107)
@@ -95,6 +95,15 @@
                                                 **kwargs)
 
 
+class CleanWorkingDirectory(shell.ShellCommand):
+    name = 'clean-working-directory'
+    description = ['clean-working-directory running']
+    descriptionDone = ['clean-working-directory']
+    flunkOnFailure = True
+    haltOnFailure = True
+    command = ['Tools/Scripts/clean-webkit']
+
+
 class ApplyPatch(shell.ShellCommand, CompositeStepMixin):
     name = 'apply-patch'
     description = ['applying-patch']
@@ -344,7 +353,7 @@
         return None
 
 
-class UnApplyPatchIfRequired(CheckOutSource):
+class UnApplyPatchIfRequired(CleanWorkingDirectory):
     name = 'unapply-patch'
 
     def doStepIf(self, step):

Modified: trunk/Tools/ChangeLog (240106 => 240107)


--- trunk/Tools/ChangeLog	2019-01-17 08:10:53 UTC (rev 240106)
+++ trunk/Tools/ChangeLog	2019-01-17 13:52:45 UTC (rev 240107)
@@ -1,3 +1,14 @@
+2019-01-17  Aakash Jain  <aakash_j...@apple.com>
+
+        [ews-build] unapply-patch step should not update source
+        https://bugs.webkit.org/show_bug.cgi?id=193234
+
+        Reviewed by Lucas Forschler.
+
+        * BuildSlaveSupport/ews-build/steps.py:
+        (CleanWorkingDirectory): Use clean-webkit script to clean the working directory.
+        (UnApplyPatchIfRequired): Use CleanWorkingDirectory as base class.
+
 2019-01-16  Youenn Fablet  <you...@apple.com>
 
         Add a new SPI for controlling getUserMedia
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to