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

Change license to (Apache-2.0 WITH LLVM-exception) OR Apache-2.0 OR MIT #10

Merged
merged 1 commit into from
Aug 29, 2019

Conversation

sunfishcode
Copy link
Member

See discussion in #9.

Fixes #9.

@mati865
Copy link

mati865 commented Aug 29, 2019

Hey @sunfishcode.

Hate to ask on ETA but rust-lang/rust#63806 is waiting on relicensed version of wasi or adding wasi to the exception list.

If you want more time it's fine, just let me know so I'll split libc update from that PR (to fix bug).

@sunfishcode sunfishcode merged commit 8430279 into master Aug 29, 2019
@mati865
Copy link

mati865 commented Aug 29, 2019

Thank you for merging it. Do you mind publishing new release on crates.io?

@sunfishcode sunfishcode deleted the license branch August 29, 2019 15:08
@sunfishcode
Copy link
Member Author

cargo publish is failing, with this message:

error: api errors (status 200 OK): unknown license or other term: (Apache-2.0; see http://opensource.org/licenses for options, and http://spdx.org/licenses/ for their identifiers

At first glance, this looks like a bug in cargo, because "Apache-2.0" is a valid license identifier. Anyone have any ideas?

@mati865
Copy link

mati865 commented Aug 29, 2019

No idea, maybe it doesn't like ( before license, I'll check the code soon.

@sunfishcode
Copy link
Member Author

Looks like it indeed doesn't like (: ehuss/license-exprs#3. But it also looks like WITH has a higher precedence than OR, so I'll try removing the parens.

@sunfishcode
Copy link
Member Author

That worked! 0.6.0 is up, with the new license.

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.

License
2 participants