-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Add macOS arm64 to Cirrus CI #14028
base: master
Are you sure you want to change the base?
Add macOS arm64 to Cirrus CI #14028
Conversation
dont know why linux is failing 🤔 |
I will squash all these commits when I am done testing and fixing. |
If it helps, this I what I do to compile locally:
|
3e394c8
to
677d2d7
Compare
anyone have any ideas whats going on with homebrew to cause |
7d6c442
to
7d75916
Compare
This also optimizes the build-mac script Reduces from 1 hour + to less than 30 minutes Co-Authored-By: UltraHDR <[email protected]> Co-Authored-By: shinra-electric <[email protected]>
I'm getting a library not loaded error, even though the dylib is packaged.
|
Actually, since the problem is FAudio dylib, why not just disable it with FAudio is ignored in Windows builds (see #14128) , and it's not necessary for macOS (Cubeb works fine). Edit: removing FAudio would also have the added benefit of reducing downloading & build times Edit 2: Well, I think FAudio is ignored on Windows, though that code looks like it's just not updating the submodule. Regardless, it's not required on macOS Edit 3: I've removed the code suggestions about removing FAudio (though that is still an option), since I noticed a couple of issues with |
Co-Authored-By: shinra-electric <[email protected]> Co-Authored-By: Margen67 <[email protected]>
f7db5fa
to
878c120
Compare
Maybe have a look at what Nas did in #14563, that PR fixes a similar issue on master. |
FYI we've moved away from Cirrus for our CI (other than FreeBSD) because of their new limits. |
Attempts to Resolve #12599