Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge bitcoin-core/secp256k1#1412: ci: Switch macOS from Ventura to M…
…onterey and add Valgrind c223d7e ci: Switch macOS from Ventura to Monterey and add Valgrind (Hennadii Stepanov) Pull request description: This PR switches the macOS native job from Ventura to Monterey, which allows to support Valgrind. Both runners--`macos-12` and `macos-13`--have the same clang compilers installed: - https://github.com/actions/runner-images/blob/main/images/macos/macos-12-Readme.md - https://github.com/actions/runner-images/blob/main/images/macos/macos-13-Readme.md But Valgrind works fine on macOS Monterey, but not on Ventura. See: bitcoin-core/secp256k1#1392 (comment). The Homebrew's Valgrind package is cached once it has been built (as it was before bitcoin-core/secp256k1#1152). Therefore, the `actions/cache@*` action is needed to be added to the list of the allowed actions. bitcoin-core/secp256k1#1412 (comment): > By the way, this solves bitcoin#1151. ACKs for top commit: real-or-random: ACK c223d7e I tested that a cttest failure makes CI fail: https://github.com/real-or-random/secp256k1/actions/runs/6010365844 Tree-SHA512: 5e72d89fd4d82acbda8adeda7106db0dad85162cca03abe8eae9a40393997ba36a84ad7b12c4b32aec5e9230f275738ef12169994cd530952e2b0b963449b231
- Loading branch information