Re: cond5.test spurious failure

[Peter Rosin]

Hi Stefano,

Den 2010-08-06 13:54 skrev Stefano Lattarini:
> At Friday 06 August 2010, Peter Rosin wrote:
> > [CUT]
> > I did get a spurious failure in cond5.test, but it went away when
> > I ran it separately.
> Did you by chance run many tests in parallel?  If yes, you might have hit
> a known issue:
>   "The test cond5.test can fail spuriously due to timeouts when it's run
>   in low priority (e.g. with `nice -n19'), especially on loaded systems,
>   or when many tests are run in parallel (I myself routinely run up to
>   50 tests in parallel with low priority)."
>
> This problem should have been solved by this patch:
>   <http://www.mail-archive.com/address@hidden/msg02095.html>
>
> Is you automake repository up-to-date? Do you have this commit installed?
>   36fe4f389c0a835dfcdb0f58a8909eab43189af1 "Improve and extend test cond5.test."
> If yes, then my patch didn't fix the problem after all...

I was not running in parallel and I was running off of the msvc branch
(basically maint I think) plus the commit in the previous message so I
am up to date. I had a look at the cond5.log file (but it is now gone)
and from what I could see the
   if kill -0 $pid; then
failed on the first attempt (which I found odd) and then the following
grep in the else statement must have come up empty. But when I looked
at the stderr file I think it had the expected content so the grep
should have been ok.

Maybe the process was gone (kill failed) but the stderr content was
not yet flushed or something? This is on MSYS/MinGW so don't expect
full POSIX adherence...

Hmmm, maybe I should just try to run the test again? Ok, below is the
output from a failed run (it failed directly). Maybe we should add
an extra "sleep $arbitrary" before the "cat stderr"?

Cheers,
Peter

=== Running test ./cond5.test
++ pwd
/home/peda/automake/git/automake/tests/cond5.dir
+ set -e
+ cat
+ cat
+ aclocal-1.11 -Werror
+ pid=7240
+ automake-1.11 --foreign -Werror -Wall
+ try=1
+ test 1 -le 30
+ kill -0 7240
./cond5.test: line 54: kill: (7240) - No such process
+ cat stderr
+ grep 'variable.*OPT_SRC.*recursively defined' stderr
+ exit_status=1
+ set +e
+ cd /home/peda/automake/git/automake/tests
+ case $exit_status,$keep_testdirs in
+ test 0 '!=' 0
+ echo 'cond5: exit 1'
cond5: exit 1
+ exit 1