Change type of extraRecipients to a list....

master.chromium/master.cfg

 # -*- python -*-
 # ex: set syntax=python:
 
 # Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 # This is the buildmaster config file for the 'chromium' bot. It must
 # be installed as 'master.cfg' in your buildmaster's base directory
 # (although the filename can be changed with the --basedir option to
@@ skipping 1766 matching lines @@
   exclusions = { }
   forgiving_steps = ['update scripts', 'update', 'svnkill', 'taskkill',
                      'archived build', 'Start Crash Handler']
   c['status'].append(gatekeeper.GateKeeper(
       fromaddr=ActiveMaster.from_address,
       categories_steps=categories_steps,
       exclusions=exclusions,
       relayhost=config.Master.smtp,
       subject='buildbot %(result)s in %(projectName)s on %(builder)s, '
               'revision %(revision)s',
-      extraRecipients=tuple(ActiveMaster.tree_closing_notification_recipients),
+      extraRecipients=ActiveMaster.tree_closing_notification_recipients,
       lookup='google.com',
       forgiving_steps=forgiving_steps,
       tree_status_url=ActiveMaster.tree_status_url))
 
 if GOOD_REVISIONS:
   import goodrevisions
   # This is the list of builders with their respective list of critical steps
   # that all need to succeed to mark a revision as successful. A single failure
   # in any of the steps of any of the builders will mark the revision as failed.
   good_revision_steps = {
@@ skipping 73 matching lines @@
   forgiving_steps = ['update scripts', 'update', 'svnkill', 'taskkill',
                      'archived build', 'Start Crash Handler']
   c['status'].append(chromium_notifier.ChromiumNotifier(
       fromaddr=ActiveMaster.from_address,
       categories_steps=categories_steps,
       exclusions=exclusions,
       relayhost=config.Master.smtp,
       status_header='Perf alert on "%(builder)s":\n%(steps)s',
       subject='buildbot %(result)s in %(projectName)s on %(builder)s, '
               'revision %(revision)s',
-      extraRecipients=tuple(['cmp@google.com']),
+      extraRecipients=['cmp@google.com'],
       lookup='google.com',
       forgiving_steps=forgiving_steps))
 
 
 # Keep last build logs, the default is too low.
 c['buildHorizon'] = 1000
 c['logHorizon'] = 500
 # Must be at least 2x the number of slaves.
 c['eventHorizon'] = 200
 # Must be at least 1x the number of builds listed in console.
@@ skipping 30 matching lines @@
 c['projectName'] = ActiveMaster.project_name
 c['projectURL'] = config.Master.project_url
 
 # the 'buildbotURL' string should point to the location where the buildbot's
 # internal web server (usually the html.Waterfall page) is visible. This
 # typically uses the port number set in the Waterfall 'status' entry, but
 # with an externally-visible host name which the buildbot cannot figure out
 # without some help.
 
 c['buildbotURL'] = 'http://build.chromium.org/buildbot/waterfall/'