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

fix: support for upcoming Boost 1.87.0 #245

Merged
merged 1 commit into from
Nov 19, 2024
Merged

fix: support for upcoming Boost 1.87.0 #245

merged 1 commit into from
Nov 19, 2024

Conversation

cho-m
Copy link
Contributor

@cho-m cho-m commented Nov 19, 2024

io_service was deprecated and replaced by io_context in 1.66.01. The upcoming Boost 1.87.0 will remove the deprecated API2.


BOOST_REQUIRED_VERSION is 1.67.0 so io_context should be available https://www.boost.org/doc/libs/1_67_0/doc/html/boost_asio/net_ts.html

I didn't change variable names for just the minimum diff needed to compile, but could modify those to ioc if preferred.

Footnotes

  1. https://github.com/boostorg/asio/commit/b60e92b13ef68dfbb9af180d76eae41d22e19356

  2. https://github.com/boostorg/asio/commit/ec0908c562102915423d8bd7aefd3079efbb6c86

io_service was deprecated and replaced by io_context in 1.66.0[^1].
The upcoming Boost 1.87.0 will remove the deprecated API[^2].

[^1]: boostorg/asio@b60e92b
[^2]: boostorg/asio@ec0908c
@cho-m cho-m mentioned this pull request Nov 19, 2024
7 tasks
@mhx
Copy link
Owner

mhx commented Nov 19, 2024

Nice catch, thanks! I'm surprised this didn't trigger any deprecation warnings.

@mhx mhx merged commit b14b3b0 into mhx:main Nov 19, 2024
1 check passed
@cho-m cho-m deleted the boost-1.87 branch November 19, 2024 22:09
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