DO NOT REPLY [Bug 29977] New: - topoSort() logic is broken with multiple targets

[bu...@apache.org]

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

           Summary: topoSort() logic is broken with multiple targets
           Product: Ant
           Version: 1.7Alpha (nightly)
          Platform: Other
        OS/Version: Other
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: Core
        AssignedTo: dev@ant.apache.org
        ReportedBy: alexeys@inventigo.com


ANT (and <ant>, and <antcall>) trims result from topoSort() by searching 
the last mentioned target. It does not work. topoSort() should return 
only targets that should be executed, so no trimming is required.

This is a test example:

<project name="test" default="all" basedir=".">
  <target name="t1">
    <echo>t1</echo>
  </target>
  <target name="t2" depends="t1">
    <echo>t2</echo>
  </target>
  <target name="all">
    <antcall>
      <target name="t2"/>
      <target name="t1"/>
    </antcall>
  </target>
  <target name="all2">
    <antcall>
      <target name="t1"/>
      <target name="t2"/>
    </antcall>
  </target>
</project>

Only "t1" is executed when "all" is called, but both "t1" and "t2" with "all2".

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