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

Fix aarch64 build and test aarch64 on macOS #806

Merged
merged 2 commits into from
Mar 15, 2024
Merged

Conversation

thedataking
Copy link
Collaborator

@thedataking thedataking commented Mar 15, 2024

  • delete .cargo/config.toml since it can cause cargo build without --target to fail. Closes rav1d fails to build on aarch64 #773.
  • test aarch64 on macOS/arm64 which runs much faster than under QEMU.
    • Old QEMU/arm64 workflow restricted to specific branches so it is effectively disabled.
  • test armv7 without docker which avoids the time spent provisioning and starting docker on each run.

The feature flags were causing build failures with
aarch64 GNU/Linux. Closes #773.
@thedataking thedataking force-pushed the perl/issue-773 branch 14 times, most recently from 733e1c8 to aad6244 Compare March 15, 2024 10:02
@thedataking thedataking requested a review from rinon March 15, 2024 10:02
@thedataking thedataking force-pushed the perl/issue-773 branch 5 times, most recently from d54a3fa to 9c66a59 Compare March 15, 2024 10:51
@thedataking thedataking marked this pull request as ready for review March 15, 2024 10:59
Copy link
Collaborator

@kkysen kkysen left a comment

Choose a reason for hiding this comment

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

We need to keep a good way to run tests with qemu locally. Is it possible to do this in build.rs somehow, maybe using cargo:rustc-flags=FLAGS? Maybe we can have a local-only solution for testing, but if build.rs can accommodate this, I think that'd be ideal.

@thedataking thedataking force-pushed the perl/issue-773 branch 3 times, most recently from 857d9b3 to 38887a5 Compare March 15, 2024 18:53
@thedataking thedataking merged commit f5bee55 into main Mar 15, 2024
34 checks passed
@kkysen kkysen deleted the perl/issue-773 branch March 16, 2024 00:20
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.

rav1d fails to build on aarch64
3 participants