-
Notifications
You must be signed in to change notification settings - Fork 247
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
Fix the build with rmw_fastrtps_dynamic. #1416
Fix the build with rmw_fastrtps_dynamic. #1416
Conversation
When building with *only* rmw_fastrtps_dynamic, there is no test named "test_play_services__rmw_fastrtps_cpp" to mark as xfail. Instead, it is called "test_play_services__rmw_fastrtps_dynamic_cpp", so make sure to add a different xfail marking for that test. Signed-off-by: Chris Lalancette <[email protected]>
FYI: the Rpr job here is failing because All of that is to say is that I don't think that it should prevent this PR from going in. |
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
|
When building with *only* rmw_fastrtps_dynamic, there is no test named "test_play_services__rmw_fastrtps_cpp" to mark as xfail. Instead, it is called "test_play_services__rmw_fastrtps_dynamic_cpp", so make sure to add a different xfail marking for that test. Signed-off-by: Chris Lalancette <[email protected]> (cherry picked from commit 744cdf0)
https://github.com/Mergifyio backport humble |
✅ Backports have been created
|
When building with *only* rmw_fastrtps_dynamic, there is no test named "test_play_services__rmw_fastrtps_cpp" to mark as xfail. Instead, it is called "test_play_services__rmw_fastrtps_dynamic_cpp", so make sure to add a different xfail marking for that test. Signed-off-by: Chris Lalancette <[email protected]> (cherry picked from commit 744cdf0)
When building with *only* rmw_fastrtps_dynamic, there is no test named "test_play_services__rmw_fastrtps_cpp" to mark as xfail. Instead, it is called "test_play_services__rmw_fastrtps_dynamic_cpp", so make sure to add a different xfail marking for that test. Signed-off-by: Chris Lalancette <[email protected]> (cherry picked from commit 744cdf0) Co-authored-by: Chris Lalancette <[email protected]>
When building with *only* rmw_fastrtps_dynamic, there is no test named "test_play_services__rmw_fastrtps_cpp" to mark as xfail. Instead, it is called "test_play_services__rmw_fastrtps_dynamic_cpp", so make sure to add a different xfail marking for that test. Signed-off-by: Chris Lalancette <[email protected]> (cherry picked from commit 744cdf0) Co-authored-by: Chris Lalancette <[email protected]>
When building with only rmw_fastrtps_dynamic, there is no test named "test_play_services__rmw_fastrtps_cpp" to mark as xfail. Instead, it is called
"test_play_services__rmw_fastrtps_dynamic_cpp", so make sure to add a different xfail marking for that test.
This should fix the failing build at https://build.ros2.org/view/Rci/job/Rci__nightly-fastrtps-dynamic_ubuntu_jammy_amd64/