Revert of Fix pylint warnings in webkitpy/common/.

third_party/WebKit/Tools/Scripts/webkitpy/common/checkout/scm/scm.py

 # Copyright (c) 2009, Google Inc. All rights reserved.
 # Copyright (c) 2009 Apple Inc. All rights reserved.
 #
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
 # met:
 #
 #     * Redistributions of source code must retain the above copyright
 # notice, this list of conditions and the following disclaimer.
 #     * Redistributions in binary form must reproduce the above
@@ skipping 16 matching lines @@
 # (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 #
 # Python module for interacting with an SCM system (like SVN or Git)
 
 import logging
 import re
 import sys
 
 from webkitpy.common.system.executive import Executive
+from webkitpy.common.system.executive import ScriptError
 from webkitpy.common.system.filesystem import FileSystem
 
 _log = logging.getLogger(__name__)
 
 
 # SCM methods are expected to return paths relative to self.checkout_root.
-class SCM(object):
+class SCM:
 
     def __init__(self, cwd, executive=None, filesystem=None):
         self.cwd = cwd
         self._executive = executive or Executive()
         self._filesystem = filesystem or FileSystem()
         self.checkout_root = self.find_checkout_root(self.cwd)
 
     # A wrapper used by subclasses to create processes.
-    def _run(self, args, cwd=None, input_func=None, error_handler=None,
-             return_exit_code=False, return_stderr=True, decode_output=True):
+    def _run(self, args, cwd=None, input=None, error_handler=None, return_exit_code=False, return_stderr=True, decode_output=True):
         # FIXME: We should set cwd appropriately.
         return self._executive.run_command(args,
                                            cwd=cwd,
-                                           input_func=input_func,
+                                           input=input,
                                            error_handler=error_handler,
                                            return_exit_code=return_exit_code,
                                            return_stderr=return_stderr,
                                            decode_output=decode_output)
 
     # SCM always returns repository relative path, but sometimes we need
     # absolute paths to pass to rm, etc.
     def absolute_path(self, repository_relative_path):
         return self._filesystem.join(self.checkout_root, repository_relative_path)
 
     def _run_status_and_extract_filenames(self, status_command, status_regexp):
         filenames = []
         # We run with cwd=self.checkout_root so that returned-paths are root-relative.
         for line in self._run(status_command, cwd=self.checkout_root).splitlines():
             match = re.search(status_regexp, line)
             if not match:
                 continue
             # status = match.group('status')
             filename = match.group('filename')
             filenames.append(filename)
         return filenames
 
     @staticmethod
     def _subclass_must_implement():
         raise NotImplementedError("subclasses must implement")
 
-    # Most methods here are abstract methods which don't use their arguments.
-    # FIXME: Merge Git and SCM and remove the abstract base methods.
-    # pylint: disable=unused-argument
-
     @classmethod
     def in_working_directory(cls, path, executive=None):
         SCM._subclass_must_implement()
 
     def find_checkout_root(self, path):
         SCM._subclass_must_implement()
 
     def add(self, path, return_exit_code=False, recurse=True):
         self.add_list([path], return_exit_code, recurse)
 
@@ skipping 40 matching lines @@
     #--------------------------------------------------------------------------
     # Subclasses must indicate if they support local commits,
     # but the SCM baseclass will only call local_commits methods when this is true.
     @staticmethod
     def supports_local_commits():
         SCM._subclass_must_implement()
 
     def commit_locally_with_message(self, message):
         _log.error("Your source control manager does not support local commits.")
         sys.exit(1)