mariusmuja/flann — Issue #17: TBB CMake problems
Author: xeri0n
Created: 2011-10-21 00:14:24+00:00
State: open

Marius,
There is a small mistake at the FindTBB.cmake file over at line 44.

I think
set(TBB_INCLUDE_DIRS ${PC_TBB_INCLUDEDIR})
should be
set(TBB_INCLUDE_DIRS ${TBB_INCLUDE_DIR})
otherwise its left as an empty string :)

Also was there any reason of not using something like this ?

if(TBB_FOUND AND TBB_DEBUG_FOUND)
target_link_libraries(flann_cpp_s ${TBB_LIBRARIES})
add_definitions("-DTBB")
endif(TBB_FOUND AND TBB_DEBUG_FOUND)

If found why not just add the definition ?
I guess its limits the options but why would not you wanna use TBB.