search

smanders / externpro

build external projects with cmake
MIT License
13 stars 12 forks source link

eigen requires additional include directory for internal includes #253

Closed smanders closed 4 years ago

smanders commented 4 years ago

this was found by a project using ceres, which includes eigen headers

it would be a big change to existing Vantage and Plugin code, but I would prefer this be the base include directory (include/eigen_3.2.7/eigen3 instead of include/eigen_3.2.7) so that code would include eigen

#include <Eigen/Array>

instead of

#include <eigen3/Eigen/Array>

so that if it ever becomes eigen4 we would simply update the base include directory in the use script and not have to modify every include in all of the projects that use eigen

smanders commented 4 years ago

completed with commit to dev branch referenced above