ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Christopher Elkins" <celk...@scardini.com>
Subject Re: [PATCH] Suggestion for new commandline switch -targets
Date Mon, 14 Aug 2000 02:01:16 GMT
> 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


Mime
View raw message