maven-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Bentmann <benjamin.bentm...@udo.edu>
Subject Re: svn commit: r950747 - /maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/DefaultCheckstyleExecutor.java
Date Thu, 03 Jun 2010 09:43:51 GMT
Hi Dan,

> Author: dkulp
> Date: Wed Jun  2 20:23:49 2010
> New Revision: 950747
>
> URL: http://svn.apache.org/viewvc?rev=950747&view=rev
> Log:
> Since the DefaultCheckstyleExecutor contains an object (ResourceManager)
> that holds onto state and must be per-lookup, the DefaultCheckstyleExecutor
> must also be per-lookup to be thread safe.
>
> Modified:
>      maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/DefaultCheckstyleExecutor.java

This appears to have broken the integration tests [0], can you 
double-check this?


Benjamin


[0] https://grid.sonatype.org/ci/job/maven-plugins-ITs/454/

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


Mime
View raw message