Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Enhancement] Remove artifacts with incompatible licenses from the classpath #4842

Open
6 of 8 tasks
Pil0tXia opened this issue Apr 15, 2024 · 2 comments
Open
6 of 8 tasks
Labels
dependencies Pull requests that update a dependency file enhancement New feature or request help wanted Extra attention is needed license

Comments

@Pil0tXia
Copy link
Member

Pil0tXia commented Apr 15, 2024

Search before asking

  • I had searched in the issues and found no similar issues.

Enhancement Request

According to ASF 3RD PARTY LICENSE POLICY, artifacts with specific licenses may not be distributed with EventMesh binary.

Artifacts licensed under multiple licenses and one of these licenses compatible with Apache 2.0 can be excluded from this issue's task.

Describe the solution you'd like

Run generateDistLicense Gradle task (will be merged in #4831) to collect all artifacts' licenses, and remove artifacts with incompatible licenses from the classpath.

Personal summary of license violations (There may be omissions, feel free to add):

LGPL-2.1-only

  • hibernate-commons-annotations (within JDBC Connector)
  • hibernate-core (within JDBC Connector)

The GNU General Public License, v2 with Universal FOSS Exception, v1.0

  • mysql-connector-j (within JDBC Connector)

Solution examples:

BSD-4-Clause

  • ST4 (within JDBC Connector) The https://www.antlr.org/license.html is actually the BSD-3-Clause license.
  • xsdlib (within DingTalk Connector, com.aliyun:tea-xml artifact)

Related PRs

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@Pil0tXia Pil0tXia added enhancement New feature or request good first issue Issues for first-time contributors labels Apr 15, 2024
@Pil0tXia
Copy link
Member Author

@mxsm @yanrongzhen May you please help take a look? I'm not sure what the impact of excluding them would be.

@mxsm
Copy link
Member

mxsm commented Apr 15, 2024

@mxsm @yanrongzhen May you please help take a look? I'm not sure what the impact of excluding them would be.

I will

@Pil0tXia Pil0tXia added dependencies Pull requests that update a dependency file license and removed good first issue Issues for first-time contributors labels Apr 16, 2024
mxsm added a commit to mxsm/incubator-eventmesh that referenced this issue Apr 24, 2024
mxsm added a commit to mxsm/incubator-eventmesh that referenced this issue May 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file enhancement New feature or request help wanted Extra attention is needed license
Projects
None yet
Development

No branches or pull requests

2 participants