Skip to content

Commit

Permalink
Adding checkbashsms call to test target of the Makefile as suggested …
Browse files Browse the repository at this point in the history
…by Trent W. Buck <trentbuck@gmail.com>.
  • Loading branch information
daniel-baumann committed Nov 13, 2007
1 parent ab9a90b commit 9217c8a
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions Makefile
Expand Up @@ -5,11 +5,20 @@ TRANSLATIONS="it"
all: build

test:
# Checking for syntax errors
set -e; for SCRIPT in bin/* hooks/* scripts/live scripts/live-functions scripts/live-helpers scripts/*/*; \
do \
sh -n $$SCRIPT; \
done

# Checking for bashisms (temporary not failing, but only listing)
if [ -x /usr/bin/checkbashisms ]; \
then \
checkbashisms bin/* hooks/* scripts/live scripts/live-functions scripts/live-helpers scripts/*/* || true; \
else \
echo "bashism test skipped - you need to install devscripts."; \
fi

build:

install: test build
Expand Down

0 comments on commit 9217c8a

Please sign in to comment.