sun_checks is used instead of custom checkstyle configuration

[GoogleCodeExporter]

I have a project using a checkstyle configuration from a dependency like 
this:
<plugin>
  <groupId>org.apache.maven.plugins</groupId>
  <artifactId>maven-checkstyle-plugin</artifactId>
  <version>2.5</version>
  <executions>
    <execution>
      <id>do checkstyle</id>
      <phase>process-sources</phase>
      <configuration>
        <configLocation>checkstyle.xml</configLocation>
      </configuration>
      <goals>
    <goal>check</goal>
      </goals>
    </execution>
  </executions>
  <dependencies>
    <dependency>
      <groupId>se.idainfront.project</groupId>
      <artifactId>util</artifactId>
      <version>${version}</version>
    </dependency>
  </dependencies>
</plugin> 

This is working fine when building with Maven 3.0-alpha-7.
However the generated checktyle-eclipse configuration looks like this:

<?xml version="1.0" encoding="UTF-8"?>

<fileset-config file-format-version="1.2.0" simple-config="false">
  <local-check-config name="maven-chekstyle-plugin" 
location="jar:file:/c:/maven/repository/org/apache/maven/plugins/maven-
checkstyle-plugin/2.5/maven-checkstyle-plugin-
2.5.jar!/config/sun_checks.xml" type="remote" description="maven-
checkstyle-plugin configuration">
    <property name="checkstyle.cache.file" 
value="${project_loc}/checkstyle-cachefile"/>
  </local-check-config>
  <fileset name="java-sources" enabled="true" check-config-name="maven-
chekstyle-plugin" local="true">
    <file-match-pattern match-pattern="src/main/java/" include-
pattern="true"/>
  </fileset>
</fileset-config> 

Original issue reported on code.google.com by grims...@gmail.com on 12 Apr 2010 at 11:45

[GoogleCodeExporter]

After some investigation the problem only exists if the checkstyle 
configuration is 
placed inside the execution element.

Original comment by grims...@gmail.com on 15 Apr 2010 at 8:44

[GoogleCodeExporter]

I see will see if this can be fixed, although I have not seen a mutliple 
execution checkstyle configuration or why you would need two in the same build 
lifecycle?

Original comment by mkann...@gmail.com on 28 Sep 2010 at 11:23

• Changed state: Accepted