Revert configure quoting.

Turns out the suposedly extraneous quoting is required.

Also append success message to build so testbed can catch it and find
any error without having to enumerate all failure cases.

diff --git a/Makefile.am b/Makefile.am
index b3290881a186735ebac68ca0eb2c6b7b79b0b8a5..ff3a26b9368f916575d1ad7cfe61ffe726a40ed5 100644 (file)
--- a/Makefile.am
+++ b/Makefile.am
@@ -42,3 +42,6 @@ EXTRA_DIST = \
 install-pinger:
        chown root $(DEFAULT_PINGER) 
        chmod 4711 $(DEFAULT_PINGER)
+
+all-am:
+       @echo "Build Successful."

diff --git a/configure.in b/configure.in
index b9ff773317ace884ab537366e3240b80d03799d4..bb4831e9c17654f2fd08b2834600e63ca52ea9be 100644 (file)
--- a/configure.in
+++ b/configure.in
@@ -815,7 +815,7 @@ then
     dnl eCAP support requires libecap
     AC_CHECK_LIB([ecap], [main],
         [ECAP_LIBS="-lecap"],
-        AC_MSG_FAILURE([eCAP support requires libecap library, but no usable library was found])
+        [AC_MSG_FAILURE([eCAP support requires libecap library, but no usable library was found])]
     )
 fi
 

diff --git a/test-builds.sh b/test-builds.sh
index 76a6d2961ccc5f93437c53c234de7b1eb475c05a..a8a9d2f4ed054b213e1523cd3e9c5d4cbb731564 100755 (executable)
--- a/test-builds.sh
+++ b/test-builds.sh
@@ -37,6 +37,8 @@ for f in `ls -1 ./test-suite/buildtests/layer*.opts` ; do
        echo "TESTING: ${arg}"
        ../test-suite/buildtest.sh ".${arg}"
        grep -E "${errors}" buildtest_*.log && exit 1
+       result=`tail -2 buildtest_*.log | head -1`
+       test "${result}" = "Build Successful." || ( tail -5 buildtest_*.log ; exit 1 )
        cd ..
        if test "${cleanup}" = "yes" ; then
                echo "REMOVE: bt${layer}"