dejan2609 commented on a change in pull request #10606:
URL: https://github.com/apache/kafka/pull/10606#discussion_r632083522
##########
File path: build.gradle
##########
@@ -849,7 +849,7 @@ project(':core') {
compile.exclude module: 'mail'
// To prevent a UniqueResourceException due the same resource existing in
both
// org.apache.directory.api/api-all and
org.apache.directory.api/api-ldap-schema-data
- testCompile.exclude module: 'api-ldap-schema-data'
+ all*.exclude module: 'api-ldap-schema-data'
Review comment:
> You misunderstood my point. compile was also removed, so those lines
may not be having any impact.
Aha, I see now.... I was focused on `testCompile` only... You have a point,
`compile` is removed in Gradle 7 also and given a fact that no error is thrown
I assume that Gradle 7 just skips in silence (at the surface, at least) this
entire block of exclusions:
https://github.com/apache/kafka/blob/ebb1d6e21cc9213071ee1c6a15ec3411fc215b81/build.gradle#L818
Well, this could really be a Gradle regression; I filed a ticket here:
https://github.com/gradle/gradle/issues/17187 so I will just update it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
