Don't fail in Makefile when there's no checkbashism.
authorDaniel Baumann <daniel@debian.org>
Mon, 26 Jul 2010 21:05:09 +0000 (23:05 +0200)
committerDaniel Baumann <daniel@debian.org>
Mon, 26 Jul 2010 21:05:09 +0000 (23:05 +0200)
Makefile

index b24d4f8..915f864 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -19,27 +19,24 @@ test:
 
        @echo " done."
 
-       @if [ ! -x "$$(which checkbashisms 2>/dev/null)" ]; \
+       @# We can't just fail yet on bashisms (FIXME)
+       @if [ -x "$$(which checkbashisms 2>/dev/null)" ]; \
        then \
+               echo -n "Checking for bashisms"; \
+               for SCRIPT in $(SCRIPTS); \
+               do \
+                       checkbashisms -f -x $${SCRIPT} || true; \
+                       echo -n "."; \
+               done; \
+               echo " done."; \
+       else \
                echo "W: checkbashisms - command not found"; \
                echo "I: checkbashisms can be optained from: "; \
                echo "I:   http://git.debian.org/?p=devscripts/devscripts.git"; \
                echo "I: On Debian systems, checkbashisms can be installed with:"; \
                echo "I:   apt-get install devscripts"; \
-               exit 0; \
        fi
 
-       @echo -n "Checking for bashisms"
-
-       @# We can't just fail yet on bashisms (FIXME)
-       @for SCRIPT in $(SCRIPTS); \
-       do \
-               checkbashisms -f -x $${SCRIPT} || true; \
-               echo -n "."; \
-       done
-
-       @echo " done."
-
 build:
        @echo "Nothing to build."