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 <[email protected]>
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
