adoroszlai commented on code in PR #9909:
URL: https://github.com/apache/ozone/pull/9909#discussion_r2920448960


##########
hadoop-ozone/dist/src/main/license/jar-report.txt:
##########
@@ -158,6 +161,7 @@ share/ozone/lib/jsch.jar
 share/ozone/lib/json-simple.jar
 share/ozone/lib/jsp-api.jar
 share/ozone/lib/jspecify.jar
+share/ozone/lib/jsr305.jar

Review Comment:
   Please add exclusion for `jsr305`, since it has incompatible license.
   
   https://apache.org/legal/resolved.html#category-x



##########
hadoop-ozone/dist/src/main/license/jar-report.txt:
##########
@@ -106,11 +105,15 @@ share/ozone/lib/jakarta.validation-api.jar
 share/ozone/lib/jakarta.ws.rs-api.jar
 share/ozone/lib/jakarta.xml.bind-api.jar
 share/ozone/lib/javassist-GA.jar
+share/ozone/lib/javax.activation-api.jar

Review Comment:
   Please add exclusion for `javax.activation-api`, we use 
`jakarta.activation-api` (which includes the same classes).



##########
hadoop-ozone/dist/src/main/license/bin/LICENSE.txt:
##########
@@ -246,15 +246,19 @@ EPL 2.0
 CDDL 1.1 + GPLv2 with classpath exception
 =====================
 
+   com.sun.xml.bind:jaxb-core
+   com.sun.xml.bind:jaxb-impl
    com.sun.istack:istack-commons-runtime
    com.sun.jersey:jersey-client
    com.sun.jersey:jersey-core
+   javax.activation:javax.activation-api
    javax.annotation:javax.annotation-api
    javax.el:javax.el-api
    javax.interceptor:javax.interceptor-api
    javax.servlet:javax.servlet-api
    javax.servlet.jsp:jsp-api
    javax.ws.rs:jsr311-api
+   javax.xml.bind:jaxb-api

Review Comment:
   Please exclude this, we already have `jakarta.xml.bind-api`, which has the 
same classes.



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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to