mirror of https://github.com/apache/kafka.git
MINOR: srcJar should depend on processMessages task (#13316)
This fixes the following `./gradlew install` issue: ```text * What went wrong: A problem was found with the configuration of task ':storage:srcJar' (type 'Jar'). - Gradle detected a problem with the following location: '/Users/ijuma/src/kafka/storage/src/generated/java'. Reason: Task ':storage:srcJar' uses this output of task ':storage:processMessages' without declaring an explicit or implicit dependency. This can lead to incorrect results being produced, depending on what order the tasks are executed. Possible solutions: 1. Declare task ':storage:processMessages' as an input of ':storage:srcJar'. 2. Declare an explicit dependency on ':storage:processMessages' from ':storage:srcJar' using Task#dependsOn. 3. Declare an explicit dependency on ':storage:processMessages' from ':storage:srcJar' using Task#mustRunAfter. Please refer to https://docs.gradle.org/8.0.1/userguide/validation_problems.html#implicit_dependency for more details about this problem. ``` Reviewers: David Jacot <david.jacot@gmail.com>
This commit is contained in:
parent
a1b8586a57
commit
dcc1799951
|
@ -982,6 +982,7 @@ project(':core') {
|
|||
}
|
||||
|
||||
compileJava.dependsOn 'processMessages'
|
||||
srcJar.dependsOn 'processMessages'
|
||||
|
||||
task genProtocolErrorDocs(type: JavaExec) {
|
||||
classpath = sourceSets.main.runtimeClasspath
|
||||
|
@ -1213,6 +1214,7 @@ project(':metadata') {
|
|||
}
|
||||
|
||||
compileJava.dependsOn 'processMessages'
|
||||
srcJar.dependsOn 'processMessages'
|
||||
|
||||
sourceSets {
|
||||
main {
|
||||
|
@ -1287,6 +1289,7 @@ project(':group-coordinator') {
|
|||
}
|
||||
|
||||
compileJava.dependsOn 'processMessages'
|
||||
srcJar.dependsOn 'processMessages'
|
||||
}
|
||||
|
||||
project(':examples') {
|
||||
|
@ -1425,6 +1428,7 @@ project(':clients') {
|
|||
}
|
||||
|
||||
compileJava.dependsOn 'processMessages'
|
||||
srcJar.dependsOn 'processMessages'
|
||||
|
||||
compileTestJava.dependsOn 'processTestMessages'
|
||||
|
||||
|
@ -1527,6 +1531,7 @@ project(':raft') {
|
|||
}
|
||||
|
||||
compileJava.dependsOn 'processMessages'
|
||||
srcJar.dependsOn 'processMessages'
|
||||
|
||||
jar {
|
||||
dependsOn createVersionFile
|
||||
|
@ -1749,6 +1754,7 @@ project(':storage') {
|
|||
}
|
||||
|
||||
compileJava.dependsOn 'processMessages'
|
||||
srcJar.dependsOn 'processMessages'
|
||||
|
||||
jar {
|
||||
dependsOn createVersionFile
|
||||
|
@ -1987,6 +1993,7 @@ project(':streams') {
|
|||
}
|
||||
|
||||
compileJava.dependsOn 'processMessages'
|
||||
srcJar.dependsOn 'processMessages'
|
||||
|
||||
javadoc {
|
||||
include "**/org/apache/kafka/streams/**"
|
||||
|
|
Loading…
Reference in New Issue