Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 141645 - gnome-power-manager dependency on libnotify should be optional
Summary: gnome-power-manager dependency on libnotify should be optional
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] GNOME (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Saleem Abdulrasool (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-07-24 16:26 UTC by Jose Marino
Modified: 2006-08-27 11:53 UTC (History)
0 users

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jose Marino 2006-07-24 16:26:16 UTC 
From the gnome-power-manager home page I find that dependency on libnotify is actually optional. I've tried to compile it without libnotify and it compiles and runs fine.

The current ebuild in portage forces the installation of libnotify. I believe the ebuild should use the 'libnotify' USE flag.

Here's a patch to "gnome-power-manager-2.14.3.ebuild" that adds a libnotify USE flag:

# diff -u gnome-power-manager-2.14.3.ebuild gnome-power-manager-2.14.3.ebuild.new
--- gnome-power-manager-2.14.3.ebuild   2006-07-24 17:24:04.000000000 -0600
+++ gnome-power-manager-2.14.3.ebuild.new       2006-07-24 17:21:31.000000000 -0600
@@ -12,7 +12,7 @@
 LICENSE="GPL-2"
 SLOT="0"
 KEYWORDS="amd64 ppc x86"
-IUSE="doc"
+IUSE="doc libnotify"

 RDEPEND=">=dev-libs/glib-2.6.0
                 >=x11-libs/gtk+-2.6.0
@@ -21,9 +21,9 @@
                 >=sys-apps/dbus-0.61
                 >=sys-apps/hal-0.5.7-r1
                 >=gnome-base/libglade-2.5.0
-                >=x11-libs/libnotify-0.2.2
+                libnotify? ( >=x11-libs/libnotify-0.2.2 )
                 >=x11-libs/libwnck-2.10.0
-                >=x11-misc/notification-daemon-0.2.1
+                libnotify? ( >=x11-misc/notification-daemon-0.2.1 )
                 || (
                                (
                                        x11-libs/libX11
@@ -41,5 +41,5 @@
 DOCS="AUTHORS ChangeLog NEWS README TODO"

 pkg_setup() {
-       G2CONF="${G2CONF} --enable-actions-menu --enable-libnotify --with-dpms-ext"
+       G2CONF="${G2CONF} --enable-actions-menu --with-dpms-ext"
 }
Comment 1 Saleem Abdulrasool (RETIRED) gentoo-dev 2006-08-27 11:53:28 UTC 
made libnotify optional with 2.14.3-r1 (now in ~arch)