Stream: general

Topic: Cranelift-simplejit build issue


view this post on Zulip Graham Scheaffer (Jul 04 2020 at 20:46):

I've found an issue with building cranelift-simplejit, is a github issue with the cranelift bugreport template the best way to report it?

view this post on Zulip bjorn3 (Jul 04 2020 at 20:47):

What is the issue?

view this post on Zulip Graham Scheaffer (Jul 04 2020 at 20:48):

cranelift-simplejit specifies the dependency region = "2.1.0", but It only builds with version 2.2.0 (latest).

view this post on Zulip Graham Scheaffer (Jul 04 2020 at 20:49):

Using cranelift v0.65.0 btw

view this post on Zulip bjorn3 (Jul 04 2020 at 20:50):

:confused: I recently had to change the 2.0.0 requirement to 2.1.0: https://github.com/bytecodealliance/wasmtime/pull/1827 It is suprising that it already needs 2.2.0.

Using region 2.0.0 fails with: error[E0599]: no associated item named `READ_WRITE` found for struct `region::protect::Protection` in the current scope --> /home/bjorn/.cargo/git/checkouts/was...

view this post on Zulip bjorn3 (Jul 04 2020 at 20:51):

What is the exact error?

view this post on Zulip Graham Scheaffer (Jul 04 2020 at 20:52):

Here is the cargo output: http://ix.io/2qUQ

view this post on Zulip Graham Scheaffer (Jul 04 2020 at 20:53):

Seems to be a very similar error

view this post on Zulip bjorn3 (Jul 04 2020 at 20:53):

I am pretty sure I tested it with 2.1.0.

view this post on Zulip Graham Scheaffer (Jul 04 2020 at 20:55):

To test it I just made an emtpy cargo project and added the following dependencies:

cranelift-simplejit="0.65.0"
region = "=2.1.0"

view this post on Zulip Graham Scheaffer (Jul 04 2020 at 20:56):

It also happens with region version 2.1.2

view this post on Zulip bjorn3 (Jul 04 2020 at 20:58):

In that case could you please make a PR to update the region requirement to 2.2.0 everywhere? Both cranelift-sinplejit and the crate of wasmtime that uses region.

view this post on Zulip Graham Scheaffer (Jul 04 2020 at 20:58):

Yes, I can do that.

view this post on Zulip Graham Scheaffer (Jul 04 2020 at 21:03):

Are there tests I should run/add to?

view this post on Zulip Graham Scheaffer (Jul 04 2020 at 21:09):

I submitted the PR.

view this post on Zulip bjorn3 (Jul 04 2020 at 21:15):

I don't think a test is necessary.


Last updated: Nov 22 2024 at 17:03 UTC