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

ci: Update MacOS to latest OS #218

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

spencer-lunarg
Copy link

The macOS 12 runner image will be removed by December 3rd, 2024.
...
Jobs using the macos-12 YAML workflow label should be updated to macos-15, macos-14, macos-13, or macos-latest.

@SaschaWillems
Copy link
Owner

Thanks for the PR. Very much appreciated 👍🏻

@spencer-lunarg
Copy link
Author

spencer-lunarg commented Oct 17, 2024

found architecture 'arm64', required architecture 'x86_64'

o no... @richard-lunarg @aitor-lunarg help here... my 🍎 knowledge doesn't go much farther than just updating YAML files

@SaschaWillems
Copy link
Owner

It might be possible that the newer MacOS images are now arm based instead of x86? Which would explain the error message. Don't know enough about Mac myself.

@richard-lunarg
Copy link
Contributor

Yes, I've seen this error message. You are either building for arm64 and linking to x86_64 or vice-a-versa. If you are using prebuilt Qt binaries, the upgrade may be running now on an Apple Silicon machine and you will need to rebuilt the Qt libraries for Apple Silicon. Are you using qmake or cmake to build in your CI workflow (I am close to an idiot on CI stuff)? You may also be able to set a compiler flag or command line switch to build for x86_64 again. If you want to build both platforms, I have instructions from when I figured out how to build Qt5 as a dual platform static library (this is how I build caps viewer for the SDK, it's dual, x86_64/Apple Silicon).

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