ci: Compile with minimum supported g++ in olddeps - #330
Conversation
|
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. ReviewsSee the guideline and AI policy for information on the review process.
If your review is incorrectly listed, please copy-paste ConflictsReviewers, this pull request conflicts with the following ones:
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first. |
It seems g++10 is the minimum supported. |
ryanofsky
left a comment
There was a problem hiding this comment.
Code review ACK edf6343. Would maybe change title to "Compile with older GCC in olddeps" because it's not clear what the real minimum is. (Or if it is clear it would be nice to document).
Also this is only testing libmultiprocess build with the older GCC, so using older GCC for capnproto as well could be more realistic, but this PR is probably the simplest change
Happy to do this, but not sure how. Should I cherry-pick bitcoin/bitcoin#34580 into this repo ?
This seems like something for upstream to do. Recall they already have |
Looks like there is no CI coverage for the minimum supported g++ version?