Skip to content

Correct the license to that of upstream compiler-rt #98

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

Merged
merged 1 commit into from
Oct 13, 2016

Conversation

brson
Copy link
Contributor

@brson brson commented Oct 12, 2016

As this is a derivative work we must maintain the same license terms.

This changes the license to NCSA or MIT instead of Apache 2 or MIT. Since @japaric, @mattico, and @Amanieu have not contributed their work under NCSA they need to sign off on this PR before merging.

Alternately we can simply drop Apache 2 and continue with solely MIT, but that restricts any contributions back upstream, which are unlikely, but could theoretically happen. Better to be compatible if we can.

As this is a derivative work we must maintain the same license terms.
@mattico
Copy link
Contributor

mattico commented Oct 12, 2016

👍 I meant to bring this up earlier.

I agree to relicense my contributions under the MIT+NCSA license with no other restrictions or conditions.

@japaric
Copy link
Member

japaric commented Oct 12, 2016

I agree to relicense my contributions under the MIT+NCSA license with no other restrictions or conditions.

1 similar comment
@Amanieu
Copy link
Member

Amanieu commented Oct 13, 2016

I agree to relicense my contributions under the MIT+NCSA license with no other restrictions or conditions.

@japaric
Copy link
Member

japaric commented Oct 13, 2016

@alexcrichton also needs to sign off

@alexcrichton
Copy link
Member

I agree to relicense my contributions under the MIT+NCSA license with no other restrictions or conditions.

@japaric japaric merged commit 1221ff3 into rust-lang:master Oct 13, 2016
@japaric
Copy link
Member

japaric commented Oct 13, 2016

Thank you all!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants