<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/108556/">http://git.reviewboard.kde.org/r/108556/</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;">This review has been submitted with commit 855878fee5dff88363ad8db7fcc836234d02384b by Aleix Pol on behalf of Sergey Vidyuk to branch 4.5.</pre>

 <br />

<p>- Commit</p>

<br />

<p>On January 22nd, 2013, 7:20 p.m. UTC, Sergey Vidyuk wrote:</p>

<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">

 <tr>

  <td>

<div>Review request for KDevelop.</div>

<div>By Sergey Vidyuk.</div>

<p style="color: grey;"><i>Updated Jan. 22, 2013, 7:20 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;">Calculation of the executable path is done through the

ProjectExecutableTargetItem which respects RUNTIME_OUTPUT_DIRECTORY

target property and CMAKE_RUNTIME_OUTPUT_DIRECTORY variable values.

Addtionally Test::executable is not modified if kde4_add_unit_test

detection logick failed (target name calculated in the detection block

dies not exists in the code).

Note: this commit is made for 4.5 since at least for me wrong detection of unit test executable path is critical bug. If it's too late to add this fix to the upcommig release I can rebase it on master.</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;"> * Manually tested on a project modifying runtime target locations through the CMAKE_RUNTIME_OUTPUT_DIRECTORY cmake variable.

 * Run modified unit tests.</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>projectmanagers/cmake/parser/cmakeprojectvisitor.cpp <span style="color: grey">(27d81e1)</span></li>

 <li>projectmanagers/cmake/parser/cmaketypes.h <span style="color: grey">(be09d67)</span></li>

 <li>projectmanagers/cmake/testing/ctestutils.cpp <span style="color: grey">(09c83af)</span></li>

 <li>projectmanagers/cmake/tests/ctestfindsuitestest.cpp <span style="color: grey">(9e81e3b)</span></li>

 <li>projectmanagers/cmake/tests/manual/unit_tests/CMakeLists.txt <span style="color: grey">(21ba34c)</span></li>

 <li>projectmanagers/cmake/tests/manual/unit_tests_kde/CMakeLists.txt <span style="color: grey">(cc49da0)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/108556/diff/" style="margin-left: 3em;">View Diff</a></p>

  </td>

 </tr>

</table>

  </div>

 </body>

</html>