tags 11909 + patch
close 11909
thanks

On 07/14/2012 01:38 PM, Stefano Lattarini wrote:
>
> On 07/11/2012 02:37 PM, Max Horn wrote:
>>
>> turns out there is one more test failure on Mac OS X 10.7.4,
>> in t/self-check-explicit-skips.sh.
>>
> I can reproduce the issue with bash 3.2 on Debian.
> 
>> Looking at the code, I don't understand it at all, so please don't
>> torch me if this isn't a bug in the automake test suite but rather
>> in Mac OS X :).
>>
> It's probably a limitation on older bash versions, that breaks an
> invariant the Automake testsuite framework expects.  Probably nothing
> serious, but I will attempt a fix since I can reproduce the problem.
> 
>> I just want to know what is causing this and whether I should ignore
>> it, disable the test, or apply some patch.
>>
> I hope I'll manage to fix it soon, but for the moment, just ignore it.
>
I've pushed the attached patch, that fixes the problem for me.  I'm thus
closing this bug report.  In case the problem persist on Mac OS X, feel
free to re-open it.

Thanks,
  Stefano

>From 7519d2634bdac6763ef2d57b3aff00f37c086a30 Mon Sep 17 00:00:00 2001
Message-Id: <7519d2634bdac6763ef2d57b3aff00f37c086a30.1343036892.git.stefano.lattar...@gmail.com>
From: Stefano Lattarini <stefano.lattar...@gmail.com>
Date: Sat, 14 Jul 2012 13:56:34 +0200
Subject: [PATCH] self tests: avoid spurious failures on older bash

Fixes automake bug#11909.

* t/self-check-explicit-skips.sh: Remove or rework few checks for use
cases that are no more relevant nor supported now that we've got rid
of the need to use the 'Exit' function explicitly.  Make the other
existing checks stricter.

Signed-off-by: Stefano Lattarini <stefano.lattar...@gmail.com>
---
 t/self-check-explicit-skips.sh |   11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

diff --git a/t/self-check-explicit-skips.sh b/t/self-check-explicit-skips.sh
index eb60dc2..97f5288 100755
--- a/t/self-check-explicit-skips.sh
+++ b/t/self-check-explicit-skips.sh
@@ -36,22 +36,19 @@ run_dummy_test ()
   " dummy.sh
 }
 
-run_dummy_test '(exit 77); exit 77'
+run_dummy_test 'exit 77'
 test $? -eq 77 || exit 1
 
 run_dummy_test 'sh -c "exit 77"' am_explicit_skips=no
 test $? -eq 77 || exit 1
 
-run_dummy_test '(exit 77); exit 77' am_explicit_skips=yes 
+run_dummy_test '$PERL -e "exit 77"; true' am_explicit_skips=yes
 test $? -eq 78 || exit 1
 
-run_dummy_test 'sh -c "exit 77"' am_explicit_skips=y
+run_dummy_test 'sh -c "exit 77"; exit 0' am_explicit_skips=y
 test $? -eq 78 || exit 1
 
-run_dummy_test 'exit 77' am_explicit_skips=yes
-test $? -eq 77 || exit 1
-
-run_dummy_test 'skip_ "foo"' am_explicit_skips=y
+run_dummy_test 'skip_ "foo"; :' am_explicit_skips=y
 test $? -eq 77 || exit 1
 
 :
-- 
1.7.10.4

Reply via email to