Re: What about Kfloppy and SuperKaramba
Am Freitag, 11. Dezember 2009 21:03:47 schrieb Rolf Eike Beer: I just read that KPilot was dropped from kdepim. That reminded me of some corpses in kdeutils ... I must confess I never used any of the two. I you don't count my krazy fixes from two weeks ago both applications have not seen any developer activity for month, just some general cleanups that were applied on many projects in the whole repository at once. Is it time to move them to tags/unmaintained? I doubt anyone has a strong need for Kfloppy these days as USB sticks are cheaper, more robust and e.g. no computer I bought in the last 2 years even had a floppy drive. Those we ordered at work 3 years ago were probably never used. I'm not sure the availability of USB sticks is the same all over the world. There might be places where floppies are still needed/used. Can't really comment on that - I don't even know if Kfloppy does what it's intended to do as I don't have a floppy disc drive myself :-) For SuperKaramba I'm less sure but given the speed of plasma development I think there can't be much that this project is still needed for. Plasma seems to include a way to use SuperKaramba widgets. Unfortunately I'm not sure to what extent this requires SuperKaramba. Maybe a plasma developer could comment on that (CCing for that purpose). Currently they are just bitrotting and spitting warnings about using K3Process and the like. When Friedrich took over kdeutils he tried to find new maintainers but I can't see that anyone ever stepped up. Maybe it's time to let them rest in peace somewhere else. Any comments on this? I agree that we should take a closer look if those utils are still needed and in good shape to ship them. btw, it seems kdessh is meant to run an application on a remote host, providing means to gather and store the password using a kde gui. however it seems it doesn't really work (I couldn't observe anything running kdesshing into my machine). Regards, Michael ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Review Request: Adding KMessageBox instead of KNotification
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2361/ --- Review request for Plasma and Anne-Marie Mahfouf. Summary --- Implementing a TODO in the timer.cpp. Removing the KNotification with KMessageBox. Diffs - trunk/KDE/kdeplasma-addons/applets/timer/timer.cpp 1061089 Diff: http://reviewboard.kde.org/r/2361/diff Testing --- Testing is done. Thanks, Sujith ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Adding KMessageBox instead of KNotification
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2361/#review3404 --- A few comments: - keep the same string if you want to get this in 4.4 - remove the kDebug() line - If the user checks 'Do not show this message' then the message should not been shown again (see in Settings) - I expect a sound like a ring (quite intrusive) if I put a timer on as an option. With notification we had a sound but it was too tiny. Should be thought of for 4.5 Overall I think more work should be done in this timer for 4.5 (maybe a bit of usability study should be conducted): how do users expect to set the timer? couldn't some mouse scrolling set it? how do users expect to be notified of the elapsed time? sound, message, ... Should the timer stay at 00:00 or come back to last time set after finishing? - Anne-Marie On 2009-12-12 10:15:49, Sujith H wrote: --- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2361/ --- (Updated 2009-12-12 10:15:49) Review request for Plasma and Anne-Marie Mahfouf. Summary --- Implementing a TODO in the timer.cpp. Removing the KNotification with KMessageBox. Diffs - trunk/KDE/kdeplasma-addons/applets/timer/timer.cpp 1061089 Diff: http://reviewboard.kde.org/r/2361/diff Testing --- Testing is done. Thanks, Sujith ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Adding KMessageBox instead of KNotification
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2361/ --- (Updated 2009-12-12 12:04:32.331590) Review request for Plasma and Anne-Marie Mahfouf. Changes --- The timer applet will now always start with 00:00:00. And the i18n string is reverted back. Summary --- Implementing a TODO in the timer.cpp. Removing the KNotification with KMessageBox. Diffs (updated) - trunk/KDE/kdeplasma-addons/applets/timer/timer.cpp 1061089 Diff: http://reviewboard.kde.org/r/2361/diff Testing --- Testing is done. Thanks, Sujith ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Adding KMessageBox instead of KNotification
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2361/ --- (Updated 2009-12-12 12:13:50.835279) Review request for Plasma and Anne-Marie Mahfouf. Changes --- Removed KNotification from the earlier patch. Summary --- Implementing a TODO in the timer.cpp. Removing the KNotification with KMessageBox. Diffs (updated) - trunk/KDE/kdeplasma-addons/applets/timer/timer.cpp 1061089 Diff: http://reviewboard.kde.org/r/2361/diff Testing --- Testing is done. Thanks, Sujith ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Adding KMessageBox instead of KNotification
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2361/ --- (Updated 2009-12-12 13:11:26.536267) Review request for Plasma and Anne-Marie Mahfouf. Changes --- Removed the enableAllMessages. Now its users wish, whether he/she wants to see the message. Summary --- Implementing a TODO in the timer.cpp. Removing the KNotification with KMessageBox. Diffs (updated) - trunk/KDE/kdeplasma-addons/applets/timer/timer.cpp 1061089 Diff: http://reviewboard.kde.org/r/2361/diff Testing --- Testing is done. Thanks, Sujith ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Adding KMessageBox instead of KNotification
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2361/ --- (Updated 2009-12-12 13:13:42.435258) Review request for Plasma and Anne-Marie Mahfouf. Summary --- Implementing a TODO in the timer.cpp. Removing the KNotification with KMessageBox. Diffs (updated) - trunk/KDE/kdeplasma-addons/applets/timer/timer.cpp 1061089 Diff: http://reviewboard.kde.org/r/2361/diff Testing --- Testing is done. Thanks, Sujith ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Adding KMessageBox instead of KNotification
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2361/#review3406 --- replacing something that doesn't interrupt the workflow with something that does? it's just the opposite of what we're trying to do everywhere else - Marco On 2009-12-12 13:13:42, Sujith H wrote: --- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2361/ --- (Updated 2009-12-12 13:13:42) Review request for Plasma and Anne-Marie Mahfouf. Summary --- Implementing a TODO in the timer.cpp. Removing the KNotification with KMessageBox. Diffs - trunk/KDE/kdeplasma-addons/applets/timer/timer.cpp 1061089 Diff: http://reviewboard.kde.org/r/2361/diff Testing --- Testing is done. Thanks, Sujith ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Ruby applet dataEngine service operation call
Hi richard, Do you think this is corrected now and if it is in which release will it be available ? Thanks Cedric On Tue, Aug 18, 2009 at 2:59 PM, Richard Dale richard.j.d...@gmail.comwrote: On Tue, Aug 18, 2009 at 1:45 PM, Cédrick...@xfou.com wrote: Thanks Richard, That seems to be the problem. What I was telling in my earlier mail is that the javascript binding seems to have a similar problem with plasmoid.dataEngine(engine name).serviceForSource(source name) and maybe they solved it by implementing the service function: plasmoid.service(engine name, source name) (and method Plasma::Applet#service is not implemented in ruby ) . I found the problem. It is a bug in how the smoke library works with overriden virtual methods in classes that are don't have bindings generated for them. In this case the 'TasksEngine' class isn't part of the bindings, and when the serviceForSource() method is called on the DataEngine, the code to invoke it looks like this: void x_10(Smoke::Stack x) { // serviceForSource(const QString) Plasma::Service* xret = this-Plasma::DataEngine::serviceForSource(*(const QString *)x[1].s_voidp); x[0].s_class = (void*)xret; } When it should probably look like this, and invoke the method in the TasksEngine class: void x_10(Smoke::Stack x) { // serviceForSource(const QString) Plasma::Service* xret = this-serviceForSource(*(const QString *)x[1].s_voidp); x[0].s_class = (void*)xret; } Arno Rehn has just written a new smoke library bindings generator, and I think we need to discuss this issue and see if we can fix it in that tool. -- Richard ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Adding KMessageBox instead of KNotification
On 2009-12-12 14:02:03, Marco Martin wrote: replacing something that doesn't interrupt the workflow with something that does? it's just the opposite of what we're trying to do everywhere else You are absolutely right. Hence I am closing this request. - Sujith --- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2361/#review3406 --- On 2009-12-12 13:13:42, Sujith H wrote: --- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2361/ --- (Updated 2009-12-12 13:13:42) Review request for Plasma and Anne-Marie Mahfouf. Summary --- Implementing a TODO in the timer.cpp. Removing the KNotification with KMessageBox. Diffs - trunk/KDE/kdeplasma-addons/applets/timer/timer.cpp 1061089 Diff: http://reviewboard.kde.org/r/2361/diff Testing --- Testing is done. Thanks, Sujith ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel