-
Notifications
You must be signed in to change notification settings - Fork 197
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
Export tf2 dependency from tf2_ros #72
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.
Yep, good call.
CI is failing because this needs to be rebased after ros2/rcutils#113 was merged. @crdelsey would you mind doing a quick rebase? Thanks. |
tf2_ros brings in tf2 headers. This dependency was not being exported before. As a consequence of this, if a package included the tf2_ros package and included the `transform_listener.h` header, the package would fail to compile.
b8b9987
to
d91d6a0
Compare
@clalancette The branch is rebased now. |
|
No worries :). |
The unstable builds on all of this is because of a different problem unrelated to this PR. I'm going to merge this up. Thanks for your contribution! |
tf2_ros brings in tf2 headers. This dependency was not being
exported before. As a consequence of this, if a package
included the tf2_ros package and included the
transform_listener.h
header, the package would fail to compile.