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

Cross-compile for Linux #1152

Merged
merged 11 commits into from
Apr 20, 2023
Merged

Cross-compile for Linux #1152

merged 11 commits into from
Apr 20, 2023

Conversation

RebeccaMahany
Copy link
Contributor

@RebeccaMahany RebeccaMahany commented Apr 20, 2023

First part of #1149

Without setting -linkmode external, we see errors on build for grpc.ext including runtime.gcdata: missing Go type information for global symbol .dynsym: size 72, which indicates per golang/go#52863 that external linking is required.

@RebeccaMahany RebeccaMahany changed the title cross-compile for linux Cross-compile for Linux Apr 20, 2023
@RebeccaMahany RebeccaMahany marked this pull request as ready for review April 20, 2023 15:50
Copy link
Contributor

@James-Pickett James-Pickett left a comment

Choose a reason for hiding this comment

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

this looks okay to me, the only thing I noticed was new warning when building

https://github.com/kolide/launcher/actions/runs/4756282010/jobs/8452110679#step:10:13

seejdev
seejdev previously approved these changes Apr 20, 2023
@RebeccaMahany
Copy link
Contributor Author

this looks okay to me, the only thing I noticed was new warning when building

https://github.com/kolide/launcher/actions/runs/4756282010/jobs/8452110679#step:10:13

I think that's this: ziglang/zig#13385

Downgrading didn't help, and 0.11.0 isn't available yet, so I guess I'll try setting -Wno-deprecated-non-prototype!

@RebeccaMahany RebeccaMahany force-pushed the becca/arm branch 2 times, most recently from 7a981f6 to ab73f2d Compare April 20, 2023 19:42
James-Pickett
James-Pickett previously approved these changes Apr 20, 2023
@RebeccaMahany RebeccaMahany merged commit 5dd744b into kolide:main Apr 20, 2023
@RebeccaMahany RebeccaMahany deleted the becca/arm branch April 20, 2023 20:36
James-Pickett pushed a commit to James-Pickett/launcher that referenced this pull request Apr 27, 2023
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.

3 participants