gnome-base/gnome-applets-2.12.2 does not build the battstat applet without the useflags 'apm' or 'acpi' On the platform other than ppc (and probably ppc64) this might be right, but on ppc neither apm nor acpi is needed. For me the following worked: --- /usr/portage/gnome-base/gnome-applets/gnome-applets-2.12.2.ebuild 2006-01-23 00:05:43.000000000 +0100 +++ /usr/local/portage/gnome-base/gnome-applets/gnome-applets-2.12.2.ebuild 2006-01-24 16:01:40.000000000 +0100 @@ -53,10 +53,10 @@ --enable-flags \ $(use_enable ipv6) $(use_with hal)" - - if ! use apm && ! use acpi; then - G2CONF="${G2CONF} --disable-battstat" - fi +# we do not need this on ppc +# if ! use apm && ! use acpi; then +# G2CONF="${G2CONF} --disable-battstat" +# fi } This just disables the check for the apm and acpi use-flags. Of course this would be wrong on platforms other than ppc, but I don't know how to determine the platform in an ebuild. The same applies to the 2.12.1* ebuilds
flipping assign and cc to ppc folks. can you guys verify this and possibly provide an ebuild patch? Please assign back to gnome when done. Thanks!
As far as I can tell from looking at the source, battstat does not have pmu specific code. There is an apm emulation option in the kernel, perhaps the reporter is using this. If so, the apm use flag should be enabled. The other option is to get the battery info from hal. As far as I know, hal does use the pmu and can report battery life without the apm layer, so in this case, it could be enabled without apm and acpi, but would depend on hal. I guess I should pass this back to the gnome folks, what do you think considering the above?
(In reply to comment #2) > As far as I can tell from looking at the source, battstat does not have pmu > specific code. There is an apm emulation option in the kernel, perhaps the > reporter is using this. If so, the apm use flag should be enabled. The other > option is to get the battery info from hal. As far as I know, hal does use the > pmu and can report battery life without the apm layer, so in this case, it > could be enabled without apm and acpi, but would depend on hal. Ok, you're right. Of course I had the apm emulation switched on in the kernel, but now that I have removed the apm emulation from the kernel battstat still works. But if I stop either hald or dbus it dosn't work anymore which suggests a hal use flag.
gnome team, do you have any further input? Should there be a hal dep?
gnome-applets alreay has a dep on hal, based on the hal use flag.
The point is that on PPC, hal is enough for battery info, apm or acpi isn't required. Can we add hal to the check?
Sure. Something like this: Index: gnome-applets-2.12.2.ebuild =================================================================== RCS file: /var/cvsroot/gentoo-x86/gnome-base/gnome-applets/gnome-applets-2.12.2.ebuild,v retrieving revision 1.12 diff -u -p -r1.12 gnome-applets-2.12.2.ebuild --- gnome-applets-2.12.2.ebuild 3 Feb 2006 19:52:35 -0000 1.12 +++ gnome-applets-2.12.2.ebuild 10 Jul 2006 01:57:18 -0000 @@ -54,7 +54,11 @@ pkg_setup() { $(use_enable ipv6) $(use_with hal)" - if ! use apm && ! use acpi; then + if ! use ppc && ! use apm && ! use acpi; then + G2CONF="${G2CONF} --disable-battstat" + fi + + if use ppc && ! use apm && ! use hal; then G2CONF="${G2CONF} --disable-battstat" fi } Could someone with a ppc test this?
(In reply to comment #7) I have tested the patch on ppc and it works very well.
This works for me as well. Please apply it! :)
applied.
