[ http://jira.codehaus.org/browse/MNG-1113?page=comments#action_51309 ] 

Michael Böckling commented on MNG-1113:
---------------------------------------

I second that, also it would be really nice if checkstyle-4.0.beta6 could be 
used. The Eclipse checkstyle plugin uses that since some time now.
Of course the above was just a quick proof-of-concept, I certainly wouldn't use 
an absolute path in a production build. Besides, the presence of a checkstyle 
config could easily be checked by the verifier plugin in the entry-test phase.

> Plugin does not support using custom checkstyle XML config file
> ---------------------------------------------------------------
>
>          Key: MNG-1113
>          URL: http://jira.codehaus.org/browse/MNG-1113
>      Project: Maven 2
>         Type: Bug
>   Components: maven-checkstyle-plugin
>     Versions: 2.0-beta-3
>     Reporter: Vincent Massol
>     Assignee: Edwin Punzalan
>  Attachments: CheckstyleReport.java, MNG-1113-maven-checkstyle-plugin.patch, 
> checkstyle-config-file.patch, recommended.txt
>
>
> The plugin source code says:
>     /**
>      * Specifies the location of the checkstyle properties that will be used 
> to check the source.
>      *
>      * @parameter
>      */
>     private File propertiesFile;
> But in practice it's only loading *properties* and not the XML. The 
> getConfigFile() method is not right and needs to be modified to take into 
> account a custom checkstyle config file.

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


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to