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

Use macos-13 for Intel build #2508

Merged
merged 1 commit into from
Sep 24, 2024
Merged

Conversation

davegarthsimpson
Copy link
Contributor

@davegarthsimpson davegarthsimpson commented Sep 19, 2024

Motivation

We noticed the CI builds are no longer producing artifacts for intel mac.

This is due to the fact that Github Actions runners referenced with "macos-latest", i.e. MacOS v14 OS runners, will now default to arm systems.

At the moment the only intel based Mac OS 14 runner image is macos-14-large (ref) which is a paid solution.

We've decided to change the referenced image to "macos-13" as a work-around, to start reproducing the intel builds of IDE 2.x whilst avoiding additional cost.

Change description

Change the referenced image to "macos-13" in build.yml

Other information

actions/runner-images#9741

Reviewer checklist

  • PR addresses a single concern.
  • The PR has no duplicates (please search among the Pull Requests before creating one)
  • PR title and description are properly filled.
  • Docs have been added / updated (for bug fixes / features)

@davegarthsimpson davegarthsimpson added topic: infrastructure Related to project infrastructure os: macos Specific to macOS operating system labels Sep 19, 2024
@davegarthsimpson davegarthsimpson merged commit 2921979 into main Sep 24, 2024
29 of 39 checks passed
@davegarthsimpson davegarthsimpson deleted the use-intel-mac-in-build-runner branch September 24, 2024 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
os: macos Specific to macOS operating system topic: infrastructure Related to project infrastructure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants