ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 29977] - topoSort() logic is broken with multiple targets
Date Fri, 23 Jul 2004 20:23:28 GMT
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=29977>.
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=29977

topoSort() logic is broken with multiple targets

gudnabrsam@yahoo.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From gudnabrsam@yahoo.com  2004-07-23 20:23 -------
I didn't want to change the results of topoSort(String, Vector).  However, I 
modified the signature of topoSort(String[], Vector) to topoSort(String[], 
Vector, boolean)--the boolean "returnAll" indicates whether to return the 
entire list.  The preexisting method will specify true for this parameter, 
which should retain compatibility in the event that anyone is currently using 
that method externally.  Ant's internals have been modified to use topoSort 
with returnAll equal to false, simplifying execution of dependency lists.  A 
convenience method topoSort(String, Vector, boolean) also exists.  New 
testcases illustrating this bugrep have been added.  I believe this works as 
intended now in CVS HEAD.  Thanks!

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org


Mime
View raw message