| Index: third_party/gsutil/pylintrc
 | 
| diff --git a/pylintrc b/third_party/gsutil/pylintrc
 | 
| similarity index 50%
 | 
| copy from pylintrc
 | 
| copy to third_party/gsutil/pylintrc
 | 
| index b76c81d71321625d5a71fb1ac889353d4af92bdf..44ed07796f5fa5e4cb89d6bff3128d29306e16c2 100644
 | 
| --- a/pylintrc
 | 
| +++ b/third_party/gsutil/pylintrc
 | 
| @@ -1,22 +1,30 @@
 | 
| +# lint Python modules using external checkers.
 | 
| +# 
 | 
| +# This is the main checker controlling the other ones and the reports
 | 
| +# generation. It is itself both a raw checker and an astng checker in order
 | 
| +# to:
 | 
| +# * handle message activation / deactivation at the module level
 | 
| +# * handle some basic but necessary stats'data (number of classes, methods...)
 | 
| +# 
 | 
|  [MASTER]
 | 
|  
 | 
| +
 | 
|  # Specify a configuration file.
 | 
|  #rcfile=
 | 
|  
 | 
| -# Python code to execute, usually for sys.path manipulation such as
 | 
| -# pygtk.require().
 | 
| -#init-hook=
 | 
| -
 | 
|  # Profiled execution.
 | 
|  profile=no
 | 
|  
 | 
| -# Add files or directories to the blacklist. They should be base names, not
 | 
| -# paths.
 | 
| -ignore=CVS
 | 
| +# Add <file or directory> to the black list. It should be a base name, not a
 | 
| +# path. You may set this option multiple times.
 | 
| +ignore=.svn
 | 
|  
 | 
|  # Pickle collected data for later comparisons.
 | 
|  persistent=yes
 | 
|  
 | 
| +# Set the cache size for astng objects.
 | 
| +cache-size=500
 | 
| +
 | 
|  # List of plugins (as comma separated values of python modules names) to load,
 | 
|  # usually to register additional checkers.
 | 
|  load-plugins=
 | 
| @@ -24,51 +32,32 @@ load-plugins=
 | 
|  
 | 
|  [MESSAGES CONTROL]
 | 
|  
 | 
| -# Enable the message, report, category or checker with the given id(s). You can
 | 
| -# either give multiple identifier separated by comma (,) or put this option
 | 
| -# multiple time.
 | 
| -#enable=
 | 
| -
 | 
| -# Disable the message, report, category or checker with the given id(s). You
 | 
| -# can either give multiple identifier separated by comma (,) or put this option
 | 
| -# multiple time (only on the command line, not in the configuration file where
 | 
| -# it should appear only once).
 | 
| -# CHANGED:
 | 
| -# C0103: Invalid name ""
 | 
| -# C0111: Missing docstring
 | 
| -# C0302: Too many lines in module (N)
 | 
| -# I0010: Unable to consider inline option ''
 | 
| -# I0011: Locally disabling WNNNN
 | 
| -#
 | 
| -# R0801: Similar lines in N files
 | 
| -# R0901: Too many ancestors (8/7)
 | 
| -# R0902: Too many instance attributes (N/7)
 | 
| -# R0903: Too few public methods (N/2)
 | 
| -# R0904: Too many public methods (N/20)
 | 
| -# R0911: Too many return statements (N/6)
 | 
| -# R0912: Too many branches (N/12)
 | 
| -# R0913: Too many arguments (N/5)
 | 
| -# R0914: Too many local variables (N/15)
 | 
| -# R0915: Too many statements (N/50)
 | 
| -# R0921: Abstract class not referenced
 | 
| -# R0922: Abstract class is only referenced 1 times
 | 
| -# W0122: Use of the exec statement
 | 
| -# W0141: Used builtin function ''
 | 
| -# W0142: Used * or ** magic
 | 
| -# W0402: Uses of a deprecated module 'string'
 | 
| -# W0404: 41: Reimport 'XX' (imported line NN)
 | 
| -# W0511: TODO
 | 
| -# W0603: Using the global statement
 | 
| -# W0703: Catch "Exception"
 | 
| -# W1201: Specify string format arguments as logging function parameters
 | 
| -disable=C0103,C0111,C0302,I0010,I0011,R0801,R0901,R0902,R0903,R0904,R0911,R0912,R0913,R0914,R0915,R0921,R0922,W0122,W0141,W0142,W0402,W0404,W0511,W0603,W0703,W1201
 | 
| +# Enable only checker(s) with the given id(s). This option conflict with the
 | 
| +# disable-checker option
 | 
| +#enable-checker=
 | 
| +
 | 
| +# Enable all checker(s) except those with the given id(s). This option conflict
 | 
| +# with the disable-checker option
 | 
| +#disable-checker=
 | 
| +
 | 
| +# Enable all messages in the listed categories.
 | 
| +#enable-msg-cat=
 | 
| +
 | 
| +# Disable all messages in the listed categories.
 | 
| +#disable-msg-cat=
 | 
|  
 | 
| +# Enable the message(s) with the given id(s).
 | 
| +#enable-msg=
 | 
| +
 | 
| +# Disable the message(s) with the given id(s).
 | 
| +# disable-msg=C0323,W0142,C0301,C0103,C0111,E0213,C0302,C0203,W0703,R0201
 | 
| +disable-msg=C0111,C0103,W0703,W0702
 | 
|  
 | 
|  [REPORTS]
 | 
|  
 | 
| -# Set the output format. Available formats are text, parseable, colorized, msvs
 | 
| -# (visual studio) and html
 | 
| -output-format=text
 | 
| +# set the output format. Available formats are text, parseable, colorized and
 | 
| +# html
 | 
| +output-format=colorized
 | 
|  
 | 
|  # Include message's id in output
 | 
|  include-ids=yes
 | 
| @@ -78,29 +67,38 @@ include-ids=yes
 | 
|  # written in a file name "pylint_global.[txt|html]".
 | 
|  files-output=no
 | 
|  
 | 
| -# Tells whether to display a full report or only the messages
 | 
| -# CHANGED:
 | 
| -reports=no
 | 
| +# Tells wether to display a full report or only the messages
 | 
| +reports=yes
 | 
|  
 | 
|  # Python expression which should return a note less than 10 (10 is the highest
 | 
| -# note). You have access to the variables errors warning, statement which
 | 
| -# respectively contain the number of errors / warnings messages and the total
 | 
| +# note).You have access to the variables errors warning, statement which
 | 
| +# respectivly contain the number of errors / warnings messages and the total
 | 
|  # number of statements analyzed. This is used by the global evaluation report
 | 
| -# (RP0004).
 | 
| +# (R0004).
 | 
|  evaluation=10.0 - ((float(5 * error + warning + refactor + convention) / statement) * 10)
 | 
|  
 | 
|  # Add a comment according to your evaluation note. This is used by the global
 | 
| -# evaluation report (RP0004).
 | 
| +# evaluation report (R0004).
 | 
|  comment=no
 | 
|  
 | 
| +# Enable the report(s) with the given id(s).
 | 
| +#enable-report=
 | 
| +
 | 
| +# Disable the report(s) with the given id(s).
 | 
| +#disable-report=
 | 
|  
 | 
| +# checks for
 | 
| +# * unused variables / imports
 | 
| +# * undefined variables
 | 
| +# * redefinition of variable from builtins or from an outer scope
 | 
| +# * use of variable before assigment
 | 
| +# 
 | 
|  [VARIABLES]
 | 
|  
 | 
| -# Tells whether we should check for unused import in __init__ files.
 | 
| -init-import=no
 | 
| +# Tells wether we should check for unused import in __init__ files.
 | 
| +init-import=yes
 | 
|  
 | 
| -# A regular expression matching the beginning of the name of dummy variables
 | 
| -# (i.e. not used).
 | 
| +# A regular expression matching names used for dummy variables (i.e. not used).
 | 
|  dummy-variables-rgx=_|dummy
 | 
|  
 | 
|  # List of additional names supposed to be defined in builtins. Remember that
 | 
| @@ -108,71 +106,47 @@ dummy-variables-rgx=_|dummy
 | 
|  additional-builtins=
 | 
|  
 | 
|  
 | 
| +# try to find bugs in the code using type inference
 | 
| +# 
 | 
|  [TYPECHECK]
 | 
|  
 | 
| -# Tells whether missing members accessed in mixin class should be ignored. A
 | 
| +# Tells wether missing members accessed in mixin class should be ignored. A
 | 
|  # mixin class is detected if its name ends with "mixin" (case insensitive).
 | 
|  ignore-mixin-members=yes
 | 
|  
 | 
| -# List of classes names for which member attributes should not be checked
 | 
| -# (useful for classes with attributes dynamically set).
 | 
| -ignored-classes=SQLObject,twisted.internet.reactor,hashlib,google.appengine.api.memcache
 | 
| -
 | 
| -# When zope mode is activated, add a predefined set of Zope acquired attributes
 | 
| -# to generated-members.
 | 
| +# When zope mode is activated, consider the acquired-members option to ignore
 | 
| +# access to some undefined attributes.
 | 
|  zope=no
 | 
|  
 | 
| -# List of members which are set dynamically and missed by pylint inference
 | 
| -# system, and so shouldn't trigger E0201 when accessed. Python regular
 | 
| -# expressions are accepted.
 | 
| -generated-members=REQUEST,acl_users,aq_parent,multiprocessing.managers.SyncManager
 | 
| -
 | 
| -
 | 
| -[MISCELLANEOUS]
 | 
| -
 | 
| -# List of note tags to take in consideration, separated by a comma.
 | 
| -notes=FIXME,XXX,TODO
 | 
| -
 | 
| -
 | 
| -[SIMILARITIES]
 | 
| -
 | 
| -# Minimum lines number of a similarity.
 | 
| -min-similarity-lines=4
 | 
| -
 | 
| -# Ignore comments when computing similarities.
 | 
| -ignore-comments=yes
 | 
| -
 | 
| -# Ignore docstrings when computing similarities.
 | 
| -ignore-docstrings=yes
 | 
| -
 | 
| -
 | 
| -[FORMAT]
 | 
| -
 | 
| -# Maximum number of characters on a single line.
 | 
| -max-line-length=80
 | 
| -
 | 
| -# Maximum number of lines in a module
 | 
| -max-module-lines=1000
 | 
| -
 | 
| -# String used as indentation unit. This is usually " " (4 spaces) or "\t" (1
 | 
| -# tab).
 | 
| -# CHANGED:
 | 
| -indent-string='  '
 | 
| -
 | 
| -
 | 
| +# List of members which are usually get through zope's acquisition mecanism and
 | 
| +# so shouldn't trigger E0201 when accessed (need zope=yes to be considered).
 | 
| +acquired-members=REQUEST,acl_users,aq_parent
 | 
| +
 | 
| +
 | 
| +# checks for :
 | 
| +# * doc strings
 | 
| +# * modules / classes / functions / methods / arguments / variables name
 | 
| +# * number of arguments, local variables, branches, returns and statements in
 | 
| +# functions, methods
 | 
| +# * required module attributes
 | 
| +# * dangerous default values as arguments
 | 
| +# * redefinition of function / method / class
 | 
| +# * uses of the global statement
 | 
| +# 
 | 
|  [BASIC]
 | 
|  
 | 
|  # Required attributes for module, separated by a comma
 | 
|  required-attributes=
 | 
|  
 | 
| -# List of builtins function names that should not be used, separated by a comma
 | 
| -bad-functions=map,filter,apply,input
 | 
| +# Regular expression which should only match functions or classes name which do
 | 
| +# not require a docstring
 | 
| +no-docstring-rgx=__.*__
 | 
|  
 | 
|  # Regular expression which should only match correct module names
 | 
|  module-rgx=(([a-z_][a-z0-9_]*)|([A-Z][a-zA-Z0-9]+))$
 | 
|  
 | 
|  # Regular expression which should only match correct module level names
 | 
| -const-rgx=(([A-Z_][A-Z0-9_]*)|(__.*__))$
 | 
| +const-rgx=(([A-Z_][A-Z1-9_]*)|(__.*__))$
 | 
|  
 | 
|  # Regular expression which should only match correct class names
 | 
|  class-rgx=[A-Z_][a-zA-Z0-9]+$
 | 
| @@ -202,78 +176,130 @@ good-names=i,j,k,ex,Run,_
 | 
|  # Bad variable names which should always be refused, separated by a comma
 | 
|  bad-names=foo,bar,baz,toto,tutu,tata
 | 
|  
 | 
| -# Regular expression which should only match functions or classes name which do
 | 
| -# not require a docstring
 | 
| -no-docstring-rgx=__.*__
 | 
| +# List of builtins function names that should not be used, separated by a comma
 | 
| +bad-functions=apply,input
 | 
|  
 | 
|  
 | 
| +# checks for sign of poor/misdesign:
 | 
| +# * number of methods, attributes, local variables...
 | 
| +# * size, complexity of functions, methods
 | 
| +# 
 | 
|  [DESIGN]
 | 
|  
 | 
|  # Maximum number of arguments for function / method
 | 
| -max-args=5
 | 
| -
 | 
| -# Argument names that match this expression will be ignored. Default to name
 | 
| -# with leading underscore
 | 
| -ignored-argument-names=_.*
 | 
| +max-args=12
 | 
|  
 | 
|  # Maximum number of locals for function / method body
 | 
| -max-locals=15
 | 
| +max-locals=30
 | 
|  
 | 
|  # Maximum number of return / yield for function / method body
 | 
| -max-returns=6
 | 
| +max-returns=12
 | 
|  
 | 
|  # Maximum number of branch for function / method body
 | 
| -max-branchs=12
 | 
| +max-branchs=30
 | 
|  
 | 
|  # Maximum number of statements in function / method body
 | 
| -max-statements=50
 | 
| +max-statements=60
 | 
|  
 | 
|  # Maximum number of parents for a class (see R0901).
 | 
|  max-parents=7
 | 
|  
 | 
|  # Maximum number of attributes for a class (see R0902).
 | 
| -max-attributes=7
 | 
| +max-attributes=20
 | 
|  
 | 
|  # Minimum number of public methods for a class (see R0903).
 | 
| -min-public-methods=2
 | 
| +min-public-methods=0
 | 
|  
 | 
|  # Maximum number of public methods for a class (see R0904).
 | 
|  max-public-methods=20
 | 
|  
 | 
|  
 | 
| +# checks for
 | 
| +# * external modules dependencies
 | 
| +# * relative / wildcard imports
 | 
| +# * cyclic imports
 | 
| +# * uses of deprecated modules
 | 
| +# 
 | 
| +[IMPORTS]
 | 
| +
 | 
| +# Deprecated modules which should not be used, separated by a comma
 | 
| +deprecated-modules=regsub,string,TERMIOS,Bastion,rexec
 | 
| +
 | 
| +# Create a graph of every (i.e. internal and external) dependencies in the
 | 
| +# given file (report R0402 must not be disabled)
 | 
| +import-graph=
 | 
| +
 | 
| +# Create a graph of external dependencies in the given file (report R0402 must
 | 
| +# not be disabled)
 | 
| +ext-import-graph=
 | 
| +
 | 
| +# Create a graph of internal dependencies in the given file (report R0402 must
 | 
| +# not be disabled)
 | 
| +int-import-graph=
 | 
| +
 | 
| +
 | 
| +# checks for :
 | 
| +# * methods without self as first argument
 | 
| +# * overridden methods signature
 | 
| +# * access only to existant members via self
 | 
| +# * attributes not defined in the __init__ method
 | 
| +# * supported interfaces implementation
 | 
| +# * unreachable code
 | 
| +# 
 | 
|  [CLASSES]
 | 
|  
 | 
|  # List of interface methods to ignore, separated by a comma. This is used for
 | 
|  # instance to not check methods defines in Zope's Interface base class.
 | 
| -ignore-iface-methods=isImplementedBy,deferred,extends,names,namesAndDescriptions,queryDescriptionFor,getBases,getDescriptionFor,getDoc,getName,getTaggedValue,getTaggedValueTags,isEqualOrExtendedBy,setTaggedValue,isImplementedByInstancesOf,adaptWith,is_implemented_by
 | 
| +# ignore-iface-methods=isImplementedBy,deferred,extends,names,namesAndDescriptions,queryDescriptionFor,getBases,getDescriptionFor,getDoc,getName,getTaggedValue,getTaggedValueTags,isEqualOrExtendedBy,setTaggedValue,isImplementedByInstancesOf,adaptWith,is_implemented_by
 | 
|  
 | 
|  # List of method names used to declare (i.e. assign) instance attributes.
 | 
|  defining-attr-methods=__init__,__new__,setUp
 | 
|  
 | 
| -# List of valid names for the first argument in a class method.
 | 
| -valid-classmethod-first-arg=cls
 | 
|  
 | 
| +# checks for similarities and duplicated code. This computation may be
 | 
| +# memory / CPU intensive, so you should disable it if you experiments some
 | 
| +# problems.
 | 
| +# 
 | 
| +[SIMILARITIES]
 | 
|  
 | 
| -[IMPORTS]
 | 
| +# Minimum lines number of a similarity.
 | 
| +min-similarity-lines=5
 | 
|  
 | 
| -# Deprecated modules which should not be used, separated by a comma
 | 
| -deprecated-modules=regsub,string,TERMIOS,Bastion,rexec
 | 
| +# Ignore comments when computing similarities.
 | 
| +ignore-comments=yes
 | 
|  
 | 
| -# Create a graph of every (i.e. internal and external) dependencies in the
 | 
| -# given file (report RP0402 must not be disabled)
 | 
| -import-graph=
 | 
| +# Ignore docstrings when computing similarities.
 | 
| +ignore-docstrings=yes
 | 
|  
 | 
| -# Create a graph of external dependencies in the given file (report RP0402 must
 | 
| -# not be disabled)
 | 
| -ext-import-graph=
 | 
|  
 | 
| -# Create a graph of internal dependencies in the given file (report RP0402 must
 | 
| -# not be disabled)
 | 
| -int-import-graph=
 | 
| +# checks for:
 | 
| +# * warning notes in the code like FIXME, XXX
 | 
| +# * PEP 263: source code with non ascii character but no encoding declaration
 | 
| +# 
 | 
| +[MISCELLANEOUS]
 | 
| +
 | 
| +# List of note tags to take in consideration, separated by a comma.
 | 
| +notes=FIXME,XXX,TODO,BUG:
 | 
|  
 | 
|  
 | 
| -[EXCEPTIONS]
 | 
| +# checks for :
 | 
| +# * unauthorized constructions
 | 
| +# * strict indentation
 | 
| +# * line length
 | 
| +# * use of <> instead of !=
 | 
| +# 
 | 
| +[FORMAT]
 | 
|  
 | 
| -# Exceptions that will emit a warning when being caught. Defaults to
 | 
| -# "Exception"
 | 
| -overgeneral-exceptions=Exception
 | 
| +# Maximum number of characters on a single line.
 | 
| +max-line-length=80
 | 
| +
 | 
| +# Maximum number of lines in a module
 | 
| +max-module-lines=1000
 | 
| +
 | 
| +# String used as indentation unit. This is usually " " (4 spaces) or "\t" (1
 | 
| +# tab).
 | 
| +indent-string='    '
 | 
| +
 | 
| +
 | 
| +[MESSAGES CONTROL]
 | 
| +disable-msg=C0301,C0111,C0103,R0201,W0702,C0324
 | 
| 
 |