-
Notifications
You must be signed in to change notification settings - Fork 33
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
ament_export_dependencies any package with targets we linked against #89
Conversation
Signed-off-by: Shane Loretz <sloretz@openrobotics.org>
Note this doesn't completely fix the build described in #82 (comment) because other packages have the same issue, but it resolves the issue in this package. |
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 with green CI (though I'll suggest running with --packages-above rcl_logging_spdlog
, to make sure downstreams pick up these new exports properly).
I wouldn't expect that to tell us much because CI builds shared libraries, and that works with or without this PR. I can't make CI build static libraries because other packages with the same issue prevent building |
@Mergifyio backport humble galactic |
✅ Backports have been created
|
Resolves #82
When building static libraries CMake will link PRIVATE targets downstream as if they were PUBLIC. This calls
ament_export_dependencies
on all packages providing targets that are linked against so downstream packages have those targets available too.