search

bytecodealliance / wasi-rs

Experimental WASI API bindings for Rust
Apache License 2.0
273 stars 48 forks source link

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

Closed sunfishcode closed 5 years ago

sunfishcode commented 5 years ago

See discussion in #9.

Fixes #9.

mati865 commented 5 years ago

Hey @sunfishcode.

Hate to ask on ETA but https://github.com/rust-lang/rust/pull/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).

mati865 commented 5 years ago

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

sunfishcode commented 5 years ago

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 commented 5 years ago

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

sunfishcode commented 5 years ago

Looks like it indeed doesn't like (: https://github.com/rust-lang-nursery/license-exprs/issues/3. But it also looks like WITH has a higher precedence than OR, so I'll try removing the parens.

sunfishcode commented 5 years ago

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