Merge pull request #1249 from djmitche/subscribeConsumer-assert

Remove assert from subscribeConsumer
buildbot · Oct 10, 2014 · 3c52778 · 3c52778
2 parents f43059f + 7248778
commit 3c52778
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/master/buildbot/status/logfile.py b/master/buildbot/status/logfile.py
@@ -431,7 +431,8 @@ def unsubscribe(self, receiver):
             self.watchers.remove(receiver)
 
     def subscribeConsumer(self, consumer):
-        assert not self._isNewStyle, "not available in new-style steps"
+        # NOTE: this method is called by WebStatus, so it must remain available
+        # even for new-style steps
         p = LogFileProducer(self, consumer)
         p.resumeProducing()