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

gclient.py

Index: gclient.py
diff --git a/gclient.py b/gclient.py
index 0d1e2f93f2f950066e8969fc208cba195a067b1a..054f52d93bbd76040372adb835333f6140666ddf 100644
--- a/gclient.py
+++ b/gclient.py
@@ -58,22 +58,21 @@ import os
 import posixpath
 import pprint
 import re
-import subprocess
 import sys
 import urlparse
 import urllib
 
-import breakpad
+import breakpad  # pylint: disable=W0611
 
 import gclient_scm
 import gclient_utils
 from third_party.repo.progress import Progress
 
 
-def attr(attr, data):
+def attr(attribute, data):
   """Sets an attribute on a function."""
   def hook(fn):
-    setattr(fn, attr, data)
+    setattr(fn, attribute, data)
     return fn
   return hook