<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/103551/">http://git.reviewboard.kde.org/r/103551/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On January 10th, 2012, 6:42 p.m. UTC, <b>Thomas Capricelli</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;">So.... is it still a no-go?</pre>
</blockquote>
<p>On January 11th, 2012, 2:01 p.m. UTC, <b>Bernd Buschinski</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;">I fear yes, still a no go as it currently is.
These things are welcome in another cleanup patch, but cleanup is unrelated to this topic.
As you said yourself our code is not the nicest, but it wont improve if we let other things slip into it under unrelated topics. And you wanted the patch to be minimal too</pre>
</blockquote>
<p>On February 6th, 2013, 10:46 p.m. UTC, <b>Albert Astals Cid</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;">Should we then discard this request so it does not clutter the reviewboard dashboard?</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;">I've spent a lot of time making this patch, testing it and then sending it here. But it seems "Bernd" is nitpicking on some oneliner for some reason. I guess it will never be merged, you can discard the request. That's frustrating and really not welcoming external contributors.</pre>
<br />
<p>- Thomas</p>
<br />
<p>On January 7th, 2012, 8:54 a.m. UTC, Thomas Capricelli 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 Konversation.</div>
<div>By Thomas Capricelli.</div>
<p style="color: grey;"><i>Updated Jan. 7, 2012, 8:54 a.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;">For people outthere like me that do not use KDE, the dependency on kdepim is a very huge one. We would very happily make the tradeoff of NOT being able to link nicknames to kde addressbook.
I guess you see the points, depending on kdepim brings two big problems
* installation/compilation : depending on kdepim brings half of kde packages, kdepim package and hence ugliness such as nepomuk/akonadi.
* at runtime, starting konversation, by the magic of dbus and KDE spaghetti runtime dependencies, it starts lot of unrelated KDE stuff daemons that takes both cpu and ram. I know this arg is denied by KDE zealot, but i stand on it.
So I did it. This patch allows compilation without kdepim. I've tried to keep the patch as minimal as possible of course.
I'm always on irc/freenode as 'orzel', and i'll be on #konversation for the next days.
Also, I do have commit rights on KDE (not sure about konversation proper, i did no follow the git migration very closely)</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 with kdepim installed and with kdepim not installed
compiles tested with kdepim not installed: compiles, link and work fine</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">(c8df038)</span></li>
<li>config-konversation.h.cmake <span style="color: grey">(4aaf9d1)</span></li>
<li>src/CMakeLists.txt <span style="color: grey">(a7ba9f4)</span></li>
<li>src/application.cpp <span style="color: grey">(038b403)</span></li>
<li>src/irc/nick.cpp <span style="color: grey">(298eafb)</span></li>
<li>src/irc/nickinfo.h <span style="color: grey">(20b8ef1)</span></li>
<li>src/irc/nickinfo.cpp <span style="color: grey">(e768d0d)</span></li>
<li>src/irc/nicksonline.h <span style="color: grey">(fbdd2cd)</span></li>
<li>src/irc/nicksonline.cpp <span style="color: grey">(8c9a4b1)</span></li>
<li>src/irc/outputfilter.cpp <span style="color: grey">(a7701e7)</span></li>
<li>src/irc/query.h <span style="color: grey">(3685a53)</span></li>
<li>src/irc/query.cpp <span style="color: grey">(1a432d4)</span></li>
<li>src/irc/server.h <span style="color: grey">(775cb7c)</span></li>
<li>src/irc/server.cpp <span style="color: grey">(f2d3168)</span></li>
<li>src/irc/serverison.h <span style="color: grey">(4500564)</span></li>
<li>src/irc/serverison.cpp <span style="color: grey">(f737b51)</span></li>
<li>src/konsolepanel.cpp <span style="color: grey">(14b32cc)</span></li>
<li>src/main.cpp <span style="color: grey">(f2c7a22)</span></li>
<li>src/mainwindow.cpp <span style="color: grey">(112dd39)</span></li>
<li>src/viewer/irccontextmenus.h <span style="color: grey">(40756d1)</span></li>
<li>src/viewer/irccontextmenus.cpp <span style="color: grey">(b735843)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/103551/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>