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

Add audit for wasmtime-math #10059

Merged

Conversation

saulecabrera
Copy link
Member

I noticed that CI is failing given that an audit and policy for wasmtime-math is missing.

wasmtime-math was introduced in
https://github.com/bytecodealliance/wasmtime/pull/9808/files.

I followed a similar approach to what it's used for all the other wasmtime-* crates.

I noticed that CI is failing given that an audit and policy for
`wasmtime-math` is missing.

`wasmtime-math` was introduced in
https://github.com/bytecodealliance/wasmtime/pull/9808/files.

I followed a similar approach to what it's used for all the other
`wasmtime-*` crates.
@saulecabrera saulecabrera requested a review from a team as a code owner January 21, 2025 12:49
@saulecabrera saulecabrera requested review from abrown and alexcrichton and removed request for a team January 21, 2025 12:49
@abrown abrown added this pull request to the merge queue Jan 21, 2025
@alexcrichton
Copy link
Member

Thanks! I still don't know how to prevent this from happening when we add a new crate alas...

Merged via the queue into bytecodealliance:main with commit e3898f1 Jan 21, 2025
39 checks passed
alexcrichton pushed a commit to alexcrichton/wasmtime that referenced this pull request Jan 21, 2025
I noticed that CI is failing given that an audit and policy for
`wasmtime-math` is missing.

`wasmtime-math` was introduced in
https://github.com/bytecodealliance/wasmtime/pull/9808/files.

I followed a similar approach to what it's used for all the other
`wasmtime-*` crates.
alexcrichton added a commit that referenced this pull request Jan 21, 2025
* Add audit for `wasmtime-math` (#10059)

I noticed that CI is failing given that an audit and policy for
`wasmtime-math` is missing.

`wasmtime-math` was introduced in
https://github.com/bytecodealliance/wasmtime/pull/9808/files.

I followed a similar approach to what it's used for all the other
`wasmtime-*` crates.

* Fix a missing increment in p1-to-p2 adapter (#10064)

This commit fixes a bug in the WASIp1-to-WASIp2 adapter during
`fd_prestat_dir_name` where an iterator variable was forgotten to be
incremented. That means that getting the path for anything other than
the first preopen didn't work correctly.

Closes #10058

* Downgrade `wasip2` dep to 0.13.0

Compat with 1.81.0 MSRV

---------

Co-authored-by: Saúl Cabrera <saulecabrera@gmail.com>
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.

3 participants