Skip to content
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

Open
wants to merge 13 commits into
base: hotfix
Choose a base branch
from

Conversation

2001herne
Copy link

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.

CyberTailor and others added 13 commits February 11, 2025 03:06
"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.
@jamescowens
Copy link
Member

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.

@2001herne
Copy link
Author

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.

@jamescowens
Copy link
Member

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants