The sysutils/gkrellm2 port works fine on amd64 machines, but has no support for the battery state on my laptop. This is because the battery code is only compiled if __i386__ is defined. But on __amd64__, the ACPI code for the battery is usable. I've modified the existing patch for freebsd.c so that it will compile a part of the battery code for __amd64__, which actually yields the correct results on my laptop. Fix: Patch attached which is intended to replace the existing /usr/ports/sysutils/gkrellm2/files/patch-src::sysdeps::freebsd.c file. Patch attached with submission follows:
Responsible Changed From-To: freebsd-ports-bugs->ume Over to maintainer (via the GNATS Auto Assign Tool)
ume 2008-01-31 17:32:17 UTC FreeBSD ports repository Modified files: sysutils/gkrellm2 Makefile sysutils/gkrellm2/files patch-src::sysdeps::freebsd.c Log: - Enable battery support on amd64 using ACPI. [1] - Suppress compile time warnings. PR: ports/120149 [1] Submitted by: Sven Berkvens-Matthijsse <sven__at__berkvens.net> [1] Revision Changes Path 1.130 +1 -1 ports/sysutils/gkrellm2/Makefile 1.10 +121 -7 ports/sysutils/gkrellm2/files/patch-src::sysdeps::freebsd.c _______________________________________________ cvs-all@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/cvs-all To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
State Changed From-To: open->closed Thanks! Committed with some modification.