intel / collision-avoidance-library

A framework for testing and benchmarking collision avoidance strategies
Apache License 2.0
81 stars 39 forks source link

Realsense #33

Closed rchiossi closed 7 years ago

mbelluzzo commented 7 years ago

I tried to compile this branch without having librealsense in my path, cmake went OK but compilation failed because of missing header:

n file included from /collision-avoidance-library/sensors/RealSenseCamera.cc:20:0:
/collision-avoidance-library/sensors/RealSenseCamera.hh:18:31: fatal error: librealsense/rs.hpp: No such file or directory
 #include <librealsense/rs.hpp>

So either cmake check got a false positive or we are missing a ifdef

mbelluzzo commented 7 years ago

I think I said compilation went ok too fast, but may be something here as well, now that I have librealsense installed, linker is missing symbols during linkage. Tomorrow I will investigate more.

rchiossi commented 7 years ago

yeah, actually that's a problem with the other dependencies as well. They are not marked as required, but it won't compile without then. I wanted to compile the realsense stuff only if librealsense was available, but I still don't know how to do it in cmake. Anyway, we need to review our dependencies.

rchiossi commented 7 years ago

@mbelluzzo Also, can you paste the lines related to realsense when you run "cmake .."? and what are the mssing symbols you get when compiling?

rchiossi commented 7 years ago

see #35