Subject: Re: CVS commit: basesrc
To: enami tsugutomo <enami@but-b.or.jp>
From: Simon Gerraty <sjg@juniper.net>
List: source-changes
Date: 06/09/2001 20:30:31
On 10 Jun 2001 12:00:57 +0900, enami tsugutomo wrote:
>What's the intention of following change?  Even if this is necessary,
>why inserted between list operation?  And, it makes make -j1 cleandir
>under bin/cat slow.

While testing the -dx code, I found that output was being lost without
this change (though that was with bmake, but on *BSD they should be
the same).  I don't recall which test makefile I was using, and it
wasn't 100% reproducable, so I figured it better to put it in our make
as well.

You'll note that the Job_CatchOutput() is made just prior to removing
it from the jobs list so this is the last chance to gather output for
it.

--sjg

>enami.
>
>@@ -2352,6 +2360,7 @@
> 	    }
> 	} else {
> 	    job = (Job *) Lst_Datum(jnode);
>+	    Job_CatchOutput();
> 	    (void) Lst_Remove(jobs, jnode);
> 	    nJobs -= 1;
> #ifdef REMOTE