You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by gr...@apache.org on 2018/05/07 21:00:28 UTC

kudu git commit: [java] Disable javadoc generation in Java 10+

Repository: kudu
Updated Branches:
  refs/heads/master 8cd7c05b6 -> 3d344ecef


[java] Disable javadoc generation in Java 10+

Javadoc is currently broken in Java 10+. This patch does
not fix javadocs but instead disables them when compiling
with Java 10 or higher.

We should revisit this in the future (perhaps when Java 11
is out) and hopefully  by then Java, Gradle, or Yetus will
have made this easy.

Change-Id: I9a5803dec97713ff564e7169da3825dbf3231806
Reviewed-on: http://gerrit.cloudera.org:8080/10326
Reviewed-by: Adar Dembo <ad...@cloudera.com>
Tested-by: Grant Henke <gr...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/3d344ece
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/3d344ece
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/3d344ece

Branch: refs/heads/master
Commit: 3d344ecefee7764955dca186e2624357faab69af
Parents: 8cd7c05
Author: Grant Henke <gr...@apache.org>
Authored: Mon May 7 10:55:14 2018 -0500
Committer: Grant Henke <gr...@apache.org>
Committed: Mon May 7 21:00:04 2018 +0000

----------------------------------------------------------------------
 java/gradle/docs.gradle | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/3d344ece/java/gradle/docs.gradle
----------------------------------------------------------------------
diff --git a/java/gradle/docs.gradle b/java/gradle/docs.gradle
index 19720b7..82060b1 100644
--- a/java/gradle/docs.gradle
+++ b/java/gradle/docs.gradle
@@ -30,11 +30,15 @@ dependencies {
 // in the projects build script have been applied
 afterEvaluate {
   tasks.withType(Javadoc) {
+    if(JavaVersion.current().isJava10Compatible()) {
+      // Disable Javadoc generation on Java 10+ until it is fixed.
+      enabled = false
+    }
     // Configure the Yetus doclet
     options.docletpath = configurations.doclet.files.asType(List)
     options.doclet = "org.apache.yetus.audience.tools.IncludePublicAnnotationsStandardDoclet"
     // Link to Java SE javadoc
-    options.links "https://docs.oracle.com/javase/7/docs/api/"
+    options.links "https://docs.oracle.com/javase/$javaCompatibility/docs/api/"
     // Hacky workaround to YETUS-627. Replace with options.noTimestamp
     // once this workaround is no longer needed.
     options.addStringOption("notimestamp", "org.apache.kudu")