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

chore: default to zigbuild for cross-compilation #4103

Merged
merged 4 commits into from
Jul 18, 2024

Conversation

EstebanBorai
Copy link
Contributor

@EstebanBorai EstebanBorai commented Jul 15, 2024

While working on building the Fluvio image to run in Kubernetes on macOS I found that
cross-compilation of Fluvio might have some incompatibility issues.

In order to speed up development I ended up using zigbuild as an alternative to cross-compile Fluvio.

In the future, perhaps it would be nice to detect host's OS and conditionally use zigbuild if running on macOS for such step.

@sehz
Copy link
Contributor

sehz commented Jul 16, 2024

maybe we should convert default build to use zigbuild instead?

@EstebanBorai
Copy link
Contributor Author

maybe we should convert default build to use zigbuild instead?

Lets try that!

@EstebanBorai EstebanBorai changed the title chore(docs): hint on using zigbuild for osx/linux cross-compilation chore(docs): default to zigbuild for cross-compilation Jul 16, 2024
@EstebanBorai EstebanBorai changed the title chore(docs): default to zigbuild for cross-compilation chore: default to zigbuild for cross-compilation Jul 16, 2024
@EstebanBorai EstebanBorai added this pull request to the merge queue Jul 18, 2024
Copy link
Contributor

@sehz sehz left a comment

Choose a reason for hiding this comment

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

can remove all zig related flags

@nacardin
Copy link
Contributor

can remove all zig related flags

I don't think so, I believe zigbuild only replaces linker, not CC.

@sehz
Copy link
Contributor

sehz commented Jul 18, 2024

got it

Merged via the queue into master with commit a9cd127 Jul 18, 2024
104 checks passed
@EstebanBorai EstebanBorai deleted the chore/developer-docs branch July 18, 2024 15:59
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.

4 participants