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 28941] - [patch] javac && onCompile support
Date Thu, 13 May 2004 13:06:19 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=28941>.
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=28941

[patch] javac && onCompile support





------- Additional Comments From tquas@yahoo.com  2004-05-13 13:06 -------
Other code generators I'm using don't do such checks. XDoclet was just an
example. A badly chosen one, I admit.

You're probably right, and it would be more in Ant's spirit to use 'depends'
instead of <antcall>; however, I don't see how this would solve the problem.
Using 'depends' the only change in the script above would be in the "build" target:

    <target name="build" depends="compile,gen.hibernate.mappings,build.jar"/>

Still, the major issue here is that Ant must not execute
"gen.hibernate.mappings" or "build.jar" if nothing has been compiled in the
previous step. IMHO, a good way to communicate that is via properties and
<target if="x">.

Just for curiousity: why is 'depends' much faster? This particular build script
compiles 9 modules in about 30 secs. How much faster than <antcall> could
'depends' be? Are we talking milliseconds? Seconds?

Using <dependset> can obviously not be faster because it has to check timestamps
on files. And since it removes target files it doesn't look like a good match
anyways.

I'd be happy to use Ant's existing mechanisms, but I don't see a straightforward
solution using its current capabilities. Hence the patch.

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


Mime
View raw message