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

Support Boost 1.87 #368

Merged
merged 12 commits into from
Jan 7, 2025
Merged

Support Boost 1.87 #368

merged 12 commits into from
Jan 7, 2025

Conversation

bmerry
Copy link
Contributor

@bmerry bmerry commented Jan 7, 2025

Boost 1.87 dropped a lot of deprecated functionality that spead2 was still using. Modernise to use the new forms, which seem to have been available since at least Boost 1.70, which is now the minimum.

bmerry added 12 commits January 7, 2025 08:37
Homebrew deleted 1.86, so a bump was needed.
Boost.Asio made this rename ages ago, but kept the old name as an alias.
Boost 1.87 drops the alias though, breaking all the spead2 code.

Apart from switching the use of the Boost type, also rename classes
(io_service_ref) and methods (get_io_service) to match, keeping
deprecated aliases so that spead2 users have a migration path.
io_context::post and ip::address[_v4/6]::from_string have been removed
and replaced by boost::asio::post and ip::make_address[_v4/6], and
io_context::work has been replaced by executor_work_guard.
The code was already using features introduced by 1.70, but the
meson.build indicated 1.69 as the minimum. Nobody has complained, so I'm
just bumping the minimum to 1.70.
One function wasn't actually used, so delete it.
The minimum version that works is 1.70, so this code is no longer
useful.
This was breaking cibuildwheel.
@bmerry bmerry requested a review from james-smith-za January 7, 2025 12:51
Copy link
Contributor

@james-smith-za james-smith-za left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bmerry bmerry merged commit 61d4750 into master Jan 7, 2025
74 checks passed
@bmerry bmerry deleted the bump-boost-1.87 branch January 7, 2025 14:21
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.

2 participants