Bug#841204: liblaser-geometry-dev: laser_geometry.h includes Eigen/Core instead of eigen3/Eigen/Core

[Johannes Schauer]

Hi,

Quoting Jochen Sprickerhof (2016-10-19 09:37:11)
> * Johannes Schauer <josch at debian.org> [2016-10-18 15:36]:
> > because libeigen3-dev installs to /usr/include/eigen3/Eigen/Core.
> > Otherwise, all users of liblaser-geometry-dev must manually add
> > /usr/include/eigen3 to their include path.
> 
> That's normally intentional, because cmake or pkg-config add the correct
> include path. If you don't use any of these, you have to add it manually.

right, the problem indeed seems to be a different one. Check the following
code:

cmake_minimum_required(VERSION 3.6)
project("test")
find_package(catkin REQUIRED COMPONENTS
      laser_geometry
  )
MESSAGE(${catkin_INCLUDE_DIRS})

I would expect that ${catkin_INCLUDE_DIRS} would include /usr/include/eigen3.
Instead I get:

-- catkin 0.7.4
/usr/include/usr/include/eigen3
-- Configuring done

Users of laser_geometry should not know that it is also requiring eigen3 and
then have to add that manually. Cmake/catkin should take care of adding the
right include directories, but the above looks wrong.

Thanks!

cheers, josch
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: signature
URL: <http://lists.alioth.debian.org/pipermail/debian-science-maintainers/attachments/20161019/7404abef/attachment.sig>