search

ros2 / ros_astra_camera

ROS wrapper for Astra camera
9 stars 12 forks source link

local_setup.bash doesn't put install/lib into LD_LIBRARY_PATH #7

Closed SecretaryBirds closed 7 years ago

SecretaryBirds commented 7 years ago

For whatever reason, this doesn't seem to work. Maybe a line missing from CMakeLists? I have to add it manually to get the node to launch.

dhood commented 7 years ago

this reminds me of https://github.com/ros2/navigation/pull/7. Probably a line like ament_export_libraries(astra_driver_lib) is missing, if you could try that?

mikaelarguedas commented 7 years ago

I think that @Kukanani (correct me if I;m wrong) faced this recently and has a fix for this and a few other missing exports or shared libs.

clalancette commented 7 years ago

What's weird is that it works perfectly fine for me like it is. Any thoughts as to what is going on?

dhood commented 7 years ago

perhaps you weren't using isolated install

SecretaryBirds commented 7 years ago

I'm just doing a straight 'ament build'. No flags. Nothing special. I am using CoreDX as the rmw implementation.

Kukanani commented 7 years ago

I fixed this problem recently on my machine, as well as correctly exporting include directories and everything else needed to build against this package. I'll make a PR shortly.