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 2004/04/14 02:55:25 UTC
cvs commit: maven-plugins/pmd/xdocs changes.xml
brett 2004/04/13 17:55:25
Modified: pmd plugin.jelly
pmd/xdocs changes.xml
Log:
PR: MPPMD-7
fixed issues after clean. Fixed handling of cpd.enable property.
Revision Changes Path
1.14 +6 -3 maven-plugins/pmd/plugin.jelly
Index: plugin.jelly
===================================================================
RCS file: /home/cvs/maven-plugins/pmd/plugin.jelly,v
retrieving revision 1.13
retrieving revision 1.14
diff -u -r1.13 -r1.14
--- plugin.jelly 12 Mar 2004 09:42:29 -0000 1.13
+++ plugin.jelly 14 Apr 2004 00:55:25 -0000 1.14
@@ -23,7 +23,7 @@
PMD Plugin. Generate PMD reports using the PMD framework.
=============================================================================
-->
-<project xmlns:j="jelly:core" xmlns:doc="doc">
+<project xmlns:j="jelly:core" xmlns:doc="doc" xmlns:ant="jelly:ant">
<!--
========================================================================
@@ -37,7 +37,7 @@
pluginName="pmd"
description="Verification of coding rules."
link="pmd-report"/>
- <j:if test="${maven.pmd.cpd.enable}">
+ <j:if test="${context.getVariable('maven.pmd.cpd.enable') == 'true'}">
<doc:registerReport
name="CPD Report"
pluginName=""
@@ -55,7 +55,7 @@
<goal name="maven-pmd-plugin:deregister">
<j:if test="${sourcesPresent}">
<doc:deregisterReport name="PMD Report"/>
- <j:if test="${maven.pmd.cpd.enable}">
+ <j:if test="${context.getVariable('maven.pmd.cpd.enable') == 'true'}">
<doc:deregisterReport name="CPD Report"/>
</j:if>
</j:if>
@@ -170,6 +170,8 @@
<!-- Run the CPD task -->
<echo>Running the CPD task with minimumTokenCount = ${maven.pmd.cpd.minimumtokencount} ...</echo>
+ <ant:mkdir dir="${maven.build.dir}" />
+
<cpd minimumTokenCount="${maven.pmd.cpd.minimumtokencount}"
outputFile="${maven.build.dir}/cpd-raw-report.txt">
<!-- cut and paste from pmd:report -->
@@ -196,6 +198,7 @@
<!-- Convert raw text report into XDOC -->
<j:set var="genDocs" value="${maven.gen.docs}" />
+ <ant:mkdir dir="${genDocs}" />
<doc:text-xdoc
title="CPD Report"
section="CPD Report"
1.19 +4 -0 maven-plugins/pmd/xdocs/changes.xml
Index: changes.xml
===================================================================
RCS file: /home/cvs/maven-plugins/pmd/xdocs/changes.xml,v
retrieving revision 1.18
retrieving revision 1.19
diff -u -r1.18 -r1.19
--- changes.xml 10 Mar 2004 12:23:01 -0000 1.18
+++ changes.xml 14 Apr 2004 00:55:25 -0000 1.19
@@ -25,6 +25,10 @@
<author email="vmassol@apache.org">Vincent Massol</author>
</properties>
<body>
+ <release version="1.4-SNAPSHOT" date="in CVS">
+ <action dev="brett" type="fix" issue="MPPMD-7">Create directories that exist under maven.build.dir before using them so it works after clean.</action>
+ <action dev="brett" type="fix">Fix usage of the maven.pmd.cpd.enable property so it is disabled when set to false (the default)</action>
+ </release>
<release version="1.3" date="2004-03-10">
<action dev="dion" type="fix" issue="MAVEN-1044">Allow multiple includes/excludes.</action>
<action dev="dion" type="update">Apply MAVEN-1008. Allow for custom rulesets in the classpath.</action>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org