> Reposted with my suggestion in a diff file. Also, see comments below. > Much better. But what are those "MS:begin" and "MS:end" lines? Is your editor inserting those?
> > > Hi, > > > > > > In the attached version of Main.java there are some simple modifications > > > which make ant recognize a -targets option. > > > > > > This will print out a list of all the targets available in the selected > > > build-file. It is meant as some sort of documentation on what > > the build-file > > > can do. Execution is suppressed. > > > > > > Do you guys like this suggestion, and if so, how do I go about > > incorporating > > > it in ant? > > > > > First, you should submit your patch in diff form > > <http://jakarta.apache.org/guidelines/source.html>. It makes it > > much easier to > > evaluate. > > > > Second, I think this would be a useful addition. Many ant-using > > projects create > > their own targets (e.g., "usage") to provide similar functionality as your > > patch. Standardizing such behavior would be a very good thing. > > > > However, not all targets in a build file are suitable and/or relevant for > > calling from the command-line. For example, calling the "prepare" > > target in > > jakarta-ant/build.xml from the command-line would have limited utility. > > Therefore, it might be useful to add an attribute to the <target> tag that > > allowed one to specify whether that target should be output in > > Main.printTargets(). > > Or perhaps a 'private' attribute, which can make a target invisible from the > outside and only visible to 'depends' clauses. +1 (Although my vote doesn't count for much, because I'm not a committer. ;-) > Another optional attribute could be 'description', to be printed as part of > the target list. +1 > Let me know what you think of this, > > Regards, > Marcel Schutte > -- Christopher Elkins
