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
Link and compile against rosbag2_storage_mcap: Fixed issue 1492 #1496
Conversation
@ahcorde Could you please retarget your PR to the Rolling branch? |
739ad00
to
06ef74e
Compare
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
06ef74e
to
5944744
Compare
Sure @MichaelOrlov, Done |
@MichaelOrlov some tests are failing is this expected? |
Actually not. Those types of test failures are not expected. |
@ros-pull-request-builder retest this please |
CI is green, thank you @MichaelOrlov for launching CI again |
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.
@ahcorde What was a rationale for installing and exporting content of the include
folder?
There only one file with visibility macros.
https://github.com/ros2/rosbag2/tree/rolling/rosbag2_storage_mcap/include/rosbag2_storage_mcap
We didn't do this originally for a reason. i.e. it doesn't make any sense.
I would appreciate if you will remove it from delta.
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
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.
Looks good to me, but let's wait for @MichaelOrlov to approve before merging.
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.
https://github.com/Mergifyio backport iron |
✅ Backports have been created
|
https://github.com/Mergifyio backport humble |
* Link and compile against rosbag2_storage_mcap: Fixed issue 1492 Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com> * Don't install include floder Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com> --------- Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com> (cherry picked from commit 7fcb703)
✅ Backports have been created
|
* Link and compile against rosbag2_storage_mcap: Fixed issue 1492 Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com> * Don't install include floder Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com> --------- Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com> (cherry picked from commit 7fcb703)
Fixed issue #1492
How to test it
FYI @EricCousineau-TRI