<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/102869/">http://git.reviewboard.kde.org/r/102869/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On October 16th, 2011, 9:43 a.m., <b>Alexander Neundorf</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Please don't commit this.
This is on purpose.
We install a file which includes CMakeParseArguments.cmake, so we *must* also install CMakeParseArguments.cmake itself, otherwise we may get in trouble if the version in cmake changes.
Alex</pre>
</blockquote>
<p>On October 16th, 2011, 9:55 a.m., <b>Ralf Jung</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The problem is that this results in a warning about CMake files being overridden when running cmake for kde-workspace. Plus, the files is not even exactly the same as the CMakeParseArgument.cmake that is shipped with my cmake version - it is missing the double-include guard.
If you do not want to use the system version of the file even if it exists, then why not rename this file to KDEParseAguments and fix the cmake files using it? After all you obviously don't even want the behaviour of the cmake default version, but specifically the behaviour of the version shipped in kdelibs. So there is no need to make the filename collide with the "official" version.
I could write up such a patch - is FindPackageHandleStandardArgs.cmake the only user?</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The issue was actually caused by a faulty cmake file in kde-workspace, so I am closing the review request here.</pre>
<br />
<p>- Ralf</p>
<br />
<p>On October 15th, 2011, 2:47 p.m., Ralf Jung 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 kdelibs.</div>
<div>By Ralf Jung.</div>
<p style="color: grey;"><i>Updated Oct. 15, 2011, 2:47 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;">Only install CMakeParseArguments.cmake for cmake versions that do not have it.
This fixes a warning when running cmake for kde-workspace, because the same file is shipped by cmake and installed into a modules folder.</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;">re-ran cmake and "make install" for kdelibs and kde-workspace, all working fine and the file is no longer installed (I am using cmake 2.8.5)</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>cmake/modules/CMakeLists.txt <span style="color: grey">(65379db)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/102869/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>