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

Remove macOS 12 CI build job from the actions workflow #956

Merged
merged 1 commit into from
Oct 27, 2024

Conversation

garybuhrmaster
Copy link
Contributor

Per actions/runner-images#10721
macOS 12 will be removed from github action runners on
12/03/24.

Remove the CI build from the workflow

Thank you for contributing to MythTV!

Please review the checklist below to ensure that your contribution
to MythTV is ready for review by our developers.

It is helpful to regularly rebase your pull request to ensure changes
apply cleanly to the current MythTV development branch.

Checklist

Per actions/runner-images#10721
macOS 12 will be removed from github action runners
on 12/03/24.  Remove the CI build from the workflow
@garybuhrmaster
Copy link
Contributor Author

FWIW, if you look at the current CI runs, you will see a warning that mentions the macOS 12 removal, and the impending brownouts (with build failures) that will start occurring in November in advance of the full removal.

As a future improvement, macos 15 is now available as a runner image.

@jhoyt4
Copy link
Contributor

jhoyt4 commented Oct 27, 2024

agree - we should remove macOS 12. I'd hold off on macOS 15 until macports starts to cache built packages else we'll get constant timeouts from the runners at the 6 hour mark due to the runner compiling all the dependencies (qt5, mysql8 etc).

@billmeek billmeek merged commit 133cdfa into MythTV:master Oct 27, 2024
10 checks passed
@garybuhrmaster garybuhrmaster deleted the remove-macos12 branch October 28, 2024 01:59
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