<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="https://git.reviewboard.kde.org/r/117379/">https://git.reviewboard.kde.org/r/117379/</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 de8d5859dcf93e52abb1c1fbfc40c95f962a6e57 by Aleix Pol on behalf of Dāvis Mosāns to branch master.</pre>
 <br />









<p>- Commit Hook</p>


<br />
<p>On April 11th, 2014, 11:10 a.m. UTC, Dāvis Mosāns wrote:</p>








<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://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 Dāvis Mosāns.</div>


<p style="color: grey;"><i>Updated April 11, 2014, 11:10 a.m.</i></p>









<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
kdev-mercurial
</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;">Various fixes for mercurial support. Basically updated so it can be built against latest kdevplatform. Also fixed tests so they pass.


Changes:

1. Removed DVCS view factory

  * Kdevplatform change commit 41db303595d55d355a80d896a25543f449819d30

2. Changed MercurialHeadsModel constructor to match VcsEventsWidget

  * Kdevplatform change commit a6d910861906365f0ef11b99badbbde01e6f3c7b

3. Fixed tests

4. Other changes
</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;">All tests pass, but they don't cover all functionality.

Manually verified that adding files, commits does work. Also commit history viewing works fine. Didn't tried remotes nor branches. They might not or might work in some cases.


I consider this in Alpha quality so basically it can be used to play a bit, but would require more work to be finished.</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">(c4f91a394215c8401579af38feba4982bed055dc)</span></li>

 <li>kdevmercurial.desktop <span style="color: grey">(7cc0db2e8cacbd8029d726414549fd694d4b7b2a)</span></li>

 <li>mercurialplugin.h <span style="color: grey">(a2817e3f8e534b09353574c64b1c6e0b701117b6)</span></li>

 <li>mercurialplugin.cpp <span style="color: grey">(e2aba19a1743318bdb849bff15c9a9bae17006ae)</span></li>

 <li>models/mercurialheadsmodel.h <span style="color: grey">(8e2ed09dbac64d03af68d936365adbbea41a58a9)</span></li>

 <li>models/mercurialheadsmodel.cpp <span style="color: grey">(41e6c7bbfcabc437932faf8d48cf73e831d03080)</span></li>

 <li>tests/CMakeLists.txt <span style="color: grey">(018e9fcec1f5d2ba0d865bccf82f07ddd5078436)</span></li>

 <li>tests/initTest.cpp <span style="color: grey">(44d050cedab71b1418f38a1ff03aa7ad26ca7dc6)</span></li>

 <li>ui/mercurialheadswidget.h <span style="color: grey">(8952b7b2c15e2606b82e58980f81d16e9df391f3)</span></li>

 <li>ui/mercurialheadswidget.cpp <span style="color: grey">(e35b5faa8fc2f1bfb9b40fafb676088849d224cf)</span></li>

 <li>ui/mercurialqueuesmanager.cpp <span style="color: grey">(bc8568c0c0911f7a4d21dfbe80824bf8a6116616)</span></li>

</ul>

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







  </td>
 </tr>
</table>








  </div>
 </body>
</html>