-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
Comments
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>
I've added light-weight license checking to rust-analyzer itself s well: |
I think I won't be working on this directly myself in the nearest future:
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, |
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. |
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. |
Also mentioned on Zulip |
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
The text was updated successfully, but these errors were encountered: