Skip to content
This repository was archived by the owner on Aug 25, 2025. It is now read-only.

all: update the x/tools dependency to fix the build with Go 1.25#432

Merged
findleyr merged 1 commit intogoogle:mainfrom
findleyr:fixbuild
Aug 21, 2025
Merged

all: update the x/tools dependency to fix the build with Go 1.25#432
findleyr merged 1 commit intogoogle:mainfrom
findleyr:fixbuild

Conversation

@findleyr
Copy link
Contributor

@findleyr findleyr commented Aug 20, 2025

An unfortunate bug caused a number of packages in old x/tools versions to fail to build with Go 1.25 (https://go.dev/issue/74462). Fix this minimally by updating to the lowest patched version with the fix: v0.24.1. This should allow wire to continue building with the same range of Go versions as it could build with before (down to 1.19).

Also:

  • Update the go.mod go directive to Go 1.19 to allow for module graph pruning, and to be consistent with the minimum build version.
  • Fix a test that is broken in recent Go versions due to an error message change.

Fixes #431

@findleyr findleyr force-pushed the fixbuild branch 2 times, most recently from be25772 to 0b783bc Compare August 20, 2025 21:20
An unfortunate bug caused a number of packages in old x/tools versions to fail to build with Go 1.25 (https://go.dev/issue/74462). Fix this minimally by updating to the lowest patched version with the fix: v0.24.1. This should allow wire to continue building with the same range of Go versions as it could build with before (down to 1.19). Also: - Update the go.mod go directive to Go 1.19 to allow for module graph pruning, and to be consistent with the minimum build version. - Fix a test that is broken in recent Go versions due to an error message change. - Update tests to run on Go 1.25.x (the most recent version of Go). - Remove coverage, since coverage upload wasn't working anyway. Fixes google#431
Copy link
Collaborator

@cherrymui cherrymui left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@findleyr findleyr merged commit 5c5c92a into google:main Aug 21, 2025
4 checks passed
@findleyr findleyr deleted the fixbuild branch August 21, 2025 14:44
tbxark added a commit to wireinject/wire that referenced this pull request Aug 21, 2025
tbxark added a commit to wireinject/wire that referenced this pull request Aug 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

2 participants