diff options
author | Tom Ryder <tom@sanctum.geek.nz> | 2017-10-31 21:52:24 +1300 |
---|---|---|
committer | Tom Ryder <tom@sanctum.geek.nz> | 2017-10-31 21:52:24 +1300 |
commit | a469c1ef50d0c03b93add44e50afa5d791855106 (patch) | |
tree | 1c849ff1bde6a28add6efe43afb44735c367ce9b /lint | |
parent | Revert "Drastically simplify `lint` scripts" (diff) | |
download | dotfiles-a469c1ef50d0c03b93add44e50afa5d791855106.tar.gz dotfiles-a469c1ef50d0c03b93add44e50afa5d791855106.zip |
Adjust `check-bin`, `lint-bin` for built scripts
Make the `$(BINS)` target a prerequisite of `check-bin` so that all of
the scripts with a #!/bin/sh shebang are built, and then check them all
by iterating through a glob (and hence an order according to LC_COLLATE)
and stripping the `.sh` suffix to find the name of the matching
shebanged script.
Leverage `shellcheck`'s support of multiple check arguments to build an
argument list of the binscripts first before passing all of those to a
single call, simply for speed.
We don't have anything in this target to test the scripts of any other
type, such as the `.awk` or `.sed` scripts. `gawk` has a `--lint` mode
that might apply.
Diffstat (limited to 'lint')
-rw-r--r-- | lint/bin.sh | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/lint/bin.sh b/lint/bin.sh index 0fe82d7a..16facb6a 100644 --- a/lint/bin.sh +++ b/lint/bin.sh @@ -1 +1,5 @@ -find bin -type f -name '*.sh' -print -exec shellcheck -e SC1090 -s sh -- {} + +set -- +for sh in bin/*.sh ; do + set "$@" "${sh%.sh}" +done +shellcheck -e SC1090 -- "$@" |