Merge branch 'tests-man-fixes'

diff --cc ChangeLog
index 3e3105b,e39dc11..24b20ae
--- 1/ChangeLog
--- 2/ChangeLog
+++ b/ChangeLog
@@@ -1,96 -1,24 +1,117 @@@
  2010-11-08  Stefano Lattarini  <stefano.lattarini@gmail.com>
  
+       Improve and extend tests on man pages support.
+       * tests/man.test: Enable `errexit' shell flag, and related changes.
+       Make grepping of generated Makefile.in slighty stricter.
+       * tests/man3.test:  Add trailing `:' command.
+       * tests/man5.test: Prefer cat + here-doc over echo to append to
+       configure.in.
+       * tests/man2.test: Likewise, and add trailing `:' command.
+       * tests/man4.test: More thorough and consistent checking of make
+       error messages.  Place fake `help2man' program in a new `bin'
+       directory rather than in `.'.  Move the checks using the real
+       `help2man' program to ...
+       * tests/man6.test: ... this new test, and extend them.  This test
+       passes with GNU make and Solaris make, still fails with BSD make.
+       * tests/man7.test: New test, extracted from old man4.test, which
+       checks for a bug in maintainer-clean w.r.t. generated manpages.
+       * tests/man8.test: New test, extracted from old man4.test, which
+       checks for a bug in distcheck w.r.t. generated manpages.  Passes
+       with GNU make and Solaris make, still fails with BSD make.
+ 
++2010-11-08  Stefano Lattarini  <stefano.lattarini@gmail.com>
++
 +      Minor improvements and extensions to various tests.
 +      * tests/defun.test: Also run autoconf and grep the generated
 +      configure to make sure that aclocal truly picks up all the
 +      required macros.
 +      * tests/compile_f_c_cxx.test: Prefer trailing `:' over trailing
 +      `Exit 0'.  Do not create useless dummy source files.  Do not set
 +      useless `$(foo_LDADD)' variable in `Makefile.am'.  Do not call
 +      useless macro `AC_F77_LIBRARY_LDFLAGS' in `configure.in'.
 +      * tests/compile_f90_c_cxx.test: Likewise.
 +      * tests/suffix10.test: Slighty stricter grepping of make output.
 +      * tests/compile.test: Add trailing `:' command.
 +      * tests/defun2.test: Likewise.
 +      * tests/vars3.test: Likewise.
 +      * tests/vartar.test: Likewise.
 +      * tests/vars.test: Likewise.  Also, extend test by checking
 +      that the definition of `MY_FLAGS*' variables is preserved in
 +      the generated `Makefile.in'.
 +      * tests/stamph2.test: Prefer trailing `:' over trailing `Exit 0'.
 +      Use proper m4 quoting in `configure.in'.
 +
 +2010-11-11  Stefano Lattarini  <stefano.lattarini@gmail.com>
 +
 +      Tests defs: improve messages for skipped tests.
 +      * tests/defs: Give meaningful messages about the reasons of a
 +      test skip; this is especially useful as this file is run without
 +      verbose xtraces on.  Related reorderings in the code and new
 +      comments.
 +
 +2010-11-10  Stefano Lattarini  <stefano.lattarini@gmail.com>
 +
 +      Tests defs: move static definitions in a new file `defs-static'.
 +      The new file is meant to be idempotent w.r.t. mutiple inclusions.
 +      * tests/defs.in: Removed, its contents split among ...
 +      * tests/defs-static.in: ... this new file ...
 +      * tests/defs: ... and this new file, including the former.
 +      * configure.ac (AC_CONFIG_FILES): Remove `tests/defs', add
 +      `tests/defs-static'.
 +      (AC_CONFIG_LINKS): Add `tests/defs'.
 +      * tests/Makefile.am ($(parallel_tests)): Update.
 +      ($(instspc_tests)): Likewise.
 +      * tests/.gitignore: Update.
 +
 +      Tests defs: $testsbuilddir is now AC_SUBST'ed.
 +      * tests/defs.in ($testsbuilddir): Substitute from @abs_builddir@.
 +      Add sanity check on $testsbuilddir, similar to those on
 +      $testsrcdir and $top_testsrcdir.
 +
 +      Tests defs: do not print message "Running test $0" anymore.
 +      * tests/defs.in: Printing the message "=== Running test $0" at
 +      the beginning of each tests made sense when Automake used the old
 +      test-driver, which sent all the output directly to stdout/stderr.
 +      Now that the parallel test-driver is used, which saves output of
 +      each test in its correspoding log file, that old message is just
 +      useless noise.
 +
 +      Tests defs: rename $curdir -> $testbuilddir
 +      * tests/defs.in: Rename $curdir to $testbuildir, for clarity and
 +      consistency with $testsrcdir and $top_testsrcdir.
 +
 +      Tests defs: prefer "$curdir" over "`pwd`".
 +      * tests/defs.in: We already save the value of `pwd` in $curdir
 +      early in the file, so there no need to recalculate it later, when
 +      the current working directory is not changed.
 +
 +      Tests defs: use `$me' in more error messages.
 +      * tests/defs.in: Also use `$me' in error messages referring to
 +      missing `defs' or `defs.in', since that variable is now defined
 +      before those checks.
 +
 +      Tests defs: do not use `Exit' where plain `exit' suffices.
 +      * tests/defs.in: Use "exit 77" rather than "Exit 77" to skip the
 +      test when required libtool/gettext macros are not found, since
 +      such skips would take place before the exit trap is installed.
 +
 +      Tests defs: improve and extends comments.
 +      * tests/defs.in: Improve and extends some comments, especially in
 +      relation with the changes introduced by the previous reordering.
 +
 +      Tests defs: various reorderings.
 +      * tests/defs.in: Reordered various snippets of code in a
 +      clearer way.
 +
 +2010-11-09  Stefano Lattarini  <stefano.lattarini@gmail.com>
 +
 +      Fix bug in test `backcompat4.test' (stale autom4te cache).
 +      * tests/backcompat4.test: Remove stale autom4te cache directory
 +      before re-running aclocal and automake.  Also, since we are at
 +      it, be more verbose in displaying contents of generated files,
 +      to ease debuggability.
 +      Reported by Ralf Wildenhues.
 +
  2010-11-07  Stefano Lattarini  <stefano.lattarini@gmail.com>
  
        Fix a bug in variable concatanation with `+='.