]> git.proxmox.com Git - mirror_zfs.git/blame - config/Shellcheck.am
autoconf: use include directives instead of recursing down etc
[mirror_zfs.git] / config / Shellcheck.am
CommitLineData
5cdca5b1 1# Global ShellCheck exclusions:
8df0bde3
DS
2#
3# ShellCheck can't follow non-constant source. Use a directive to specify location. [SC1090]
4# Not following: a was not specified as input (see shellcheck -x). [SC1091]
5# Prefer putting braces around variable references even when not strictly required. [SC2250]
6# In POSIX sh, 'local' is undefined. [SC2039] # older ShellCheck versions
7# In POSIX sh, 'local' is undefined. [SC3043] # newer ShellCheck versions
5cdca5b1
AZ
8
9SHELLCHECKSCRIPTS =
10
11JUST_SHELLCHECK_OPTS = $(addprefix shellcheck-here-,$(subst /,^,$(1)))
12JUST_CHECKBASHISMS_OPTS = $(addprefix checkbashisms-here-,$(subst /,^,$(1)))
13SHELLCHECK_OPTS = $(call JUST_SHELLCHECK_OPTS,$(1)) $(call JUST_CHECKBASHISMS_OPTS,$(1))
14
48f43799 15PHONY += shellcheck shellcheck-recursive
5cdca5b1
AZ
16
17shellcheck-recursive:
18 @set -e; for dir in $(SHELLCHECKDIRS); do $(MAKE) -C $$dir shellcheck; done
19
20_STGT = $(subst ^,/,$(subst shellcheck-here-,,$@))
21shellcheck-here-%:
c3ef9f75 22if HAVE_SHELLCHECK
5cdca5b1 23 shellcheck --format=gcc --enable=all --exclude=SC1090,SC1091,SC2039,SC2250,SC3043 $$([ -n "$(SHELLCHECK_SHELL)" ] && echo "--shell=$(SHELLCHECK_SHELL)") "$$([ -e "$(_STGT)" ] || echo "$(srcdir)/")$(_STGT)"
c3ef9f75 24else
5cdca5b1 25 @echo "skipping shellcheck of" $(_STGT) "because shellcheck is not installed"
c3ef9f75 26endif
5cdca5b1
AZ
27
28shellcheck: $(SHELLCHECKSCRIPTS) $(call JUST_SHELLCHECK_OPTS,$(SHELLCHECKSCRIPTS)) shellcheck-recursive
f7d7ee05
AZ
29
30
48f43799 31PHONY += checkbashisms checkbashisms-recursive
5cdca5b1
AZ
32
33checkbashisms-recursive:
34 @set -e; for dir in $(SHELLCHECKDIRS); do $(MAKE) -C $$dir checkbashisms; done
35
f7d7ee05 36# command -v *is* specified by POSIX and every shell in existence supports it
5cdca5b1
AZ
37_BTGT = $(subst ^,/,$(subst checkbashisms-here-,,$@))
38checkbashisms-here-%:
f7d7ee05 39if HAVE_CHECKBASHISMS
5cdca5b1
AZ
40 ! { [ -n "$(SHELLCHECK_SHELL)" ] && echo '#!/bin/$(SHELLCHECK_SHELL)'; cat "$$([ -e "$(_BTGT)" ] || echo "$(srcdir)/")$(_BTGT)"; } | \
41 checkbashisms -npx 2>&1 | grep -vFe "'command' with option other than -p" -e 'command -v' -e 'any possible bashisms' $(CHECKBASHISMS_IGNORE) >&2
f7d7ee05 42else
5cdca5b1 43 @echo "skipping checkbashisms of" $(_BTGT) "because checkbashisms is not installed"
f7d7ee05 44endif
5cdca5b1
AZ
45
46checkbashisms: $(SHELLCHECKSCRIPTS) $(call JUST_CHECKBASHISMS_OPTS,$(SHELLCHECKSCRIPTS)) checkbashisms-recursive