2019-12-07 18:28:55 +01:00
|
|
|
#!/usr/bin/env python3
|
2019-04-17 12:06:00 +02:00
|
|
|
|
2020-07-14 14:34:44 +02:00
|
|
|
import argparse
|
2021-08-09 22:43:18 +02:00
|
|
|
import json
|
2019-04-17 12:06:00 +02:00
|
|
|
import multiprocessing
|
|
|
|
import os
|
2021-07-25 23:54:32 +02:00
|
|
|
import queue
|
2019-04-17 12:06:00 +02:00
|
|
|
import re
|
|
|
|
import shutil
|
|
|
|
import subprocess
|
|
|
|
import sys
|
|
|
|
import tempfile
|
2020-07-14 14:34:44 +02:00
|
|
|
import threading
|
2019-04-17 12:06:00 +02:00
|
|
|
|
|
|
|
import click
|
2020-07-14 14:34:44 +02:00
|
|
|
import pexpect
|
2019-04-17 12:06:00 +02:00
|
|
|
|
2019-05-12 23:04:36 +02:00
|
|
|
sys.path.append(os.path.dirname(__file__))
|
2021-09-13 18:11:27 +02:00
|
|
|
from helpers import shlex_quote, get_output, filter_grep, \
|
2021-09-24 18:02:28 +02:00
|
|
|
build_all_include, temp_header_file, git_ls_files, filter_changed, load_idedata, basepath
|
2019-05-12 23:04:36 +02:00
|
|
|
|
2019-04-17 12:06:00 +02:00
|
|
|
|
2021-08-09 22:43:18 +02:00
|
|
|
def clang_options(idedata):
|
|
|
|
cmd = [
|
|
|
|
# target 32-bit arch (this prevents size mismatch errors on a 64-bit host)
|
|
|
|
'-m32',
|
|
|
|
# disable built-in include directories from the host
|
|
|
|
'-nostdinc',
|
|
|
|
'-nostdinc++',
|
2021-09-13 18:55:04 +02:00
|
|
|
# replace pgmspace.h, as it uses GNU extensions clang doesn't support
|
|
|
|
# https://github.com/earlephilhower/newlib-xtensa/pull/18
|
|
|
|
'-D_PGMSPACE_H_',
|
|
|
|
'-Dpgm_read_byte(s)=(*(const uint8_t *)(s))',
|
|
|
|
'-Dpgm_read_byte_near(s)=(*(const uint8_t *)(s))',
|
2021-11-10 19:35:31 +01:00
|
|
|
'-Dpgm_read_word(s)=(*(const uint16_t *)(s))',
|
2021-09-13 18:55:04 +02:00
|
|
|
'-Dpgm_read_dword(s)=(*(const uint32_t *)(s))',
|
|
|
|
'-DPROGMEM=',
|
|
|
|
'-DPGM_P=const char *',
|
|
|
|
'-DPSTR(s)=(s)',
|
|
|
|
# this next one is also needed with upstream pgmspace.h
|
|
|
|
# suppress warning about identifier naming in expansion of this macro
|
|
|
|
'-DPSTRN(s, n)=(s)',
|
|
|
|
# suppress warning about attribute cannot be applied to type
|
|
|
|
# https://github.com/esp8266/Arduino/pull/8258
|
|
|
|
'-Ddeprecated(x)=',
|
2021-09-13 18:11:27 +02:00
|
|
|
# pretend we're an Xtensa compiler, which gates some features in the headers
|
|
|
|
'-D__XTENSA__',
|
2021-08-09 22:43:18 +02:00
|
|
|
# allow to condition code on the presence of clang-tidy
|
2021-09-20 11:47:51 +02:00
|
|
|
'-DCLANG_TIDY',
|
|
|
|
# (esp-idf) Disable this header because they use asm with registers clang-tidy doesn't know
|
|
|
|
'-D__XTENSA_API_H__',
|
|
|
|
# (esp-idf) Fix __once_callable in some libstdc++ headers
|
|
|
|
'-D_GLIBCXX_HAVE_TLS',
|
2021-08-09 22:43:18 +02:00
|
|
|
]
|
|
|
|
|
|
|
|
# copy compiler flags, except those clang doesn't understand.
|
|
|
|
cmd.extend(flag for flag in idedata['cxx_flags'].split(' ')
|
2021-09-21 17:12:17 +02:00
|
|
|
if flag not in ('-free', '-fipa-pta', '-fstrict-volatile-bitfields',
|
|
|
|
'-mlongcalls', '-mtext-section-literals',
|
|
|
|
'-mfix-esp32-psram-cache-issue', '-mfix-esp32-psram-cache-strategy=memw'))
|
2021-08-09 22:43:18 +02:00
|
|
|
|
|
|
|
# defines
|
|
|
|
cmd.extend(f'-D{define}' for define in idedata['defines'])
|
|
|
|
|
|
|
|
# add include directories, using -isystem for dependencies to suppress their errors
|
|
|
|
for directory in idedata['includes']['toolchain']:
|
2021-09-20 11:47:51 +02:00
|
|
|
if 'xtensa-esp32s2-elf' not in directory:
|
|
|
|
cmd.extend(['-isystem', directory])
|
2021-08-09 22:43:18 +02:00
|
|
|
for directory in sorted(set(idedata['includes']['build'])):
|
|
|
|
dependency = "framework-arduino" in directory or "/libdeps/" in directory
|
|
|
|
cmd.extend(['-isystem' if dependency else '-I', directory])
|
|
|
|
|
|
|
|
return cmd
|
|
|
|
|
|
|
|
|
|
|
|
def run_tidy(args, options, tmpdir, queue, lock, failed_files):
|
2019-04-17 12:06:00 +02:00
|
|
|
while True:
|
|
|
|
path = queue.get()
|
2021-08-09 22:43:18 +02:00
|
|
|
invocation = ['clang-tidy-11']
|
|
|
|
|
2019-04-17 12:06:00 +02:00
|
|
|
if tmpdir is not None:
|
2021-08-09 22:43:18 +02:00
|
|
|
invocation.append('--export-fixes')
|
2019-04-17 12:06:00 +02:00
|
|
|
# Get a temporary file. We immediately close the handle so clang-tidy can
|
|
|
|
# overwrite it.
|
|
|
|
(handle, name) = tempfile.mkstemp(suffix='.yaml', dir=tmpdir)
|
|
|
|
os.close(handle)
|
|
|
|
invocation.append(name)
|
2021-08-09 22:43:18 +02:00
|
|
|
|
2019-04-17 12:06:00 +02:00
|
|
|
if args.quiet:
|
|
|
|
invocation.append('-quiet')
|
2021-08-09 22:43:18 +02:00
|
|
|
|
2019-04-17 12:06:00 +02:00
|
|
|
invocation.append(os.path.abspath(path))
|
2021-09-24 18:02:28 +02:00
|
|
|
invocation.append(f"--header-filter={os.path.abspath(basepath)}/.*")
|
2021-08-09 22:43:18 +02:00
|
|
|
invocation.append('--')
|
|
|
|
invocation.extend(options)
|
2019-04-17 12:06:00 +02:00
|
|
|
invocation_s = ' '.join(shlex_quote(x) for x in invocation)
|
|
|
|
|
|
|
|
# Use pexpect for a pseudy-TTY with colored output
|
|
|
|
output, rc = pexpect.run(invocation_s, withexitstatus=True, encoding='utf-8',
|
2020-07-14 14:34:44 +02:00
|
|
|
timeout=15 * 60)
|
2021-07-25 23:54:32 +02:00
|
|
|
if rc != 0:
|
|
|
|
with lock:
|
2019-04-17 12:06:00 +02:00
|
|
|
print()
|
|
|
|
print("\033[0;32m************* File \033[1;32m{}\033[0m".format(path))
|
|
|
|
print(output)
|
|
|
|
print()
|
|
|
|
failed_files.append(path)
|
|
|
|
queue.task_done()
|
|
|
|
|
|
|
|
|
|
|
|
def progress_bar_show(value):
|
|
|
|
if value is None:
|
|
|
|
return ''
|
|
|
|
|
|
|
|
|
2020-07-14 14:34:44 +02:00
|
|
|
def split_list(a, n):
|
|
|
|
k, m = divmod(len(a), n)
|
|
|
|
return [a[i * k + min(i, m):(i + 1) * k + min(i + 1, m)] for i in range(n)]
|
|
|
|
|
|
|
|
|
2019-04-17 12:06:00 +02:00
|
|
|
def main():
|
|
|
|
parser = argparse.ArgumentParser()
|
|
|
|
parser.add_argument('-j', '--jobs', type=int,
|
|
|
|
default=multiprocessing.cpu_count(),
|
|
|
|
help='number of tidy instances to be run in parallel.')
|
2021-09-30 16:24:02 +02:00
|
|
|
parser.add_argument('-e', '--environment', default='esp32-tidy',
|
2021-09-13 18:11:27 +02:00
|
|
|
help='the PlatformIO environment to run against (esp8266-tidy or esp32-tidy)')
|
2019-04-17 12:06:00 +02:00
|
|
|
parser.add_argument('files', nargs='*', default=[],
|
|
|
|
help='files to be processed (regex on path)')
|
|
|
|
parser.add_argument('--fix', action='store_true', help='apply fix-its')
|
|
|
|
parser.add_argument('-q', '--quiet', action='store_false',
|
2021-07-25 23:54:32 +02:00
|
|
|
help='run clang-tidy in quiet mode')
|
2019-04-17 12:06:00 +02:00
|
|
|
parser.add_argument('-c', '--changed', action='store_true',
|
2021-07-25 23:54:32 +02:00
|
|
|
help='only run on changed files')
|
2021-09-13 18:11:27 +02:00
|
|
|
parser.add_argument('-g', '--grep', help='only run on files containing value')
|
2021-07-25 23:54:32 +02:00
|
|
|
parser.add_argument('--split-num', type=int, help='split the files into X jobs.',
|
2020-07-14 14:34:44 +02:00
|
|
|
default=None)
|
2021-07-25 23:54:32 +02:00
|
|
|
parser.add_argument('--split-at', type=int, help='which split is this? starts at 1',
|
2020-07-14 14:34:44 +02:00
|
|
|
default=None)
|
2019-04-17 12:06:00 +02:00
|
|
|
parser.add_argument('--all-headers', action='store_true',
|
2021-07-25 23:54:32 +02:00
|
|
|
help='create a dummy file that checks all headers')
|
2019-04-17 12:06:00 +02:00
|
|
|
args = parser.parse_args()
|
|
|
|
|
|
|
|
try:
|
2021-06-08 22:16:17 +02:00
|
|
|
get_output('clang-tidy-11', '-version')
|
2019-04-17 12:06:00 +02:00
|
|
|
except:
|
|
|
|
print("""
|
2021-06-08 22:16:17 +02:00
|
|
|
Oops. It looks like clang-tidy-11 is not installed.
|
2020-07-27 12:07:05 +02:00
|
|
|
|
2021-06-08 22:16:17 +02:00
|
|
|
Please check you can run "clang-tidy-11 -version" in your terminal and install
|
|
|
|
clang-tidy (v11) if necessary.
|
2020-07-27 12:07:05 +02:00
|
|
|
|
2019-04-17 12:06:00 +02:00
|
|
|
Note you can also upload your code as a pull request on GitHub and see the CI check
|
|
|
|
output to apply clang-tidy.
|
|
|
|
""")
|
|
|
|
return 1
|
|
|
|
|
2021-09-13 18:11:27 +02:00
|
|
|
idedata = load_idedata(args.environment)
|
2021-08-09 22:43:18 +02:00
|
|
|
options = clang_options(idedata)
|
2019-04-17 12:06:00 +02:00
|
|
|
|
|
|
|
files = []
|
2021-07-25 23:54:32 +02:00
|
|
|
for path in git_ls_files(['*.cpp']):
|
|
|
|
files.append(os.path.relpath(path, os.getcwd()))
|
|
|
|
|
|
|
|
if args.files:
|
|
|
|
# Match against files specified on command-line
|
|
|
|
file_name_re = re.compile('|'.join(args.files))
|
|
|
|
files = [p for p in files if file_name_re.search(p)]
|
2019-04-17 12:06:00 +02:00
|
|
|
|
|
|
|
if args.changed:
|
|
|
|
files = filter_changed(files)
|
|
|
|
|
2021-09-13 18:11:27 +02:00
|
|
|
if args.grep:
|
|
|
|
files = filter_grep(files, args.grep)
|
|
|
|
|
2019-04-17 12:06:00 +02:00
|
|
|
files.sort()
|
|
|
|
|
2020-07-14 14:34:44 +02:00
|
|
|
if args.split_num:
|
|
|
|
files = split_list(files, args.split_num)[args.split_at - 1]
|
|
|
|
|
|
|
|
if args.all_headers and args.split_at in (None, 1):
|
2021-08-09 22:43:18 +02:00
|
|
|
build_all_include()
|
2019-04-17 12:06:00 +02:00
|
|
|
files.insert(0, temp_header_file)
|
|
|
|
|
|
|
|
tmpdir = None
|
|
|
|
if args.fix:
|
|
|
|
tmpdir = tempfile.mkdtemp()
|
|
|
|
|
|
|
|
failed_files = []
|
|
|
|
try:
|
|
|
|
task_queue = queue.Queue(args.jobs)
|
|
|
|
lock = threading.Lock()
|
|
|
|
for _ in range(args.jobs):
|
|
|
|
t = threading.Thread(target=run_tidy,
|
2021-08-09 22:43:18 +02:00
|
|
|
args=(args, options, tmpdir, task_queue, lock, failed_files))
|
2019-04-17 12:06:00 +02:00
|
|
|
t.daemon = True
|
|
|
|
t.start()
|
|
|
|
|
|
|
|
# Fill the queue with files.
|
|
|
|
with click.progressbar(files, width=30, file=sys.stderr,
|
|
|
|
item_show_func=progress_bar_show) as bar:
|
|
|
|
for name in bar:
|
|
|
|
task_queue.put(name)
|
|
|
|
|
|
|
|
# Wait for all threads to be done.
|
|
|
|
task_queue.join()
|
|
|
|
|
|
|
|
except KeyboardInterrupt:
|
|
|
|
print()
|
|
|
|
print('Ctrl-C detected, goodbye.')
|
|
|
|
if tmpdir:
|
|
|
|
shutil.rmtree(tmpdir)
|
|
|
|
os.kill(0, 9)
|
|
|
|
|
|
|
|
if args.fix and failed_files:
|
|
|
|
print('Applying fixes ...')
|
|
|
|
try:
|
2021-06-08 22:16:17 +02:00
|
|
|
subprocess.call(['clang-apply-replacements-11', tmpdir])
|
2019-04-17 12:06:00 +02:00
|
|
|
except:
|
|
|
|
print('Error applying fixes.\n', file=sys.stderr)
|
|
|
|
raise
|
|
|
|
|
2021-07-25 23:54:32 +02:00
|
|
|
sys.exit(len(failed_files))
|
2019-04-17 12:06:00 +02:00
|
|
|
|
|
|
|
|
|
|
|
if __name__ == '__main__':
|
2021-07-25 23:54:32 +02:00
|
|
|
main()
|