Skip to content

Go: remove the Go implementation in favor of bytecodealliance/go-modules #1194

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

Closed
Mossaka opened this issue Mar 6, 2025 · 1 comment · Fixed by #1195
Closed

Go: remove the Go implementation in favor of bytecodealliance/go-modules #1194

Mossaka opened this issue Mar 6, 2025 · 1 comment · Fixed by #1195
Labels
gen-tinygo Related to the TinyGo bindings generator

Comments

@Mossaka
Copy link
Member

Mossaka commented Mar 6, 2025

The new Go bindings generator is now actively developed and maintained in a separate repository: bytecodealliance/go-modules (Thanks to @ydnar!). I’d like to propose sunsetting the old CGo-based implementation in this repository.

My plan is to remove all Go-related source code and tests from this repo while keeping the Go subcommand in wit-bindgen. If users attempt to generate bindings with it, the tool will panic with a message directing them to use the new wit-bindgen-go CLI instead.

e.g.

> wit-bindgen tiny-go try.wit
Error: Go bindgen has been moved to a separate repository. Please visit https://github.com/bytecodealliance/go-modules for the new Go bindings generator `wit-bindgen-go`.

I will contribute all the existing runtime tests from this repo to the new repo, and @alexcrichton 's work on refactoring the test cases should help this process a lot, thank you!

Love to hear your thoughts on this one.

@Mossaka Mossaka changed the title go: remove the Go implementation in favor of https://github.com/bytecodealliance/go-modules go: remove the Go implementation in favor of bytecodealliance/go-modules Mar 6, 2025
@Mossaka Mossaka added the gen-tinygo Related to the TinyGo bindings generator label Mar 6, 2025
@Mossaka Mossaka changed the title go: remove the Go implementation in favor of bytecodealliance/go-modules Go: remove the Go implementation in favor of bytecodealliance/go-modules Mar 6, 2025
@Mossaka Mossaka linked a pull request Mar 6, 2025 that will close this issue
@alexcrichton
Copy link
Member

(responded over on the associated PR)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gen-tinygo Related to the TinyGo bindings generator
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants