Hi, Rohan:
Good job! I update some information inline.
Regards,
Aegir
Rohan Ranade wrote:
>Hi all,
>
>Since I underwent training on Ant a couple of days
>back, I thought of looking at the build.xml which came
>with quickget1.
>
>I observe the following points :
>
>1. Build.xml complains when there is no "bin"
>directory. As a solution, i have added a target called
>"prepare". In the prepare target, i issue an mkdir
>task to create the bin directory. The "all" target
>depends on prepare target.Similarly the "clean" target
>has been modified by me to delete the bin directory.
>
>2. The "lib" property is cumbersome. I think there is
>something called <path> which does the job cleanly.
>Will investigate it further.
>
>3. The "all" target should be replaced with separate
>targets for each discrete task such as a target for
>"compile", "jar" etc.
>
>4. Versioning for generated JAR files?
>
>5. A task for running the jar file?
>
>
It seems to be a good plus. And we can trigger the running of jar file by
something like "ant run"...
And I also suggest to add the necessary information in the meta file in
jar file to
support the "java -jar quickget.jar".
>6. (OT) Author's name please (for rants) ;-)
>
>
Alex is the author.
>Please let me know your views on the same. I plan on
>cleaning up the build file tomorrow and will forward
>it to the author (who is the author?). (lets keep this
>policy of submitting changes to modules to the
>original author who will then review them and check
>them in).
>
>
Sounds good. And Alex please check them into the quickget2 module. I
am now refactoring quickget2 to use new patterns (state pattern) to add the
management ability to start/stop/resume the task.
>Regards,
>Rohan
>
>
>
>__________________________________
>Yahoo! Mail - PC Magazine Editors' Choice 2005
>http://mail.yahoo.com
>
>
>-------------------------------------------------------
>SF.Net email is sponsored by:
>Tame your development challenges with Apache's Geronimo App Server. Download
>it for free - -and be entered to win a 42" plasma tv or your very own
>Sony(tm)PSP. Click here to play: http://sourceforge.net/geronimo.php
>_______________________________________________
>Quickget-develop mailing list
>Qui...@li...
>https://lists.sourceforge.net/lists/listinfo/quickget-develop
>
>
|
