Skip to content

Prepare go version bump to 1.25#829

Open
cbosdo wants to merge 4 commits into
uyuni-project:mainfrom
cbosdo:go-bump
Open

Prepare go version bump to 1.25#829
cbosdo wants to merge 4 commits into
uyuni-project:mainfrom
cbosdo:go-bump

Conversation

@cbosdo

@cbosdo cbosdo commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

What does this PR change?

Go 1.24+ already have a few additional checks that will become mandatory in 1.25. Let's prepare the bump before it's too late.

Test coverage

  • No tests: already covered

  • DONE

Links

Issue(s): #

  • DONE

Changelogs

Make sure the changelogs entries you are adding are compliant with https://github.com/uyuni-project/uyuni/wiki/Contributing#changelogs and https://github.com/uyuni-project/uyuni/wiki/Contributing#uyuni-projectuyuni-repository

If you don't need a changelog check, please mark this checkbox:

  • No changelog needed

If you uncheck the checkbox after the PR is created, you will need to re-run changelog_test (see below)

Before you merge

Check How to branch and merge properly!

Comment thread .devcontainer/Dockerfile Fixed
@cbosdo
cbosdo force-pushed the go-bump branch 5 times, most recently from 0d57f2e to d7218a7 Compare July 21, 2026 18:28
@cbosdo

cbosdo commented Jul 21, 2026

Copy link
Copy Markdown
Contributor Author

cbosdo added 2 commits July 21, 2026 20:34
Changing the go.mod version to 1.25.0 fails the build for older distros
like Debian 13 or Ubuntu 24.04. In order to still build for them, the
mingo_build changes the go.mod of the fly. The spec file has also been
adjusted for a similar hack.

So, the code needs to build on 1.25.0, but not use features introduced
after go1.19 to still build on Debian 12. The minimum go version will be
raised as those distros become end of life
Go 1.25 is more picky about those: fix them all.
@sonarqubecloud

Copy link
Copy Markdown

@cbosdo
cbosdo requested a review from a team July 22, 2026 09:31
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.

2 participants