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

feat: switch hello-cosmo-go to WASI preview2 tinygo w/ wash build #14

Conversation

vados-cosmonic
Copy link

Primary Reviewer

@ricochet

Feature or Problem

Work is underway in wash to enable building components with the tinygo toolchain. This means that running wash build should be enough for people to get a preview2 tinygo component built and signed.

However, the support in this repository is pre WASI resources landing, so a slightly older version of some
dependencies (specifically, wasi:io/streams) must be used.

This commit updates the hello-cosmo-go example to use the options that will work soon (tm) in wash, and the necessary WIT to make the actor compile properly.

Related Issues

wasmCloud/wash#758

Release Information

Consumer Impact

Consumers should be able to build WASI preview2 components for TinyGo with wash build

Testing

Built on platform(s)

  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
  • x86_64-windows

Tested on platform(s)

  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
  • x86_64-windows

Unit Test(s)

Acceptance or Integration

Manual Verification

Verified manually locally

Work is underway in wash to enable building components
with the tinygo toolchain. This means that running `wash build` should
be enough for people to get a preview2 tinygo component built and
signed.

However, the support in this repository is *pre* WASI resources
landing, so a slightly older version of some
dependencies (specifically, wasi:io/streams) must be used.

This commit updates the hello-cosmo-go example to use the options that
will work soon (tm) in wash, and the necessary WIT to make the actor
compile properly.

Signed-off-by: Victor Adossi <vadossi@cosmonic.com>
Signed-off-by: Victor Adossi <vadossi@cosmonic.com>
@connorsmith256
Copy link
Contributor

@vados-cosmonic I don't think the wasmcloud host we use with wash is compatible with the changes here(?)

As far as I know, we should be using wasmtime v0.13 stuff everywhere: wit definitions, adapter, and so on

@lachieh lachieh closed this Dec 13, 2024
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.

3 participants