Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 155602

Summary: sys-power/powersave needs loving for > dbus-0.95
Product: Gentoo Linux Reporter: Avuton Olrich <avuton>
Component: New packagesAssignee: Stefan Schweizer (RETIRED) <genstef>
Status: RESOLVED FIXED    
Severity: minor    
Priority: High    
Version: 2006.1   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 154521    
Attachments: Seems that my inline patch got chewed up. Here's a diff

Description Avuton Olrich 2006-11-18 11:45:52 UTC
Noticed when going to the masked dbus that things didn't work the way I thought they should. The newer powersave and dbus>0.95 require dbus-glib. Here's the fix:

--- /usr/portage/sys-power/powersave/powersave-0.14.0.ebuild    2006-11-02 13:06:24.000000000 -0800
+++ powersave-0.14.0.ebuild     2006-11-18 11:43:13.000000000 -0800
@@ -13,7 +13,7 @@
 IUSE="pam_console doc"


-RDEPEND="|| ( dev-libs/dbus-glib >=sys-apps/dbus-0.30 )
+RDEPEND="|| ( ( dev-libs/dbus-glib >=sys-apps/dbus-0.95 ) <sys-apps/dbus-0.95 )
        >=sys-apps/hal-0.5.3
        >=sys-power/cpufrequtils-001
        pam_console? ( || ( sys-auth/pam_console <sys-libs/pam-0.99 ) )"
Comment 1 Jakub Moc (RETIRED) gentoo-dev 2006-11-18 11:51:46 UTC
Should actually be 

RDEPEND="|| ( dev-libs/dbus-glib ( <sys-apps/dbus-0.90 >=sys-apps/dbus-0.34 ) )
Comment 2 Avuton Olrich 2006-11-18 11:53:18 UTC
Created attachment 102291 [details, diff]
Seems that my inline patch got chewed up. Here's a diff
Comment 3 Stefan Schweizer (RETIRED) gentoo-dev 2006-11-18 13:22:48 UTC
thanks, fixed
Comment 4 Jakub Moc (RETIRED) gentoo-dev 2006-11-18 14:08:23 UTC
Well, as noted in Comment #1 - dbus-glib already depends on dbus, so that part of the dependency is redundant; <sys-apps/dbus-0.95 is not the correct minimal "non-modular" version; should be <sys-apps/dbus-0.90. See Bug 154521 for porting howto.
Comment 5 Avuton Olrich 2006-11-18 14:23:24 UTC
Whoops. I did test it on both dbus>0.95 and dbus<0.95 and it worked as advertised. I'm unsure there's actually a different functionality that occurs in comment 1 but I'm no ebuild ninja.
Comment 6 Avuton Olrich 2006-11-18 14:40:18 UTC
Comment 1 doesnt work here, changing my diff to 0.90 would definitely be a fix, if there were versions 0.90-0.95 (sorry didn't realize):
--- /usr/portage/sys-power/powersave/powersave-0.14.0.ebuild    2006-11-02 13:06:24.000000000 -0800
+++ powersave-0.14.0.ebuild     2006-11-18 14:36:55.876408088 -0800
@@ -13,7 +13,7 @@
 IUSE="pam_console doc"


-RDEPEND="|| ( dev-libs/dbus-glib >=sys-apps/dbus-0.30 )
+RDEPEND="|| ( dev-libs/dbus-glib ( <sys-apps/dbus-0.90 >=sys-apps/dbus-0.34 ) )
        >=sys-apps/hal-0.5.3
        >=sys-power/cpufrequtils-001
        pam_console? ( || ( sys-auth/pam_console <sys-libs/pam-0.99 ) )"


tulip powersave # echo 'sys-apps/dbus' >> /etc/portage/package.unmask
tulip powersave # emerge -pv dbus powersave

These are the packages that would be merged, in order:

Calculating dependencies... done!
[ebuild     U ] sys-apps/dbus-1.0.1 [0.62-r1] USE="X -debug -doc (-selinux) (-gtk%) (-mono%) (-python%*) (-qt3%*) (-qt4%*)" 0 kB
[ebuild   R   ] sys-power/powersave-0.14.0  USE="pam_console -doc" 0 kB [1]
Comment 7 Stefan Schweizer (RETIRED) gentoo-dev 2006-11-19 09:18:49 UTC
really fixed now, Thanks