<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/119413/">https://git.reviewboard.kde.org/r/119413/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Kipi-plugins won't link against OpenCV :</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Undefined symbols for architecture x86_64:<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
"_cvArcLength", referenced from:<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
KIPIRemoveRedEyesPlugin::CBlobGetHullPerimeter::operator()(KIPIRemoveRedEyesPlugin::CBlob const&) const in Blob.cpp.o<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
"_cvContourArea", referenced from:<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
KIPIRemoveRedEyesPlugin::CBlobGetHullArea::operator()(KIPIRemoveRedEyesPlugin::CBlob const&) const in Blob.cpp.o<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
"_cvConvexHull2", referenced from:<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
KIPIRemoveRedEyesPlugin::CBlob::GetConvexHull(CvSeq<em style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;"><em style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">) const in Blob.cpp.o<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
KIPIRemoveRedEyesPlugin::CBlobGetHullPerimeter::operator()(KIPIRemoveRedEyesPlugin::CBlob const&) const in Blob.cpp.o<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
KIPIRemoveRedEyesPlugin::CBlobGetHullArea::operator()(KIPIRemoveRedEyesPlugin::CBlob const&) const in Blob.cpp.o<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
"_cvCvtColor", referenced from:<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
KIPIRemoveRedEyesPlugin::HaarClassifierLocator::findPossibleEyes(double, int, char const</em>) in haarclassifierlocator.cpp.o<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
"_cvDilate", referenced from:<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
KIPIRemoveRedEyesPlugin::HaarClassifierLocator::generateMask(int, CvSeq</em>) in haarclassifierlocator.cpp.o<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
"_cvErode", referenced from:<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
KIPIRemoveRedEyesPlugin::HaarClassifierLocator::generateMask(int, CvSeq<em style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">) in haarclassifierlocator.cpp.o<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
"_cvFillImage", referenced from:<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
KIPIRemoveRedEyesPlugin::HaarClassifierLocator::findBlobs(_IplImage</em>, int) in haarclassifierlocator.cpp.o<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
KIPIRemoveRedEyesPlugin::HaarClassifierLocator::allocateBuffers() in haarclassifierlocator.cpp.o<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
"_cvFitEllipse2", referenced from:<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
KIPIRemoveRedEyesPlugin::CBlob::GetEllipse() const in Blob.cpp.o<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
"_cvHaarDetectObjects", referenced from:<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
KIPIRemoveRedEyesPlugin::HaarClassifierLocator::findPossibleEyes(double, int, char const<em style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">) in haarclassifierlocator.cpp.o<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
"_cvLoadImage", referenced from:<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
KIPIRemoveRedEyesPlugin::HaarClassifierLocator::startCorrection(QString const&, QString const&) in haarclassifierlocator.cpp.o<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
"_cvReleaseHaarClassifierCascade", referenced from:<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
KIPIRemoveRedEyesPlugin::HaarClassifierLocator::findPossibleEyes(double, int, char const</em>) in haarclassifierlocator.cpp.o<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
"_cvSaveImage", referenced from:<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
KIPIRemoveRedEyesPlugin::HaarClassifierLocator::saveImage(QString const&, KIPIRemoveRedEyesPlugin::Locator::SaveResult) in haarclassifierlocator.cpp.o<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
"_cvSmooth", referenced from:<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
KIPIRemoveRedEyesPlugin::HaarClassifierLocator::removeRedEyes() in haarclassifierlocator.cpp.o<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
"_cvThreshold", referenced from:<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
KIPIRemoveRedEyesPlugin::HaarClassifierLocator::generateMask(int, CvSeq*) in haarclassifierlocator.cpp.o<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
[ 49%] ld: symbol(s) not found for architecture x86_64<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
clang: error: linker command failed with exit code 1 (use -v to see invocation)</p></pre>
<br />
<p>- Gilles Caulier</p>
<br />
<p>On July 23rd, 2014, 9:13 a.m. UTC, Vadim Zhukov wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for Digikam.</div>
<div>By Vadim Zhukov.</div>
<p style="color: grey;"><i>Updated July 23, 2014, 9:13 a.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kipi-plugins
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><ol style="padding: 0;text-rendering: inherit;margin: 0 0 0 2em;line-height: inherit;white-space: normal;">
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">Force resetting OpenCV_LIBS before calling find_package() to make sure no extra libraries sneak in target_link_libraries() calls.</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">Limit KIPI plugins linking to only OpenCV components it really uses.</li>
</ol>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This, in particular, unbreaks building Digikam on OpenBSD, where different OpenCV components live in different packages. Before the patch, this forced to always have all OpenCV components installed, even if they aren't really needed while neither building or running Digikam.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Related reviews: https://git.reviewboard.kde.org/r/119201/ and https://git.reviewboard.kde.org/r/119414/ . All those three patches should be applied to have an effect on Digikam SC build.</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">OpenBSD/i386-CURRENT</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>CMakeLists.txt <span style="color: grey">(8a4a0d4)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/119413/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>