Skip to content

Conversation

@aarzilli
Copy link
Member

Once upon a time Go version strings for development builds did not
reference the major/minor version they belonged to, therefore the best
we could do was to always assume that a development build belonged to
the most recently released version of Go.

This commit changes pkg/goversion so that the new style of development
version string is fully parsed.

@aarzilli
Copy link
Member Author

I want to use this in the (future) PR to support swisstables. I want to add a check to the swisstables test so that it will start failing when 1.25 is released, so that we can remember to change it when swisstables become the default but I can't do that if we can't distinguish 1.24 development builds from 1.26 development builds (which we can not without this change).

Once upon a time Go version strings for development builds did not
reference the major/minor version they belonged to, therefore the best
we could do was to always assume that a development build belonged to
the most recently released version of Go.

This commit changes pkg/goversion so that the new style of development
version string is fully parsed.
Copy link
Member

@derekparker derekparker left a comment

Choose a reason for hiding this comment

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

LGTM

@derekparker derekparker merged commit 1a9bd03 into go-delve:master Oct 21, 2024
1 of 2 checks passed
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