You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Svetlomira Manova (JIRA)" <ji...@codehaus.org> on 2011/04/04 10:01:22 UTC
[jira] Commented: (MCHECKSTYLE-152) encoding property in maven
plugin is never set correctly to charset property of the checkstyle itself.
[ http://jira.codehaus.org/browse/MCHECKSTYLE-152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=262431#action_262431 ]
Svetlomira Manova commented on MCHECKSTYLE-152:
-----------------------------------------------
Hi guys,
Any update here?
The patch is provided.
Is there any chance this patch to be applyed soon?
Thanks!
Best regards,
Svetlomira
> encoding property in maven plugin is never set correctly to charset property of the checkstyle itself.
> ------------------------------------------------------------------------------------------------------
>
> Key: MCHECKSTYLE-152
> URL: http://jira.codehaus.org/browse/MCHECKSTYLE-152
> Project: Maven 2.x Checkstyle Plugin
> Issue Type: Bug
> Affects Versions: 2.4, 2.5, 2.6
> Environment: Windows x
> Reporter: Svetlomira Manova
> Attachments: DefaultCheckstyleExecutor.java, fix_CodeDifference.jpg, mychanges.patch, src.zip
>
>
> 1. In the pom.xml i set property <encoding>:
> <plugin>
> <groupId>org.apache.maven.plugins</groupId>
> <artifactId>maven-checkstyle-plugin</artifactId>
> <version>2.6</version>
> <configuration>
> <encoding>UTF-8</encoding>
> </configuration>
> </plugin>
> 2. When i run the checkstyle (mvn checkstyle:checkstyle) this property is not set to "charset" attribute.
> 3. I noticed that this functionality works for version 2.2. However it does not work for versions 2.4 and above. I think that this functionality does not work after some refactoring is done and this functionality is moved in org.apache.maven.plugin.checkstyle.DefaultCheckstyleExecutor -> "public Configuration getConfiguration( CheckstyleExecutorRequest request )".
> The problem is that in this method the developer tries to find the Checker module and to set its "charset" attribute as a child of the "config" object. However the "config" object it the Checker module itself.
> Fix is simple - just take out adding of "charset" attribute value from the for cycle.
> I will attach the class with the new code and difference between the new and old code.
> I hope this bug can be fixed soon.
> Thanks!
> Best regards,
> Svetlomira
--
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