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

Update CI #74

Merged
merged 2 commits into from
Feb 26, 2023
Merged

Update CI #74

merged 2 commits into from
Feb 26, 2023

Conversation

hv15
Copy link
Member

@hv15 hv15 commented Feb 19, 2023

We no longer build for < Xcode 13 and we are building with a newer macos SDK, so this PR updates to the latest GitHub Actions runner for macos.

Additionally we update to Ubuntu 20.04 runner as 18.04 is deprecated (we will also drop this package upstream anyway in the near future).

@hv15 hv15 added the hotfix When things need fix ASAP. label Feb 19, 2023
@hv15 hv15 self-assigned this Feb 19, 2023
@hv15
Copy link
Member Author

hv15 commented Feb 19, 2023

Crap, this has nothing to do with us not supporting a specific version of Xcode, rather its that actions only supports x64! No ETA on when this will be changed to (extended?) to M1/aarm64... actions/runner-images#2187

@hv15
Copy link
Member Author

hv15 commented Feb 19, 2023

OK the fix is reasonably easy, we extend our latest release to include a short link for the x64 package... at some point I made the latest package default to aarm64.... this maybe wasn't a great idea.

@hv15
Copy link
Member Author

hv15 commented Feb 19, 2023

OK Fixed!

@hv15 hv15 requested a review from sbscholz February 26, 2023 13:30
Copy link
Member

@sbscholz sbscholz left a comment

Choose a reason for hiding this comment

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

seems to do the trick.

@sbscholz sbscholz merged commit a9a7a69 into SacBase:master Feb 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hotfix When things need fix ASAP.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants