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

.cargo/config.toml: remove #746

Merged

Conversation

billatarm
Copy link
Contributor

When compiling on an aarch64 host, the override of the linker to the cross "compiler" (uses the gcc frontend) causes a build failure. Theirs no reason to set these globally for all developers as their build host may be different as well as their toolchain preference and naming conventions.

Instead override the target specific linker variable in a --config option string in the cross compilation tests.

@billatarm
Copy link
Contributor Author

This hasn't been tested yet, I didn't look into firing up the CI. But it looks like the .cargo/config.toml file was only needed for the cross compilation CI tests and can be nuked.

This fixes building on an aarch64 host.

@gowthamsk-arm gowthamsk-arm self-requested a review December 6, 2023 10:55
@billatarm
Copy link
Contributor Author

Ahh OK, I looked through the docker hub testing locally now will upload a fix.

When compiling on an aarch64 host, the override of the linker to the
cross "compiler" (uses the gcc frontend) causes a build failure. Theirs
no reason to set these globally for all developers as their build host
may be different as well as their toolchain preference and naming
conventions.

Instead override the target specific linker variable in a --config
option string in the cross compilation tests.

Signed-off-by: Bill Roberts <[email protected]>
@billatarm billatarm force-pushed the fix-linking-on-aarch64-host branch from 0540e12 to cf2707f Compare December 6, 2023 19:29
@billatarm
Copy link
Contributor Author

Ahh OK, I looked through the docker hub testing locally now will upload a fix.

Had the key wrong and some minor TOML formatting issues. I have working locally now, lets see what the CI thinks.

Copy link
Member

@tgonzalezorlandoarm tgonzalezorlandoarm left a comment

Choose a reason for hiding this comment

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

LGTM, thanks a lot for this!

Copy link
Contributor

@gowthamsk-arm gowthamsk-arm left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you :)

@gowthamsk-arm gowthamsk-arm merged commit bfa6341 into parallaxsecond:main Dec 7, 2023
16 checks passed
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