cmake: fix WITH_VTK usage
authorAlexander Alekhin <alexander.alekhin@intel.com>
Wed, 29 Nov 2017 13:52:42 +0000 (16:52 +0300)
committerAlexander Alekhin <alexander.a.alekhin@gmail.com>
Wed, 29 Nov 2017 18:43:09 +0000 (21:43 +0300)
'WITH_' variables is intended to enable CMake scripts with some autodetection logic.
'WITH_' can be off, but components is really enabled via command-line options
with proper variables setup (including 'HAVE_').

CMakeLists.txt
cmake/OpenCVDetectVTK.cmake
modules/viz/CMakeLists.txt

index f73c8c8..4af166d 100644 (file)
@@ -656,7 +656,9 @@ if(WITH_MATLAB)
   include(cmake/OpenCVFindMatlab.cmake)
 endif()
 
-include(cmake/OpenCVDetectVTK.cmake)
+if(WITH_VTK)
+  include(cmake/OpenCVDetectVTK.cmake)
+endif()
 
 if(WITH_OPENVX)
   include(cmake/FindOpenVX.cmake)
index cf4eb2b..74f075e 100644 (file)
@@ -1,7 +1,3 @@
-if(NOT WITH_VTK)
-  return()
-endif()
-
 # VTK 6.x components
 find_package(VTK QUIET COMPONENTS vtkInteractionStyle vtkRenderingLOD vtkIOPLY vtkFiltersTexture vtkRenderingFreeType vtkIOExport NO_MODULE)
 IF(VTK_FOUND)
index a5468c5..3575a0c 100644 (file)
@@ -1,4 +1,4 @@
-if(NOT WITH_VTK OR NOT DEFINED HAVE_VTK OR NOT HAVE_VTK)
+if(NOT HAVE_VTK)
   ocv_module_disable(viz)
 endif()