Hi Peter, thanks for the testing.

On 05/02/2012 03:19 PM, Peter Breitenlohner wrote:
> On Tue, 1 May 2012, Stefano Lattarini wrote:
> 
>> From: Peter Breitenlohner <p...@mppmu.mpg.de>
>>
>> * t/objcxx-basic.sh: New test.
> 
> Hi Stefano,
> 
> I have just build the automake git experimental/objc++ branch
> (from a freshly checked out tree) and found three FAILs:
>   t/objc-basic.sh
>   t/objcxx-basic.sh
>   t/suffix.sh
> 
> In all three cases the reason seems to be that the sequence
>   $ACLOCAL
>   $AUTOMAKE
>   echo <some thing> >> configure.ac
>   $ACLOCAL --force
> doesn't work as expected.  When autom4te.cache is removed after modifying
> configure.ac all three test succeed.
>
Your diagnosis and proposed solutions are right.  I've squashed in the fix
into 't/objc-basic.sh' and 't/objcxx-basic.sh', and fixed 't/suffix.sh' on
main with the attached patch.  I plan to merge the 'objc++-support' branch
into 'maint' by tomorrow if there are no more errors.
Regards,
  Stefano
>From b78cb35dcad438908c652a80c64bf9732f7251f6 Mon Sep 17 00:00:00 2001
Message-Id: <b78cb35dcad438908c652a80c64bf9732f7251f6.1335973824.git.stefano.lattar...@gmail.com>
From: Stefano Lattarini <stefano.lattar...@gmail.com>
Date: Wed, 2 May 2012 17:47:56 +0200
Subject: [PATCH]  tests: fix spurious failure due to autom4te caching

Reported by eter Breitenlohner :
<http://lists.gnu.org/archive/html/automake-patches/2012-05/msg00045.html>

* t/suffix.sh: Remove stale autom4te.cache directories, to prevent racy,
spurious failures (using 'aclocal --force' was not enough, since the cache
was still picked up by the following automake call).

Signed-off-by: Stefano Lattarini <stefano.lattar...@gmail.com>
---
 t/suffix.sh |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/t/suffix.sh b/t/suffix.sh
index f9944cb..600aedf 100755
--- a/t/suffix.sh
+++ b/t/suffix.sh
@@ -32,6 +32,8 @@ END
 
 for use_arlib in false :; do
 
+  rm -rf autom4te*.cache
+
   if $use_arlib; then
     am_warns=
     echo AM_PROG_AR >> configure.ac
@@ -40,7 +42,7 @@ for use_arlib in false :; do
     am_warns=-Wno-extra-portability
   fi
 
-  $ACLOCAL --force
+  $ACLOCAL
 
   $AUTOMAKE $am_warns -i
   grep '^ *\.c' Makefile.in # For debugging.
-- 
1.7.9.5

Reply via email to