On 6 March 2012 09:15, <simonetrip...@apache.org> wrote: > Author: simonetripodi > Date: Tue Mar 6 09:15:10 2012 > New Revision: 1297393 > > URL: http://svn.apache.org/viewvc?rev=1297393&view=rev > Log: > explicit use of the surefire-report plugin to aggregate test results > > Modified: > commons/proper/chain/trunk/pom.xml > Modified: commons/proper/chain/trunk/pom.xml > URL: > http://svn.apache.org/viewvc/commons/proper/chain/trunk/pom.xml?rev=1297393&r1=1297392&r2=1297393&view=diff > ============================================================================== > --- commons/proper/chain/trunk/pom.xml (original) > +++ commons/proper/chain/trunk/pom.xml Tue Mar 6 09:15:10 2012 > @@ -122,6 +122,8 @@ > <commons.jira.pid>12310462</commons.jira.pid> > <commons.osgi.import>!javax.portlet,*</commons.osgi.import> > <commons.osgi.dynamicImport>javax.portlet</commons.osgi.dynamicImport> > + > + <surefire.version>2.12</surefire.version>
Commons Parent 23 has <commons.surefire.version>2.9</commons.surefire.version> Would it work to override this instead? If so, I think the build/plugin section could be dropped. > </properties> > > <dependencyManagement> > @@ -169,12 +171,17 @@ > </execution> > </executions> > </plugin> > + > + <plugin> > + <groupId>org.apache.maven.plugins</groupId> > + <artifactId>maven-surefire-plugin</artifactId> > + <version>${surefire.version}</version> > + </plugin> Possibly not needed, see above. > </plugins> > </build> > > <reporting> > <plugins> > - > <plugin> > <groupId>org.apache.maven.plugins</groupId> > <artifactId>maven-changes-plugin</artifactId> > @@ -194,6 +201,15 @@ > > <plugin> > <groupId>org.apache.maven.plugins</groupId> > + <artifactId>maven-surefire-report-plugin</artifactId> > + <version>${surefire.version}</version> > + <configuration> > + <aggregate>true</aggregate> > + </configuration> The config section is not in CP23. Perhaps need to consider if it should be added? Should be done as a property, so can be overridden, but what would be the best default? > + </plugin> > + > + <plugin> > + <groupId>org.apache.maven.plugins</groupId> > <artifactId>maven-checkstyle-plugin</artifactId> > <version>2.9.1</version> > <configuration> > > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org