-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Add macos-14 to the push and nightly workflows #13299
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
Conversation
Once you have a chance, please have a look at the build failure. It fails when building without capstone as |
This should be fixed via 2bf9f7e |
5ac0e7e
to
91d4fc7
Compare
Btw could you target this against PHP-8.2? I believe it's a good idea to run arm64 tests on currently supported versions of PHP as well as the master branch :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Shivam!
f799c10
to
1a04dc2
Compare
Since we build with the configuration option --with-sodium, adding libsodium to make sure it is installed
1a04dc2
to
a92770e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM otherwise
Co-authored-by: Ilija Tovilo <[email protected]>
@shivammathur Thanks! Feel free to merge yourself. |
Adding
macos-14
to the push and nightly workflows to test with arm64 architecture.Ref: https://github.blog/changelog/2024-01-30-github-actions-macos-14-sonoma-is-now-available/
This PR also does the following
libsodium
to thebrew
action as it is not installed onmacos-14
by default.configure-macos
action as brew's prefix is different inmacos-14
.