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

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


The following commit(s) were added to refs/heads/2.8 by this push:
     new e978f72  MINOR: fix aggregatedJavadoc dep on compileJava (#10386)
e978f72 is described below

commit e978f72b0f43b2ae24a75a82ea9f0a9af3472745
Author: John Roesler <[email protected]>
AuthorDate: Wed Mar 24 14:18:05 2021 -0500

    MINOR: fix aggregatedJavadoc dep on compileJava (#10386)
    
    The root gradle project depends on the `java` plugin, but we were only
    applying the plugin explicitly to subprojects. We didn't notice because
    when on a git fork, we run the `rat` plugin, which transitively applies the
    `java` plugin to the root project.
    
    This fixes the problem by explicitly applying the `java` plugin to the root
    and all subprojects. See also 
https://github.com/apache/kafka/pull/10394/files
    for fixes in later branches.
    
    Reviewers: Chia-Ping Tsai <[email protected]>, Alexandre Dupriez 
<[email protected]>, Ismael Juma <[email protected]>, Guozhang Wang 
<[email protected]>
---
 build.gradle | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/build.gradle b/build.gradle
index 1648605..29b3a74 100644
--- a/build.gradle
+++ b/build.gradle
@@ -59,6 +59,7 @@ allprojects {
   apply plugin: 'idea'
   apply plugin: 'org.owasp.dependencycheck'
   apply plugin: 'com.github.ben-manes.versions'
+  apply plugin: 'java'
 
   dependencyUpdates {
     revision="release"
@@ -167,7 +168,6 @@ if (file('.git').exists()) {
   }
 }
 
-
 subprojects {
 
   // enable running :dependencies task recursively on all subprojects
@@ -177,7 +177,6 @@ subprojects {
   // eg: ./gradlew allDepInsight --configuration runtime --dependency 
com.fasterxml.jackson.core:jackson-databind
   task allDepInsight(type: DependencyInsightReportTask) doLast {}
 
-  apply plugin: 'java'
   // apply the eclipse plugin only to subprojects that hold code. 'connect' is 
just a folder.
   if (!project.name.equals('connect')) {
     apply plugin: 'eclipse'

Reply via email to