fix pylint warnings

native_client_sdk/src/build_tools/update_nacl_manifest.py

 #!/usr/bin/env python
 # Copyright (c) 2012 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.
 
 """Script that reads omahaproxy and gsutil to determine version of SDK to put
 in manifest.
 """
 
+# pylint is convinced the email module is missing attributes
+# pylint: disable=E1101
+
 import buildbot_common
 import csv
 import cStringIO
 import email
 import manifest_util
 import optparse
 import os
 import posixpath
 import re
 import smtplib
@@ skipping 65 matching lines @@
   platforms = []
   for url in archive_urls:
     platform = GetPlatformFromArchiveUrl(url)
     if platform:
       platforms.append(platform)
   return platforms
 
 
 class Delegate(object):
   """Delegate all external access; reading/writing to filesystem, gsutil etc."""
+
   def GetRepoManifest(self):
     """Read the manifest file from the NaCl SDK repository.
 
     This manifest is used as a template for the auto updater; only pepper
     bundles with no archives are considered for auto updating.
 
     Returns:
       A manifest_util.SDKManifest object read from the NaCl SDK repo."""
     raise NotImplementedError()
 
@@ skipping 45 matching lines @@
   def GsUtil_cp(self, src, dest, stdin=None):
     """Runs gsutil cp |src| |dest|
 
     Args:
       src: The file path or url to copy from.
       dest: The file path or url to copy to.
       stdin: If src is '-', this is used as the stdin to give to gsutil. The
           effect is that text in stdin is copied to |dest|."""
     raise NotImplementedError()
 
-
   def Print(self, *args):
     """Print a message."""
     raise NotImplementedError()
 
 
 class RealDelegate(Delegate):
   def __init__(self, dryrun=False, gsutil=None):
+    super(RealDelegate, self).__init__()
     self.dryrun = dryrun
     if gsutil:
       self.gsutil = gsutil
     else:
       self.gsutil = buildbot_common.GetGsutil()
 
   def GetRepoManifest(self):
     """See Delegate.GetRepoManifest"""
     with open(REPO_MANIFEST, 'r') as sdk_stream:
       sdk_json_string = sdk_stream.read()
@@ skipping 128 matching lines @@
       shared_version_generator: A generator that will yield (version, channel)
           tuples in order of most recent to least recent.
       allow_trunk_revisions: If True, will search for archives using the
             trunk revision that matches the branch version.
     Returns:
       A tuple (version, channel, archives). The version is a string such as
       "19.0.1084.41". The channel is one of ('stable', 'beta', 'dev',
       'canary'). |archives| is a list of archive URLs."""
     version = None
     skipped_versions = []
+    channel = ''
     while True:
       try:
         version, channel = shared_version_generator.next()
       except StopIteration:
         msg = 'No shared version for platforms: %s\n' % (', '.join(platforms))
         msg += 'Last version checked = %s.\n' % (version,)
         if skipped_versions:
           msg += 'Versions skipped due to missing archives:\n'
           for version, channel, archives in skipped_versions:
             if archives:
@@ skipping 296 matching lines @@
       subject = '[%s] Failed to update manifest' % (scriptname,)
       text = '%s failed.\n\nSTDERR:\n%s\n' % (scriptname, sys.stderr.getvalue())
       SendMail(options.mailfrom, options.mailto, subject, text)
       sys.exit(1)
     else:
       raise
 
 
 if __name__ == '__main__':
   sys.exit(main(sys.argv))