Changeset 51275 in webkit

Timestamp:

Nov 20, 2009 7:14:51 PM (14 years ago)

Author:

eric@webkit.org

Message:

2009-11-20 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Fix exception thrown when running the commit-queue.

• Scripts/modules/statusbot.py: patch is optional.
• Scripts/modules/workqueue.py: WorkQUeue requires a name.

Location:

trunk/WebKitTools

Files:

• ChangeLog (modified) (1 diff)
• Scripts/modules/statusbot.py (modified) (1 diff)
• Scripts/modules/workqueue.py (modified) (1 diff)

trunk/WebKitTools/ChangeLog

@@ +1 @@
+2009-11-20  Eric Seidel  <eric@webkit.org>
+
+        Reviewed by Adam Barth.
+
+        Fix exception thrown when running the commit-queue.
+
+        * Scripts/modules/statusbot.py: patch is optional.
+        * Scripts/modules/workqueue.py: WorkQUeue requires a name.
+
 2009-11-20  Eric Seidel  <eric@webkit.org>
 

trunk/WebKitTools/Scripts/modules/statusbot.py

@@ -58 +58 @@
         self.browser = Browser()
 
-    def update_status(self, queue_name, status, patch):
+    def update_status(self, queue_name, status, patch=None):
         # During unit testing, statusbot_host is None
         if not self.statusbot_host:

trunk/WebKitTools/Scripts/modules/workqueue.py

@@ -149 +149 @@
     def _update_status_and_sleep(self, message):
         status_message = self._sleep_message(message)
-        self.status_bot.update_status(status_message)
+        self.status_bot.update_status(self._name, status_message)
         log(status_message)
         time.sleep(self.seconds_to_sleep)