-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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 bson #17553
Add bson #17553
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's already a key python-xdot
Line 3922 in c075a34
python-xdot: |
After a bit of archaeology: At the time of the renaming, issues have been opened on the repos using it to notify of the change: If you want to compile these packages from source you will need to rename the key in their package.xml. If you can submit PRs on these repos with the change that would be valuable as well 😄 The bson change can stay in this PR though 👍 but the xdot rule should be removed. Thanks! |
@mikaelarguedas Done, removed the commit about xdot. Sorry for the noise! I didn't find that information at the moment. Interesting. I'll need to patch that for the Gentoo packages building. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, thanks @awesomebytes
@tfoote All good now? |
@awesomebytes Please actually submit the patch upstream so that the code uses the correct rosdep keys. It will be needed in the next release anyway and it will be better to have it upstream instead of just in gentoo. |
@tfoote Done:
|
bson
is a dependency forrosbridge_library
, needed for Gentoo (as seen in ros/ros-overlay#560).xdot
is a run depend of ecto.