commit c281e587bb1b4ac1bdaa97ea68c1ab0c80fe4bab
parent b169c8c14f23394b40305f38ee4ce08add278e27
Author: Nick Mathewson <nickm@torproject.org>
Date: Tue, 15 Jan 2019 08:08:41 -0500
Merge branch 'ticket28058_squashed'
Diffstat:
3 files changed, 14 insertions(+), 1 deletion(-)
diff --git a/.travis.yml b/.travis.yml
@@ -131,6 +131,7 @@ addons:
- docbook-xsl
- docbook-xml
- xmlto
+ - shellcheck
## (OSX only)
homebrew:
packages:
@@ -154,6 +155,7 @@ addons:
## Always installed, because manual brew installs are hard to get right
- asciidoc
- xmlto
+ - shellcheck
## (OSX only) Use the default OSX image
## See https://docs.travis-ci.com/user/reference/osx#os-x-version
diff --git a/Makefile.am b/Makefile.am
@@ -215,7 +215,16 @@ doxygen:
test: all
$(top_builddir)/src/test/test
-check-local: check-spaces check-changes check-includes
+shellcheck:
+ # Only use shellcheck if it is present
+ if command -v shellcheck; then \
+ find $(top_srcdir)/scripts/ -name "*.sh" -exec shellcheck {} +; \
+ if [ -d "$(top_srcdir)/scripts/test" ]; then \
+ shellcheck $(top_srcdir)/scripts/test/cov-diff $(top_builddir)/scripts/test/coverage; \
+ fi; \
+ fi
+
+check-local: check-spaces check-changes check-includes shellcheck
need-chutney-path:
@if test ! -d "$$CHUTNEY_PATH"; then \
diff --git a/changes/ticket28058 b/changes/ticket28058
@@ -0,0 +1,2 @@
+ o Testing:
+ - Run shellcheck for stuff in scripts/ directory. Closes ticket 28058.