-
Notifications
You must be signed in to change notification settings - Fork 10
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 rosbag2 to ros variants #24
Conversation
I wasn't sure whether it best belongs to `ros_core`, but ultimately settled on `ros_base`.
from pre-waffle triagge: @Karsten1987 Can you assign yourself in this one? |
@nuclearsandwich just a friendly ping |
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.
Seems like a reasonable place to add it. It would be good to get others to weigh in.
This looks good to me! I was just thinking "we should get rosbag in a variant" and here you're way ahead of me :) |
Forgive me for asking rather than digging for myself, but does rosbag2 depend (either directly or indirectly) on the ros1_bridge and therefore ROS 1? If so that's quite heavyweight for ros_base. |
It does not. The rosbag2_bag_v2 plugin lives in its own repository. |
I wasn't sure whether it best belongs to
ros_core
, but ultimately settled onros_base
.