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

Disable Automatic Activation of externref #4213

Closed
wants to merge 4 commits into from

Conversation

CryZe
Copy link
Contributor

@CryZe CryZe commented Oct 19, 2024

Starting with Rust 1.82 reference-types is always present in the target_features section. The problem is that JS bundlers like Webpack can't currently handle the reference-types proposal. So for now there should not be any automatic detection of the feature and instead it should explicitly be enabled by the user.

Fixes #4211

@CryZe CryZe force-pushed the webpack-rust-1-82-mess branch 2 times, most recently from 77ee2fd to a2c62f0 Compare October 19, 2024 20:13
Starting with Rust 1.82 `reference-types` is always present in the
`target_features` section. The problem is that JS bundlers like Webpack
can't currently handle the `reference-types` proposal. So for now there
should not be any automatic detection of the feature and instead it
should explicitly be enabled by the user.
@CryZe CryZe force-pushed the webpack-rust-1-82-mess branch from a2c62f0 to 37502b9 Compare October 19, 2024 20:16
@CryZe
Copy link
Contributor Author

CryZe commented Oct 19, 2024

Due to multivalue also being broken, my PR is rebased ontop of #4210 for now.

@daxpedda
Copy link
Collaborator

Replaced by #4235.

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.

Rust v1.82 enables reference type transformations by default
3 participants