DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG· RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT <http://issues.apache.org/bugzilla/show_bug.cgi?id=36545>. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND· INSERTED IN THE BUG DATABASE.
http://issues.apache.org/bugzilla/show_bug.cgi?id=36545 Summary: [exec] potential threading issue when closing process.getInputStream() Product: Commons Version: unspecified Platform: Other OS/Version: other Status: NEW Severity: normal Priority: P1 Component: Sandbox AssignedTo: [email protected] ReportedBy: [EMAIL PROTECTED] When I started to look into finding common code between ant, plexus, m2 and cruisecontrol, I found a couple of differences that I think need to be merged back into commons-exec. In particular it seems like this plexus-util change [1] contains a threading fix which should also be backported to commons-exec. I think we are hitting it now in CruiseControl in some rare cases: executed process is ant which forks and spawns a separate process. This makes getInputStream().close() hang, and I think your change would fix it. I haven't yet worked on it, nor wrote a test case. Would appreciate input from trygvis as he's the one who made the fix for plexus. [1] http://svn.codehaus.org/trunk/plexus-utils/src/main/java/org/codehaus/plexus/util/cli/StreamPumper.java?rev=1298&root=plexus&view=rev -- Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee. --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
