-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Fix issue 1244 #1245
Fix issue 1244 #1245
Conversation
This might happen because of LTO. Fixes rustwasm#1244
whelp |
Looks like a bug in webpack/webassembly-js: https://travis-ci.com/rustwasm/wasm-bindgen/jobs/177248729#L859-L895
|
+cc @xtuc ^ Going to merge this. |
This looks familiar to me, i'll take a look at it. |
Thanks Sven!
|
Did this get merged into |
@lrlna this should be published yeah, but you may be running into #1267 which is unfortunately still a bug in webpack from what I can tell. It should be fixed by xtuc/webassemblyjs#493 purportedly but I'm trying to figure out how those changes can propagate out to us |
Hmmm, possibly! The error I get though is the same as above. But! It seems y'all got a fix in for 1267 (yay!), how do I test it out? |
@lrlna in theory, if it's the same issue, you can just make sure that |
No description provided.