Compatibility

[tinywombat765]

It seems that this only builds on asmjs-unkown-emscripten in debug mode

[therustmonk]

Thank you for this issue report. Right, It doesn't work for every target, but only for asmjs debug. While there are cases when even asmjs fails, like this one: rust-lang/rust#46730
I'm going to do it compatible with all targets. With your permission I will use this issue for tracking support of:

• asmjs-unkown-emscripten - debug
• asmjs-unkown-emscripten - release
• wasm32-unknown-emscripten - debug
• wasm32-unknown-emscripten - release
• wasm32-unknown-unknown - debug
• wasm32-unknown-unknown - release
• add checks to TravisCI

Related: koute/stdweb#36

[tinywombat765]

Go for it.

…

On Mon, Dec 25, 2017, 2:42 AM Denis Kolodin ***@***.***> wrote: Thank you for this issue report. Right, It doesn't work for every target, but only for asmjs debug. While there are cases when even asmjs fails, like this one: rust-lang/rust#46730 <rust-lang/rust#46730> I'm going to do it compatible with all targets. With your permission I will use this issue for tracking support of: - asmjs-unkown-emscripten - debug - asmjs-unkown-emscripten - release - wasm32-unknown-emscripten - debug - wasm32-unknown-emscripten - release - wasm32-unknown-unknown - debug - wasm32-unknown-unknown - release - add checks to TravisCI Related: koute/stdweb#36 <koute/stdweb#36> — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#6 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AFwTPseqmyEQli7Jvbkm291WYFt6cI0Iks5tD1HOgaJpZM4RMBZn> .

[koute]

The wasm32-unknown-unknown builds work (FYI, those are all release builds as rustc itself isn't able to properly do a debug build for that target yet), as do all of the debug *-emscripten ones. The Emscripten release builds check the generated .js file's syntax, which is why they fail.

You need to change this:

let callback = @{callback};

into this:

var callback = @{callback};

in html.rs#48

[therustmonk]

@koute Thank you for the explanation! You've helped us to fix that.
Huge and Special Thanks for stdweb and carg-web! It's vital to this framework.

[jstarry]

Closing since asmjs-unkown-emscripten, wasm32-unknown-emscripten, and wasm32-unknown-unknown - debug have been supported for some time (debug and release)