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

Add rosdep rules for python3-bson #22333

Merged
merged 2 commits into from
Sep 24, 2019
Merged

Add rosdep rules for python3-bson #22333

merged 2 commits into from
Sep 24, 2019

Conversation

hidmic
Copy link
Contributor

@hidmic hidmic commented Sep 23, 2019

Added rules for:

to match python-bson platform coverage. For Gentoo and OSX, the keys are the same as IIUC the Python version being targeted depends on OS configuration and package manager respectively, but I don't have boxes at hand to try this out.

Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
@hidmic hidmic requested a review from a team as a code owner September 23, 2019 15:53
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
Copy link
Member

@tfoote tfoote left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The gentoo ones looks odd, but matches the existing rule so I'll assum it's fine. @allenh1 FYI

Copy link
Contributor

@allenh1 allenh1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Gentoo version is actually a custom ebuild we have in the overlay, dev-python/bson.

Added in ros/ros-overlay#574.

@hidmic
Copy link
Contributor Author

hidmic commented Sep 24, 2019

@allenh1 I see dev-python/bson made it here before and was shortly dropped in favor of dev-python/pymongo in #17593, days after ros/ros-overlay#574. I presume you mean that dev-python/bson is the right version for Python3+ installations. Can you confirm? I haven't personally dug enough into bson packaging nuisances myself.

@allenh1
Copy link
Contributor

allenh1 commented Sep 24, 2019

@hidmic I'm honestly not sure myself at this point... Let's leave it at dev-python/pymongo and just see if things break, I think. I'll definitely hear about it shortly if things aren't working. ;)

@dirk-thomas dirk-thomas merged commit fe39ef0 into ros:master Sep 24, 2019
@hidmic hidmic deleted the bson branch September 24, 2019 22:07
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.

4 participants