Comment 23 for bug 406302

Revision history for this message
Steven Sheehy (steven-sheehy) wrote :

Committed to trunk. I added the #ifdef HAVE_LIBNOTIFY so that it's only enabled if libnotify is present at the time of compilation. I also removed the pixbuf in favor of using icon name. The reason it wasn't working is because we're not storing the application icon in the icon theme search path visible to other programs (our custom path is only visible to linuxdcpp, not notification-daemon). I will be fixing it in an upcoming commit to install the application icon to /usr/share/icons and keep all other icons at /usr/share/linuxdcpp/icons/.

Thanks for the patch Jakh Daven! If you're looking for possible things to work on next, best place would be to take a look at the open bugs on launchpad. We generally try to put more of a focus on bugs before features, but you're of course free to work on whatever you like. Hope we'll see more from you. :)