Review Request: Implement simple Growl support in KNotify

Olivier Goffart ogoffart at kde.org
Tue Aug 10 19:05:49 BST 2010


Le Tuesday 10 August 2010, Markus Slopianka a écrit :
> > 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.

KNotify is already a wrapper service.  We do not need more :-)




More information about the kde-core-devel mailing list