#!/usr/bin/python # -*- coding:utf-8 -*- # Copyright 2016 The Android Open Source Project # # 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. """Wrapper to run git-clang-format and parse its output.""" from __future__ import print_function import argparse import os import sys _path = os.path.realpath(__file__ + '/../..') if sys.path[0] != _path: sys.path.insert(0, _path) del _path # We have to import our local modules after the sys.path tweak. We can't use # relative imports because this is an executable program, not a module. # pylint: disable=wrong-import-position import rh.shell import rh.utils # Since we're asking git-clang-format to print a diff, all modified filenames # that have formatting errors are printed with this prefix. DIFF_MARKER_PREFIX = '+++ b/' def get_parser(): """Return a command line parser.""" parser = argparse.ArgumentParser(description=__doc__) parser.add_argument('--clang-format', default='clang-format', help='The path of the clang-format executable.') parser.add_argument('--git-clang-format', default='git-clang-format', help='The path of the git-clang-format executable.') parser.add_argument('--style', metavar='STYLE', type=str, help='The style that clang-format will use.') parser.add_argument('--extensions', metavar='EXTENSIONS', type=str, help='Comma-separated list of file extensions to ' 'format.') parser.add_argument('--fix', action='store_true', help='Fix any formatting errors automatically.') scope = parser.add_mutually_exclusive_group(required=True) scope.add_argument('--commit', type=str, default='HEAD', help='Specify the commit to validate.') scope.add_argument('--working-tree', action='store_true', help='Validates the files that have changed from ' 'HEAD in the working directory.') parser.add_argument('files', type=str, nargs='*', help='If specified, only consider differences in ' 'these files.') return parser def main(argv): """The main entry.""" parser = get_parser() opts = parser.parse_args(argv) cmd = [opts.git_clang_format, '--binary', opts.clang_format, '--diff'] if opts.style: cmd.extend(['--style', opts.style]) if opts.extensions: cmd.extend(['--extensions', opts.extensions]) if not opts.working_tree: cmd.extend(['%s^' % opts.commit, opts.commit]) cmd.extend(['--'] + opts.files) # Fail gracefully if clang-format itself aborts/fails. try: result = rh.utils.run(cmd, capture_output=True) except rh.utils.CalledProcessError as e: print('clang-format failed:\n%s' % (e,), file=sys.stderr) print('\nPlease report this to the clang team.', file=sys.stderr) return 1 stdout = result.stdout if stdout.rstrip('\n') == 'no modified files to format': # This is always printed when only files that clang-format does not # understand were modified. return 0 diff_filenames = [] for line in stdout.splitlines(): if line.startswith(DIFF_MARKER_PREFIX): diff_filenames.append(line[len(DIFF_MARKER_PREFIX):].rstrip()) if diff_filenames: if opts.fix: result = rh.utils.run(['git', 'apply'], input=stdout, check=False) if result.returncode: print('Error: Unable to automatically fix things.\n' ' Make sure your checkout is clean first.\n' ' If you have multiple commits, you might have to ' 'manually rebase your tree first.', file=sys.stderr) return result.returncode else: print('The following files have formatting errors:') for filename in diff_filenames: print('\t%s' % filename) print('You can try to fix this by running:\n%s --fix %s' % (sys.argv[0], rh.shell.cmd_to_str(argv))) return 1 return 0 if __name__ == '__main__': sys.exit(main(sys.argv[1:]))