ijuma commented on a change in pull request #10606:
URL: https://github.com/apache/kafka/pull/10606#discussion_r632062402



##########
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:
       > No, not at all. It only modifies exclusion level for a single 
dependency (referenced here as a module 'api-ldap-schema-data'). Isolated 
change, atomic level.
   
   You misunderstood my point. `compile` was also removed, so those lines may 
not be having any impact.




-- 
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


Reply via email to