Skip to content
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] rviz_default_plugins package deps #814

Closed
wants to merge 1 commit into from

Conversation

wodtko
Copy link

@wodtko wodtko commented Jan 4, 2022

image_transport switched to modern CMake style.
by this change, a missing dependency export led to errors while linking the rviz_default_plugin package to other custom rviz_plugins.
I.e., the linker was not able to find -limage_transport::image_transport.
CMake resolves the issue and finds the package if it is added to the dependency list of rviz_default_plugins.

@iuhilnehc-ynos
Copy link

Fixed in the #813

@clalancette
Copy link
Contributor

Closing as a duplicate of the (now merged) #813

@clalancette clalancette closed this Jan 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants