-
Notifications
You must be signed in to change notification settings - Fork 173
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
Use Go 1.21 WASI GOOS in examples #653
Comments
I guess it is not so straightforward. A simple Go app built to wasip1 requires the following imports:
Also, when I stub them with the following changes: diff --git a/runtimes/web/src/runtime.ts b/runtimes/web/src/runtime.ts
index bf83d76..01c81d5 100644
--- a/runtimes/web/src/runtime.ts
+++ b/runtimes/web/src/runtime.ts
@@ -159,8 +159,21 @@ export class Runtime {
tracef: this.tracef.bind(this),
};
+ const wasi_snapshot_preview1 = {
+ sched_yield: () => {},
+ proc_exit: () => {},
+ args_get: () => {},
+ args_sizes_get: () => {},
+ clock_time_get: () => {},
+ environ_get: () => {},
+ environ_sizes_get: () => {},
+ fd_write: () => {},
+ random_get: () => {},
+ poll_oneoff: () => {},
+ };
+
await this.bluescreenOnError(async () => {
- const module = await WebAssembly.instantiate(wasmBuffer, { env });
+ const module = await WebAssembly.instantiate(wasmBuffer, { env, wasi_snapshot_preview1 });
this.wasm = module.instance;
// Call the WASI _start/_initialize function (different from WASM-4's start callback!) It still doesn't run, and produces this error in the browser:
|
Also, it seems Go 1.21 wasip1 won't allow exporting functions to WASM: |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Go 1.21 will ship with a new GOARCH=wasm GOOS=wasip1 target. More details at https://tip.golang.org/doc/go1.21.
It should be possible to use that target instead of using tinygo.
The text was updated successfully, but these errors were encountered: