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 python-bson to pull pymongo in Gentoo #17593

Merged

Conversation

awesomebytes
Copy link
Contributor

Apparently for the other distros the bson dependency has been generated into a different package from the main pymongo package. For example, Ubuntu Xenial: https://packages.ubuntu.com/xenial/python-bson comes from the same source than all pymongo.

And the library python-bson says it is stripped from pymongo bson and made a package itself but is not compatible with who uses it... in our case rosbridge_library.

Also python-bson seems to be in a unstable state currently, they seem to be changing the name of the library itself (to py-bson):
py-bson/bson#41

And rosbridge_library (who actually depends on it) checks for the correct library here:
https://github.com/RobotWebTools/rosbridge_suite/blob/develop/rosbridge_library/src/rosbridge_library/util/__init__.py#L19-L27

There are plenty of issues with things that are different from the pymongo bson: https://github.com/py-bson/bson/issues

Until this py-bson doesn't become a real alternative, or Gentoo, or someone in ros-overlay with more time to dedicate to this headache fixes it... I think this is quite a good workaround.

As a note, I noticed all this when actually running rosbrige_server in Gentoo. I never was able to run it until today, so I could find all this only today. What a mess.

Sorry for all the noise.

Apparently for the other distros the bson part has been generated into a different package.

And the library that says it stripped from pymongo bson and made a package itself is not compatible
with those neither the code that uses it.
@awesomebytes
Copy link
Contributor Author

awesomebytes commented Apr 24, 2018

@allenh1 when this gets merged, can we have another sync please? Thank you. It will still need ros/ros-overlay#578 too

@mikaelarguedas mikaelarguedas merged commit db66e61 into ros:master Apr 25, 2018
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.

3 participants