arpg / HAL

Hardware Abstraction Layer library.
Other
36 stars 24 forks source link

freenect issue #217 has been resolved #98

Closed Algomorph closed 9 years ago

Algomorph commented 9 years ago

See https://github.com/OpenKinect/libfreenect2/issues/131

We now have to remove the temp fix from HAL/Drivers/Freenect/CMakeLists.txt, otherwise tinythreads.h is not found again. Here's the source which reflects this change (already tested):

find_package( freenect2 QUIET )

set( BUILD_Freenect2 "${freenect2_FOUND}" CACHE BOOL
  "Build Freenect2 driver for Kinect2" )

if( BUILD_Freenect2 )

  set(freenect2_DEFINITIONS "-DLIBFREENECT2_THREADING_TINYTHREAD")

  if( freenect2_FOUND )
    add_to_hal_include_dirs( ${freenect2_INCLUDE_DIRS} )
    add_to_hal_libraries( ${freenect2_LIBRARY} )
    add_to_hal_sources( Freenect2Driver.h Freenect2Driver.cpp
      Freenect2Factory.cpp depth_registration.h depth_registration.cpp depth_registration_cpu.h depth_registration_cpu.cpp)
    hal_set_compile_flags( ${CMAKE_CURRENT_SOURCE_DIR}/Freenect2Driver.cpp ${freenect2_DEFINITIONS} )
      hal_set_compile_flags( ${CMAKE_CURRENT_SOURCE_DIR}/Freenect2Factory.cpp ${freenect2_DEFINITIONS} )
  else()
      message("libfreenect2 not found. Freenect2 driver cannot be built")
  endif()
endif()
crheckman commented 9 years ago

@nimski you asked me about tinythreads.h recently, so I am guessing this is related?

Algomorph commented 9 years ago

I believe this was fixed, closing.