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

fix tinygo build caused by go toolchain #109

Merged

Conversation

flavio
Copy link
Member

@flavio flavio commented Feb 5, 2024

Latest version of Go introduces the concept of toolchain.
This brings a new keyword inside of go.mod. This keyword can be understood using Go 1.21 or later.

Even though our Go policies are built with TinyGo, TinyGo requires the official Go compiler to perform some tasks.
Because of that, we have to ensure the latest stable release of the Go compiler is installed inside of the build environment. We cannot rely on what is shipped out of the box with the GH runner image.

This PR contains two commits:

  • fix: ensure TinyGo policies build without errors
  • Prepare for 3.1.16 tagging

Once merged, I'll create the v3.1.16 tag.

Latest version of Go introduces the concept of toolchain [1]. This
brings a new keyword inside of `go.mod`. This keyword can be understood
using Go 1.21 or later.

Even though our Go policies are built with TinyGo, TinyGo requires
the official Go compiler to perform some tasks.
Because of that, we have to ensure the latest stable release of the Go
compiler is installed inside of the build environment. We cannot rely on
what is shipped out of the box with the GH runner image.

[1] https://go.dev/blog/toolchain

Signed-off-by: Flavio Castelli <fcastelli@suse.com>
```
sed --in-place 's/3\.1\.15/3\.1\.16/' $(find . -maxdepth 3 -name "*.yml")
````

Signed-off-by: Flavio Castelli <fcastelli@suse.com>
@flavio flavio requested a review from a team as a code owner February 5, 2024 08:39
@viccuad viccuad merged commit fec014f into kubewarden:main Feb 5, 2024
@flavio flavio deleted the fix-tinygo-build-caused-by-go-toolchain branch February 5, 2024 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants