[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Remote applets
From:       Shantanu Tushar Jha <jhahoneyk () gmail ! com>
Date:       2010-01-28 9:55:29
Message-ID: ec556b641001280143v4a68d481m45eeb2596d71d38 () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


Recently on IRC someone pointed out that Remote Applets aren't working as
they should and the plasmoidviewer --list-remote ain't working either. For
plasmoidviewer, it was two changed SIGNAL names which I've fixed (patch
attached). Now, plasmoidviewer is able to list the remote applets, but i'm
not able to test if adding actually works.
I tried to share an applet on a 4.4 RC2 system and tried to add it to my
machine (running trunk) but after sometime plasma-desktop crashed. Here is
the terminal output for plasma-desktop http://pastebin.com/m2f8bb5ca and
backtrace at http://pastebin.com/m103e1a99 . Further, I don't know how to
tell plasmoidviewer to load a remote applet.
As I was not able to check it successfully, I was wondering if the the patch
is OK to commit ?

-- 
Shantanu Tushar    (UTC +0530)
http://www.shantanutushar.com

[Attachment #5 (text/html)]

Recently on IRC someone pointed out that Remote Applets aren&#39;t working as they \
should and the plasmoidviewer --list-remote ain&#39;t working either. For \
plasmoidviewer, it was two changed SIGNAL names which I&#39;ve fixed (patch \
attached). Now, plasmoidviewer is able to list the remote applets, but i&#39;m not \
able to test if adding actually works.<br>


I tried to share an applet on a 4.4 RC2 system and tried to add it to my machine \
(running trunk) but after sometime plasma-desktop crashed. Here is the terminal \
output for plasma-desktop <a href="http://pastebin.com/m2f8bb5ca" \
target="_blank">http://pastebin.com/m2f8bb5ca</a> and backtrace at <a \
href="http://pastebin.com/m103e1a99" \
target="_blank">http://pastebin.com/m103e1a99</a> . Further, I don&#39;t know how to \
tell plasmoidviewer to load a remote applet.<br> As I was not able to check it \
successfully, I was wondering if the the patch is OK to commit ?<br \
clear="all"><br>-- <br>Shantanu Tushar    (UTC +0530)<br><a \
href="http://www.shantanutushar.com" \
target="_blank">http://www.shantanutushar.com</a><br>



--0016e6d9a317fa11ab047e365974--


["remote-applets.diff" (text/x-diff)]

Index: main.cpp
===================================================================
--- main.cpp	(revision 1081288)
+++ main.cpp	(working copy)
@@ -57,9 +57,9 @@
         : QObject(manager)
     {
         kDebug();
-        connect(manager, SIGNAL(remotePlasmoidAdded(Plasma::PackageMetadata)),
+        connect(manager, SIGNAL(remoteAppletAnnounced(Plasma::PackageMetadata)),
                 this, SLOT(slotServiceAdded(Plasma::PackageMetadata)));
-        connect(manager, SIGNAL(remotePlasmoidRemoved(Plasma::PackageMetadata)),
+        connect(manager, SIGNAL(remoteAppletUnannounced(Plasma::PackageMetadata)),
                 this, SLOT(slotServiceRemoved(Plasma::PackageMetadata)));
     }
 


_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[prev in list] [next in list] [prev in thread] [next in thread]