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

ko install failed by Go 1.18 #650

Closed
sonatard opened this issue Mar 15, 2022 · 7 comments · Fixed by #657
Closed

ko install failed by Go 1.18 #650

sonatard opened this issue Mar 15, 2022 · 7 comments · Fixed by #657

Comments

@sonatard
Copy link

$ go version
go version go1.18 darwin/arm64

$ go install github.com/google/ko@latest
...
...
# github.com/google/ko/internal/sbom
go/pkg/mod/github.com/google/[email protected]/internal/sbom/mod_1.18.go:28:16: dbi.UnmarshalText undefined (type *debug.BuildInfo has no field or method UnmarshalText)
@sonatard
Copy link
Author

sonatard commented Mar 15, 2022

I found this issue fixed.
#631

@sonatard sonatard reopened this Mar 15, 2022
@sonatard
Copy link
Author

sonatard commented Mar 15, 2022

Do you have a v0.11.0 release plan?
Today Go 1.18 is released.
https://go.dev/dl/

@imjasonh
Copy link
Member

imjasonh commented Mar 16, 2022

I'll probably cut a v0.11 some time tomorrow to pick up the fix in #631.

In the meantime, you should be able to get around this with go install github.com/google/ko@main (note: not @latest), which will include the fix.

@mchmarny
Copy link
Contributor

Thanks for pushing the new version @imjasonh. Seems to fixed the issues with install under go 1.18.

@mchmarny
Copy link
Contributor

I spoke too soon. I am getting this now using build under 1.18

Error: failed to publish images: error building "ko://github.com/mchmarny/<repo>/cmd/app": 
could not parse Go build info: line 43: expected 2 columns for build setting; got 1

Here is the CI command I'm using

KO_DOCKER_REPO=gcr.io/<project>/<name> \
  GOFLAGS="-ldflags=-X=main.version=v0.7.8" \
  ko build ./cmd/app/ --image-refs ./image-digest --bare --tags v0.7.8,latest

@sonatard
Copy link
Author

@imjasonh Thanks for releasing v0.11. My issue is fixed by v0.11. Please close this issue on your own time.

@imjasonh
Copy link
Member

@imjasonh Thanks for releasing v0.11. My issue is fixed by v0.11. Please close this issue on your own time.

There's actually a fix still needed, which will become v0.11.1

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 a pull request may close this issue.

3 participants