Merge branch 'maint'
authorStefano Lattarini <stefano.lattarini@gmail.com>
Thu, 28 Jun 2012 15:41:27 +0000 (17:41 +0200)
committerStefano Lattarini <stefano.lattarini@gmail.com>
Thu, 28 Jun 2012 15:41:27 +0000 (17:41 +0200)
* maint:
  tests: one test was not executable, make it so
  readme: fix typo in t/README: s/$((...)/$((...))/
  tests: don't skip if $(abs_builddir) or $(abs_srcdir) contain whitespace
  tests: avoid spurious maintainer-check failures
  tests: remove obsolete hacks around maintainer-check false positives

1  2 
t/ax/test-init.sh

diff --combined t/ax/test-init.sh
@@@ -591,7 -591,8 +591,8 @@@ fetch_tap_driver (
    # TODO: we should devise a way to make the shell TAP driver tested also
    # TODO: with /bin/sh, for better coverage.
    case $am_tap_implementation in
-     perl)
+     # Extra quoting required to avoid maintainer-check spurious failures.
+    'perl')
        $PERL -MTAP::Parser -e 1 \
          || skip_all_ "cannot import TAP::Parser perl module"
        sed "1s|#!.*|#! $PERL -w|" "$am_scriptdir"/tap-driver.pl >tap-driver
@@@ -797,6 -798,13 +798,6 @@@ d
        makedepend -f- \
          || skip_all_ "required program 'makedepend' not available"
        ;;
 -    makeinfo-html)
 -      # Make sure we have makeinfo, and it understands '--html'.
 -      echo "$me: running makeinfo --html --version"
 -      makeinfo --html --version \
 -        || skip_all_ "cannot find a makeinfo program that groks" \
 -                     "the '--html' option"
 -      ;;
      mingw)
        uname_s=$(uname -s || echo UNKNOWN)
        echo "$me: system name: $uname_s"
        fi
        unset priv_check_temp overwrite_status
        ;;
-     perl-threads)
+     # Extra quoting required to avoid maintainer-check spurious failures.
+     'perl-threads')
        if test "$WANT_NO_THREADS" = "yes"; then
          skip_all_ "Devel::Cover cannot cope with threads"
        fi
          skip_all_ "TeX is required, but it wasn't found by configure"
        fi
        ;;
 -    texi2dvi-o)
 -      # Texi2dvi supports '-o' since Texinfo 4.1.
 -      echo "$me: running texi2dvi -o /dev/null --version"
 -      texi2dvi -o /dev/null --version \
 -        || skip_all_ "required program 'texi2dvi' not available"
 -      ;;
      lex)
        test x"$LEX" = x"false" && skip_all_ "lex not found or disabled"
        export LEX
    esac
  done
  
- # Using just $am_top_builddir for the check here is ok, since the
- # further temporary subdirectory where the test will be run is
- # ensured not to contain any whitespace character.
- case $am_top_builddir in
-   *\ *|*\     *)
-     case " $required " in
-       *' libtool '* | *' libtoolize '* )
-         skip_all_ "libtool has problems with spaces in builddir name";;
-     esac
-     ;;
- esac
- # This test is necessary, although Automake's configure script bails out
- # when $srcdir contains spaces.  This is because $am_top_srcdir is in not
- # configure-time $srcdir, but is instead configure-time $abs_srcdir, and
- # that is allowed to contain spaces.
- case $am_top_srcdir in
-   *\ * |*\    *)
-     case " $required " in
-       *' libtool '* | *' libtoolize '* | *' gettext '* )
-         skip_all_ "spaces in srcdir name: libtool/gettext tests won't work";;
-    esac
-    ;;
- esac
  # We might need extra macros, e.g., from Libtool or Gettext.
  case " $required " in *\ libtool*) . ./t/libtool-macros.dir/get.sh;; esac
  case " $required " in *\ gettext*) . ./t/gettext-macros.dir/get.sh;; esac
@@@ -1028,9 -1018,9 +1005,9 @@@ els
      {
        echo "AC_INIT([$me], [1.0])"
        if test x"$am_serial_tests" = x"yes"; then
 -        echo "AM_INIT_AUTOMAKE"
 +        echo "AM_INIT_AUTOMAKE([serial-tests])"
        else
 -        echo "AM_INIT_AUTOMAKE([parallel-tests])"
 +        echo "AM_INIT_AUTOMAKE"
        fi
        echo "AC_CONFIG_FILES([Makefile])"
      } >configure.ac || framework_failure_ "creating configure.ac skeleton"