<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/124304/">https://git.reviewboard.kde.org/r/124304/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On July 10th, 2015, 9:03 a.m. UTC, <b>David Faure</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Guys, shortening this value won't make anything faster. Instead, in case an app takes 26 seconds to start (slow machine, busy system, lots of initialization code...) the caller will get an error message erroneously.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Did you never see that in kdelibs4? Type kmail twice in a terminal, the first one goes through some slow path for some reason, and the second one tells you "DBus communication error, couldn't communicate with running instance blah blah". There's no error though.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Yes it's a blocking call, but it's done by the just-starting second-instance of the app, which has shown no GUI yet, so this isn't going to block some GUI for the user. Waiting is better than a wrong error IMHO.</p></pre>
</blockquote>
<p>On July 10th, 2015, 9:22 a.m. UTC, <b>Martin Klapetek</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I would say that error is correct though, because it does fail to communicate with the running instance within a reasonable limit (and same error will come if app startup takes 301 seconds...old machines/mobile devices etc). I'd say such application should be fixed to not block itself during startup; receiving activation request after 2 minutes and suddenly popping to the front seems...kinda broken anyway. Also application that is unusable for minutes after launching is a sign of bad design and this feels like providing a workaround for it.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Nevertheless, this was just something I stumbled upon while reading the code and I wasn't sure if that was intended or not. If you, the maintainer, believe this should stay, I'll just discard this.</p></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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Yes it's intended, the comment in the source code even made it quite explicit :-)</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">26 seconds, I have seen in the past. 301 never.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">It's not necessarily bad design for an app to take 26 seconds to initialize, if it does provide some feedback (e.g. a popup "please wait, I'm reindexing your mail folders"). There is actually a good argument for not being available on DBus before doing that, since the app isn't ready to handle incoming calls, in this "being repaired" state. This is a made up example though, I don't remember which exact cases we had where this happened.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I maintain that the error is incorrect, because it creates two problems instead of one (the first instance is slow ok, but now on top of that we have a failing second instance).</p></pre>
<br />
<p>- David</p>
<br />
<p>On July 9th, 2015, 11:48 a.m. UTC, Martin Klapetek wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for KDE Frameworks and David Faure.</div>
<div>By Martin Klapetek.</div>
<p style="color: grey;"><i>Updated July 9, 2015, 11:48 a.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kdbusaddons
</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Now I don't know if that was perhaps intended, but 5 minute timeout on dbus call to activate an app seems a bit too much?</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">So I've reduced it to standard 25 seconds.</p></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/kdbusservice.cpp <span style="color: grey">(ea7727d)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/124304/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>