Return-Path: 
 <ant-user-return-26072-qmlist-jakarta-archive-ant-user=jakarta.apache.org@jakarta.apache.org>
Delivered-To: apmail-jakarta-ant-user-archive@apache.org
Received: (qmail 49124 invoked from network); 18 Oct 2002 06:55:15 -0000
Received: from unknown (HELO nagoya.betaversion.org) (192.18.49.131)
  by daedalus.apache.org with SMTP; 18 Oct 2002 06:55:15 -0000
Received: (qmail 7922 invoked by uid 97); 18 Oct 2002 06:56:06 -0000
Delivered-To: qmlist-jakarta-archive-ant-user@jakarta.apache.org
Received: (qmail 7876 invoked by uid 97); 18 Oct 2002 06:56:05 -0000
Mailing-List: contact ant-user-help@jakarta.apache.org; run by ezmlm
Precedence: bulk
List-Unsubscribe: <mailto:ant-user-unsubscribe@jakarta.apache.org>
List-Subscribe: <mailto:ant-user-subscribe@jakarta.apache.org>
List-Help: <mailto:ant-user-help@jakarta.apache.org>
List-Post: <mailto:ant-user@jakarta.apache.org>
List-Id: "Ant Users List" <ant-user.jakarta.apache.org>
Reply-To: "Ant Users List" <ant-user@jakarta.apache.org>
Delivered-To: mailing list ant-user@jakarta.apache.org
Received: (qmail 7861 invoked by uid 98); 18 Oct 2002 06:56:04 -0000
X-Antivirus: nagoya (v4218 created Aug 14 2002)
X-Authentication-Warning: bodewig.bost.de: bodewig set sender to
 bodewig@bost.de using -f
To: ant-user@jakarta.apache.org
Subject: Re: create cab in linux; patch request
References: <H3QASG00.6J0@ace.mail.wipro.com> <3DA49A3C.6040503@dallaway.com>
	<3DA4A1CD.2000406@hotu.com> <m3k7kqg62y.fsf@bodewig.bost.de>
	<3DA60B52.90508@hotu.com> <m3u1jtbexg.fsf@bodewig.bost.de>
	<3DA71F69.1020701@hotu.com> <m3elatblhq.fsf@bodewig.bost.de>
	<3DAB5E1B.9050502@hotu.com> <m3it01qljo.fsf@bodewig.bost.de>
	<3DAF20AF.5080500@hotu.com>
From: Stefan Bodewig <bodewig@apache.org>
Date: 18 Oct 2002 08:55:01 +0200
Message-ID: <m3r8eonsmy.fsf@bodewig.bost.de>
Lines: 39
User-Agent: Gnus/5.0808 (Gnus v5.8.8) XEmacs/21.4 (Honest Recruiter)
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N
X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N

On Thu, 17 Oct 2002, <newsham@hotu.com> wrote:

> This is reasonable and I'm happy to discuss this with whoever may be
> interested.

Thanks.

> The modified behavior:
> 
> 1)  main thread spawns two threads to read stdout, stderr (async)
> 2)  main thread writes entire input into process's stdin
> 3)  main thread waits for process to complete
> 
> Since we have three threads servicing the process' three streams
> simultaneously, we shouldn't have the potential problem described in
> the javadoc.

Maybe we need to spawn yet another thread to pump in the input, but
I'm not sure.

Anyway, I asked Ilya Kriveshko (who has been responsible for the
latest Linux related patches to <cab>) to try your patch and see
whether it worked for him, the most important excerpts from his
answer:

>> I tried the patch on Linux RH7 and it appears to function correctly
>> in our build system.
>>
>> it appears to be a correct fix for the problem.
>>
>> Thanks to Jim Newsham for catching this.

8-)

I'm going to commit your patch to CVS HEAD today and ask all <cab>
users to give one of the next Ant nightlies (starting with 2002-10-19)
a try.

Stefan

--
To unsubscribe, e-mail:   <mailto:ant-user-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:ant-user-help@jakarta.apache.org>