Author: dennisl
Date: Thu Jun 25 12:23:17 2015
New Revision: 1687493

URL: http://svn.apache.org/r1687493
Log:
[MPMD-206] Make the sourceDirectories configurable
I have changed the parameters so that they are not read-only any more.
However due to MNG-5440 you cannot yet set values for source directories in 
your own POM. Once a fix for that issue has been made to Maven core I will add 
an integration test for this issue, that will require at least that version of 
Maven to run. I have not committed the provided documentation changes yet. That 
will have to wait until it is actually possible to configure things that way.

Modified:
    
maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/AbstractPmdReport.java

Modified: 
maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/AbstractPmdReport.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/AbstractPmdReport.java?rev=1687493&r1=1687492&r2=1687493&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/AbstractPmdReport.java
 (original)
+++ 
maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/AbstractPmdReport.java
 Thu Jun 25 12:23:17 2015
@@ -128,13 +128,13 @@ public abstract class AbstractPmdReport
     /**
      * The directories containing the sources to be compiled.
      */
-    @Parameter( property = "project.compileSourceRoots", required = true, 
readonly = true )
+    @Parameter( defaultValue = "${project.compileSourceRoots}" )
     private List<String> compileSourceRoots;
 
     /**
      * The directories containing the test-sources to be compiled.
      */
-    @Parameter( property = "project.testCompileSourceRoots", required = true, 
readonly = true )
+    @Parameter( defaultValue = "${project.testCompileSourceRoots}" )
     private List<String> testSourceRoots;
 
     /**


Reply via email to