-
Notifications
You must be signed in to change notification settings - Fork 280
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
Add support for cross-compiling via the Makefile #80
Conversation
6c49671
to
d962bfd
Compare
d962bfd
to
9c3cb04
Compare
@rockdaboot I fixed the Docker build and also ensured that test binaries are built with static flag now, so they should no longer need the dynamic libc installed. |
|
554c609
to
215ec05
Compare
Good catch -- there was a problem with the |
Can you document cross-building in README.md? |
|
Done.
Yeah, the test case is broken when running under emulation. I created #81 to fix this. |
There is one little inconsistency...
And |
83b784b
to
cbc100c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM with a small nit.
Takes a while & not actually required for any of the targets.
961e8ee
to
153b232
Compare
Co-authored-by: Timo Teräs <[email protected]>
Ah yes, good catch. I also didn't update the |
Continuing #72 that got auto-closed due to #76.