ant-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 5003] exec task does not return after executed command finished on Windows only
Date Thu, 06 Nov 2008 14:34:28 GMT
https://issues.apache.org/bugzilla/show_bug.cgi?id=5003


Stefan Bodewig <bodewig@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|1.6                         |1.8.0




--- Comment #29 from Stefan Bodewig <bodewig@apache.org>  2008-11-06 06:34:25 PST ---
a loooooooooooooooong time, I know.

Ant's code has changed a bit, so some extra work has become necessary.  That
other classes are now using StreamPumper as well didn't help either.

With the original patch (even if adapted) several unit tests of Ant would hang
and never return - I guess this has been true seven years ago as well.

One major problem I faced was that available() returns 0 on a closed stream
on some VMs (it did on Suns 1.4.2 for Windows, for example) and thus the
available trick doesn't work unless you are sure you are going to interrupt
the thread running StreamPumper eventually.

I've also noticed that the approach using available impacts performance
considerably, so I've restricted it to the platform (Windows) where it is
needed (like the original patch did, but for a different reason).

svn revision 711860 ( https://svn.apache.org/viewcvs.cgi?view=rev&rev=711860 )


-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

Mime
View raw message