-
Notifications
You must be signed in to change notification settings - Fork 1k
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
ci: Optimize build matrix #1046
ci: Optimize build matrix #1046
Conversation
3c7e0bb
to
4899e0d
Compare
Also simplify #ifdefs in generated file.
This has been overlooked in bitcoin-core#988.
4899e0d
to
5c100cd
Compare
Added the commit "precompute_ecmult: Always compute all tables up to default WINDOW_G" to make sure the new CI check that the build does not modify the precomp files passed (introduced by bb36fe9). |
Oh I think the reason is that we never update brew (and the catalina image provided by Cirrus is old and deprecated). Since the brew binary URL has moved, it always needs to fall back to building deps (autotools) from source, which currently fails because gnu.org is down... Let's see if a newer brew fixes this. Then I may reconsider the commit that trims the macOS matrix. We're stuck to old macOS Catalina because valgrind doesn't work on the latest versions and even the unoffical fork we use does only support Catalina (LouisBrunner/valgrind-macos#19). Do you think we should stick to Catalina or move on (and drop Valgrind builds on macOS)? |
94fe07c
to
1fbf1d9
Compare
1fbf1d9
to
a45d0c8
Compare
This is WIP. I unmarked and marked again as draft trigger CI (which does not run on draft PRs...) |
Opened cirruslabs/macos-image-templates#41 to avoid adding |
... and remove non-existing --shallow argument from brew tap.
66d7269
to
dacab1b
Compare
I'm working on a further PR that adds MSVCs build on native Windows. This will give us a real new dimension in the configuration space. And there's still the idea to build with multiple versions of gcc and clang.