autoconf-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: parallel autotest ineffective for zsh


From: Ralf Wildenhues
Subject: Re: parallel autotest ineffective for zsh
Date: Thu, 9 Jul 2009 08:23:01 +0200
User-agent: Mutt/1.5.20 (2009-06-15)

* Eric Blake wrote on Wed, Jul 08, 2009 at 07:02:06AM CEST:
> According to Eric Blake on 7/7/2009 3:12 PM:
> > Meanwhile, I think the best course of action is to just teach 
> > AT_SKIP_PARALLEL_TESTS to recognize 'set -m' as another reason to skip 
> > rather 
> > than fail.  Hmm.  Maybe I should factor the autotest/general.m4 conditions 
> > into 
> > something that can be more easily reused in autotest.at, rather than having 
> > to 
> > keep the two filters in sync.
> 
> Done as follows.

> +     Skip parallel tests when zsh 'set -m' fails.
> +     * tests/autotest.at (AT_SKIP_PARALLEL_TESTS): Skip test if set -m
> +     is not supported.
> +     Reported by Ralf Wildenhues.

Thanks.  I've added in this patch, so that things don't unnecessarily
fail with zsh 4.4.x.

Cheers,
Ralf

    Ignore messages on stderr when testing for the zsh issue.
    
    * tests/autotest.at (AT_SKIP_PARALLEL_TESTS): Ignore stderr.

diff --git a/tests/autotest.at b/tests/autotest.at
index 9bdb728..438c01d 100644
--- a/tests/autotest.at
+++ b/tests/autotest.at
@@ -994,7 +994,8 @@ AT_CHECK([${CONFIG_SHELL-$SHELL} -c 'test -n 
"${BASH_VERSION+set}]]dnl
 [[${ZSH_VERSION+set}${TEST_PARALLEL_AUTOTEST+set}"' || exit 77])
 # The parallel scheduler requires mkfifo and job control to work.
 AT_CHECK([mkfifo fifo || exit 77])
-AT_CHECK([${CONFIG_SHELL-$SHELL} -c '(set -m && set +m) || exit 77'])
+AT_CHECK([${CONFIG_SHELL-$SHELL} -c '(set -m && set +m) || exit 77'],
+        [], [], [ignore])
 ])
 
 




reply via email to

[Prev in Thread] Current Thread [Next in Thread]