You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@celeborn.apache.org by "FMX (via GitHub)" <gi...@apache.org> on 2024/06/11 12:52:33 UTC

[PR] [CELEBORN-1456][FOLLOWUP] Fix license issue [celeborn]

FMX opened a new pull request, #2556:
URL: https://github.com/apache/celeborn/pull/2556

   ### What changes were proposed in this pull request?
   Fix license issues.
   
   
   ### Why are the changes needed?
   Fix license and notice for release 0.5.0.
   
   
   ### Does this PR introduce _any_ user-facing change?
   NO.
   
   
   ### How was this patch tested?
   NO.
   


-- 
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: issues-unsubscribe@celeborn.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [CELEBORN-1456][FOLLOWUP] Fix license issue [celeborn]

Posted by "pan3793 (via GitHub)" <gi...@apache.org>.
pan3793 commented on code in PR #2556:
URL: https://github.com/apache/celeborn/pull/2556#discussion_r1634845746


##########
LICENSE-binary:
##########
@@ -317,11 +323,14 @@ org.fusesource.leveldbjni:leveldbjni-all
 
 MIT License
 ------------
+See licenses/LICENSE-ClassGraph.txt for details.
+io.github.classgraph:classgraph
 See licenses/LICENSE-jdktools.txt for details.
 com.github.olivergondza:maven-jdk-tools-wrapper
 See licenses/LICENSE-slf4j.txt for details.
 org.slf4j:jul-to-slf4j
 org.slf4j:slf4j-api
+

Review Comment:
   ```suggestion
   
   MOZILLA PUBLIC LICENSE (MPL) 1.1
   ```



-- 
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: issues-unsubscribe@celeborn.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [CELEBORN-1456][FOLLOWUP] Fix license issue [celeborn]

Posted by "FMX (via GitHub)" <gi...@apache.org>.
FMX commented on PR #2556:
URL: https://github.com/apache/celeborn/pull/2556#issuecomment-2161963306

   ping @pan3793 . How about now?


-- 
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: issues-unsubscribe@celeborn.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [CELEBORN-1456][FOLLOWUP] Fix license issue [celeborn]

Posted by "pan3793 (via GitHub)" <gi...@apache.org>.
pan3793 commented on code in PR #2556:
URL: https://github.com/apache/celeborn/pull/2556#discussion_r1634849284


##########
NOTICE-binary:
##########
@@ -195,3 +195,221 @@ distributed under the License is distributed on an "AS IS" BASIS,
 WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 See the License for the specific language governing permissions and
 limitations under the License.
+
+classgraph
+The MIT License (MIT)
+
+Copyright (c) 2022 Luke Hutchison
+
+Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the "Software"), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software.
+
+mimepool
+[//]: # " Copyright (c) 2018, 2022 Oracle and/or its affiliates. All rights reserved. "
+[//]: # "  "
+[//]: # " This program and the accompanying materials are made available under the "
+[//]: # " terms of the Eclipse Distribution License v. 1.0, which is available at "
+[//]: # " http://www.eclipse.org/org/documents/edl-v10.php. "
+[//]: # "  "
+[//]: # " SPDX-License-Identifier: BSD-3-Clause "
+
+# Notices for Eclipse Metro
+
+This content is produced and maintained by the Eclipse Metro project.
+
+* Project home: https://projects.eclipse.org/projects/ee4j.metro
+
+## Trademarks
+
+Eclipse Metro is a trademark of the Eclipse Foundation.
+
+## Copyright
+
+All content is the property of the respective authors or their employers. For
+more information regarding authorship of content, please consult the listed
+source code repository logs.
+
+## Declared Project Licenses
+
+This program and the accompanying materials are made available under the terms
+of the Eclipse Distribution License v1.0 which is available at
+https://www.eclipse.org/org/documents/edl-v10.php.
+
+SPDX-License-Identifier: BSD-3-Clause
+
+## Source Code
+
+The project maintains the following source code repositories:
+
+* https://github.com/eclipse-ee4j/metro-xmlstreambuffer
+* https://github.com/eclipse-ee4j/metro-policy
+* https://github.com/eclipse-ee4j/metro-wsit
+* https://github.com/eclipse-ee4j/metro-mimepull
+* https://github.com/eclipse-ee4j/metro-ws-test-harness
+* https://github.com/eclipse-ee4j/metro-package-rename-task
+* https://github.com/eclipse-ee4j/metro-jax-ws
+* https://github.com/eclipse-ee4j/metro-saaj
+* https://github.com/eclipse-ee4j/metro-jwsdp-samples
+* https://github.com/eclipse-ee4j/jax-rpc-ri
+* https://github.com/eclipse-ee4j/jaxr-ri
+
+## Third-party Content
+
+This project leverages the following third party content.
+
+addressing.xml Version: 2004/10 (n/a)
+
+* License: W3C
+* Project: https://www.w3.org/Submission/ws-addressing/
+* Source: http://schemas.xmlsoap.org/ws/2004/08/addressing/
+
+ant-launcher (1.10.2)
+
+* License: Apache-2.0 AND SAX-PD AND W3C
+* Project: https://ant.apache.org/
+* Source:
+   http://central.maven.org/maven2/org/apache/ant/ant-launcher/1.10.2/ant-launcher-1.10.2-sources.jar
+
+Apache Ant (1.6)
+
+* License: Apache-1.1
+* Project: https://ant.apache.org/
+* Source: https://repo1.maven.org/maven2/ant/ant/1.6/ant-1.6-sources.jar
+
+Apache Ant (1.10.2)
+
+* License: Apache-2.0 AND W3C AND LicenseRef-Public-Domain
+
+commons-logging (1.1.2)
+
+* License: Apache-2.0
+* Project: https://commons.apache.org/proper/commons-logging/
+* Source:
+   http://central.maven.org/maven2/commons-logging/commons-logging/1.1.2/commons-logging-1.1.2-sources.jar
+
+JUnit (4.12)

Review Comment:
   where does this part come from?
   
   the NOTICE content should be analyzed instead of copying blindly ...



-- 
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: issues-unsubscribe@celeborn.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [CELEBORN-1456][FOLLOWUP] Fix license issue [celeborn]

Posted by "FMX (via GitHub)" <gi...@apache.org>.
FMX closed pull request #2556: [CELEBORN-1456][FOLLOWUP] Fix license issue
URL: https://github.com/apache/celeborn/pull/2556


-- 
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: issues-unsubscribe@celeborn.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org