blob: bd7cb10fcc2bf3be54dc6175adea63c45946e8ed [file] [log] [blame]
Andrew Geissler9b0105c2018-01-10 10:58:35 -08001#!/bin/bash
Patrick Williamsc1513c82022-11-27 17:37:08 -06002set -e
Andrew Geissler9b0105c2018-01-10 10:58:35 -08003
Patrick Williamsc1513c82022-11-27 17:37:08 -06004# This script reformats source files using various formatters and linters.
Andrew Geissler9b0105c2018-01-10 10:58:35 -08005#
6# Files are changed in-place, so make sure you don't have anything open in an
7# editor, and you may want to commit before formatting in case of awryness.
8#
9# This must be run on a clean repository to succeed
10#
Patrick Williamsc1513c82022-11-27 17:37:08 -060011function display_help()
12{
Patrick Williams147f37a2022-12-04 14:57:10 -060013 echo "usage: format-code.sh [-h | --help] [--no-diff] [--disable <tool>]"
14 echo " [--list-tools] [<path>]"
Patrick Williamsc1513c82022-11-27 17:37:08 -060015 echo
16 echo "Format and lint a repository."
17 echo
18 echo "Arguments:"
Patrick Williams147f37a2022-12-04 14:57:10 -060019 echo " --list-tools Display available linters and formatters"
20 echo " --no-diff Don't show final diff output"
21 echo " --disable <tool> Disable linter"
Patrick Williams26d9d2c2022-12-05 15:03:07 -060022 echo " --allow-missing Run even if linters are not all present"
Patrick Williams147f37a2022-12-04 14:57:10 -060023 echo " path Path to git repository (default to pwd)"
Patrick Williamsc1513c82022-11-27 17:37:08 -060024}
Andrew Geissler9b0105c2018-01-10 10:58:35 -080025
Patrick Williams147f37a2022-12-04 14:57:10 -060026LINTERS_ALL=( \
27 commit_gitlint \
28 commit_spelling \
29 clang_format \
30 eslint \
31 pycodestyle \
32 shellcheck \
33 )
34LINTERS_DISABLED=()
35LINTERS_ENABLED=()
Patrick Williams785327a2022-12-05 16:20:43 -060036declare -A LINTERS_FAILED=()
Patrick Williams147f37a2022-12-04 14:57:10 -060037
Patrick Williams26d9d2c2022-12-05 15:03:07 -060038eval set -- "$(getopt -o 'h' --long 'help,list-tools,no-diff,disable:,allow-missing' -n 'format-code.sh' -- "$@")"
Patrick Williamsc1513c82022-11-27 17:37:08 -060039while true; do
40 case "$1" in
41 '-h'|'--help')
42 display_help && exit 0
43 ;;
Andrew Geissler9b0105c2018-01-10 10:58:35 -080044
Patrick Williams147f37a2022-12-04 14:57:10 -060045 '--list-tools')
46 echo "Available tools:"
47 for t in "${LINTERS_ALL[@]}"; do
48 echo " $t"
49 done
50 exit 0
51 ;;
52
Patrick Williamsd27ab4c2022-11-27 17:51:35 -060053 '--no-diff')
54 OPTION_NO_DIFF=1
55 shift
56 ;;
57
Patrick Williams147f37a2022-12-04 14:57:10 -060058 '--disable')
59 LINTERS_DISABLED+=("$2")
60 shift && shift
61 ;;
62
Patrick Williams26d9d2c2022-12-05 15:03:07 -060063 '--allow-missing')
64 ALLOW_MISSING=yes
65 shift
66 ;;
67
Patrick Williamsc1513c82022-11-27 17:37:08 -060068 '--')
69 shift
70 break
71 ;;
72
73 *)
74 echo "unknown option: $1"
75 display_help && exit 1
76 ;;
77 esac
78done
79
80# Detect tty and set nicer colors.
81if [ -t 1 ]; then
82 BLUE="\e[34m"
83 GREEN="\e[32m"
84 NORMAL="\e[0m"
85 RED="\e[31m"
86 YELLOW="\e[33m"
87else # non-tty, no escapes.
88 BLUE=""
89 GREEN=""
90 NORMAL=""
91 RED=""
92 YELLOW=""
93fi
94
Patrick Williamsdad9b7a2022-11-30 13:10:20 -060095# Allow called scripts to know which clang format we are using
96export CLANG_FORMAT="clang-format"
97
Patrick Williamsc1513c82022-11-27 17:37:08 -060098# Path to default config files for linters.
99CONFIG_PATH="$(git -C "$(dirname "${BASH_SOURCE[0]}")" rev-parse --show-toplevel)/config"
100
101# Find repository root for `pwd` or $1.
102if [ -z "$1" ]; then
103 DIR="$(git rev-parse --show-toplevel || pwd)"
104else
105 DIR="$(git -C "$1" rev-parse --show-toplevel)"
106fi
Patrick Williamsbc85b712022-12-05 15:16:27 -0600107if [ ! -e "$DIR/.git" ]; then
108 echo -e "${RED}Error:${NORMAL} Directory ($DIR) does not appear to be a git repository"
Patrick Williamsc1513c82022-11-27 17:37:08 -0600109 exit 1
110fi
Andrew Jeffery0cce8482018-04-30 11:37:01 +0930111
Manojkiran Edae6f120a2021-03-20 11:13:43 +0530112cd "${DIR}"
Patrick Williamsc1513c82022-11-27 17:37:08 -0600113echo -e " ${BLUE}Formatting code under${NORMAL} $DIR"
Andrew Geissler9b0105c2018-01-10 10:58:35 -0800114
Patrick Williams3d294942022-12-05 14:58:15 -0600115# Config hashes:
116# LINTER_REQUIRE - The requirements to run a linter, semi-colon separated.
117# 1. Executable.
118# 2. [optional] Configuration file.
119# 3. [optional] Global fallback configuration file.
120#
121# LINTER_IGNORE - An optional set of semi-colon separated ignore-files
122# specific to the linter.
123#
124# LINTER_TYPES - The file types supported by the linter, semi-colon separated.
125#
126# LINTER_CONFIG - The config (from LINTER_REQUIRE) chosen for the repository.
127#
Patrick Williamsdad9b7a2022-11-30 13:10:20 -0600128declare -A LINTER_REQUIRE=()
Patrick Williams71b73242022-12-02 21:07:52 -0600129declare -A LINTER_IGNORE=()
Patrick Williams71b73242022-12-02 21:07:52 -0600130declare -A LINTER_TYPES=()
Patrick Williams3d294942022-12-05 14:58:15 -0600131declare -A LINTER_CONFIG=()
Manojkiran Eda87111bb2021-08-14 11:26:16 +0530132
Patrick Williamsdad9b7a2022-11-30 13:10:20 -0600133LINTER_REQUIRE+=([commit_spelling]="codespell")
Patrick Williams71b73242022-12-02 21:07:52 -0600134LINTER_TYPES+=([commit_spelling]="commit")
Patrick Williams38515a32022-11-27 06:58:56 -0600135function do_commit_spelling() {
Patrick Williams38515a32022-11-27 06:58:56 -0600136 # Run the codespell with openbmc spcific spellings on the patchset
Patrick Williams71b73242022-12-02 21:07:52 -0600137 echo -n "openbmc-dictionary - misspelling count >> "
138 sed "s/Signed-off-by.*//" "$@" | \
Patrick Williamsc1513c82022-11-27 17:37:08 -0600139 codespell -D "${CONFIG_PATH}/openbmc-spelling.txt" -d --count -
Patrick Williams38515a32022-11-27 06:58:56 -0600140
141 # Run the codespell with generic dictionary on the patchset
Patrick Williams71b73242022-12-02 21:07:52 -0600142 echo -n "generic-dictionary - misspelling count >> "
143 sed "s/Signed-off-by.*//" "$@" | \
Patrick Williams38515a32022-11-27 06:58:56 -0600144 codespell --builtin clear,rare,en-GB_to_en-US -d --count -
145}
146
Patrick Williamsdad9b7a2022-11-30 13:10:20 -0600147LINTER_REQUIRE+=([commit_gitlint]="gitlint")
Patrick Williams71b73242022-12-02 21:07:52 -0600148LINTER_TYPES+=([commit_gitlint]="commit")
Patrick Williams38515a32022-11-27 06:58:56 -0600149function do_commit_gitlint() {
Patrick Williams71b73242022-12-02 21:07:52 -0600150 gitlint --extra-path "${CONFIG_PATH}/gitlint/" \
Patrick Williamsc1513c82022-11-27 17:37:08 -0600151 --config "${CONFIG_PATH}/.gitlint"
Patrick Williams38515a32022-11-27 06:58:56 -0600152}
153
Patrick Williamsdad9b7a2022-11-30 13:10:20 -0600154LINTER_REQUIRE+=([eslint]="eslint;.eslintrc.json;${CONFIG_PATH}/eslint-global-config.json")
Patrick Williams71b73242022-12-02 21:07:52 -0600155LINTER_IGNORE+=([eslint]=".eslintignore")
156LINTER_TYPES+=([eslint]="json")
Patrick Williams38515a32022-11-27 06:58:56 -0600157function do_eslint() {
Patrick Williams71b73242022-12-02 21:07:52 -0600158 eslint --no-eslintrc -c "${LINTER_CONFIG[eslint]}" \
Patrick Williamsdad9b7a2022-11-30 13:10:20 -0600159 --ext .json --format=stylish \
160 --resolve-plugins-relative-to /usr/local/lib/node_modules \
Patrick Williams71b73242022-12-02 21:07:52 -0600161 --no-error-on-unmatched-pattern "$@"
Patrick Williams38515a32022-11-27 06:58:56 -0600162}
Manojkiran Eda87111bb2021-08-14 11:26:16 +0530163
Patrick Williamsdad9b7a2022-11-30 13:10:20 -0600164LINTER_REQUIRE+=([pycodestyle]="pycodestyle;setup.cfg")
Patrick Williams71b73242022-12-02 21:07:52 -0600165LINTER_TYPES+=([pycodestyle]="python")
Patrick Williams38515a32022-11-27 06:58:56 -0600166function do_pycodestyle() {
Patrick Williams71b73242022-12-02 21:07:52 -0600167 pycodestyle --show-source "$@"
Patrick Williams38515a32022-11-27 06:58:56 -0600168}
James Feistf1665d62019-11-22 09:06:33 -0800169
Patrick Williamsdad9b7a2022-11-30 13:10:20 -0600170LINTER_REQUIRE+=([shellcheck]="shellcheck;.shellcheck")
Patrick Williams71b73242022-12-02 21:07:52 -0600171LINTER_TYPES+=([shellcheck]="bash;sh")
Patrick Williams38515a32022-11-27 06:58:56 -0600172function do_shellcheck() {
Patrick Williams71b73242022-12-02 21:07:52 -0600173 shellcheck --color=never -x "$@"
Patrick Williams38515a32022-11-27 06:58:56 -0600174}
James Feistf1665d62019-11-22 09:06:33 -0800175
Patrick Williamsdad9b7a2022-11-30 13:10:20 -0600176LINTER_REQUIRE+=([clang_format]="clang-format;.clang-format")
Patrick Williams71b73242022-12-02 21:07:52 -0600177LINTER_IGNORE+=([clang_format]=".clang-ignore;.clang-format-ignore")
178LINTER_TYPES+=([clang_format]="c;cpp")
Patrick Williams38515a32022-11-27 06:58:56 -0600179do_clang_format() {
Patrick Williams71b73242022-12-02 21:07:52 -0600180 "${CLANG_FORMAT}" -i "$@"
181}
Patrick Williams38515a32022-11-27 06:58:56 -0600182
Patrick Williams71b73242022-12-02 21:07:52 -0600183function get_file_type()
184{
185 case "$(basename "$1")" in
186 # First to early detect template files.
187 *.in | *.meson) echo "meson-template" && return ;;
188 *.mako | *.mako.*) echo "mako" && return ;;
Patrick Williamsd535ecb2022-12-02 14:54:18 -0600189
Patrick Williams71b73242022-12-02 21:07:52 -0600190 *.ac) echo "autoconf" && return ;;
191 *.[ch]) echo "c" && return ;;
192 *.[ch]pp) echo "cpp" && return ;;
193 *.json) echo "json" && return ;;
194 *.md) echo "markdown" && return ;;
195 *.py) echo "python" && return ;;
196 *.yaml | *.yml) echo "yaml" && return ;;
Patrick Williamsd535ecb2022-12-02 14:54:18 -0600197
Patrick Williams71b73242022-12-02 21:07:52 -0600198 # Special files.
199 .git/COMMIT_EDITMSG) echo "commit" && return ;;
200 meson.build) echo "meson" && return ;;
201 esac
202
203 case "$(file "$1")" in
204 *Bourne-Again\ shell*) echo "bash" && return ;;
205 *C++\ source*) echo "cpp" && return ;;
206 *C\ source*) echo "c" && return ;;
207 *JSON\ data*) echo "json" && return ;;
208 *POSIX\ shell*) echo "sh" && return ;;
209 *Python\ script*) echo "python" && return ;;
210 *zsh\ shell*) echo "zsh" && return ;;
211 esac
212
213 echo "unknown"
Patrick Williams38515a32022-11-27 06:58:56 -0600214}
215
Patrick Williamsdad9b7a2022-11-30 13:10:20 -0600216function check_linter()
217{
218 TITLE="$1"
219 IFS=";" read -r -a ARGS <<< "$2"
220
Patrick Williams147f37a2022-12-04 14:57:10 -0600221 if [[ "${LINTERS_DISABLED[*]}" =~ $1 ]]; then
222 return
223 fi
224
Patrick Williamsdad9b7a2022-11-30 13:10:20 -0600225 EXE="${ARGS[0]}"
226 if [ ! -x "${EXE}" ]; then
227 if ! which "${EXE}" > /dev/null 2>&1 ; then
228 echo -e " ${YELLOW}${TITLE}:${NORMAL} cannot find ${EXE}"
Patrick Williams26d9d2c2022-12-05 15:03:07 -0600229 if [ -z "$ALLOW_MISSING" ]; then
230 exit 1
231 fi
Patrick Williamsdad9b7a2022-11-30 13:10:20 -0600232 return
233 fi
234 fi
235
236 CONFIG="${ARGS[1]}"
237 FALLBACK="${ARGS[2]}"
238
239 if [ -n "${CONFIG}" ]; then
240 if [ -e "${CONFIG}" ]; then
241 LINTER_CONFIG+=( [${TITLE}]="${CONFIG}" )
242 elif [ -n "${FALLBACK}" ] && [ -e "${FALLBACK}" ]; then
243 echo -e " ${YELLOW}${TITLE}:${NORMAL} cannot find ${CONFIG}; using ${FALLBACK}"
244 LINTER_CONFIG+=( [${TITLE}]="${FALLBACK}" )
245 else
246 echo -e " ${YELLOW}${TITLE}:${NORMAL} cannot find config ${CONFIG}"
247 return
248 fi
249 fi
250
251 LINTERS_ENABLED+=( "${TITLE}" )
252}
253
Patrick Williams71b73242022-12-02 21:07:52 -0600254# Check for a global .linter-ignore file.
255GLOBAL_IGNORE=("cat")
256if [ -e ".linter-ignore" ]; then
257 GLOBAL_IGNORE=("${CONFIG_PATH}/lib/ignore-filter" ".linter-ignore")
258fi
259
260# Find all the files in the git repository and organize by type.
261declare -A FILES=()
262if [ -e .git/COMMIT_EDITMSG ]; then
263 FILES+=([commit]=".git/COMMIT_EDITMSG")
264fi
265while read -r file; do
266 ftype="$(get_file_type "$file")"
267 FILES+=([$ftype]="$(echo -ne "$file;${FILES[$ftype]:-}")")
268done < <(git ls-files | "${GLOBAL_IGNORE[@]}")
269
270# For each linter, check if there are an applicable files and if it can
271# be enabled.
Patrick Williamsdad9b7a2022-11-30 13:10:20 -0600272for op in "${LINTERS_ALL[@]}"; do
Patrick Williams71b73242022-12-02 21:07:52 -0600273 for ftype in ${LINTER_TYPES[$op]//;/ }; do
274 if [[ -v FILES["$ftype"] ]]; then
275 check_linter "$op" "${LINTER_REQUIRE[${op}]}"
276 break
277 fi
278 done
Patrick Williamsdad9b7a2022-11-30 13:10:20 -0600279done
280
Patrick Williams71b73242022-12-02 21:07:52 -0600281# Call each linter.
Patrick Williamsdad9b7a2022-11-30 13:10:20 -0600282for op in "${LINTERS_ENABLED[@]}"; do
Patrick Williams71b73242022-12-02 21:07:52 -0600283
284 # Determine the linter-specific ignore file(s).
285 LOCAL_IGNORE=("${CONFIG_PATH}/lib/ignore-filter")
286 if [[ -v LINTER_IGNORE["$op"] ]]; then
287 for ignorefile in ${LINTER_IGNORE["$op"]//;/ } ; do
288 if [ -e "$ignorefile" ]; then
289 LOCAL_IGNORE+=("$ignorefile")
290 fi
291 done
292 fi
293 if [ 1 -eq ${#LOCAL_IGNORE[@]} ]; then
294 LOCAL_IGNORE=("cat")
295 fi
296
297 # Find all the files for this linter, filtering out the ignores.
298 LINTER_FILES=()
299 while read -r file ; do
300 if [ -e "$file" ]; then
301 LINTER_FILES+=("$file")
302 fi
303 done < <(for ftype in ${LINTER_TYPES[$op]//;/ }; do
304 # shellcheck disable=SC2001
305 echo "${FILES["$ftype"]:-}" | sed "s/;/\\n/g"
306 done | "${LOCAL_IGNORE[@]}")
307
308 # Call the linter now with all the files.
309 echo -e " ${BLUE}Running $op${NORMAL}"
Patrick Williams785327a2022-12-05 16:20:43 -0600310 if ! "do_$op" "${LINTER_FILES[@]}" ; then
311 LINTERS_FAILED+=([$op]=1)
312 fi
Patrick Williams38515a32022-11-27 06:58:56 -0600313done
Andrew Jeffery457b6d12018-03-09 15:28:14 +1030314
Patrick Williams785327a2022-12-05 16:20:43 -0600315# Check for failing linters.
316if [ 0 -ne ${#LINTERS_FAILED[@]} ]; then
317 for op in "${!LINTERS_FAILED[@]}"; do
318 echo -e "$op: ${RED}FAILED${NORMAL}"
319 done
320 exit 1
321fi
322
Patrick Williams71b73242022-12-02 21:07:52 -0600323# Check for differences.
Patrick Williamsd27ab4c2022-11-27 17:51:35 -0600324if [ -z "$OPTION_NO_DIFF" ]; then
325 echo -e " ${BLUE}Result differences...${NORMAL}"
326 if ! git --no-pager diff --exit-code ; then
327 echo -e "Format: ${RED}FAILED${NORMAL}"
328 exit 1
329 else
330 echo -e "Format: ${GREEN}PASSED${NORMAL}"
331 fi
Patrick Williamsc1513c82022-11-27 17:37:08 -0600332fi
333
Andrew Jeffery457b6d12018-03-09 15:28:14 +1030334# Sometimes your situation is terrible enough that you need the flexibility.
335# For example, phosphor-mboxd.
Patrick Williams330b0772022-11-28 06:14:06 -0600336for formatter in "format-code.sh" "format-code"; do
337 if [[ -x "${formatter}" ]]; then
338 echo -e " ${BLUE}Calling secondary formatter:${NORMAL} ${formatter}"
339 "./${formatter}"
340 if [ -z "$OPTION_NO_DIFF" ]; then
341 git --no-pager diff --exit-code
342 fi
Patrick Williamsd27ab4c2022-11-27 17:51:35 -0600343 fi
Patrick Williams330b0772022-11-28 06:14:06 -0600344done