This is an automated email from the ASF dual-hosted git repository.

sijie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/bookkeeper.git


The following commit(s) were added to refs/heads/master by this push:
     new bd69dbd  Upgrade JaCoCo and fix code coverage build system
bd69dbd is described below

commit bd69dbdb8d651d6ae482c0bb02e927b3a89fc796
Author: Enrico Olivelli <eolive...@apache.org>
AuthorDate: Fri Feb 23 01:20:29 2018 -0800

    Upgrade JaCoCo and fix code coverage build system
    
    - Upgrade JaCoCo to version 0.8.0 which supports Java 9
    - Exclude generated sources from the NAR-plugin files
    - Update surefire configuration in order to not fail the build while using 
-Pcode-coverage
    
    Author: Enrico Olivelli <eolive...@apache.org>
    Author: eolivelli <eolive...@apache.org>
    
    Reviewers: Sijie Guo <si...@apache.org>
    
    This closes #1129 from eolivelli/code-coverage-2
---
 circe-checksum/pom.xml | 11 +++++++++++
 pom.xml                |  4 +++-
 2 files changed, 14 insertions(+), 1 deletion(-)

diff --git a/circe-checksum/pom.xml b/circe-checksum/pom.xml
index 9b4f084..744e4f0 100644
--- a/circe-checksum/pom.xml
+++ b/circe-checksum/pom.xml
@@ -134,6 +134,17 @@
           </execution>
         </executions>
       </plugin>
+      <plugin>
+        <groupId>org.jacoco</groupId>
+        <artifactId>jacoco-maven-plugin</artifactId>
+        <version>0.8.0</version>
+        <configuration>
+            <excludes>
+                <!-- this class is generated -->
+                <exclude>com/scurrilous/circe/checksum/NarSystem*</exclude>
+            </excludes>
+        </configuration>
+       </plugin>
     </plugins>
   </build>
 
diff --git a/pom.xml b/pom.xml
index 08b5b50..fac6f13 100644
--- a/pom.xml
+++ b/pom.xml
@@ -412,12 +412,14 @@
                  <redirectTestOutputToFile>true</redirectTestOutputToFile>
                  <reuseForks>false</reuseForks>
                  
<forkedProcessTimeoutInSeconds>1800</forkedProcessTimeoutInSeconds>
+                  <!-- we want build to complete even in case of failures -->
+                  <testFailureIgnore>true</testFailureIgnore>
                 </configuration>
              </plugin>
              <plugin>
                <groupId>org.jacoco</groupId>
                <artifactId>jacoco-maven-plugin</artifactId>
-               <version>0.7.9</version>
+               <version>0.8.0</version>
                <executions>
                  <execution>
                    <goals>

-- 
To stop receiving notification emails like this one, please contact
si...@apache.org.

Reply via email to