Changeset 690

Timestamp:

04/15/08 12:20:22 (9 months ago)

Author:

zooko@zooko.com

Message:

don't create a new missing-timer task if there is already one there
Doing that would mean that the one that is already there will not be cleaned up if the buildslave reattaches before the timer goes off, so this patch probably fixed #211.

Files:

• buildbot/buildslave.py (modified) (1 diff)

buildbot/buildslave.py

@@ -199 +199 @@
-        self.botmaster.slaveLost(self)
-        log.msg("BuildSlave.detached(%s)" % self.slavename)
-
+ and not self.missing_timer
-            self.missing_timer = reactor.callLater(self.missing_timeout,
-                                                   self._missing_timer_fired)