On 14 May 2013 18:35, Damjan Jovanovic <damjan....@gmail.com> wrote:
> On Tue, May 14, 2013 at 1:00 PM, sebb <seb...@gmail.com> wrote:
>> On 14 May 2013 06:01,  <dam...@apache.org> wrote:
>>> Author: damjan
>>> Date: Tue May 14 05:01:35 2013
>>> New Revision: 1482203
>>>
>>> URL: http://svn.apache.org/r1482203
>>> Log:
>>> Verify conformance to the Java 1.5 API using the animal-sniffer plugin.
>>
>> -1
>>
>> Does not appear to work properly on Maven 2.2.1. or Maven 3.0.4.
>> Although it generates an error when there is an invalid reference, the
>> error is in the plugin, and does not identify the source.
>> For example:
>>
>> [INFO] Checking unresolved references to 
>> org.codehaus.mojo.signature:java15:1.0
>> [INFO] 
>> ------------------------------------------------------------------------
>> [INFO] BUILD FAILURE
>> [INFO] 
>> ------------------------------------------------------------------------
>> [INFO] Total time: 1:49.531s
>> [INFO] Finished at: Tue May 14 11:48:50 BST 2013
>> [INFO] Final Memory: 13M/47M
>> [INFO] 
>> ------------------------------------------------------------------------
>> [ERROR] Failed to execute goal
>> org.codehaus.mojo:animal-sniffer-maven-plugin:1.9:check
>> (check-java-api) on project commons-compress: Execution check-java-api
>> of
>>  goal org.codehaus.mojo:animal-sniffer-maven-plugin:1.9:check failed:
>> An API incompatibility was encountered while executing
>> org.codehaus.mojo:animal-sniffer-ma
>> ven-plugin:1.9:check: java.lang.NoSuchMethodError:
>> java.nio.CharBuffer.subSequence(II)Ljava/nio/CharBuffer;
>> [ERROR] -----------------------------------------------------
>> [ERROR] realm =    plugin>org.codehaus.mojo:animal-sniffer-maven-plugin:1.9
>> [ERROR] strategy = org.codehaus.plexus.classworlds.strategy.SelfFirstStrategy
>> [ERROR] urls[0] =
>> file:/D:/repository/org/codehaus/mojo/animal-sniffer-maven-plugin/1.9/animal-sniffer-maven-plugin-1.9.jar
>> [ERROR] urls[1] =
>> file:/D:/repository/org/codehaus/mojo/animal-sniffer/1.9/animal-sniffer-1.9.jar
>> [ERROR] urls[2] = file:/D:/repository/org/ow2/asm/asm-all/4.0/asm-all-4.0.jar
>> [ERROR] urls[3] =
>> file:/D:/repository/org/codehaus/mojo/java-boot-classpath-detector/1.9/java-boot-classpath-detector-1.9.jar
>> [ERROR] urls[4] =
>> file:/D:/repository/org/apache/maven/reporting/maven-reporting-api/2.0.1/maven-reporting-api-2.0.1.jar
>> [ERROR] urls[5] =
>> file:/D:/repository/org/apache/maven/doxia/doxia-sink-api/1.0-alpha-6/doxia-sink-api-1.0-alpha-6.jar
>> [ERROR] urls[6] =
>> file:/D:/repository/commons-cli/commons-cli/1.0/commons-cli-1.0.jar
>> [ERROR] urls[7] =
>> file:/D:/repository/org/codehaus/plexus/plexus-interactivity-api/1.0-alpha-4/plexus-interactivity-api-1.0-alpha-4.jar
>> [ERROR] urls[8] =
>> file:/D:/repository/org/apache/maven/shared/maven-common-artifact-filters/1.2/maven-common-artifact-filters-1.2.jar
>> [ERROR] urls[9] =
>> file:/D:/repository/org/apache/maven/shared/maven-plugin-testing-harness/1.1/maven-plugin-testing-harness-1.1.jar
>> [ERROR] urls[10] =
>> file:/D:/repository/org/codehaus/plexus/plexus-archiver/1.0-alpha-7/plexus-archiver-1.0-alpha-7.jar
>> [ERROR] urls[11] =
>> file:/D:/repository/org/codehaus/plexus/plexus-utils/1.5.6/plexus-utils-1.5.6.jar
>> [ERROR] Number of foreign imports: 1
>> [ERROR] import: Entry[import  from realm ClassRealm[maven.api, parent: null]]
>> [ERROR]
>>
>> The above error was caused by using new IOException(String, Throwable).
>>
>> Also, the check does not work for "mvn compile", nor does it run if a
>> test fails.
>>
>> And of course the check adds extra time to every build, even those run
>> using Java 1.5.0
>>
>> What I do is I ensure that Eclipse is set up to use the appropriate JVM.
>> This catches errors at edit time.
>>
>
> Ok I've removed animal-sniffer for now, but believe that the wrong
> output bug and poor performance should rather be fixed in the plugin.

Thanks.

Also need to fix that it does not work on the compile/testCompile phases.

I suppose it might be worth considering as an optional profile in CP.
This would help developers who don't have the correct version of Java
installed, without penalising those who do.

> Regards
> Damjan
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> For additional commands, e-mail: dev-h...@commons.apache.org
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to