Closed Algomorph closed 9 years ago
Thanks for keeping us in line with OpenCV 3 and for not just grabbing all of OpenCV's libraries if not all of them are in fact required.
Since some of these changes may modify downstream packages if HAL is installed as a CMake export, I am going to test this using cmake HAL/ -DEXPORT_HAL=ON
, and not as an export using the typical make install
path, before I merge this.
1) Removed redundant calls to FindPackage(OpenCV ...) from HAL/CMakeLists.txt 2) Made sure OpenCV driver finds necessary OpenCV modules (different for OpenCV 2 & 3) 3) Restricted OpenCV modules used by the convert driver