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

Build fails with Go 1.23 #5047

Closed
stefanb opened this issue Aug 2, 2024 · 2 comments · Fixed by #5108
Closed

Build fails with Go 1.23 #5047

stefanb opened this issue Aug 2, 2024 · 2 comments · Fixed by #5108
Assignees
Labels
kind/bug Something isn't working

Comments

@stefanb
Copy link
Contributor

stefanb commented Aug 2, 2024

What happened:

Build with Go 1.23 rc1 & rc2 fails.

What you expected to happen:

Build to succeed.

How to reproduce it (as minimally and precisely as possible):

Anything else we need to know?

Build error

  ...
  # github.com/grafana/pyroscope-go/godeltaprof/internal/pprof
  /Users/brew/Library/Caches/Homebrew/go_mod_cache/pkg/mod/github.com/grafana/pyroscope-go/[email protected]/internal/pprof/delta_mutex.go:30:20: undefined: runtime_cyclesPerSecond
  /Users/brew/Library/Caches/Homebrew/go_mod_cache/pkg/mod/github.com/grafana/pyroscope-go/[email protected]/internal/pprof/proto.go:320:8: undefined: runtime_expandFinalInlineFrame
  make: *** [juicefs] Error 1

Indicating the upstream problems

The upstream fixes still needs to be released.

Environment:

@stefanb stefanb added the kind/bug Something isn't working label Aug 2, 2024
stefanb added a commit to Homebrew/homebrew-core that referenced this issue Aug 2, 2024
Use "go" again after juicedata/juicefs#5047 is resolved and released

To be merged after #175310
@zhijian-pro zhijian-pro self-assigned this Aug 2, 2024
@zhijian-pro
Copy link
Contributor

Thanks, we will fix our code after the upstream fixes

stefanb added a commit to Homebrew/homebrew-core that referenced this issue Aug 16, 2024
Use "go" again after juicedata/juicefs#5047 is resolved and released

To be merged after #175310
@stefanb
Copy link
Contributor Author

stefanb commented Aug 16, 2024

FYI, upstream pyroscope-go has released a fix for Go 1.23:
https://github.com/grafana/pyroscope-go/releases/tag/godeltaprof%2Fv0.1.8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants