[ 
https://issues.apache.org/jira/browse/MCHECKSTYLE-332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15681307#comment-15681307
 ] 

Hudson commented on MCHECKSTYLE-332:
------------------------------------

SUCCESS: Integrated in Jenkins build maven-plugins #7722 (See 
[https://builds.apache.org/job/maven-plugins/7722/])
[MCHECKSTYLE-332] maven plugin not using cache property

Starting with Checkstyle 6.16, the cache configuration was moved from the 
TreeWalker module to the Checker module, as per 
https://github.com/checkstyle/checkstyle/issues/569. To detect this change, we 
can rely on the presence of the public API method "setCacheFile(String)" in the 
Checker class. (gboue: [http://svn.apache.org/viewvc/?view=rev&rev=1770564])
* (add) maven-checkstyle-plugin/src/it/MCHECKSTYLE-332_cache-checker
* (add) 
maven-checkstyle-plugin/src/it/MCHECKSTYLE-332_cache-checker/invoker.properties
* (add) maven-checkstyle-plugin/src/it/MCHECKSTYLE-332_cache-checker/pom.xml
* (add) maven-checkstyle-plugin/src/it/MCHECKSTYLE-332_cache-checker/src
* (add) maven-checkstyle-plugin/src/it/MCHECKSTYLE-332_cache-checker/src/main
* (add) 
maven-checkstyle-plugin/src/it/MCHECKSTYLE-332_cache-checker/src/main/java
* (add) 
maven-checkstyle-plugin/src/it/MCHECKSTYLE-332_cache-checker/src/main/java/org
* (add) 
maven-checkstyle-plugin/src/it/MCHECKSTYLE-332_cache-checker/src/main/java/org/MyClass.java
* (add) 
maven-checkstyle-plugin/src/it/MCHECKSTYLE-332_cache-checker/verify.groovy
* (add) maven-checkstyle-plugin/src/it/MCHECKSTYLE-332_cache-treewalker
* (add) 
maven-checkstyle-plugin/src/it/MCHECKSTYLE-332_cache-treewalker/invoker.properties
* (add) maven-checkstyle-plugin/src/it/MCHECKSTYLE-332_cache-treewalker/pom.xml
* (add) maven-checkstyle-plugin/src/it/MCHECKSTYLE-332_cache-treewalker/src
* (add) maven-checkstyle-plugin/src/it/MCHECKSTYLE-332_cache-treewalker/src/main
* (add) 
maven-checkstyle-plugin/src/it/MCHECKSTYLE-332_cache-treewalker/src/main/java
* (add) 
maven-checkstyle-plugin/src/it/MCHECKSTYLE-332_cache-treewalker/src/main/java/org
* (add) 
maven-checkstyle-plugin/src/it/MCHECKSTYLE-332_cache-treewalker/src/main/java/org/MyClass.java
* (add) 
maven-checkstyle-plugin/src/it/MCHECKSTYLE-332_cache-treewalker/verify.groovy
* (edit) 
maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/exec/DefaultCheckstyleExecutor.java


> maven plugin not using cache property
> -------------------------------------
>
>                 Key: MCHECKSTYLE-332
>                 URL: https://issues.apache.org/jira/browse/MCHECKSTYLE-332
>             Project: Maven Checkstyle Plugin
>          Issue Type: Bug
>          Components: checkstyle:checkstyle
>    Affects Versions: 2.17
>         Environment: Windows 7, 64 bit, JDK 8
>            Reporter: richard
>
> When enabling cache in the pom for checkstyle via `cacheFile`, no cache file 
> is ever created after the run and multiple runs are not using any cache as 
> there is no speed up.
> POM: 
> https://github.com/checkstyle/checkstyle/blob/481406f2d5c07b8766757eb45bcc15fd1fa51bed/pom.xml#L453-L492
> I am specifying the cache after line 472 with the following:
> bq. <cacheFile>${project.build.directory}/cachefile-sevntu</cacheFile>
> When I specify the cache directly into the configuration file, the cache file 
> is created, so this looks like an issue directly related to the maven plugin.
> I shouldn't need to specify it into the configuration since a property is 
> available in the POM.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to