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

migrate eigen-config.cmake to the new cmake_modules package #492

Closed
wjwwood opened this issue Jul 29, 2013 · 4 comments
Closed

migrate eigen-config.cmake to the new cmake_modules package #492

wjwwood opened this issue Jul 29, 2013 · 4 comments
Assignees

Comments

@wjwwood
Copy link
Member

wjwwood commented Jul 29, 2013

Also we should convert it to a FindEigen.cmake module.

This will involve finding and fixing anyone who uses the existing catkin provided CMake config for Eigen.

@wjwwood
Copy link
Member Author

wjwwood commented Jul 29, 2013

@wjwwood
Copy link
Member Author

wjwwood commented Dec 5, 2013

One of the issues with doing this is that people currently use it via catkin so moving it would break things. We can do this for Indigo, but for Hydro and Groovy we can only add it to cmake_modules.

@dirk-thomas
Copy link
Member

Ticket filled for cmake_modules to readd the same functionality there: ros/cmake_modules#10

@dirk-thomas
Copy link
Member

Closed by a94d041

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

2 participants