Re: [PATCH] Minor testsuite size reduction.

[Ralf Wildenhues]

* Eric Blake wrote on Wed, Jul 21, 2010 at 08:26:10PM CEST:
> On 07/21/2010 11:54 AM, Ralf Wildenhues wrote:
> > * Eric Blake wrote on Wed, Jul 21, 2010 at 03:58:16PM CEST:
> >> @@ -1440,6 +1458,7 @@ dnl      kill -13 $$
> >>    wait
> >>  else
> >>    # Run serially, avoid forks and other potential surprises.
> >> +  at_jobs=1
> >>    for at_group in $at_groups; do
> >>      at_fn_group_prepare
> >>      if cd "$at_group_dir" &&
> > 
> > Rereading this again, it wasn't right, though, or, put another way, I
> > think it causes potentially worse behavior in an unusual setting (not
> > verified):
> > 
> > Say you run './testsuite -jN' on a system without mkfifo, then at_quiet
> > gets set to : but above change later sets at_jobs to 1, causing
> > $at_desc_line to never be printed.
> > 
> > Sorry about leading you down there,
> 
> Is this something we need to revert before 2.67 (seeing as how I've been
> unable to upload yet, I could rebase it in)?

Well, if, then you could revert just this one line.  It's not a huge
issue, but the change didn't fix a regression, so reverting should be
safe.

Cheers,
Ralf