[ ant-contrib-Bugs-1820975 ] Fails in <for parallel="true" keepgoing="false">

Bugs item #1820975, was opened at 2007-10-26 15:01
Message generated for change (Comment added) made by nobody
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=416920&aid=1820975&group_id=36177

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: logic
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Nobody/Anonymous (nobody)
Summary: Fails in <for parallel="true" keepgoing="false">

Initial Comment:
<for parallel="true" keepgoing="false"> does not stop execution if one of the threads fails.
<parallel> implementation does not use failonany="true" in this case.


----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2008-03-05 12:05

Message:
Logged In: NO 

clheiny,
There is an error in the code:

parallel failonany should use reverted ForTask keepgoing value, i.e. if
keepgoing is false failonany should be true
Thanks. Andrey.

----------------------------------------------------------------------

Comment By: Christopher Heiny (clheiny)
Date: 2007-11-06 15:00

Message:
Logged In: YES 
user_id=117164
Originator: NO

I've got a patch against trunk for this, but don't see a button for
attaching the patch file to the bug.  It's trivial, so here it is:


Index: src/main/java/net/sf/antcontrib/logic/ForTask.java
===================================================================
--- src/main/java/net/sf/antcontrib/logic/ForTask.java  (revision 146)
+++ src/main/java/net/sf/antcontrib/logic/ForTask.java  (working copy)
@@ -212,6 +212,7 @@
     public void execute() {
         if (parallel) {
             parallelTasks = (Parallel)
getProject().createTask("parallel");
+            parallelTasks.setFailOnAny(keepgoing);
             if (threadCount != null) {
                 parallelTasks.setThreadCount(threadCount.intValue());
             }


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=416920&aid=1820975&group_id=36177

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/