Skip to content

feat: update crossbuilder to tagged version and update go linker#26064

Merged
devanbenz merged 2 commits intomain-2.xfrom
db/update-cross-builder
Feb 26, 2025
Merged

feat: update crossbuilder to tagged version and update go linker#26064
devanbenz merged 2 commits intomain-2.xfrom
db/update-cross-builder

Conversation

@devanbenz
Copy link
Copy Markdown

@devanbenz devanbenz commented Feb 25, 2025

This PR should fix the build step failures for influxdb on windows amd64. Updates the cross builder to
go1.23.5-ba67b11de54aa0bf8f1f92a81f786205de8024b6

Adds export CGO_LDFLAGS="-lntdll -ladvapi32 -lkernel32 -luserenv -lws2_32 ${CGO_LDFLAGS}" to cgo linker to link NT specific syscalls.

Closes #26051

Copy link
Copy Markdown
Member

@gwossum gwossum left a comment

Choose a reason for hiding this comment

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

LGTM

@devanbenz devanbenz changed the title feat: update crossbuilder to tagged version feat: update crossbuilder to tagged version and update go linker Feb 25, 2025
@devanbenz devanbenz force-pushed the db/update-cross-builder branch from beea528 to 576dd4e Compare February 26, 2025 03:12
this includes a CGO_LDFLAGS env var when running go-test-compile
@devanbenz
Copy link
Copy Markdown
Author

I'm going to merge this in so at least the windows build step works on all incoming PRs.

@devanbenz devanbenz merged commit 4689f21 into main-2.x Feb 26, 2025
23 of 24 checks passed
devanbenz added a commit that referenced this pull request May 9, 2025
)

* feat: update crossbuilder and ldflags for go

* feat: Update build_windows for build-tests script
this includes a CGO_LDFLAGS env var when running go-test-compile

(cherry picked from commit 4689f21)
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