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

add cross-compilation assistance for linux aarch64 #577

Merged
merged 3 commits into from
Jun 22, 2023

Conversation

hone
Copy link
Member

@hone hone commented Jun 20, 2023

In order to cross-compile to aarch64 on Linux, cargo needs some guidance to find the compiler and linker.

@hone hone requested a review from a team as a code owner June 20, 2023 23:49
@Malax Malax self-assigned this Jun 21, 2023
@edmorley edmorley added libcnb-package enhancement New feature or request labels Jun 21, 2023
Copy link
Member

@Malax Malax left a comment

Choose a reason for hiding this comment

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

Apart from the one typo: ship it! Love having more cross-compilation assistance data :)

hone and others added 3 commits June 21, 2023 15:30
Co-authored-by: Manuel Fuchs <manuel.fuchs@salesforce.com>
@Malax Malax merged commit 2ec1b98 into main Jun 22, 2023
4 checks passed
@Malax Malax deleted the libcnb-package/aarch64 branch June 22, 2023 08:43
@Malax Malax mentioned this pull request Jun 22, 2023
@@ -59,6 +59,31 @@ The easiest way to install 'musl-gcc' is to install the 'musl-tools' package:
- https://packages.debian.org/bullseye/musl-tools"#,
)),
}
} else if target_triple.as_ref() == AARCH64_UNKNOWN_LINUX_MUSL && cfg!(target_os = "linux") {
let gcc_binary_path = "aarch64-linux-gnu-gcc";
Copy link
Member

Choose a reason for hiding this comment

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

@hone Was there a reason this is aarch64-linux-gnu-gcc not aarch64-linux-musl-gcc? We're using MUSL for everything else?

Copy link
Member

Choose a reason for hiding this comment

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

Filed #724.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request libcnb-package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants