tom/cdist
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Add shellcheck makefile target. shellcheck when doing release.

Browse source
This commit is contained in:
Darko Poljak 2018-10-06 14:57:55 +02:00
parent 877b4d2748
commit 8436f872a9
3 changed files with 68 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
Makefile
View file

@ -39,6 +39,9 @@ PYTHON_VERSION=cdist/version.py
SPHINXM=make -C $(DOCS_SRC_DIR) man SPHINXM=make -C $(DOCS_SRC_DIR) man
SPHINXH=make -C $(DOCS_SRC_DIR) html SPHINXH=make -C $(DOCS_SRC_DIR) html
SPHINXC=make -C $(DOCS_SRC_DIR) clean SPHINXC=make -C $(DOCS_SRC_DIR) clean
SHELLCHECKCMD=shellcheck -s sh -f gcc -x
SHELLCHECK_SKIP=grep -v ': __.*is referenced but not assigned.*\[SC2154\]'
################################################################################ ################################################################################
# Manpages # Manpages
# #
@ -253,3 +256,20 @@ test-remote:
pep8: pep8:
$(helper) $@ $(helper) $@
shellcheck-global-explorers:
@find cdist/conf/explorer -type f -exec $(SHELLCHECKCMD) {} + | $(SHELLCHECK_SKIP)
shellcheck-manifests:
@find cdist/conf/type -type f -name manifest -exec $(SHELLCHECKCMD) {} + | $(SHELLCHECK_SKIP)
shellcheck-local-gencodes:
@find cdist/conf/type -type f -name gencode-local -exec $(SHELLCHECKCMD) {} + | $(SHELLCHECK_SKIP)
shellcheck-remote-gencodes:
@find cdist/conf/type -type f -name gencode-remote -exec $(SHELLCHECKCMD) {} + | $(SHELLCHECK_SKIP)
shellcheck-gencodes: shellcheck-local-gencodes shellcheck-remote-gencodes
shellcheck-types: shellcheck-manifests shellcheck-gencodes
shellcheck: shellcheck-global-explorers shellcheck-types

25
bin/build-helper
View file

@ -250,6 +250,7 @@ eof
"$0" check-date "$0" check-date
"$0" check-unittest "$0" check-unittest
"$0" check-pep8 "$0" check-pep8
"$0" shellcheck
# Generate version file to be included in packaging # Generate version file to be included in packaging
"$0" target-version "$0" target-version
@ -370,7 +371,29 @@ eof
check-pep8) check-pep8)
"$0" pep8 "$0" pep8
echo "Please review pep8 report." printf "\\nPlease review pep8 report.\\n"
while true
do
echo "Continue (yes/no)?"
any=
read any
case "$any" in
yes)
break
;;
no)
exit 1
;;
*)
echo "Please answer with 'yes' or 'no' explicitly."
;;
esac
done
;;
shellcheck)
make helper=${helper} WEBDIR=${WEBDIR} shellcheck
printf "\\nPlease review shellcheck report.\\n"
while true while true
do do
echo "Continue (yes/no)?" echo "Continue (yes/no)?"

25
bin/build-helper.freebsd
View file

@ -285,6 +285,7 @@ eof
"$0" check-date "$0" check-date
"$0" check-unittest "$0" check-unittest
"$0" check-pep8 "$0" check-pep8
"$0" shellcheck
# Generate version file to be included in packaging # Generate version file to be included in packaging
"$0" target-version "$0" target-version
@ -432,7 +433,29 @@ eof
check-pep8) check-pep8)
"$0" pep8 "$0" pep8
echo "Please review pep8 report." printf "\\nPlease review pep8 report.\\n"
while true
do
echo "Continue (yes/no)?"
any=
read any
case "$any" in
yes)
break
;;
no)
exit 1
;;
*)
echo "Please answer with 'yes' or 'no' explicitly."
;;
esac
done
;;
shellcheck)
make helper=${helper} WEBDIR=${WEBDIR} shellcheck
printf "\\nPlease review shellcheck report.\\n"
while true while true
do do
echo "Continue (yes/no)?" echo "Continue (yes/no)?"