-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
can it remove dependency of boost? #60
Comments
On Windows, we only use boost for our tests and for websockets. If you disable websockets on windows, this should remove the requirement for boost. However, on other platforms we use it to provide the backing threadpool and it can't be removed. |
Is there some other way to replace Boost with something else for threadpool? |
In addition to the threadpool, we also use boost asio for our entire socket implementation on non-windows. If you were only using the library for JSON parsing, it might be possible. Otherwise, I don't see how it would be possible without rewriting most of the library. |
To be fair, asio is also available in a non boost flavour. Sent from my Nexus 5.
|
This is interesting. It's possible that we could use Non-boost ASIO, we have not looked into that. Does that actually make it easier to use the library? |
Greetings, I would like to see if any progress has been made with respect to removing boost (or at least to see if a roadmap decision or goal was made). We are intrigued by the library and it's being rather elegant and cross-platform - but boost is just too heavy of a dependency to carry. We tried and put effort forth, too - but alas it's too huge. Please kindly revisit this issue. |
Not really. Exactly the same calls. Just has asio:: and not boost::asio:: GR |
Boost is too big to use, maybe once C++17 comes out, the boost depends could be able to be removed. |
My personal opinion (which is the only one that matters) is that boost I've seen new projects where devs were using boost::shard_ptr despite being Sent from my Nexus 5. Boost is too big to use, maybe once C++17 comes out, the boost depends — |
No description provided.
The text was updated successfully, but these errors were encountered: