[Kde-imaging] Review Request 119654: Fixes for OpenCV detection (again)

[Vadim Zhukov]

> On Авг. 8, 2014, 11:10 д.п., Gilles Caulier wrote:
> > Hi Vadim,
> > 
> > "contrib" is mandatory with some distro. Look this report here :
> > 
> > https://bugs.kde.org/show_bug.cgi?id=338037
> > 
> > Other patch sound fine...
> > 
> > Gilles Caulier

Look at the actual committed patch then. ;)

 -DETECT_OPENCV(2.4.6 core highgui objdetect contrib)
 +DETECT_OPENCV(2.4.9 core highgui objdetect contrib legacy imgproc)

i.e., contrib was already there; i.e., it didn't help at all. And it couldn't, either.


- Vadim


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119654/#review64030
-----------------------------------------------------------


On Авг. 8, 2014, 11:14 д.п., Vadim Zhukov wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119654/
> -----------------------------------------------------------
> 
> (Updated Авг. 8, 2014, 11:14 д.п.)
> 
> 
> Review request for Kipi-plugins.
> 
> 
> Repository: kipi-plugins
> 
> 
> Description
> -------
> 
> Here is a patch for two problems arised between my OpenCV linking-related patches went in and 4.2.0 release:
> 
> 1. Second call to FIND_PACKAGE() in MacroOpenCV.cmake should use ${OPENCV_REQUIRED_COMPONENTS}, too. This problem exists in digikam and libkface repos as well.
> 2. "contrib" found its way back to kipi-plugins CMakeLists.txt. It looks like a copy-paste error, too. I could be wrong, but things (red eye removal tool) do work fine on OpenBSD without it.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt f242f30 
>   cmake/modules/MacroOpenCV.cmake 476607f 
> 
> Diff: https://git.reviewboard.kde.org/r/119654/diff/
> 
> 
> Testing
> -------
> 
> OpenBSD/i386 5.6-BETA, KDE 4.13.3, Digikam SC 4.2.0.
> 
> 
> Thanks,
> 
> Vadim Zhukov
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-imaging/attachments/20140808/96483045/attachment-0001.html>