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

validate rust-analyzer licenses #74269

Closed
ehuss opened this issue Jul 12, 2020 · 5 comments
Closed

validate rust-analyzer licenses #74269

ehuss opened this issue Jul 12, 2020 · 5 comments
Labels
A-licensing Area: Compiler licensing A-meta Area: Issues & PRs about the rust-lang/rust repository itself T-leadership-council Relevant to the leadership council.

Comments

@ehuss
Copy link
Contributor

ehuss commented Jul 12, 2020

License validation (https://github.com/rust-lang/rust/blob/master/src/tools/tidy/src/deps.rs) for rust-analyzer is currently not being done. I think it would probably be good to include it in the license checks, just to double-check that something inappropriate doesn't slip in.

I manually checked them, and they seem ok at this point. It would be helpful if the ra crates themselves included a license field in their Cargo.toml for automatic validation.

cc @matklad

bors bot added a commit to rust-lang/rust-analyzer that referenced this issue Jul 14, 2020
5355: Add a license field to all the crates r=matklad a=JohnTitor

Some are unnecessary but it's okay to have it, I think.
cc rust-lang/rust#74269

Co-authored-by: Yuki Okushi <huyuumi.dev@gmail.com>
@matklad
Copy link
Member

matklad commented Jul 14, 2020

I've added light-weight license checking to rust-analyzer itself s well:

https://github.com/rust-analyzer/rust-analyzer/pull/5376/files#diff-7e4576ca6f0e41b02d412c4745763e64R55-R73

@matklad
Copy link
Member

matklad commented Jul 23, 2020

I think I won't be working on this directly myself in the nearest future:

  1. we do check licenses in rust-analyzer repo
  2. rust-analyzer is somewhat provisional at the moment
  3. rust-analyzer is a separate workspace, so adding to license check means some "interesting" build system work

That is, it seems relatively low-priority, and with a high chance that something will be changed in the future.

I will, however, try to reduce the set of licenses used by rust-analyzer to, preferably, { "MIT OR Apache-2.0" }.

@ehuss
Copy link
Contributor Author

ehuss commented Jul 23, 2020

No worries, I don't think there is a rush.

Assuming some point in the future, if rust analyzer is added to the workspace and shares Cargo.lock, it will automatically get the license checks. I'm not sure if sharing the lock is feasible, but it would cut down on some compile times to share common dependencies.

@camelid camelid added A-meta Area: Issues & PRs about the rust-lang/rust repository itself T-core Relevant to the core team, which will review and decide on the PR/issue. labels Oct 20, 2020
@pietroalbini pietroalbini added the A-licensing Area: Compiler licensing label Jun 15, 2022
@ehuss ehuss added T-leadership-council Relevant to the leadership council. and removed T-core Relevant to the core team, which will review and decide on the PR/issue. labels Mar 20, 2025
@ehuss
Copy link
Contributor Author

ehuss commented Mar 20, 2025

I'm going to close as effectively resolved by #115274.

If we are not honoring any of the licenses correctly, then we can follow up in rust-lang/leadership-council#24 regarding that.

@ehuss ehuss closed this as completed Mar 20, 2025
@apiraino
Copy link
Contributor

Also mentioned on Zulip

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-licensing Area: Compiler licensing A-meta Area: Issues & PRs about the rust-lang/rust repository itself T-leadership-council Relevant to the leadership council.
Projects
None yet
Development

No branches or pull requests

5 participants