You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kafka.apache.org by ij...@apache.org on 2018/04/12 17:26:09 UTC

[kafka] 01/04: MINOR: Java 9/10 fixes, gradle and minor deps update (#4725)

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

ijuma pushed a commit to branch 1.1
in repository https://gitbox.apache.org/repos/asf/kafka.git

commit 3a5d86e3d14d0c72750cf52b73c609bbd13c394a
Author: Ismael Juma <is...@juma.me.uk>
AuthorDate: Thu Mar 22 22:01:51 2018 -0700

    MINOR: Java 9/10 fixes, gradle and minor deps update (#4725)
    
    * Added dependencies so that Trogdor and Connect work with Java 9 and 10
    * Updated Jacoco to 0.8.1 so that it works with Java 10
    * Updated Gradle to 4.6
    * A few minor version bumps (not related to Java9/10 fixes)
    
    I tested manually that we can run ./gradlew test with Java 10
    after these changes. There are test failures as EasyMock
    and PowerMock will have to be updated to use a newer
    ASM version. But compiling successfully and most tests
    passing is progress. :)
    
    I also tested manually that Trogdor can be started with Java 10.
    It previously failed with a ClassNotFoundError.
    
    Reviewers: Jason Gustafson <ja...@confluent.io>
---
 build.gradle               | 10 +++++++---
 gradle/dependencies.gradle | 16 ++++++++++------
 2 files changed, 17 insertions(+), 9 deletions(-)

diff --git a/build.gradle b/build.gradle
index f4d1fb3..a152b1e 100644
--- a/build.gradle
+++ b/build.gradle
@@ -28,7 +28,7 @@ buildscript {
     classpath 'com.github.ben-manes:gradle-versions-plugin:0.17.0'
     classpath 'org.scoverage:gradle-scoverage:2.1.0'
     classpath 'com.github.jengelman.gradle.plugins:shadow:2.0.2'
-    classpath 'org.owasp:dependency-check-gradle:3.0.2'
+    classpath 'org.owasp:dependency-check-gradle:3.1.1'
   }
 }
 
@@ -79,7 +79,7 @@ allprojects {
 }
 
 ext {
-  gradleVersion = "4.5.1"
+  gradleVersion = "4.6"
   buildVersionFileName = "kafka-version.properties"
 
   maxPermSizeArgs = []
@@ -386,7 +386,7 @@ subprojects {
   if (it.path != ':core') {
 
     jacoco {
-      toolVersion = "0.8.0"
+      toolVersion = "0.8.1"
     }
 
     // NOTE: Jacoco Gradle plugin does not support "offline instrumentation" this means that classes mocked by PowerMock
@@ -874,6 +874,8 @@ project(':tools') {
 
     compile libs.jacksonJaxrsJsonProvider
     compile libs.jerseyContainerServlet
+    compile libs.jaxbApi // Jersey dependency that was available in the JDK before Java 9
+    compile libs.activation // Jersey dependency that was available in the JDK before Java 9
     compile libs.jettyServer
     compile libs.jettyServlet
     compile libs.jettyServlets
@@ -1263,6 +1265,8 @@ project(':connect:runtime') {
 
     compile libs.jacksonJaxrsJsonProvider
     compile libs.jerseyContainerServlet
+    compile libs.jaxbApi // Jersey dependency that was available in the JDK before Java 9
+    compile libs.activation // Jersey dependency that was available in the JDK before Java 9
     compile libs.jettyServer
     compile libs.jettyServlet
     compile libs.jettyServlets
diff --git a/gradle/dependencies.gradle b/gradle/dependencies.gradle
index d82ec4b..c4f80b7 100644
--- a/gradle/dependencies.gradle
+++ b/gradle/dependencies.gradle
@@ -47,17 +47,19 @@ if (hasProperty('scalaVersion')) {
 versions["baseScala"] = versions.scala.substring(0, versions.scala.lastIndexOf("."))
 
 versions += [
+  activation: "1.1.1",
   apacheda: "1.0.0",
   apacheds: "2.0.0-M24",
   argparse4j: "0.7.0",
-  bcpkix: "1.58",
+  bcpkix: "1.59",
   easymock: "3.5.1",
   jackson: "2.9.4",
   jetty: "9.2.24.v20180105",
   jersey: "2.25.1",
-  jmh: "1.19",
+  jmh: "1.20",
   log4j: "1.2.17",
-  scalaLogging: "3.7.2",
+  scalaLogging: "3.8.0",
+  jaxb: "2.3.0",
   jopt: "5.0.4",
   junit: "4.12",
   kafka_0100: "0.10.0.1",
@@ -65,23 +67,24 @@ versions += [
   kafka_0102: "0.10.2.1",
   kafka_0110: "0.11.0.2",
   kafka_10: "1.0.1",
-  lz4: "1.4",
+  lz4: "1.4.1",
   metrics: "2.2.0",
   // PowerMock 1.x doesn't support Java 9, so use PowerMock 2.0.0 beta
   powermock: "2.0.0-beta.5",
   reflections: "0.9.11",
   rocksDB: "5.7.3",
-  scalatest: "3.0.4",
+  scalatest: "3.0.5",
   scoverage: "1.3.1",
   slf4j: "1.7.25",
   snappy: "1.1.7.1",
   zkclient: "0.10",
   zookeeper: "3.4.10",
   jfreechart: "1.0.0",
-  mavenArtifact: "3.5.2"
+  mavenArtifact: "3.5.3"
 ]
 
 libs += [
+  activation: "javax.activation:activation:$versions.activation",
   argparse4j: "net.sourceforge.argparse4j:argparse4j:$versions.argparse4j",
   apacheda: "org.apache.directory.api:api-all:$versions.apacheda",
   apachedsCoreApi: "org.apache.directory.server:apacheds-core-api:$versions.apacheds",
@@ -96,6 +99,7 @@ libs += [
   easymock: "org.easymock:easymock:$versions.easymock",
   jacksonDatabind: "com.fasterxml.jackson.core:jackson-databind:$versions.jackson",
   jacksonJaxrsJsonProvider: "com.fasterxml.jackson.jaxrs:jackson-jaxrs-json-provider:$versions.jackson",
+  jaxbApi: "javax.xml.bind:jaxb-api:$versions.jaxb",
   jettyServer: "org.eclipse.jetty:jetty-server:$versions.jetty",
   jettyClient: "org.eclipse.jetty:jetty-client:$versions.jetty",
   jettyServlet: "org.eclipse.jetty:jetty-servlet:$versions.jetty",

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