Is anyone using Clover to look at unit test coverage aside from me?  I
tried getting it working as is, but I couldn't seem to get the maven
plugin to work without modifying the main pom.  Since I'm not terribly
familiar with the pom, and I'm not sure what my changes might affect, I
figured the list should hear about it first.  The attached patch
contains my changes for 1.4.x, and should apply for 1.3.x as well since
the changes are identical.  Basically I moved the plugin definition up
in the pom file (to correspond with Atlassian's current instructions)
and updated the version.  Are there any objections to me committing this
change into the main repo?

G

P.S.  For those interested in generating the Clover output, apply the
patch, then from the top of the branch run mvn clover2:setup test
clover2:aggregate clover2:clover -Dcheckstyle.skip=true -P
admin,dist,engage,worker,workspace,serviceregistry,directory-db,capture,oaipmh,export,gstreamer-launch-service.
 The skipping of checkstyles is especially important since otherwise
checkstyles objects to the instrumented source files.
Index: pom.xml
===================================================================
--- pom.xml	(revision 12291)
+++ pom.xml	(working copy)
@@ -657,6 +657,11 @@
           </execution>
         </executions>
       </plugin>
+      <plugin>
+        <groupId>com.atlassian.maven.plugins</groupId>
+        <artifactId>maven-clover2-plugin</artifactId>
+        <version>3.1.5</version>
+      </plugin>
     </plugins>
     <pluginManagement>
       <plugins>
@@ -813,11 +818,6 @@
           <version>2.4.3</version>
         </plugin>
         <plugin>
-          <groupId>com.atlassian.maven.plugins</groupId>
-          <artifactId>maven-clover2-plugin</artifactId>
-          <version>3.0.1</version>
-        </plugin>
-        <plugin>
           <groupId>org.codehaus.mojo</groupId>
           <artifactId>chronos-maven-plugin</artifactId>
           <version>1.0-SNAPSHOT</version>

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
Matterhorn mailing list
[email protected]
http://lists.opencastproject.org/mailman/listinfo/matterhorn


To unsubscribe please email
[email protected]
_______________________________________________

Reply via email to