On 03/01/2012 10:45 PM, P. Martin wrote: > FAIL: txinfo30 > ============== > > Running from installcheck: no > Using TAP: no > PATH = /private/tmp/homebrew-automake-HEAD-DUdI/tests/ax:... > ++ pwd > /private/tmp/homebrew-automake-HEAD-DUdI/tests/txinfo30.dir > + echo info_TEXINFOS = bar.texi > + echo grepme > + sleep 2 > + cat > + echo AC_OUTPUT > + cat > + chmod +x makeinfo > ++ pwd > + PATH=/private/tmp/homebrew-automake-HEAD-DUdI/tests/txinfo30.dir:... > + export PATH > + aclocal-1.11a -Werror > + autoconf > + automake-1.11a --foreign -Werror -Wall --add-missing > Makefile.am:1: installing './texinfo.tex' > + ./configure > checking for a BSD-compatible install... /usr/bin/install -c > checking whether build environment is sane... yes > checking for a thread-safe mkdir -p... ./install-sh -c -d > checking for gawk... gawk > checking whether make sets $(MAKE)... yes > checking that generated files are newer than configure... done > configure: creating ./config.status > config.status: creating Makefile > + make > restore=: && backupdir=".am$$" && \ > am__cwd=`pwd` && CDPATH="${ZSH_VERSION+.}:" && cd . && \ > rm -rf $backupdir && mkdir $backupdir && \ > if (/usr/local/Cellar/texinfo/4.13a/bin/makeinfo --version) >/dev/null > 2>&1; then \ > for f in bar.info bar.info-[0-9] bar.info-[0-9][0-9] bar.i[0-9] > bar.i[0-9][0-9]; do \ > if test -f $f; then mv $f $backupdir; restore=mv; else :; fi; \ > done; \ > else :; fi && \ > cd "$am__cwd"; \ > if /usr/local/Cellar/texinfo/4.13a/bin/makeinfo -I . \ > -o bar.info bar.texi; \ > then \ > rc=0; \ > CDPATH="${ZSH_VERSION+.}:" && cd .; \ > else \ > rc=$?; \ > CDPATH="${ZSH_VERSION+.}:" && cd . && \ > $restore $backupdir/* `echo "./bar.info" | sed 's|[^/]*$||'`; \ > fi; \ > rm -rf $backupdir; exit $rc > + grep grepme bar.info > + exit_status=1 > ... > txinfo30: exit 1
I managed to reproduce this failure by exporting $MAKEINFO to the absolute path of a working makeinfo program before running the test. The attached patch (pushed to master) fixes the issue. Thanks, Stefano
>From 1ca53a79e4904281cf14c78b7b1234819094a3db Mon Sep 17 00:00:00 2001 Message-Id: <1ca53a79e4904281cf14c78b7b1234819094a3db.1330723490.git.stefano.lattar...@gmail.com> From: Stefano Lattarini <stefano.lattar...@gmail.com> Date: Fri, 2 Mar 2012 22:24:35 +0100 Subject: [PATCH] tests: fix spurious failure due to exported MAKEINFO * tests/txinfo30.test: Unset variable MAKEINFO, as its content could take precedence over our faked makeinfo script and cause a spurious failure. Reported as part of automake bug#10866. Add trailing ':' command since we are at it. --- tests/txinfo30.test | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/tests/txinfo30.test b/tests/txinfo30.test index 82ea856..09ebd5e 100755 --- a/tests/txinfo30.test +++ b/tests/txinfo30.test @@ -42,6 +42,10 @@ chmod +x makeinfo PATH=`pwd`$PATH_SEPARATOR$PATH export PATH +# Otherwise configure might pick up a working makeinfo from the +# environment. Seen in automake bug#10866. +unset MAKEINFO || : + $ACLOCAL $AUTOCONF $AUTOMAKE --add-missing @@ -55,3 +59,5 @@ test -f bar.info rm -f bar.info $MAKE && Exit 1 test ! -f bar.info + +: -- 1.7.9