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

extrinsic_cal: missing find_package() for yaml-cpp #14

Closed
gavanderhoorn opened this issue Jan 21, 2014 · 2 comments
Closed

extrinsic_cal: missing find_package() for yaml-cpp #14

gavanderhoorn opened this issue Jan 21, 2014 · 2 comments

Comments

@gavanderhoorn
Copy link
Member

The yaml-cpp library is added as a linking dependency in:

target_link_libraries(industrial_extrinsic_cal_ceres
  yaml-cpp ${catkin_LIBRARIES})

without first a find_package().

ROS depends on it as well, so it will probably always be present when industrial_calibration is build, but it might be a good idea to find_package() it.

@gavanderhoorn
Copy link
Member Author

This is still an issue in #29. A solution for #30 might depend on proper use of find_package(..) for yaml-cpp.

@VictorLamoine
Copy link
Contributor

@drchrislewis This (and #50) can be closed as #62 fixed this issue, here and here.

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

No branches or pull requests

3 participants