<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/103701/">http://git.reviewboard.kde.org/r/103701/</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;">Looks good in general, but: this patch makes libkexvi2 depend on cmake 2.8.6, due to the usage of WriteBasicConfigVersionFile.
kdelibs still depends on cmake 2.6.4.
So, I think this can't be committed as it is right now.
For KDE frameworks it would be ok, since this depends on cmake 2.8.7 right now and will depend on 2.8.8 once it has been released.
I'm not sure how to deal with kdelibs 4.x in this regard.
Let's discuss this on kde-buildsystem or k-c-d.</pre>
<br />
<p>- Alexander</p>
<br />
<p>On January 14th, 2012, 9:24 p.m., Yury Georgievich Kudryashov wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Build System and Digikam.</div>
<div>By Yury Georgievich Kudryashov.</div>
<p style="color: grey;"><i>Updated Jan. 14, 2012, 9:24 p.m.</i></p>
<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;">Now
find_package(LibKexiv2 NO_MODULE)
finds LibKexiv2, removing need for large FindLibKexiv2.cmake script.
I've decided to install LibKexiv2Config.cmake, not Kexiv2Config.cmake to avoid conflicts with FindKexiv2.cmake from kdelibs.
P.S.: I think about moving large parts to a macro.</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;">$ cmake .. -DCMAKE_INSTALL_PREFIX=$PWD/../install; cd ..;
$ cmake --find-package -DCMAKE_PREFIX_PATH=$PWD/i -DNAME=LibKexiv2 -DCOMPILER_ID=GNU -DLANGUAGE=CXX -DMODE=EXIST
$ cmake --find-package -DCMAKE_PREFIX_PATH=$PWD/i -DNAME=LibKexiv2 -DCOMPILER_ID=GNU -DLANGUAGE=CXX -DMODE=COMPILE
$ cmake --find-package -DCMAKE_PREFIX_PATH=$PWD/i -DNAME=LibKexiv2 -DCOMPILER_ID=GNU -DLANGUAGE=CXX -DMODE=LINK</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">(0ecd233)</span></li>
<li>LibKexiv2Config-external.cmake.in <span style="color: grey">(PRE-CREATION)</span></li>
<li>LibKexiv2Config.cmake.in <span style="color: grey">(PRE-CREATION)</span></li>
<li>libkexiv2/CMakeLists.txt <span style="color: grey">(8a6b822)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/103701/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>