I found configure could use some minor improvements:
authorAlbert Lee <trisk@acm.jhu.edu>
Fri, 18 Dec 2009 01:10:13 +0000 (02:10 +0100)
committerJelmer Vernooij <jelmer@samba.org>
Fri, 18 Dec 2009 01:10:13 +0000 (02:10 +0100)
Remove a use of 'if !' which doesn't work on plain Bourne sh.

configure.ac

index 63ec1f2..303d90d 100644 (file)
@@ -9,9 +9,7 @@ AC_DEFINE_UNQUOTED(PACKAGE,"$PACKAGE", [ Package name])
 AC_SUBST(BZR_CHECKOUT)
 BZR_CHECKOUT=no
 AC_MSG_CHECKING([whether this is a bzr checkout])
-if ! which bzr >/dev/null 2>/dev/null; then
-       AC_MSG_RESULT(no)
-else 
+if which bzr >/dev/null 2>/dev/null; then
        BZR_CHECKOUT=yes
        REVISION="`bzr revno .`"
        if test -n "$REVISION"; then
@@ -20,6 +18,8 @@ else
        else
                AC_MSG_RESULT(no)
        fi
+else 
+       AC_MSG_RESULT(no)
 fi
 
 AC_DEFINE_UNQUOTED(VERSION,"$VERSION$BZRVERSION", [ Package version])