Author: vmassol
Date: Sat May 21 04:08:50 2005
New Revision: 171202

URL: http://svn.apache.org/viewcvs?rev=171202&view=rev
Log:
Removed the <code>maven.clover.jar</code> property that was used to override 
the default Clover jar. This property was not working (it requires Ant 1.6) and 
is no needed anymore as Cenqua has cleanly separated the Clover jar from the 
Clover license. You only need to point <code>maven.clover.license.path</code> 
to your license file.

Added:
    maven/maven-1/plugins/trunk/clover/src/plugin-test/testCustomCloverLicense/
    
maven/maven-1/plugins/trunk/clover/src/plugin-test/testCustomCloverLicense/maven.xml
    
maven/maven-1/plugins/trunk/clover/src/plugin-test/testCustomCloverLicense/project.properties
    
maven/maven-1/plugins/trunk/clover/src/plugin-test/testCustomCloverLicense/project.xml
Modified:
    maven/maven-1/plugins/trunk/clover/plugin.jelly
    maven/maven-1/plugins/trunk/clover/plugin.properties
    maven/maven-1/plugins/trunk/clover/xdocs/changes.xml
    maven/maven-1/plugins/trunk/clover/xdocs/faq.fml
    maven/maven-1/plugins/trunk/clover/xdocs/properties.xml

Modified: maven/maven-1/plugins/trunk/clover/plugin.jelly
URL: 
http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/clover/plugin.jelly?rev=171202&r1=171201&r2=171202&view=diff
==============================================================================
--- maven/maven-1/plugins/trunk/clover/plugin.jelly (original)
+++ maven/maven-1/plugins/trunk/clover/plugin.jelly Sat May 21 04:08:50 2005
@@ -35,19 +35,8 @@
   -->
   <goal name="clover:init-internal">
 
-    <!-- Create an Ant <path> id that contains the Clover jar. If the user has
-         defined the maven.clover.jar property then this path will contain his
-         version of the clover jar. Otherwise use the version defined as a 
-         dependency in this plugin's project.xml. -->
     <ant:path id="clover.classpath">
-      <j:choose>
-        <j:when test="${context.getVariable('maven.clover.jar') != null}">
-          <ant:pathelement path="${maven.clover.jar}"/>
-        </j:when>
-        <j:otherwise>
-          <ant:pathelement 
path="${plugin.getDependencyPath('clover:clover')}"/>
-        </j:otherwise>
-      </j:choose>
+      <ant:pathelement path="${plugin.getDependencyPath('clover:clover')}"/>
     </ant:path>
 
     <ant:taskdef resource="clovertasks" classpathref="clover.classpath"/>
@@ -65,9 +54,10 @@
       </j:otherwise>
     </j:choose>
 
-    <!-- Allow users to override the version of the Clover jar to use  -->
+    <!-- Add the Clover jar to the CP so that it is systematically there. This 
is required
+         for example for running unit tests on Clovered code. -->
     <maven:addPath id="maven.dependency.classpath" refid="clover.classpath"/>
-
+    
     <ant:mkdir dir="${maven.clover.build.classes}"/>
     <ant:mkdir dir="${maven.clover.database.dir}"/>
 

Modified: maven/maven-1/plugins/trunk/clover/plugin.properties
URL: 
http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/clover/plugin.properties?rev=171202&r1=171201&r2=171202&view=diff
==============================================================================
--- maven/maven-1/plugins/trunk/clover/plugin.properties (original)
+++ maven/maven-1/plugins/trunk/clover/plugin.properties Sat May 21 04:08:50 
2005
@@ -34,11 +34,6 @@
 # Whether tests should be instrumented by Clover or not
 maven.clover.instrument.tests = false
 
-# Allows the user to override the Clover jar. This is especially useful
-# as some Clover jars are license-signed for a specific project so you 
-# might want to use different jars for different projects.
-#maven.clover.jar = ${maven.repo.local}/clover/jars/clover-myproject-1.2.3.jar
-
 # Location of the Clover license. Users can override this property to point to
 # their own Clover license. By default uses an eval license valid for open 
 # source projects. Set it to an empty value if you wish to use Clover's other

Added: 
maven/maven-1/plugins/trunk/clover/src/plugin-test/testCustomCloverLicense/maven.xml
URL: 
http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/clover/src/plugin-test/testCustomCloverLicense/maven.xml?rev=171202&view=auto
==============================================================================
--- 
maven/maven-1/plugins/trunk/clover/src/plugin-test/testCustomCloverLicense/maven.xml
 (added)
+++ 
maven/maven-1/plugins/trunk/clover/src/plugin-test/testCustomCloverLicense/maven.xml
 Sat May 21 04:08:50 2005
@@ -0,0 +1,38 @@
+<!-- 
+/*
+ * Copyright 2005 The Apache Software Foundation.
+ * 
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ * 
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ * 
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+ -->
+<project default="testPlugin"
+  xmlns:util="jelly:util"
+  xmlns:j="jelly:core"
+  xmlns:ant="jelly:ant"
+  xmlns:x="jelly:xml"
+  xmlns:maven="jelly:maven"
+  xmlns:assert="assert">
+
+  <goal name="testPlugin" prereqs="clean">
+
+       <ant:copy tofile="${maven.build.dir}/clover-test.license"
+           
file="${pom.getPluginContext('maven-clover-plugin').getVariable('plugin.resources')}/clover.license"/>
+       
+       <attainGoal name="clover:on"/>
+    <attainGoal name="clover:off"/>
+
+       <!-- If the license is not found, the build will fail here -->
+       
+  </goal>
+
+</project>
\ No newline at end of file

Added: 
maven/maven-1/plugins/trunk/clover/src/plugin-test/testCustomCloverLicense/project.properties
URL: 
http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/clover/src/plugin-test/testCustomCloverLicense/project.properties?rev=171202&view=auto
==============================================================================
--- 
maven/maven-1/plugins/trunk/clover/src/plugin-test/testCustomCloverLicense/project.properties
 (added)
+++ 
maven/maven-1/plugins/trunk/clover/src/plugin-test/testCustomCloverLicense/project.properties
 Sat May 21 04:08:50 2005
@@ -0,0 +1,17 @@
+# -------------------------------------------------------------------
+# Copyright 2005 The Apache Software Foundation.
+# 
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+# 
+#      http://www.apache.org/licenses/LICENSE-2.0
+#  
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+# -------------------------------------------------------------------
+
+maven.clover.license.path = ${maven.build.dir}/clover-test.license

Added: 
maven/maven-1/plugins/trunk/clover/src/plugin-test/testCustomCloverLicense/project.xml
URL: 
http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/clover/src/plugin-test/testCustomCloverLicense/project.xml?rev=171202&view=auto
==============================================================================
--- 
maven/maven-1/plugins/trunk/clover/src/plugin-test/testCustomCloverLicense/project.xml
 (added)
+++ 
maven/maven-1/plugins/trunk/clover/src/plugin-test/testCustomCloverLicense/project.xml
 Sat May 21 04:08:50 2005
@@ -0,0 +1,25 @@
+<?xml version="1.0" encoding="UTF-8"?>
+
+<!-- 
+/*
+ * Copyright 2005 The Apache Software Foundation.
+ * 
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ * 
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ * 
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+ -->
+
+<project>
+  <extend>${basedir}/../project.xml</extend>
+  <id>testCustomCloverLicense</id>
+  <name>testCustomCloverLicense</name>
+</project>

Modified: maven/maven-1/plugins/trunk/clover/xdocs/changes.xml
URL: 
http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/clover/xdocs/changes.xml?rev=171202&r1=171201&r2=171202&view=diff
==============================================================================
--- maven/maven-1/plugins/trunk/clover/xdocs/changes.xml (original)
+++ maven/maven-1/plugins/trunk/clover/xdocs/changes.xml Sat May 21 04:08:50 
2005
@@ -26,6 +26,13 @@
   </properties>
   <body>
     <release version="1.9-SNAPSHOT" date="in SVN">
+      <action dev="vmassol" type="remove">
+        Removed the <code>maven.clover.jar</code> property that was used to 
+        override the default Clover jar. This property was not working
+        (it requires Ant 1.6) and is no needed anymore as Cenqua has cleanly
+        separated the Clover jar from the Clover license. You only need to
+        point <code>maven.clover.license.path</code> to your license file.
+      </action>
       <action dev="vmassol" type="add">
         Added a quick usage guide in the documentation.
       </action>

Modified: maven/maven-1/plugins/trunk/clover/xdocs/faq.fml
URL: 
http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/clover/xdocs/faq.fml?rev=171202&r1=171201&r2=171202&view=diff
==============================================================================
--- maven/maven-1/plugins/trunk/clover/xdocs/faq.fml (original)
+++ maven/maven-1/plugins/trunk/clover/xdocs/faq.fml Sat May 21 04:08:50 2005
@@ -28,15 +28,11 @@
       </question>
       <answer>
         <p>
-          If you have a licensed clover JAR, or you want to upgrade the 
version used by the plugin, you will
-          need to use a jar override to force the clover plugin to use the 
updated JAR.
-        </p>
-        <p>eg: To use a local copy of clover-1.2.4-registered.jar, add this to 
project.properties:</p>
-          <source>maven.jar.override=on
-maven.jar.clover=lib/clover-1.2.4-registered.jar</source>
-        <p>eg: To use version 1.2.5 from the repository, add this to 
project.properties:</p>
-          <source>maven.jar.override=on
-maven.jar.clover=1.2.5</source>
+          You don't! Clover has now separated the Clover jar from it's license 
and thus 
+          there's now no need to provide a custom jar. However you'll still 
need to provide
+          your own license (unless you're using this plugin on an open source 
project). You
+          specify your own license using the 
<code>maven.clover.license.path</code> property.
+          <source>maven.clover.license.path = 
/some/path/to/my/clover.license</source>
       </answer>
     </faq>
 

Modified: maven/maven-1/plugins/trunk/clover/xdocs/properties.xml
URL: 
http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/clover/xdocs/properties.xml?rev=171202&r1=171201&r2=171202&view=diff
==============================================================================
--- maven/maven-1/plugins/trunk/clover/xdocs/properties.xml (original)
+++ maven/maven-1/plugins/trunk/clover/xdocs/properties.xml Sat May 21 04:08:50 
2005
@@ -107,19 +107,6 @@
           </td>
         </tr>
         <tr>
-          <td>maven.clover.jar</td>
-          <td>Yes</td>
-          <td>
-            Allows the user to override the Clover jar. This is especially 
useful
-            as some Clover jars are license-signed for a specific project so 
you 
-            might want to use different jars for different projects.
-          </td>
-          <td>
-            If not defined, the plugin uses a 30 day eval Clover jar which is 
-            valid for open source projects.
-          </td>
-        </tr>
-        <tr>
           <td>maven.clover.license.path</td>
           <td>Yes</td>
           <td>



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to