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

Code under the 2-clause-BSD license is not documented as a compatible license for the Rust runtime #62620

Closed
gnzlbg opened this issue Jul 12, 2019 · 4 comments
Labels
A-testsuite Area: The testsuite used to check the correctness of rustc C-bug Category: This is a bug.

Comments

@gnzlbg
Copy link
Contributor

gnzlbg commented Jul 12, 2019

The 2-clause-BSD license is not documented in the tidy script as being a compatible license for the rust runtime dependencies. We used to ship BSD licensed code as part of libstd until recently: 0b85b64#diff-4131508972c86fdd799a5f7d41fdd323 and we shipped jemalloc for a long time as part of the runtime on many targets.

We still do ship code with the MPL2.0 license in some targets, see: https://github.com/rust-lang/rust/blob/master/src/tools/tidy/src/deps.rs#L51

The deps check of the tidy script should be better documented to explain what's allowed and what isn't.

@jonas-schievink jonas-schievink added A-testsuite Area: The testsuite used to check the correctness of rustc C-bug Category: This is a bug. labels Jul 12, 2019
@Mark-Simulacrum
Copy link
Member

We discussed this a bit today and concluded that we don't need the SGX dependencies to be appropriately licensed (they're somewhat "third party") so I'm closing this issue.

@gnzlbg
Copy link
Contributor Author

gnzlbg commented Jul 17, 2019

So does that mean that it is ok to add third party libraries to the runtime under this license?

If so the documentation comment in the tidy script should be updated.

@Mark-Simulacrum
Copy link
Member

Not in general. In this case, it's specifically for SGX so that's why we're not concerned about it.

@gnzlbg
Copy link
Contributor Author

gnzlbg commented Jul 17, 2019 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-testsuite Area: The testsuite used to check the correctness of rustc C-bug Category: This is a bug.
Projects
None yet
Development

No branches or pull requests

3 participants