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 [EMAIL PROTECTED] changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED ------- Additional Comments From [EMAIL PROTECTED] 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: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
