1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
|
#!/usr/bin/env python3
# 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."""
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:]))
|