Sorry, I made a wrong conclusion. The result contained extra targets that should not be executed at all, that is why the result size was wrong. Now it creates some problems - in order to make "a,b" executed I still need an extra target named "a,b" (or similar), because the code uses topoSort() result trimming using the target name.

- Alexey.

Alexey N. Solofnenko wrote:

Hello,

I have a patch (Bug 21421) that allows to run several targets without rerunning their dependencies. Now with new <ant> is supposed to do the same and there is a new topoSort() with array of targets. I tried to use it, but it is not much better that previous implementation - all the common dependencies are re-executed. This is an example:

 <target name="t1">
   <echo>t1</echo>
 </target>
 <target name="t2">
   <echo>t2</echo>
 </target>
 <target name="t3" depends="t1,t2">
   <echo>t3</echo>
 </target>
 <target name="t4" depends="t1,t2">
   <echo>t4</echo>
 </target>

topoSort(new String["t3", "t4"], targets) returns ("t1", "t2", "t3", "t1", "t2", "t4") instead of ("t1", "t2", "t3", "t4"). I think it is important not to run the same targets over and over again. Is it an expected behaviour?

- Alexey.



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Reply via email to