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

x/tools/gopls: gopls@latest does not build with go@tip #67545

Closed
findleyr opened this issue May 21, 2024 · 3 comments
Closed

x/tools/gopls: gopls@latest does not build with go@tip #67545

findleyr opened this issue May 21, 2024 · 3 comments
Assignees
Labels
gopls Issues related to the Go language server, gopls. NeedsFix The path to resolution is known, but the work has not been done. release-blocker Tools This label describes issues relating to any tools in the x/tools repository.
Milestone

Comments

@findleyr
Copy link
Contributor

With #67182, the SetCrashOutput API changed in 1.23. This breaks the build of gopls@v0.15.3, since it uses the old SetCrashOutput API. When go1.23rc1 is cut, we want users to be able to use it to build gopls. Therefore, we must cut a new gopls release such that gopls@latest builds.

Given the timeline, I think this must be a gopls@v0.15.4 with a cherry-picked fix. We had been on the fence about cutting a v0.15.4 anyway.

Putting this in the Go1.23 milestone so that it blocks the RC. We should aim for a new release next week at the latest.

CC @adonovan

@findleyr findleyr added NeedsFix The path to resolution is known, but the work has not been done. release-blocker labels May 21, 2024
@findleyr findleyr added this to the Go1.23 milestone May 21, 2024
@findleyr findleyr changed the title x/tools/gopls: gopls@v0.15.3 does not build with go@tip x/tools/gopls: gopls@latest does not build with go@tip May 21, 2024
@gopherbot gopherbot added Tools This label describes issues relating to any tools in the x/tools repository. gopls Issues related to the Go language server, gopls. labels May 21, 2024
@adonovan adonovan self-assigned this May 22, 2024
@adonovan
Copy link
Member

adonovan commented May 22, 2024

If cherrypicking the necessary pile of changes since Feb to make 15.4 support range-over-func seems fraught (and it does), one cheaper option would be to add a go1.23-tagged file to 15.4 that simply fails to compile with a more informative error that advises the user on what to do.

@findleyr
Copy link
Contributor Author

We could do that. The v0.15.4 release branch also has a fix for a somewhat significant loopclosure analyzer bug (#66876). If the v0.15.4 release contains nothing other than a better instruction for 1.23, and a fix for loopclosure, that may be worth doing.

@findleyr
Copy link
Contributor Author

Closing this in favor of https://go.dev/issue/67616, which targets gopls@v0.16.0. We decided to move forward with v0.16.0, rather than v0.15.4, in order to include all the other work to support Go 1.23 features.

@findleyr findleyr closed this as not planned Won't fix, can't repro, duplicate, stale May 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gopls Issues related to the Go language server, gopls. NeedsFix The path to resolution is known, but the work has not been done. release-blocker Tools This label describes issues relating to any tools in the x/tools repository.
Projects
None yet
Development

No branches or pull requests

3 participants