Changeset 249148 in webkit

Timestamp:

Aug 27, 2019, 10:18:18 AM (7 years ago)

Author:

aakash_jain@apple.com

Message:

[ews-build] Use update-webkit script in Apply-WatchList EWS
​https://bugs.webkit.org/show_bug.cgi?id=201179

Reviewed by Jonathan Bedard.

• BuildSlaveSupport/ews-build/factories.py:

(WatchListFactory): Updated to use CheckOutSource step as well.

Location:

trunk/Tools

Files:

• BuildSlaveSupport/ews-build/factories.py (modified) (2 diffs)
• ChangeLog (modified) (1 diff)

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

@@ -59 +59 @@
         self.addStep(ApplyPatch())
         self.addStep(CheckStyle())
+
+
+class WatchListFactory(factory.BuildFactory):
+    def __init__(self, platform, configuration=None, architectures=None, triggers=None, additionalArguments=None, **kwargs):
+        factory.BuildFactory.__init__(self)
+        self.addStep(ConfigureBuild(platform, configuration, architectures, False, triggers, additionalArguments))
+        self.addStep(ValidatePatch())
+        self.addStep(PrintConfiguration())
+        self.addStep(CheckOutSource())
+        self.addStep(UpdateWorkingDirectory())
+        self.addStep(ApplyPatch())
+        self.addStep(ApplyWatchList())
 
 
@@ -176 +188 @@
         self.addStep(RunEWSUnitTests())
         self.addStep(RunEWSBuildbotCheckConfig())
-
-
-class WatchListFactory(Factory):
-    def __init__(self, platform, configuration=None, architectures=None, triggers=None, additionalArguments=None, **kwargs):
-        Factory.__init__(self, platform, configuration, architectures, False, triggers, additionalArguments)
-        self.addStep(ApplyWatchList())

trunk/Tools/ChangeLog

@@ +1 @@
+2019-08-27  Aakash Jain  <aakash_jain@apple.com>
+
+        [ews-build] Use update-webkit script in Apply-WatchList EWS
+        https://bugs.webkit.org/show_bug.cgi?id=201179
+
+        Reviewed by Jonathan Bedard.
+
+        * BuildSlaveSupport/ews-build/factories.py:
+        (WatchListFactory): Updated to use CheckOutSource step as well.
+
 2019-08-27  Carlos Alberto Lopez Perez  <clopez@igalia.com>