<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/106129/">http://git.reviewboard.kde.org/r/106129/</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 17th, 2012, 8:24 p.m., <b>Jekyll Wu</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;">Thanks for the update. Looks fine to me, although I really want to try the patch in a real environment before applying it. Basically I just followed the instruction from the openbsd-kde google group, but unfortunately it did not work that time. I will try it again sometime later.
Anyway, the patch in its current form won't bring any danger to users on other platforms, so I will push it so that more
OpenBSD users have a chance of trying it. And if it goes wrong badly, OpenBSD packagers can disable this feature by commenting off just two lines as temporary workaround.
Be prepared for bug reports. I might just redirect related bug reports to you :)
</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;">Don't worry, those reports will be more likely directed to me initially 'cause I'm one of the KDE-on-OpenBSD packagers you mentioned. :) And a few people already tested it on i386 and amd64 platforms.
Our project isn't "official" KDE porting project yet; I want to get 4.9.2 fully available, resulting in potentially wider test audience. And only then push all the patches we have upstream. Current patch is just one of the biggest and thus annoying ones, so I published it earlier. :)</pre>
<br />
<p>- Vadim</p>
<br />
<p>On October 4th, 2012, 10:50 a.m., Vadim Zhukov 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 Konsole and Jekyll Wu.</div>
<div>By Vadim Zhukov.</div>
<p style="color: grey;"><i>Updated Oct. 4, 2012, 10:50 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;">ATM Konsole does not support gathering process information on OpenBSD. I've a patch adding such support.
The first idea was to extend already existing FreeBSDProcessInfo class. I renamed it to a BSDProcessInfo and added a few #ifs here and there. After some input I've created separate OpenBSDProcessInfo class based on FreeBSDProcessInfo code.
I've also fixed a few nits regarding calling C routines (prepended "::" to sysctl() and such; added kWarning() printouts if they fail) in new code. If needed I can do them as a separate diff.</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;">On OpenBSD-CURRENT.</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>src/ProcessInfo.cpp <span style="color: grey">(32c86b1)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/106129/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>