Largely reduce the number of pylint warnings and fix one typo.

gclient_utils.py

 # Copyright 2009 Google Inc.  All Rights Reserved.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
 # You may obtain a copy of the License at
 #
 #      http://www.apache.org/licenses/LICENSE-2.0
 #
 # Unless required by applicable law or agreed to in writing, software
 # distributed under the License is distributed on an "AS IS" BASIS,
 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
 """Generic utils."""
 
-import copy
 import errno
 import logging
 import os
 import Queue
 import re
 import stat
 import subprocess
 import sys
 import threading
 import time
@@ skipping 265 matching lines @@
     kwargs['call_filter_on_first_line'] = True
   # Obviously.
   kwargs['print_stdout'] = True
   return CheckCallAndFilter(args, **kwargs)
 
 
 def SoftClone(obj):
   """Clones an object. copy.copy() doesn't work on 'file' objects."""
   if obj.__class__.__name__ == 'SoftCloned':
     return obj
-  class SoftCloned(object): pass
+  class SoftCloned(object):
+    pass
   new_obj = SoftCloned()
   for member in dir(obj):
     if member.startswith('_'):
       continue
     setattr(new_obj, member, getattr(obj, member))
   return new_obj
 
 
 def MakeFileAutoFlush(fileobj, delay=10):
   """Creates a file object clone to automatically flush after N seconds."""
@@ skipping 68 matching lines @@
       new_fileobj.old_annotated_write('%d>%s\n' % (index, line))
       obj[0] = remaining
 
   def full_flush():
     """Flush buffered output."""
     orphans = []
     new_fileobj.lock.acquire()
     try:
       # Detect threads no longer existing.
       indexes = (getattr(t, 'index', None) for t in threading.enumerate())
-      indexed = filter(None, indexes)
+      indexes = filter(None, indexes)
       for index in new_fileobj.output_buffers:
         if not index in indexes:
           orphans.append((index, new_fileobj.output_buffers[index][0]))
       for orphan in orphans:
         del new_fileobj.output_buffers[orphan[0]]
     finally:
       new_fileobj.lock.release()
 
     # Don't keep the lock while writting. Will append \n when it shouldn't.
     for orphan in orphans:
@@ skipping 70 matching lines @@
 
   # If we did not find the file in the current directory, make sure we are in a
   # sub directory that is controlled by this configuration.
   if path != real_from_dir:
     entries_filename = os.path.join(path, filename + '_entries')
     if not os.path.exists(entries_filename):
       # If .gclient_entries does not exist, a previous call to gclient sync
       # might have failed. In that case, we cannot verify that the .gclient
       # is the one we want to use. In order to not to cause too much trouble,
       # just issue a warning and return the path anyway.
-      print >>sys.stderr, ("%s file in parent directory %s might not be the "
+      print >> sys.stderr, ("%s file in parent directory %s might not be the "
           "file you want to use" % (filename, path))
       return path
     scope = {}
     try:
       exec(FileRead(entries_filename), scope)
     except SyntaxError, e:
       SyntaxErrorToError(filename, e)
     all_directories = scope['entries'].keys()
     path_to_check = real_from_dir[len(path)+1:]
     while path_to_check:
@@ skipping 211 matching lines @@
         logging.info('Caught exception in thread %s' % self.item.name)
         logging.info(str(sys.exc_info()))
         work_queue.exceptions.put(sys.exc_info())
       logging.info('Task %s done' % self.item.name)
 
       work_queue.ready_cond.acquire()
       try:
         work_queue.ready_cond.notifyAll()
       finally:
         work_queue.ready_cond.release()