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

Enable cross compilation for applications #4

Closed
wants to merge 1 commit into from

Conversation

rucoder
Copy link
Contributor

@rucoder rucoder commented Aug 18, 2024

Even though rust can do cross-compilation, it cannot do linking for target platform. We have to define target specific rustc options and have required toolchain installed as well as libraries for target platfrom

clang is a very good choice for cross-compilation unless we are going to build for some exotic targets

@rucoder rucoder requested a review from deitch August 18, 2024 08:08
@deitch
Copy link
Contributor

deitch commented Aug 18, 2024

What is CI complaining about?

@rucoder
Copy link
Contributor Author

rucoder commented Aug 18, 2024

What is CI complaining about?

@deitch It complains about tags, do you ALWAYS push? e.g. for PR builds before they merged to main?

@deitch
Copy link
Contributor

deitch commented Aug 18, 2024

No, it only pushes on tags. But it does check tags when building. I can check that area and see if I can fix it.

@deitch
Copy link
Contributor

deitch commented Aug 18, 2024

Try rebasing on main

Even though rust can do cross-compilation, it cannot do linking for
target platform. We have to define target specific rustc options and
have required toolchain installed as well as libraries for target
platfrom

clang is a very good choice for cross-compilation unless we are going to
build for some exotic targets

Signed-off-by: Mikhail Malyshev <[email protected]>
@rucoder rucoder force-pushed the rucoder/rust-cross branch from 32492cf to 7c6f015 Compare August 18, 2024 09:12
@deitch
Copy link
Contributor

deitch commented Aug 18, 2024

Looks like the rebasing worked.

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