Review Request: Implement simple Growl support in KNotify

[Sjors Gielen]

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

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/57d9ba56/attachment.htm>