<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/118818/">https://git.reviewboard.kde.org/r/118818/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On June 19th, 2014, 12:01 a.m. CEST, <b>Milian Wolff</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;">While cool in principle, it fails to do what I had in mind :)
I think this should be done always, whenever a bzr repository is initiated by KDevelop. This is done in the unit test - no? So as soon as that happens, it should directly thereafter check for the whoami information and, if that fails, publish it with the data from KEMailSettings.
Right now, your patch only touches the test code, so if someone would try to initiate a bzr repo without first setting the whoami globally, it would fail - no?</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 problem is with informations from KEMailSettings. These are always wrong (at least for me - it consists of correct name and empty email address - default "local" identity). If these informations were used to initialize 'bzr whoami' and then to make commit, we would get broken repository history. As a result we would have to manually fix whoami informations and then rewrite history (which in general is not the best idea).
In fact considering all informations contained in various KDE PIM applications what is clear is that it is impossible to implicitly pick the correct one (machine would have to read my mind). It still may be done explicitly (by asking user to pick appropriate identity from set of configured KDE PIM identities) but I don't see clear way to achieve this.</pre>
<br />
<p>- Maciej</p>
<br />
<p>On June 18th, 2014, 10:59 p.m. CEST, Maciej Poleski 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 Maciej Poleski.</div>
<p style="color: grey;"><i>Updated June 18, 2014, 10:59 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kdevplatform
</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;">Detect if 'bzr whoami' informations are available during testing. If not use KEMailSettings to get defaults and set them on per-branch basis.</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;">Tested on environment with and without 'bzr whoami' informations.</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>plugins/bazaar/tests/bazaartest.cpp <span style="color: grey">(86ff36f)</span></li>
<li>plugins/bazaar/tests/bazaartest.h <span style="color: grey">(29248aa)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/118818/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>