-
Notifications
You must be signed in to change notification settings - Fork 205
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
remove duplicate include dirs and link libraries #533
remove duplicate include dirs and link libraries #533
Conversation
Signed-off-by: Dirk Thomas <dirk-thomas@users.noreply.github.com>
Signed-off-by: Dirk Thomas <dirk-thomas@users.noreply.github.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.
LGTM assuming green CI
Signed-off-by: Dirk Thomas <dirk-thomas@users.noreply.github.com>
See latest builds on ament/ament_cmake#251. |
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 assuming @sloretz's concern has been addressed, which it seems like is the case given the CI on ament/ament_cmake#251 (comment)
Resolve merge conflicts from ros2#533
Otherwise the repeated include dirs / libraries exceed the maximum command line length for the linker invocation (together with ament/ament_cmake#251).
It might also help to make the packaging job turn over again.