-
Notifications
You must be signed in to change notification settings - Fork 392
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 ROS 2 RHEL build #2119
Fix ROS 2 RHEL build #2119
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #2119 +/- ##
==========================================
- Coverage 80.39% 80.31% -0.08%
==========================================
Files 416 417 +1
Lines 16143 16142 -1
Branches 2252 2252
==========================================
- Hits 12978 12965 -13
- Misses 2369 2382 +13
+ Partials 796 795 -1
Flags with carried forward coverage won't be shown. Click here to find out more. |
…ibrary is not found (e.g. ROS 2 RHEL) Signed-off-by: Simon Hoinkis <simon.hoinkis@apex.ai>
b506099
to
7a1be0d
Compare
Not sure what's going on with the integration builds |
@elBoberido @mossmaurice This is most likely related to an issue in colcon: colcon/colcon-core#602 |
@mossmaurice can you update the |
Signed-off-by: Simon Hoinkis <simon.hoinkis@apex.ai>
@mossmaurice thanks very much :) |
Pre-Review Checklist for the PR Author
Add a second reviewer for complex new features or larger refactoringsCode follows the coding style of CONTRIBUTING.mdTests follow the best practice for testingiox-123-this-is-a-branch
)iox-#123 commit text
)task-list-completed
)All touched (C/C++) source code files fromiceoryx_hoofs
are added to./clang-tidy-diff-scans.txt
Notes for Reviewer
Checklist for the PR Reviewer
iceoryx_hoofs
have been added to./clang-tidy-diff-scans.txt
Post-review Checklist for the PR Author
References