ROS Resources: Documentation | Support | Discussion Forum | Index | Service Status | ros @ Robotics Stack Exchange
Ask Your Question
5

Do Fuerte packages using Eigen need to depend on common_rosdeps?

asked 2012-05-04 17:33:11 -0500

I'm a bit confused about this. The Eigen wiki page indicates that, for Fuerte, a package's manifest.xml needs to have

  <depend package="common_rosdeps" />
  <rosdep name="eigen" />

.. but the common_rosdeps wiki page says that this "stack/package" (which is it?) is "deprecated" and "no longer necessary".

I've tried commenting out both lines above and my code still seems to build happily so maybe this only matters in a rosinstall situation (which I tend not to do). But at any rate the information on the Eigen wiki page is still a bit confusing since it recommends depending on a deprecated package.

edit retag flag offensive close merge delete

2 Answers

Sort by ยป oldest newest most voted
5

answered 2012-05-05 04:32:18 -0500

Kevin gravatar image

Eigen is now an external library so if you use Eigen, make sure your CMakeLists.txt has:

find_package(Eigen REQUIRED)
include_directories(${Eigen_INCLUDE_DIRS})

so your code will compile. You don't need to add common_rosdeps anymore for Fuerte, you are correct, it is depreciated.

edit flag offensive delete link more

Comments

Thanks. I tweaked the Eigen wiki page to hopefully make it clearer.

Patrick Bouffard gravatar image Patrick Bouffard  ( 2012-05-06 10:57:50 -0500 )edit
-1

answered 2012-05-05 12:24:45 -0500

anonymous user

Anonymous

do I have to remove <depend package="eigen"> from manifest.xml?

edit flag offensive delete link more

Comments

1

sort of ... change it to: <rosdep name="eigen">

Kevin gravatar image Kevin  ( 2012-05-05 12:26:23 -0500 )edit
1

Eigen is not longer a package, but an external library now

Kevin gravatar image Kevin  ( 2012-05-05 12:30:25 -0500 )edit

Thanks :)

anonymous userAnonymous ( 2012-05-05 14:15:39 -0500 )edit

Question Tools

1 follower

Stats

Asked: 2012-05-04 17:33:11 -0500

Seen: 1,087 times

Last updated: May 05 '12