Skip to content

Removed Boost dependency #176

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

Closed
wants to merge 12 commits into from
Closed

Removed Boost dependency #176

wants to merge 12 commits into from

Conversation

3p3r
Copy link
Contributor

@3p3r 3p3r commented Sep 10, 2017

  • Added the standalone ASIO as a replacement for Boost ASIO
  • Added Catch++ as a replacement for Boost Test framework
  • Removed dependency on Boost DateTime and Boost Lexical Cast

** I have not removed Boost from examples due to lack of time!

@jeduden
Copy link

jeduden commented Oct 3, 2017

@sepehr-laal is the plan to support both boosts asio and standalone asio ?
The project i have has multiple networking libraries ( socket
Io and cppnet ) and uses boost.
So it makes sense to use just boosts asio.

Also cppnet lib went from using boost.asio to standalone asio back to boost.asio ( see cpp-netlib/cpp-netlib#629 )

@3p3r
Copy link
Contributor Author

3p3r commented Oct 3, 2017

My original plan was to stop supporting Boost (although, I can see why supporting both can be useful). I got rid of Boost altogether since it's such a pain to work with and install on non Linux platforms.

I don't really understand why cppnet went back to Boost. Going through the thread you linked, it seems like an anecdotal reason to me (to go back to Boost).

These days C++ standard library is pretty powerful. Boost is really no longer needed for simple packages.

@jeduden
Copy link

jeduden commented Oct 5, 2017

Excep for asio ;) until networking has landed in the stl

@knro
Copy link

knro commented Mar 20, 2018

what's the status of this? I want to use this as well without Boost.

@TheOnlyJoey
Copy link

Same, boost is a burden to work with, +1 for a native method.

@cjh1
Copy link

cjh1 commented Dec 16, 2018

+1 for removing boost, any movement on this?

@ppamorim
Copy link

3 years and no feedback. This project is abandoned.

@xinthose
Copy link

@ppamorim SocketIO in general has not had an update since 2017 (blog). Although @darrachequesne still makes commits for the company.

darrachequesne pushed a commit that referenced this pull request Jan 7, 2021
darrachequesne pushed a commit that referenced this pull request Jan 7, 2021
- add the standalone ASIO as a replacement for Boost ASIO
- add Catch++ as a replacement for Boost Test framework
- remove dependency on Boost DateTime and Boost Lexical Cast
@darrachequesne
Copy link
Member

Merged as af68bf3. Sorry for the delay... Could you please ping me on issues/PR that you would like to see worked on? Thanks.

@xinthose
Copy link

xinthose commented Jan 7, 2021

3.5 years of inactivity is outside of my comfort zone for code I use in production

druckdev added a commit to DeerSoftDevelopment/socket.io-client-cpp that referenced this pull request Apr 6, 2021
* upstream/master:
  docs: update compatibility table for Socket.IO v4
  docs: fix title format (socketio#297)
  feat: allow resource path to be set in connection URI (socketio#134)
  fix: resolve client_impl::ping LOG call syntax in debug builds
  fix: resolve socketio#254: handle closing sockets upon on_fail events
  fix: lower the minimum CMake supported version
  feat: implement socketio#45: add support for logging configuration
  feat: add support for Socket.IO v3
  refactor: use correct Engine.IO protocol revision
  refactor: use standard install paths (socketio#58)
  ci: migrate to GitHub Actions
  chore: update .gitignore with cmake output
  refactor: remove Boost dependency (socketio#176)
druckdev added a commit to DeerSoftDevelopment/socket.io-client-cpp that referenced this pull request Apr 6, 2021
* tag '2.0.0':
  refactor: use correct Engine.IO protocol revision
  refactor: use standard install paths (socketio#58)
  ci: migrate to GitHub Actions
  chore: update .gitignore with cmake output
  refactor: remove Boost dependency (socketio#176)
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.

8 participants