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

ci: Freeze rustc for nightly web builds to 1.81.0 #18397

Merged
merged 1 commit into from
Oct 27, 2024

Conversation

danielhjacobs
Copy link
Contributor

@danielhjacobs danielhjacobs commented Oct 27, 2024

Note: Tested locally (in Pale Moon), but not via CI.

Rust 1.82.0 enables the reference-types and multivalue target features for the WebAssembly target by default: https://releases.rs/docs/1.82.0/#compatibility-notes. wasm-bindgen 0.2.94+ enables reference type and multivalue transformations if the module already makes use of the corresponding target features. The reference-types WebAssembly extension is not available in Safari 14.1, which we claim to support: https://caniuse.com/wasm-reference-types. It also, from testing, appears to not work in Pale Moon. Additionally, there seems to be some issues with support in Safari 15, which should support the reference-types extension but may have some bugs (which I don't expect will ever be fixed since Safari 15 isn't really supported anymore).

Hopefully, this can be reverted after a wasm-bindgen release with rustwasm/wasm-bindgen#4213 is merged.

@n0samu
Copy link
Member

n0samu commented Oct 27, 2024

Should fix #18386

@danielhjacobs danielhjacobs merged commit c8ded2e into ruffle-rs:master Oct 27, 2024
17 checks passed
@torokati44 torokati44 added A-build Area: Build scripts & CI T-chore Type: Chore (like updating a dependency, it's gotta be done) labels Oct 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-build Area: Build scripts & CI T-chore Type: Chore (like updating a dependency, it's gotta be done)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants