-
Notifications
You must be signed in to change notification settings - Fork 426
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
doesnt build with boost 1.87, asio::io_service removed #2126
Comments
1.87 is still beta. If it's removed completely without a workaround it will break too many projects. However I will try to replace io_service by io_context anyway. |
@orignal the amount of work required is quite a lot actually. The upside is that I'm taking care of it, at least in the context of patching 2.54.0 for OpenBSD (we currently have Boost 1.84.0 there, so the current patches aim to make it work for both 1.84.0 and 1.87.0). For a sneak peek, check https://gist.github.com/lgv5/1831ac7c10d6a7de5d8e7e9d825ad69f . That's missing at least changes for the resolver (doing that now). Once it's ready, I can submit a PR. |
Fixed in 3474538 |
@orignal the gist https://gist.github.com/lgv5/1831ac7c10d6a7de5d8e7e9d825ad69f includes all the work. I'm able to run i2pd 2.54.0 against Boost 1.84.0 and 1.87.0-beta1. The somewhat complete list of changes are:
I'll prepare a PR tomorrow. |
Fixed with and tested with boost 1.87. |
And few more. |
Hi! Are there any plans for a release where this is fixed, even if it is just 2.54.0 with a couple of extra commits? |
It's not "couple of extra commits". We will discuss. |
Thanks :-) |
the io_service API was deprecated a while ago, and io_context should be used instead. cf boostorg/asio#110
The text was updated successfully, but these errors were encountered: