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

Reply via email to