-
Notifications
You must be signed in to change notification settings - Fork 175
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
cmake, diagnose, rpc: Backport #2788 and #2786 to hotfix #2794
base: hotfix
Are you sure you want to change the base?
Conversation
"find_package(Boost CONFIG)" finds the upstream package directly, without the find module. BoostConfig.cmake package configuration file is provided since Boost 1.70, so earlier versions couldn't be found.
Disable MSYS2 job because there are no trivial ways to install older Boost. Conflicts: .github/workflows/cmake-ci.yml
`boost::asio::ip::tcp::resolver::query` has been removed in 1.87 which causes compilation issues. This raises the minimum supported version of Boost to 1.66 which should be safe since near-EOL focal has 1.71.
`boost::asio::ip::tcp::resolver::iterator` has been removed in Boost 1.87.
I'm at a loss for words.
Why are we doing this PR? The PR's in question have already been merged and are in the testnet branch for a release, which is imminent. No backport is necessary. |
Sorry, wasn't aware of the release schedule. Mostly I just wanted to have the (at time of writing) current release buildable on my system and I may have jumped the gun a little. |
That's ok. Thanks for going through the effort. You are welcome in the meantime to build and run your local hotfix branch on which this PR derives. This will be a production compatible release then. |
Version 5.4.8.0-hotfix-1 does not build on Arch with the latest updates. This PR backports PRs #2788 and #2786 to allow building on an updated Arch linux build.
Compiles, passes tests, and runs on my machine.