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

Update our wasmtime #6777

Open
srenatus opened this issue May 30, 2024 · 4 comments
Open

Update our wasmtime #6777

srenatus opened this issue May 30, 2024 · 4 comments

Comments

@srenatus
Copy link
Contributor

We're far behind... It's causing issues for the alpine package build, too. Besides missing out on new features and improvements.

Tried it in #6556 but hit some build problems on windows.

Copy link

stale bot commented Jun 30, 2024

This issue has been automatically marked as inactive because it has not had any activity in the last 30 days. Although currently inactive, the issue could still be considered and actively worked on in the future. More details about the use-case this issue attempts to address, the value provided by completing it or possible solutions to resolve it would help to prioritize the issue.

@stale stale bot added the inactive label Jun 30, 2024
@Juneezee
Copy link
Contributor

Juneezee commented Dec 31, 2024

At the time of writing this comment, the latest version of wasmtime-go is v28.

I attempted to upgrade wasmtime-go to v28 in my forked repository, but the Windows build also failed. Here are the job logs: go-build-windows-amd64-logs.txt.

From the logs, it appears that the MinGW compiler is incompatible with the newer version of wasmtime-go. Unfortunately, the CI logs from PR #6556 have expired, so I can't compare the errors directly. However, I suspect they are quite similar.

/usr/bin/x86_64-w64-mingw32-ld: /src/vendor/github.com/bytecodealliance/wasmtime-go/v28/build/windows-x86_64/libwasmtime.a(07c0bb04dd2aaef1-helpers.o):helpers.c:(.text$wasmtime_setjmp_inverted+0x26): undefined reference to `__intrinsic_setjmpex'

Similar issues have been reported in the wasmtime-go repository:

  1. Windows build failure: undefined reference to `_setjmp' bytecodealliance/wasmtime-go#192
  2. bytecodealliance/wasmtime-go@227ba15

@stale stale bot removed the inactive label Dec 31, 2024
@srenatus
Copy link
Contributor Author

However, I suspect they are quite similar.

Yep it was!

@srenatus
Copy link
Contributor Author

Reading up on that thread, I guess this is a starting point: bytecodealliance/wasmtime-go#192 (comment) -- so we'd need to check mingw versions and see if we can make them match in OPA's build toolchain.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants