Spaces:
Runtime error
Runtime error
#!/usr/bin/env python3 | |
""" | |
MIT License | |
Copyright (c) 2017 Guillaume Papin | |
Permission is hereby granted, free of charge, to any person obtaining a copy | |
of this software and associated documentation files (the "Software"), to deal | |
in the Software without restriction, including without limitation the rights | |
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell | |
copies of the Software, and to permit persons to whom the Software is | |
furnished to do so, subject to the following conditions: | |
The above copyright notice and this permission notice shall be included in all | |
copies or substantial portions of the Software. | |
THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR | |
IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, | |
FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE | |
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER | |
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, | |
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE | |
SOFTWARE. | |
""" | |
"""A wrapper script around clang-format, suitable for linting multiple files | |
and to use for continuous integration. | |
This is an alternative API for the clang-format command line. | |
It runs over multiple files and directories in parallel. | |
A diff output is produced and a sensible exit code is returned. | |
""" | |
import argparse # noqa: E402 | |
import difflib # noqa: E402 | |
import fnmatch # noqa: E402 | |
import io # noqa: E402 | |
import multiprocessing # noqa: E402 | |
import os # noqa: E402 | |
import signal # noqa: E402 | |
import subprocess # noqa: E402 | |
import sys # noqa: E402 | |
import traceback # noqa: E402 | |
from functools import partial # noqa: E402 | |
from subprocess import DEVNULL # noqa: E402 | |
DEFAULT_EXTENSIONS = "c,h,C,H,cpp,hpp,cc,hh,c++,h++,cxx,hxx,cu" | |
class ExitStatus: | |
SUCCESS = 0 | |
DIFF = 1 | |
TROUBLE = 2 | |
def list_files(files, recursive=False, extensions=None, exclude=None): | |
if extensions is None: | |
extensions = [] | |
if exclude is None: | |
exclude = [] | |
out = [] | |
for file in files: | |
if recursive and os.path.isdir(file): | |
for dirpath, dnames, fnames in os.walk(file): | |
fpaths = [os.path.join(dirpath, fname) for fname in fnames] | |
for pattern in exclude: | |
# os.walk() supports trimming down the dnames list | |
# by modifying it in-place, | |
# to avoid unnecessary directory listings. | |
dnames[:] = [ | |
x | |
for x in dnames | |
if not fnmatch.fnmatch(os.path.join(dirpath, x), pattern) | |
] | |
fpaths = [x for x in fpaths if not fnmatch.fnmatch(x, pattern)] | |
for f in fpaths: | |
ext = os.path.splitext(f)[1][1:] | |
if ext in extensions: | |
out.append(f) | |
else: | |
out.append(file) | |
return out | |
def make_diff(file, original, reformatted): | |
return list( | |
difflib.unified_diff( | |
original, | |
reformatted, | |
fromfile="a/{}\t(original)".format(file), | |
tofile="b/{}\t(reformatted)".format(file), | |
n=3, | |
) | |
) | |
class DiffError(Exception): | |
def __init__(self, message, errs=None): | |
super(DiffError, self).__init__(message) | |
self.errs = errs or [] | |
class UnexpectedError(Exception): | |
def __init__(self, message, exc=None): | |
super(UnexpectedError, self).__init__(message) | |
self.formatted_traceback = traceback.format_exc() | |
self.exc = exc | |
def run_clang_format_diff_wrapper(args, file): | |
try: | |
ret = run_clang_format_diff(args, file) | |
return ret | |
except DiffError: | |
raise | |
except Exception as e: | |
raise UnexpectedError("{}: {}: {}".format(file, e.__class__.__name__, e), e) | |
def run_clang_format_diff(args, file): | |
try: | |
with io.open(file, "r", encoding="utf-8") as f: | |
original = f.readlines() | |
except IOError as exc: | |
raise DiffError(str(exc)) | |
invocation = [args.clang_format_executable, file] | |
# Use of utf-8 to decode the process output. | |
# | |
# Hopefully, this is the correct thing to do. | |
# | |
# It's done due to the following assumptions (which may be incorrect): | |
# - clang-format will returns the bytes read from the files as-is, | |
# without conversion, and it is already assumed that the files use utf-8. | |
# - if the diagnostics were internationalized, they would use utf-8: | |
# > Adding Translations to Clang | |
# > | |
# > Not possible yet! | |
# > Diagnostic strings should be written in UTF-8, | |
# > the client can translate to the relevant code page if needed. | |
# > Each translation completely replaces the format string | |
# > for the diagnostic. | |
# > -- http://clang.llvm.org/docs/InternalsManual.html#internals-diag-translation | |
try: | |
proc = subprocess.Popen( | |
invocation, | |
stdout=subprocess.PIPE, | |
stderr=subprocess.PIPE, | |
universal_newlines=True, | |
encoding="utf-8", | |
) | |
except OSError as exc: | |
raise DiffError( | |
"Command '{}' failed to start: {}".format( | |
subprocess.list2cmdline(invocation), exc | |
) | |
) | |
proc_stdout = proc.stdout | |
proc_stderr = proc.stderr | |
# hopefully the stderr pipe won't get full and block the process | |
outs = list(proc_stdout.readlines()) | |
errs = list(proc_stderr.readlines()) | |
proc.wait() | |
if proc.returncode: | |
raise DiffError( | |
"Command '{}' returned non-zero exit status {}".format( | |
subprocess.list2cmdline(invocation), proc.returncode | |
), | |
errs, | |
) | |
return make_diff(file, original, outs), errs | |
def bold_red(s): | |
return "\x1b[1m\x1b[31m" + s + "\x1b[0m" | |
def colorize(diff_lines): | |
def bold(s): | |
return "\x1b[1m" + s + "\x1b[0m" | |
def cyan(s): | |
return "\x1b[36m" + s + "\x1b[0m" | |
def green(s): | |
return "\x1b[32m" + s + "\x1b[0m" | |
def red(s): | |
return "\x1b[31m" + s + "\x1b[0m" | |
for line in diff_lines: | |
if line[:4] in ["--- ", "+++ "]: | |
yield bold(line) | |
elif line.startswith("@@ "): | |
yield cyan(line) | |
elif line.startswith("+"): | |
yield green(line) | |
elif line.startswith("-"): | |
yield red(line) | |
else: | |
yield line | |
def print_diff(diff_lines, use_color): | |
if use_color: | |
diff_lines = colorize(diff_lines) | |
sys.stdout.writelines(diff_lines) | |
def print_trouble(prog, message, use_colors): | |
error_text = "error:" | |
if use_colors: | |
error_text = bold_red(error_text) | |
print("{}: {} {}".format(prog, error_text, message), file=sys.stderr) | |
def main(): | |
parser = argparse.ArgumentParser(description=__doc__) | |
parser.add_argument( | |
"--clang-format-executable", | |
metavar="EXECUTABLE", | |
help="path to the clang-format executable", | |
default="clang-format", | |
) | |
parser.add_argument( | |
"--extensions", | |
help="comma separated list of file extensions (default: {})".format( | |
DEFAULT_EXTENSIONS | |
), | |
default=DEFAULT_EXTENSIONS, | |
) | |
parser.add_argument( | |
"-r", | |
"--recursive", | |
action="store_true", | |
help="run recursively over directories", | |
) | |
parser.add_argument("files", metavar="file", nargs="+") | |
parser.add_argument("-q", "--quiet", action="store_true") | |
parser.add_argument( | |
"-j", | |
metavar="N", | |
type=int, | |
default=0, | |
help="run N clang-format jobs in parallel" " (default number of cpus + 1)", | |
) | |
parser.add_argument( | |
"--color", | |
default="auto", | |
choices=["auto", "always", "never"], | |
help="show colored diff (default: auto)", | |
) | |
parser.add_argument( | |
"-e", | |
"--exclude", | |
metavar="PATTERN", | |
action="append", | |
default=[], | |
help="exclude paths matching the given glob-like pattern(s)" | |
" from recursive search", | |
) | |
args = parser.parse_args() | |
# use default signal handling, like diff return SIGINT value on ^C | |
# https://bugs.python.org/issue14229#msg156446 | |
signal.signal(signal.SIGINT, signal.SIG_DFL) | |
try: | |
signal.SIGPIPE | |
except AttributeError: | |
# compatibility, SIGPIPE does not exist on Windows | |
pass | |
else: | |
signal.signal(signal.SIGPIPE, signal.SIG_DFL) | |
colored_stdout = False | |
colored_stderr = False | |
if args.color == "always": | |
colored_stdout = True | |
colored_stderr = True | |
elif args.color == "auto": | |
colored_stdout = sys.stdout.isatty() | |
colored_stderr = sys.stderr.isatty() | |
version_invocation = [args.clang_format_executable, str("--version")] | |
try: | |
subprocess.check_call(version_invocation, stdout=DEVNULL) | |
except subprocess.CalledProcessError as e: | |
print_trouble(parser.prog, str(e), use_colors=colored_stderr) | |
return ExitStatus.TROUBLE | |
except OSError as e: | |
print_trouble( | |
parser.prog, | |
"Command '{}' failed to start: {}".format( | |
subprocess.list2cmdline(version_invocation), e | |
), | |
use_colors=colored_stderr, | |
) | |
return ExitStatus.TROUBLE | |
retcode = ExitStatus.SUCCESS | |
files = list_files( | |
args.files, | |
recursive=args.recursive, | |
exclude=args.exclude, | |
extensions=args.extensions.split(","), | |
) | |
if not files: | |
return | |
njobs = args.j | |
if njobs == 0: | |
njobs = multiprocessing.cpu_count() + 1 | |
njobs = min(len(files), njobs) | |
if njobs == 1: | |
# execute directly instead of in a pool, | |
# less overhead, simpler stacktraces | |
it = (run_clang_format_diff_wrapper(args, file) for file in files) | |
pool = None | |
else: | |
pool = multiprocessing.Pool(njobs) | |
it = pool.imap_unordered(partial(run_clang_format_diff_wrapper, args), files) | |
while True: | |
try: | |
outs, errs = next(it) | |
except StopIteration: | |
break | |
except DiffError as e: | |
print_trouble(parser.prog, str(e), use_colors=colored_stderr) | |
retcode = ExitStatus.TROUBLE | |
sys.stderr.writelines(e.errs) | |
except UnexpectedError as e: | |
print_trouble(parser.prog, str(e), use_colors=colored_stderr) | |
sys.stderr.write(e.formatted_traceback) | |
retcode = ExitStatus.TROUBLE | |
# stop at the first unexpected error, | |
# something could be very wrong, | |
# don't process all files unnecessarily | |
if pool: | |
pool.terminate() | |
break | |
else: | |
sys.stderr.writelines(errs) | |
if outs == []: | |
continue | |
if not args.quiet: | |
print_diff(outs, use_color=colored_stdout) | |
if retcode == ExitStatus.SUCCESS: | |
retcode = ExitStatus.DIFF | |
return retcode | |
if __name__ == "__main__": | |
sys.exit(main()) | |