Review Request: Implement simple Growl support in KNotify

Markus Slopianka markus.s at kdemail.net
Tue Aug 10 18:30:29 BST 2010



> On 2010-08-10 17:04:41, Olivier Goffart wrote:
> > This is a bit hackish (the hardcoded paths, and the QSystemTrayIcon stuff)  
> > But if it works, i'm ok with it.

Amateur question: Wouldn't is make more sense if a "wrapper service" was implemented that calls Growl instead of calling Growl directly from KNotify? That way hackish code wouldn't hurt.


- Markus


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4962/#review6973
-----------------------------------------------------------


On 2010-08-10 15:05:45, Sjors Gielen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/4962/
> -----------------------------------------------------------
> 
> (Updated 2010-08-10 15:05:45)
> 
> 
> Review request for kdelibs and Olivier Goffart.
> 
> 
> Summary
> -------
> 
> This patch implements, as promised, Growl support in KNotify. Support is added in a separate class with static methods, called sometimes in NotifyByPopup. It currently uses Growl via QApplication::message(), because the GNTP protocol using which icon sending and callbacks are possible isn't stabilized in the Mac versions yet. This patch makes it possible to display the title and the message, in stripped form as with the DBus daemons after my last patch, in a Growl notification, with a (quite random) Qt information icon.
> 
> Before calling QApplication::message(), it checks if Growl is installed using QFile::exists(). There is still a TODO in the source regarding the Windows path to Growl, I will fill that in soon, but it's trivial. With this patch applied, I expect no changes on any machine where Growl is not installed, since then the older mechanism of KPassivePopup is used. (DBus notification daemons are still preferred over Growl.)
> 
> Once the newest version of Growl has stable GNTP, I will implement it in NotifyByPopupGrowl and send in another patch.
> 
> 
> This addresses bug 212751.
>     https://bugs.kde.org/show_bug.cgi?id=212751
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdebase/runtime/knotify/CMakeLists.txt 1155631 
>   /trunk/KDE/kdebase/runtime/knotify/notifybypopup.cpp 1155631 
>   /trunk/KDE/kdebase/runtime/knotify/notifybypopupgrowl.h PRE-CREATION 
>   /trunk/KDE/kdebase/runtime/knotify/notifybypopupgrowl.cpp PRE-CREATION 
> 
> Diff: http://reviewboard.kde.org/r/4962/diff
> 
> 
> Testing
> -------
> 
> Tested on Mac OS X. Nowhere else yet.
> 
> 
> Thanks,
> 
> Sjors
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20100810/0e552c88/attachment.htm>


More information about the kde-core-devel mailing list