[ http://jira.codehaus.org/browse/MCOMPILER-120?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Alex Ioffe updated MCOMPILER-120: --------------------------------- Attachment: JavacCompiler.java I think the fix is trivial, in 'parseModernError' switch the order of: isError = exitCode != 0; and: // Remove the 'warning: ' prefix String warnPrefix = getWarnPrefix( msg ); if ( warnPrefix != null ) { isError = false; msg = msg.substring( warnPrefix.length() ); } ... and then it works as expected. > Javac compiler plugin doesn't support -Werror > --------------------------------------------- > > Key: MCOMPILER-120 > URL: http://jira.codehaus.org/browse/MCOMPILER-120 > Project: Maven 2.x Compiler Plugin > Issue Type: Bug > Affects Versions: 2.1 > Reporter: Christopher Webster > Attachments: JavacCompiler.java, trial-maven.zip > > > If I write a pom file like the following: > ... > <plugin> > <groupId>org.apache.maven.plugins</groupId> > <artifactId>maven-compiler-plugin</artifactId> > <version>2.1</version> > <configuration> > <compilerId>javac</compilerId> > <source>1.6</source> > <target>1.6</target> > > <compilerArguments> > <Werror /> > <Xlint:all /> > </compilerArguments> > <showDeprecation>true</showDeprecation> > </configuration> > </plugin> > and if there are only warnings, then the build will not fail as intended by > the compiler Argument. The reason is that in compileInProcess the exit code > from javac is only considered if there are no messages. In the case of > treating warnings as errors, there will be messages but no errors so the > intention of the build failure is lost. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira