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

Dave Sag commented on MNG-1113:
-------------------------------

so what is the useFile field for?

would it be too much trouble to change the mojo such that the property is 
called configFile and add some documentation explainin that it's an optional 
config file.  then the propertiesFile can be used for its real purpose, (as per 
the comment from  Mark Kuzmycz above).  and please document what useFile is 
actually for :-)

where i work our coding standards diverge quite a bit from the sun standard. 
(something I am trying to fix)

cheers

dave


> 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: MNG-1113-maven-checkstyle-plugin.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