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: GO111MODULE broken with go 1.22 update #194

Merged
merged 1 commit into from
Jul 2, 2024

Conversation

Aflynn50
Copy link
Contributor

@Aflynn50 Aflynn50 commented Jul 2, 2024

Go no longer support go get outside of a module with the GO111MODULE environmental var: https://tip.golang.org/doc/go1.22. Change the uses of this to go install instead. A version for the modules is required since it would normally use the go.mod one.

Go no longer support go get outside of a module with the GO111MODULE
enviromental var: https://tip.golang.org/doc/go1.22. Change the uses of
this to go install instead. A version for the modules is required since
it would normally use the go.mod one.
builders:
- shell: |-
#!/bin/bash
set -eux
GO111MODULE=off go get {modules}
go install {modules}
Copy link
Member

Choose a reason for hiding this comment

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

This is the only one I don't know about. All those listed would have to include the specific version/latest.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've changed all the usages of this to be latest

@Aflynn50
Copy link
Contributor Author

Aflynn50 commented Jul 2, 2024

/merge

@Aflynn50 Aflynn50 merged commit b00d32a into juju:main Jul 2, 2024
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