[Digikam-devel] Re: [Kde-imaging] Re: Re: Git migration for 2.0.0 + code components re-structuring...

[Michael G. Hansen]

On 12/15/2010 06:41 AM, Gert Kello wrote:
>> I hope this solves all the FindOpenCV issues for now. Please let me know
>> whether it works, and thanks for trying it!
>
> Almost. If I use "external opencv find" flag then kipi-plugins do not
> find opencv:
>
> -- CMake version (cleaned): cmake version 2.8.3
>
> --
> -- ----------------------------------------------------------------------------------
> --  kipi-plugins 2.0.0-beta1 dependencies results
> <http://www.kipi-plugins.org>
> --
> --  libjpeg library found.................... YES
> --  libtiff library found.................... YES
> --  libpng library found..................... YES
> --  libkipi library found.................... YES
> --  libkexiv2 library found.................. YES
> --  libkdcraw library found.................. YES
> --  libkmap library found.................... YES
> --  libxml2 library found.................... YES (optional)
> --  libxslt library found.................... YES (optional)
> --  libexpat library found................... YES (optional)
> --  native threads support library found..... YES (optional)
> --  libopengl library found.................. YES (optional)
> --  Qt4 OpenGL module found.................. YES
> --  libopencv library found.................. NO  (optional)
>
> Maybe:
> Index: extra/kipi-plugins/CMakeLists.txt
> ===================================================================
> --- extra/kipi-plugins/CMakeLists.txt   (revision 1206613)
> +++ extra/kipi-plugins/CMakeLists.txt   (working copy)
> @@ -142,6 +142,12 @@
>   MACRO_OPTIONAL_FIND_PACKAGE(OpenCV)     # For RemoveRedEyes.
>   MACRO_OPTIONAL_FIND_PACKAGE(QJSON)      # For Debian Screenshots.
>
> +IF (NOT OPENCV_FOUND)
> +    IF (OpenCV_VERSION VERSION_GREATER "2.0.99")
> +        SET(OPENCV_FOUND TRUE)
> +    ENDIF (OpenCV_VERSION VERSION_GREATER "2.0.99")
> +ENDIF (NOT OPENCV_FOUND)
> +

Thanks for finding this one! Actually it looks like OPENCV_FOUND is the 
'legacy' name in the "Jan Woetzel" FindOpenCV.cmake, and the 'new' name 
is OpenCV_FOUND. Fixed in commit 1206767.

Michael