From 5d3ecc2dd1de0f265227de55aee4ef8bbc9475eb Mon Sep 17 00:00:00 2001
From: Tom Prince <tom.prince@ualberta.net>
Date: Wed, 18 Jan 2012 22:34:57 -0500
Subject: [PATCH] IRC: Remove some of the extraneous logging and subscriptions.

see #2124.
---
 master/buildbot/status/words.py | 23 +++++------------------
 1 file changed, 5 insertions(+), 18 deletions(-)

diff --git a/master/buildbot/status/words.py b/master/buildbot/status/words.py
index aa004ae15cb..451ada68a1e 100644
--- a/master/buildbot/status/words.py
+++ b/master/buildbot/status/words.py
@@ -318,20 +318,14 @@ def command_WATCH(self, args, who):
             self.send(r)
     command_WATCH.usage = "watch <which> - announce the completion of an active build"
 
-    def buildsetSubmitted(self, buildset):
-        log.msg('[Contact] Buildset %s added' % (buildset))
-
     def builderAdded(self, builderName, builder):
+        if (self.channel.categories != None and
+            builder.category not in self.channel.categories):
+            return
+
         log.msg('[Contact] Builder %s added' % (builder))
         builder.subscribe(self)
 
-    def builderChangedState(self, builderName, state):
-        log.msg('[Contact] Builder %s changed state to %s' % (builderName, state))
-
-    def requestSubmitted(self, brstatus):
-        log.msg('[Contact] BuildRequest %d for %s submitted' %
-            (brstatus.brid, brstatus.getBuilderName()))
-
     def builderRemoved(self, builderName):
         log.msg('[Contact] Builder %s removed' % (builderName))
 
@@ -347,7 +341,6 @@ def buildStarted(self, builderName, build):
             return
 
         if not self.notify_for('started'):
-            log.msg('Not notifying for a build when started-notification disabled')
             return
 
         if self.useRevisions:
@@ -373,9 +366,6 @@ def buildStarted(self, builderName, build):
     def buildFinished(self, builderName, build, results):
         builder = build.getBuilder()
 
-        # only notify about builders we are interested in
-        log.msg('[Contact] builder %r in category %s finished' % (builder, builder.category))
-
         if (self.channel.categories != None and
             builder.category not in self.channel.categories):
             return
@@ -435,13 +425,11 @@ def watchedBuildFinished(self, b):
 
         # only notify about builders we are interested in
         builder = b.getBuilder()
-        log.msg('builder %r in category %s finished' % (builder,
-                                                        builder.category))
         if (self.channel.categories != None and
             builder.category not in self.channel.categories):
             return
 
-        builder_name = b.getBuilder().getName()
+        builder_name = builder.getName()
         buildnum = b.getNumber()
         buildrevs = b.getRevisions()
 
@@ -876,7 +864,6 @@ def privmsg(self, user, channel, message):
         # to track users comings and goings, add code here
 
     def action(self, user, channel, data):
-        #log.msg("action: %s,%s,%s" % (user, channel, data))
         user = user.split('!', 1)[0] # rest is ~user@hostname
         # somebody did an action (/me actions) in the broadcast channel
         contact = self.getContact(channel)