tests skips: shorter and clearer messages

* tests/defs: Use shorter messages when giving reasons for test
skipping; it turns out these shorter messages are also clearer.
If more info might be useful, send them to the log file only.

diff --git a/ChangeLog b/ChangeLog
index c629fe0290e1f06c31d4104856a4375c30ea6636..0851b77d58d6ed40bf7ffef3ee1453b01559c01a 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2011-05-01  Stefano Lattarini  <stefano.lattarini@gmail.com>
+
+       tests skips: shorter and clearer messages
+       * tests/defs: Use shorter messages when giving reasons for test
+       skipping; it turns out these shorter messages are also clearer.
+       If more info might be useful, send them to the log file only.
+
 2011-05-01  Stefano Lattarini  <stefano.lattarini@gmail.com>
 
        tests: fix self check spurious failure with older bash versions

diff --git a/tests/defs b/tests/defs
index 9f862af7effc6b603c9ae239e97105d8974ed72f..8c2b084803ea0a3609db467dc3c0c0d4660ad22e 100644 (file)
--- a/tests/defs
+++ b/tests/defs
@@ -441,8 +441,7 @@ case $testbuilddir in
   *\ *|*\      *)
     case " $required " in
       *' libtool '* | *' libtoolize '* )
-        skip_ "libtool and libtoolize cannot cope correctly" \
-              "with spaces in the build tree";;
+        skip_ "libtool has problems with spaces in builddir name";;
     esac
     ;;
 esac
@@ -455,8 +454,7 @@ case $testsrcdir in
   *\ * |*\     *)
     case " $required " in
       *' libtool '* | *' libtoolize '* | *' gettext '* )
-        skip_ "our testsuite setup cannot cope with spaces in the" \
-              "source tree name for libtool/gettext tests.";;
+        skip_ "spaces in srcdir name: libtool/gettext tests won't work";;
    esac
    ;;
 esac
@@ -493,14 +491,18 @@ case " $required " in
     case " $required " in
       *' libtool '*|*' libtoolize '*)
         if test $libtool_found != yes; then
-          skip_ "libtool is required, but libtool.m4 wasn't found in" \
-                "directories $aclocaldir $extra_includes"
+          # Write more diagnostic to the log file than to the console.
+          echo "$me: \`libtool.m4' wasn't found in directories" \
+               "$aclocaldir $extra_includes"
+          skip_ "libtool m4 macros won't be found by aclocal"
         fi
         ;;
       *' gettext '*)
         if test $gettext_found != yes; then
-          skip_ "gettext is required, but gettext.m4 wasn't found in" \
-                "directories $aclocaldir $extra_includes"
+          # Write more diagnostic to the log file than to the console.
+          echo "$me: \`gettext.m4' wasn't found in directories" \
+               "$aclocaldir $extra_includes"
+          skip_ "gettext m4 macros won't be found by aclocal"
         fi
         ;;
     esac