<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/101316/">http://git.reviewboard.kde.org/r/101316/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On May 9th, 2011, 7:42 a.m., <b>Ben Cooksley</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;">Does this patch ensure that Plasma will still register as org.kde.plasma-desktop? If not, it should include such capabilities as there are an extensive number of times where commands to stop Plasma Desktop and remove it's configuration have been posted, and switching to KApplication will break these commands.
Further, it will make supporting the various versions of KDE in use harder, as seperate sets of commands will be needed involving more complicated steps for newer versions.</pre>
</blockquote>
<p>On May 9th, 2011, 5:42 p.m., <b>Martin Gräßlin</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;">kquitapp plasma-desktop
unnamed app(24717): "Application plasma-desktop could not be found using service org.kde.plasma-desktop and path /MainApplication."
seems not like it, though I am sure we can readd this to the dbus interface.</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;">this requires taking the org.kde.plasma-desktop name. right now the dbus service name is automatically created by KApplication without any opportunity of override (aside from saying "no dbus", perhaps? need to look again if even that is possible.)
there's more digging to do :)</pre>
<br />
<p>- Aaron J.</p>
<br />
<p>On May 8th, 2011, 7:43 p.m., Martin Gräßlin 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 Plasma.</div>
<div>By Martin Gräßlin.</div>
<p style="color: grey;"><i>Updated May 8, 2011, 7:43 p.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;">Replaces KUniqueApplication by a KApplication and adds a selection owner for Plasma Shells like the selection owner used by KWin (WMs in general). If plasma-desktop is already running, it is possible to restart with "plasma-desktop --replace". If plasma-desktop is running and you don't add the --replace, it will only yell (like KWin). The same could be added to the other shells like netbook, to easily restart/switch during development. I got the idea from this thread on wm-spec mailinglist: http://mail.gnome.org/archives/wm-spec-list/2011-January/msg00002.html
Current state: it works, plasma-desktop is replaced, but sometimes the WeatherEngine is causing a crash when exiting the previous one. So it is possible that the change from KUniqueApplication to KApplication is opening Pandora's Box. You decide whether it is worth to spend more time on it.</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>plasma/desktop/shell/main.cpp <span style="color: grey">(6f04e98)</span></li>
<li>plasma/desktop/shell/plasmaapp.h <span style="color: grey">(06ee523)</span></li>
<li>plasma/desktop/shell/plasmaapp.cpp <span style="color: grey">(58e8947)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/101316/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>