You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ah...@apache.org on 2006/02/03 23:50:41 UTC

svn commit: r374770 - in /maven/maven-1/plugins/trunk/pmd: plugin.jelly plugin.properties xdocs/changes.xml xdocs/properties.xml

Author: aheritier
Date: Fri Feb  3 14:50:30 2006
New Revision: 374770

URL: http://svn.apache.org/viewcvs?rev=374770&view=rev
Log:
MPPMD-21 : New properties "maven.pmd.failonerror" and "maven.pmd.failonruleviolation" to fail the build if any errors or problems are found.

Modified:
    maven/maven-1/plugins/trunk/pmd/plugin.jelly
    maven/maven-1/plugins/trunk/pmd/plugin.properties
    maven/maven-1/plugins/trunk/pmd/xdocs/changes.xml
    maven/maven-1/plugins/trunk/pmd/xdocs/properties.xml

Modified: maven/maven-1/plugins/trunk/pmd/plugin.jelly
URL: http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/pmd/plugin.jelly?rev=374770&r1=374769&r2=374770&view=diff
==============================================================================
--- maven/maven-1/plugins/trunk/pmd/plugin.jelly (original)
+++ maven/maven-1/plugins/trunk/pmd/plugin.jelly Fri Feb  3 14:50:30 2006
@@ -114,7 +114,11 @@
 
     <j:choose>
       <j:when test="${sourcesPresent == 'true'}">
-        <pmd rulesetfiles="${maven.pmd.rulesetfiles}" printToConsole="${maven.pmd.console}">
+        <pmd 
+          rulesetfiles="${maven.pmd.rulesetfiles}" 
+          printToConsole="${maven.pmd.console}"
+          failonerror="${maven.pmd.failonerror}"
+          failOnRuleViolation="${maven.pmd.failonruleviolation}">
           <formatter type="xml" toFile="${maven.build.dir}/pmd-raw-report.xml"/>
           <fileset dir="${pom.build.sourceDirectory}" 
             includes="${maven.pmd.includes}"

Modified: maven/maven-1/plugins/trunk/pmd/plugin.properties
URL: http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/pmd/plugin.properties?rev=374770&r1=374769&r2=374770&view=diff
==============================================================================
--- maven/maven-1/plugins/trunk/pmd/plugin.properties (original)
+++ maven/maven-1/plugins/trunk/pmd/plugin.properties Fri Feb  3 14:50:30 2006
@@ -44,4 +44,8 @@
 maven.pmd.check.tests = true
 
 # Print the output to the console
-maven.pmd.console = false
\ No newline at end of file
+maven.pmd.console = false
+
+# Fail the build ?
+maven.pmd.failonerror = false
+maven.pmd.failonruleviolation = false
\ No newline at end of file

Modified: maven/maven-1/plugins/trunk/pmd/xdocs/changes.xml
URL: http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/pmd/xdocs/changes.xml?rev=374770&r1=374769&r2=374770&view=diff
==============================================================================
--- maven/maven-1/plugins/trunk/pmd/xdocs/changes.xml (original)
+++ maven/maven-1/plugins/trunk/pmd/xdocs/changes.xml Fri Feb  3 14:50:30 2006
@@ -25,6 +25,7 @@
   </properties>
   <body>
     <release version="1.8-SNAPSHOT" date="In SVN">
+      <action dev="aheritier" type="add" issue="MPPMD-21">New properties "maven.pmd.failonerror" and "maven.pmd.failonruleviolation" to fail the build if any errors or problems are found.</action>
       <action dev="aheritier" type="add" issue="MPPMD-13">New property "maven.pmd.console" to display pmd errors to the console.</action>
       <action dev="aheritier" type="fix">Do not generate links to JXR files if they are not created.</action>
       <action dev="aheritier" type="update">Use properties maven.jxr.destdir and maven.jxr.destdir.test to generate links from the PMD report to jxr files.</action>

Modified: maven/maven-1/plugins/trunk/pmd/xdocs/properties.xml
URL: http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/pmd/xdocs/properties.xml?rev=374770&r1=374769&r2=374770&view=diff
==============================================================================
--- maven/maven-1/plugins/trunk/pmd/xdocs/properties.xml (original)
+++ maven/maven-1/plugins/trunk/pmd/xdocs/properties.xml Fri Feb  3 14:50:30 2006
@@ -101,6 +101,20 @@
             Whether or not to display PMD errors to the console. Defaults to false.
           </td>
         </tr>
+        <tr>
+          <td>maven.pmd.failonerror</td>
+          <td>Yes</td>
+          <td>
+            Whether or not to fail the build if any errors occur while processing the files. Defaults to false.
+          </td>
+        </tr>
+        <tr>
+          <td>maven.pmd.failonruleviolation</td>
+          <td>Yes</td>
+          <td>
+            Whether or not to fail the build if PMD finds any problems. Defaults to false.
+          </td>
+        </tr>
       </table>
     </section>
   </body>