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

Use boost instead of arpa/inet for cross platform compatibility #152

Open
agutenkunst opened this issue Feb 17, 2021 · 0 comments
Open

Use boost instead of arpa/inet for cross platform compatibility #152

agutenkunst opened this issue Feb 17, 2021 · 0 comments
Labels
enhancement New feature or request

Comments

@agutenkunst
Copy link
Contributor

Everything network related should use boost::asio instead of <arpa/inet.h> for better Win compile compatibility.

I changed did this to some degree in #141 but there are still two places left after the merge.
On drawback is that the various boost versions have a slightly different interface with regards to boost::address_v4 but I think they can be easily abstracted away and may even fade away by time.

This should (hopefully) make the transition to ROS2 easier.

@agutenkunst agutenkunst added the enhancement New feature or request label Feb 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant