[ 
http://jira.codehaus.org/browse/MCHECKSTYLE-132?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=242223#action_242223
 ] 

Paul Nyheim commented on MCHECKSTYLE-132:
-----------------------------------------

Did not have time to actually fix them now, but managed to track down the error.
The tests that fail are related to this change in Checkstyle 5.3 (all tests run 
fine with 5.2)
[changeset:DefaultConfiguration.java|http://checkstyle.hg.sourceforge.net/hgweb/checkstyle/checkstyle/diff/917266ff93a4/src/checkstyle/com/puppycrawl/tools/checkstyle/DefaultConfiguration.java]

As far as I can find out, cacheFile is specified in both min-plugin-config.xml 
and in sun_checks.xml (with substitution property), and this creates problems 
due to the previous mentioned change, where the absolute path would be added to 
the property with a comma to separate them. This obviously fails when trying to 
create the file...

So the bottom line is that so long that you do not specify e.g. {{cacheFile}} 
several times in your config, you should be all clear.

I am not sure whether this is in fact a bug with 5.3, or if it is a result of 
maven-checkstyle-plugin using the Checkstyle API in the wrong way.

> Upgrade to Checkstyle 5.1
> -------------------------
>
>                 Key: MCHECKSTYLE-132
>                 URL: http://jira.codehaus.org/browse/MCHECKSTYLE-132
>             Project: Maven 2.x Checkstyle Plugin
>          Issue Type: Improvement
>            Reporter: Simon Brandhof
>         Attachments: checkstyle-5.3.patch
>
>
> [Release notes|http://checkstyle.sourceforge.net/releasenotes.html]

-- 
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

        

Reply via email to