Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 323593 - x11-libs/libnotify PDEPEND should include kde-base/knotify as an option
Summary: x11-libs/libnotify PDEPEND should include kde-base/knotify as an option
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Library (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Freedesktop bugs
URL:
Whiteboard:
Keywords:
: 346319 (view as bug list)
Depends on:
Blocks:
 
Reported: 2010-06-11 17:37 UTC by Ian Abbott
Modified: 2011-01-24 23:49 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ian Abbott 2010-06-11 17:37:16 UTC
The ebuild for x11-libs/libnotify-0.4.5 has the following PDEPEND

PDEPEND="|| ( x11-misc/notification-daemon
    xfce-extra/xfce4-notifyd )"

The KDE desktop uses kde-base/knotify to display notifications, not notification-daemon or xfce4-notifyd, so it would be nice if the PDEPEND could be extended to allow knotify as an alternative to avoid pulling in unnecessary dependencies:

PDEPEND="|| ( x11-misc/notification-daemon
    xfce-extra/xfce4-notifyd
    kde-base/knotify )"


Reproducible: Always

Steps to Reproduce:
Comment 1 Samuli Suominen (RETIRED) gentoo-dev 2010-06-11 17:50:25 UTC
If you run "notify-send test" in KDE, does the "test" message appear inside KDE with no notification-daemon installed ?

If (and only if) that works, this bug is valid... kde@ maintainers can fix the dep then, if it works.
Comment 2 Samuli Suominen (RETIRED) gentoo-dev 2010-06-11 17:52:26 UTC
... because normally you need the /usr/share/dbus-1/services/org.freedesktop.Notifications.service file installed, and I don't think knotify installs it
Comment 3 Ian Abbott 2010-06-14 13:19:02 UTC
(In reply to comment #1)
> If you run "notify-send test" in KDE, does the "test" message appear inside KDE
> with no notification-daemon installed ?
> 
> If (and only if) that works, this bug is valid... kde@ maintainers can fix the
> dep then, if it works.

I ran "emerge -C notification-daemon" and rebooted to get rid of anything lurking in any dbus-daemon's memory.  Then, running "notify-send test" in KDE still results in the notification message being displayed by the KDE system tray when "Application notifications" are enabled in the KDE system tray settings.
Comment 4 Ian Abbott 2010-06-14 13:41:54 UTC
... however, kde-base/knotify isn't the correct package for it to PDEPEND on, as application notifications are still displayed by the KDE system tray when this package is non installed.

Hopefully, the KDE maintainers can determine which package it _really_ should PDEPEND on (perhaps it's kde-base/ksystraycmd but I don't really know!).
Comment 5 Maciej Mrozowski gentoo-dev 2010-06-29 05:12:33 UTC
Just kdelibs is sufficient as org.freedesktop.Notifications interface is implemented there. Still you need something that actually displays those notifications (former libknotificationitem, now in kdelibs, I suppose works just as dispatcher).
So something that listens to and displays *KDE* notifications is needed. And I'm only afraid plasma-workspace is the one we have.
Comment 6 Ben Sagal 2010-11-23 15:09:25 UTC
*** Bug 346319 has been marked as a duplicate of this bug. ***
Comment 7 Theo Chatzimichos (RETIRED) archtester gentoo-dev Security 2011-01-13 16:18:54 UTC
i'm reassigning the bug to freedesktop, i'll let them decide, IMO it isn't worth it
Comment 8 Ben Sagal 2011-01-20 07:02:04 UTC
a similar bug concerning x11-wm/awesome (bug #342259) was reported much later than this one and has already been dealt with, why is there a reason for not dealing with this bug?
Comment 9 Gilles Dartiguelongue (RETIRED) gentoo-dev 2011-01-20 09:15:51 UTC
if there are so many notification agents, we might be better off switching to a virtual. @fdo, what do you think ?
Comment 10 Gilles Dartiguelongue (RETIRED) gentoo-dev 2011-01-24 23:49:08 UTC
+*libnotify-0.7.1 (24 Jan 2011)
+*libnotify-0.5.2 (24 Jan 2011)
+
+  24 Jan 2011; Gilles Dartiguelongue <eva@gentoo.org> libnotify-0.4.5.ebuild,
+  +libnotify-0.5.2.ebuild, +libnotify-0.7.1.ebuild,
+  +files/libnotify-0.7.1-gtk3-tests.patch:
+  Version bump. 0.5 for last of gtk+:2 only release and 0.7 for testing API
+  breakages, bug #340501. Add knotify to PDEPEND, bug #323593.

Added. Thanks for reporting.