Author: gk
Date: Fri Jun 12 14:29:05 2020
New Revision: 1878775
URL: http://svn.apache.org/viewvc?rev=1878775&view=rev
Log:
- update org.owasp dependency-check-maven plugin to 5.3.3, log4j2 to 2.13.3
- add cobertura profile to include if needed, activate jacoco by default
- add java12 profile
Modified:
turbine/maven/turbine-parent/trunk/README.txt
turbine/maven/turbine-parent/trunk/pom.xml
Modified: turbine/maven/turbine-parent/trunk/README.txt
URL:
http://svn.apache.org/viewvc/turbine/maven/turbine-parent/trunk/README.txt?rev=1878775&r1=1878774&r2=1878775&view=diff
==============================================================================
--- turbine/maven/turbine-parent/trunk/README.txt (original)
+++ turbine/maven/turbine-parent/trunk/README.txt Fri Jun 12 14:29:05 2020
@@ -36,3 +36,5 @@ CAVEAT
RELEASE
- Normally, this module could be released lazily.
+
+- Java 8: include explicitely cobertura if needed: mvn clean site install
-Papache-release,cobertura
Modified: turbine/maven/turbine-parent/trunk/pom.xml
URL:
http://svn.apache.org/viewvc/turbine/maven/turbine-parent/trunk/pom.xml?rev=1878775&r1=1878774&r2=1878775&view=diff
==============================================================================
--- turbine/maven/turbine-parent/trunk/pom.xml (original)
+++ turbine/maven/turbine-parent/trunk/pom.xml Fri Jun 12 14:29:05 2020
@@ -331,13 +331,7 @@
<artifactId>maven-surefire-report-plugin</artifactId>
<version>${turbine.surefire.version}</version>
</plugin>
- <!-- get test coverage for regression tests -->
- <!-- [WARN] JavaNCSS got an error while parsing the java file containing
an ERROR ParseException, which is caused by Java8 syntax - reports are still
generated - and it's already a registered issue:
https://github.com/cobertura/cobertura/issues/176 -->
- <plugin>
- <groupId>org.codehaus.mojo</groupId>
- <artifactId>cobertura-maven-plugin</artifactId>
- <version>2.7</version>
- </plugin>
+ <!-- using jacoco since java 8 to get test coverage for regression tests
-->
<plugin>
<groupId>org.jacoco</groupId>
<artifactId>jacoco-maven-plugin</artifactId>
@@ -429,7 +423,7 @@
<plugin>
<groupId>org.owasp</groupId>
<artifactId>dependency-check-maven</artifactId>
- <version>5.1.1</version><!--
+ <version>5.3.2</version><!--
- find all configuration parameters here:
https://jeremylong.github.io/DependencyCheck/dependency-check-maven/.
- if process hangs up, check or remove all files including
odc.mv.db in maven repo in folder org\owasp\dependency-check-data\4.0\.
- requires mvn version > 3.3! For older version try to check v
3.2.1 or 3.1.2 -->
@@ -448,13 +442,13 @@
</suppressionFiles-->
</configuration>
</plugin>
- <!-- jacoco is skipped by default -->
+ <!-- jacoco is since java 8 enabled by default -->
<plugin>
<groupId>org.jacoco</groupId>
<artifactId>jacoco-maven-plugin</artifactId>
<version>${turbine.jacoco.version}</version>
<configuration>
- <skip>${turbine.jacoco.skip}</skip>
+ <skip>${jacoco.skip}</skip>
<excludes>
<exclude>*/*Test*</exclude>
</excludes>
@@ -516,17 +510,54 @@
</plugins>
</build>
</profile>
+ <!-- usng profile to exclude if required, java 12 incompatible -->
+ <profile>
+ <id>cobertura</id>
+ <reporting>
+ <plugins>
+ <!-- [WARN] JavaNCSS got an error while parsing the java file
containing an ERROR ParseException, which is caused by Java8 syntax - reports
are still generated - and it's already a registered issue:
https://github.com/cobertura/cobertura/issues/176 -->
+ <plugin>
+ <groupId>org.codehaus.mojo</groupId>
+ <artifactId>cobertura-maven-plugin</artifactId>
+ <version>2.7</version>
+ <!--reportSets>
+ <reportSet>
+ <reports>
+ <report>cobertura</report>
+ </reports>
+ </reportSet>
+ </reportSets-->
+ </plugin>
+ </plugins>
+ </reporting>
+ <properties>
+ <cobertura.skip>false</cobertura.skip>
+ </properties>
+ </profile>
<!-- ignore doclint errors (should be warnings) for site generation, if
using java 8 or above -->
<!-- Use java 8 compatible version of FindBugs -->
<profile>
<id>java8</id>
<activation>
- <jdk>[1.8,)</jdk>
+ <jdk>8</jdk>
</activation>
<properties>
<doclint>none</doclint><!-- since javadoc v.3 this is required
instead of -Xdoclint:none -->
</properties>
</profile>
+ <profile>
+ <id>java12</id>
+ <activation>
+ <jdk>[12,)</jdk>
+ </activation>
+ <properties>
+ <java.version>12</java.version>
+ <!-- maven.compiler.* is now different from compilerVersion,
which requires to fork -->
+ <turbine.compiler.fork>true</turbine.compiler.fork>
+
<turbine.compiler.compilerVersion>1.12</turbine.compiler.compilerVersion>
+ <maven.compiler.release>8</maven.compiler.release>
+ </properties>
+ </profile>
</profiles>
@@ -557,8 +588,9 @@
<turbine.findbugs.version>3.0.5</turbine.findbugs.version>
<turbine.jacoco.version>0.8.5</turbine.jacoco.version>
<!-- may replace local settings -->
- <turbine.log4j2.version>2.13.0</turbine.log4j2.version>
- <turbine.jacoco.skip>true</turbine.jacoco.skip>
+ <turbine.log4j2.version>2.13.3</turbine.log4j2.version>
+ <jacoco.skip>false</jacoco.skip>
+ <cobertura.skip>true</cobertura.skip>
<!--
Encoding of Java source files: Make sure, that the compiler and