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 multipass_tests target on Ubuntu ARM64 (#3739) #3740

Closed
wants to merge 2 commits into from

Conversation

jason-erb
Copy link
Contributor

@jason-erb jason-erb commented Oct 17, 2024

This PR:

  • Excludes QEMU code from tests if QEMU not enabled.
  • Ensures that all <BACKEND>_ENABLED definition checks occur at compile time, not run time, using #ifdef.

Fixes #3739 (Building target multipass_tests fails on Ubuntu ARM64)


MULTI-1585

@jason-erb jason-erb changed the title Properly exclude QEMU code if QEMU not enabled (#3739) Fix multipass_tests target on Ubuntu ARM6 (#3739) Oct 18, 2024
@jason-erb jason-erb changed the title Fix multipass_tests target on Ubuntu ARM6 (#3739) Fix multipass_tests target on Ubuntu ARM64 (#3739) Oct 18, 2024
This change also ensures that all `<BACKEND>_ENABLED` definition checks
occur at compile time, not run time, using #ifdef.

Fixes canonical#3739 (Building target multipass_tests fails on Ubuntu ARM64)
ricab
ricab previously approved these changes Nov 27, 2024
Copy link
Collaborator

@ricab ricab left a comment

Choose a reason for hiding this comment

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

This looks good to me, thanks @jason-erb!

tests/linux/test_platform_linux.cpp Outdated Show resolved Hide resolved
@ricab ricab requested a review from levkropp November 28, 2024 00:00
levkropp
levkropp previously approved these changes Nov 28, 2024
This addresses a review comment for canonical#3739 (Building target multipass_tests fails on Ubuntu ARM64)
@ricab
Copy link
Collaborator

ricab commented Dec 2, 2024

Thank you @jason-erb! I have transported that to our internal branch, to have CI run on it.

ricab added a commit that referenced this pull request Dec 3, 2024
      Clone of #3740: "Fix multipass_tests target on Ubuntu ARM64 (#3739)"
@ricab
Copy link
Collaborator

ricab commented Dec 3, 2024

Your commits are merged @jason-erb. Thanks again!

@ricab ricab closed this Dec 3, 2024
@jason-erb
Copy link
Contributor Author

@ricab You're welcome! Thanks for reviewing and merging. I'm in the running for a position on the Canonical Multipass team, so please feel free to put in a good word... 🙂

@ricab
Copy link
Collaborator

ricab commented Dec 3, 2024

Thank you for your interest. Best of luck with your application.

ricab added a commit that referenced this pull request Feb 7, 2025
      Clone of #3740: "Fix multipass_tests target on Ubuntu ARM64 (#3739)"
ricab added a commit that referenced this pull request Feb 7, 2025
      Clone of #3740: "Fix multipass_tests target on Ubuntu ARM64 (#3739)"
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.

Building target multipass_tests fails on Ubuntu ARM64
3 participants