Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Changes for Boost 1_70_0 #7288

Merged
merged 2 commits into from
May 7, 2019
Merged

Changes for Boost 1_70_0 #7288

merged 2 commits into from
May 7, 2019

Conversation

heifner
Copy link
Contributor

@heifner heifner commented May 6, 2019

Change Description

Consensus Changes

  • Consensus Changes

API Changes

  • API Changes

Documentation Additions

  • Documentation Additions

@spoonincode
Copy link
Contributor

This isn't enough for me when I tried it on macos, I still needed to pass -DBoost_NO_BOOST_CMAKE=ON. It's not yet clear to me why this is needed & what changed.

@heifner
Copy link
Contributor Author

heifner commented May 7, 2019

I did not see the cmake issue on Ubuntu.

As to what changed, the deprecated get_io_service of socket was removed and they modified the constructor to take a templated type instead of an io_context.

https://www.boost.org/doc/libs/1_67_0/doc/html/boost_asio/reference/basic_stream_socket/basic_stream_socket.html
vs
https://www.boost.org/doc/libs/1_70_0/doc/html/boost_asio/reference/basic_stream_socket/basic_stream_socket.html

@spoonincode
Copy link
Contributor

boost 1.70 appears to be the first version that includes its own cmake Config files. The FindBoost function in cmake (either the ancient one we still have in fc or even the newest in cmake 3.14+) will completely delegate boost configuration to the Config files if it finds them. The Config files aren't entirely compatible with the older FindBoost usage -- they only set up imported targets; something we don't use -- so our current cmake usage fails.

However, it seems like it only searches for the Config files in the default search paths, not in BOOST_ROOT. So I think you'd have to install 1.70 such that you don't need to specify BOOST_ROOT to suffer from the problem.

These changes look good but I don't think we can clear #7098 until we fix the cmake issues. There are a number of approaches we could take.

@heifner heifner merged commit 45d8b1e into develop May 7, 2019
@heifner heifner deleted the boost-1-70-0 branch May 7, 2019 14:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants