You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by br...@apache.org on 2005/05/16 02:20:01 UTC

svn commit: r170287 - /maven/components/trunk/maven-reports/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReport.java

Author: brett
Date: Sun May 15 17:20:00 2005
New Revision: 170287

URL: http://svn.apache.org/viewcvs?rev=170287&view=rev
Log:
check style report should save cache file to the correct output directory

Modified:
    maven/components/trunk/maven-reports/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReport.java

Modified: maven/components/trunk/maven-reports/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReport.java
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-reports/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReport.java?rev=170287&r1=170286&r2=170287&view=diff
==============================================================================
--- maven/components/trunk/maven-reports/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReport.java (original)
+++ maven/components/trunk/maven-reports/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReport.java Sun May 15 17:20:00 2005
@@ -145,8 +145,9 @@
 
         try
         {
+            Properties overridingProperties = createOverridingProperties();
             config = ConfigurationLoader.loadConfiguration( configFile.toString(),
-                                                            new PropertiesExpander( createOverridingProperties() ) );
+                                                            new PropertiesExpander( overridingProperties ) );
         }
         catch ( CheckstyleException e )
         {
@@ -233,7 +234,8 @@
     {
         Properties props = new Properties();
         props.setProperty( "checkstyle.header.file", "LICENSE.txt" );
-        props.setProperty( "checkstyle.cache.file", "target/checkstyle-cachefile" );
+        // TODO: explicit output directory when it is back
+        props.setProperty( "checkstyle.cache.file", getConfiguration().getModel().getBuild().getDirectory() + "/checkstyle-cachefile" );
         return props;
     }
 }



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org