[ 
https://jira.codehaus.org/browse/MCHECKSTYLE-172?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=297601#comment-297601
 ] 

Andrew Hunt edited comment on MCHECKSTYLE-172 at 5/1/12 3:19 AM:
-----------------------------------------------------------------

Not sure that the reported fix to the count problem MCHECKSTYLE-167 has been 
fixed in the checkstyle-aggregate report. If you have a look at 
http://maven.apache.org/plugins/maven-checkstyle-plugin/project-reports.html, 
you will see the report repetition AND that the first report has double the 
issues of the second report.  In the checkstyle-aggregate report, each error is 
reported twice.

The workaround works great (in terms of providing one report and removing the 
repeats), though I had to make a minor adjustment (my full config shown)...
{code}
  <reporting>
      ...
        <plugin>
           <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-checkstyle-plugin</artifactId>
           <version>2.9</version>
           <inherited>false</inherited>
           <reportSets>
             <reportSet>
                <reports>
                  <report>checkstyle</report>        <!-- This is changed from 
above -->
                </reports>
             </reportSet>
           </reportSets>
           <configuration>
             <configLocation>group_default_checkstyles.xml</configLocation>
           </configuration>
         </plugin>
      ...
{code}
                
      was (Author: andrew.hunt):
    Not sure that the reported fix to the count problem MCHECKSTYLE-167 has 
been fixed in the checkstyle-aggregate report. If you have a look at 
http://maven.apache.org/plugins/maven-checkstyle-plugin/project-reports.html, 
you will see the report repetition AND that the first report has double the 
issues of the second report.  In the checkstyle-aggregate report, each error is 
reported twice.

The workaround works great, though with this minor adjustment (full 
definition)...
{code}
  <reporting>
      ...
        <plugin>
           <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-checkstyle-plugin</artifactId>
           <version>2.9</version>
           <inherited>false</inherited>
           <reportSets>
             <reportSet>
                <reports>
                  <report>checkstyle</report>        <!-- This is changed from 
above -->
                </reports>
             </reportSet>
           </reportSets>
           <configuration>
             <configLocation>group_default_checkstyles.xml</configLocation>
           </configuration>
         </plugin>
      ...
{code}
                  
> Checkstyle Plugin 2.8+ generates an additional aggregate report
> ---------------------------------------------------------------
>
>                 Key: MCHECKSTYLE-172
>                 URL: https://jira.codehaus.org/browse/MCHECKSTYLE-172
>             Project: Maven 2.x Checkstyle Plugin
>          Issue Type: Bug
>    Affects Versions: 2.8, 2.9
>         Environment: {{{
> mvn --version
> Apache Maven 3.0.4 (r1206075; 2011-11-25 09:20:29+0100)
> Maven home: /usr/local/Cellar/maven/current/libexec
> Java version: 1.6.0_29, vendor: Apple Inc.
> Java home: 
> /Library/Java/JavaVirtualMachines/1.6.0_29-b11-402.jdk/Contents/Home
> Default locale: de_DE, platform encoding: MacRoman
> OS name: "mac os x", version: "10.7.2", arch: "x86_64", family: "mac"
> }}}
>            Reporter: SebbASF
>
> Using a very simple single module project, the aggregated report is created 
> by default.
> Both the {{checkstyle}} and {{checkstyle-aggregate}} report are generated.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://jira.codehaus.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to