Hi all,
Diane's was the culminating suggestion in answer to my request to set
Jikes to stop on warnings as if they were errors. The original thread
was on ant-user.
I've coded this up and attached the cvs diff -u output. Can we add this
to ant? Is there more work I need to do to bring it up to snuff?
Should I have posted this directly to bugzilla, instead?
This is my first real open source contribution, so I'm really looking
forward to your feedback, be it positive, or negative.
Thanks!
-Steve-
PS: Ant rocks. I'm never going back.
Diane Holt wrote:
>
> --- Conor MacNeill <[EMAIL PROTECTED]> wrote:
> > > But maybe we've simply reached the point to introduce
> > >
> > > <javac additionalArgs="..." />
> >
> > I think this capability should probably be there but it does have some
> > problems. Such a build file will fail with a different compiler since
> > the compiler adapter cannot know whether the arg applies to its
> > particular compiler (unless it is really smart about it).
> >
> > It seems a balance between a mega-interface and non-portable build
> > files. Thoughts about where that balance lies?
>
> How about a nested <flags> element that takes a "compiler" and "args"
> attribute, and only uses those that are for the compiler currently being
> used? For example:
>
> <flags compiler="jikes" args="+Z"/>
> <flags compiler="modern" args="-g:lines"/>
>
> Either that, or I could see people specifying the value of the
> "additionalArgs" attribute as a property (probably use <condition> to test
> ${build.compiler} and set [a default value for] the property for
> "additionalArgs" accordingly).
>
> Diane
>
> =====
> ([EMAIL PROTECTED])
>
> __________________________________________________
> Do You Yahoo!?
> Make a great connection at Yahoo! Personals.
> http://personals.yahoo.comM src/main/org/apache/tools/ant/taskdefs/Javac.java
M src/main/org/apache/tools/ant/taskdefs/compilers/DefaultCompilerAdapter.java
M src/main/org/apache/tools/ant/taskdefs/compilers/Gcj.java
M src/main/org/apache/tools/ant/taskdefs/compilers/Jikes.java
M src/main/org/apache/tools/ant/taskdefs/compilers/Jvc.java
M src/main/org/apache/tools/ant/taskdefs/compilers/Kjc.java
