-
Notifications
You must be signed in to change notification settings - Fork 570
Unify GopherJS's syscall/js
and GopherWasm
#899
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
Comments
I'll get started soon. |
@hajimehoshi thanks for your great work for gopherjs, may I ask in this context what would you recommend if I'm actively using |
Hi, I have no idea about these projects. Please ask the owners. |
Well it's not about these projects, but generally about your vision of moving towards wasm in the context of this particular adoption keeping in mind that many ppl still (and will) be using other libs as well. Ok your point is clear. |
@inliquid I don't wish to speak for @dominikh but in the past he has expressed his intent not to spend anymore time on adapting the existing libraries to webassembly. I think there are alternative efforts to have a syscall/js-based DOM interface such as https://github.com/dennwc/dom. Ideally for the DOM specifically we would generate Go bindings directly from the WebIDL spec like Rust does. @dennwc was doing some work in this area last year sure if that effort is still ongoing. Would you be interested in contributing to this? |
Hi @johanbrandhorst! Thank you for detailed comments, I will follow mentioned repo activity and try to contribute when possible. |
This PR adds a new package `syscall/js`. Originally `syscall/js` is for WebAssembly, but in GopherJS this package works as a wrapper of `github.com/gopherjs/gopherjs/js`. With this package, developers can use `syscall/js` both for GopherJS and WebAssembly. The implementation is basically same as `github.com/gohperjs/gopherwasm`. Fixes #899
Will we get another release with this? |
I guess GopherJS does not have a 'release' policy. Not sure though. |
The release policy is we increment the version number by 1 whenever there's a notable or large change. It's just for convenience, so people can tell if their GopherJS version includes said notable/large change (and we can ask them in bug reports). It's also so we can blog/tweet about a new version being released. See issue #500 for background. I think adding support for |
Sure, let me update the version soon :-) |
Introducing `syscall/js` (#899) is so significant that we update the version.
Introducing `syscall/js` (#899) is so significant that we update the version.
Uh oh!
There was an error while loading. Please reload this page.
@neelance suggested to unify GopherWasm and GopherJS by making GopherJS recognize
syscall/js
as a wrapper like GopherWasm. This would enable the users to replacewith
while the programs still work both for GopherJS and Wasm. This simplifies things very much and encourages users to migrate from GopherJS to Wasm more smoothly than the current GopherWasm.
Any thoughts?
The text was updated successfully, but these errors were encountered: