bug#643: 23.0.60; battery.el needs deprecated ACPI_PROCFS_POWER interface

bug#643: 23.0.60; battery.el needs deprecated ACPI_PROCFS_POWER interface

[Sven Joachim]

In recent Linux kernels, the information about battery status has been
moved from /proc/acpi/battery to /sys/class/power_supply.  It is only
available from the former place if the kernel has been built with the
ACPI_PROCFS_POWER option, which is deprecated:

,----[ /usr/src/linux-2.6.26.1/drivers/acpi/Kconfig ]
| config ACPI_PROCFS_POWER
| 	bool "Deprecated power /proc/acpi directories"
| 	depends on PROC_FS
| 	default y
| 	---help---
| 	  For backwards compatibility, this option allows
|           deprecated power /proc/acpi/ directories to exist, even when
|           they have been replaced by functions in /sys.
|           The deprecated directories (and their replacements) include:
| 	  /proc/acpi/battery/* (/sys/class/power_supply/*)
| 	  /proc/acpi/ac_adapter/* (sys/class/power_supply/*)
| 	  This option has no effect on /proc/acpi/ directories
| 	  and functions, which do not yet exist in /sys
| 
| 	  Say N to delete power /proc/acpi/ directories that have moved to /sys/
| 
`----

At least Debian ships kernels that are built without ACPI_PROCFS_POWER,
so battery.el will not work with them.

Sven

bug#643: marked as done (23.0.60; battery.el needs deprecated ACPI_PROCFS_POWER interface)

[Emacs bug Tracking System]

[-- Attachment #1: Type: text/plain, Size: 893 bytes --]


Your message dated Fri, 08 Aug 2008 13:49:56 -0400
with message-id <87projiegr.fsf@stupidchicken.com>
and subject line Re: 23.0.60; battery.el needs deprecated ACPI_PROCFS_POWER interface
has caused the Emacs bug report #643,
regarding 23.0.60; battery.el needs deprecated ACPI_PROCFS_POWER interface
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact don@donarmstrong.com
immediately.)


-- 
643: http://emacsbugs.donarmstrong.com/cgi-bin/bugreport.cgi?bug=643
Emacs Bug Tracking System
Contact don@donarmstrong.com with problems

[-- Attachment #2: Type: message/rfc822, Size: 3124 bytes --]

From: Sven Joachim <svenjoac@gmx.de>
To: emacs-pretest-bug@gnu.org
Subject: 23.0.60; battery.el needs deprecated ACPI_PROCFS_POWER interface
Date: Sun, 03 Aug 2008 09:04:05 +0200
Message-ID: <87zlnuinq2.fsf@gmx.de>

In recent Linux kernels, the information about battery status has been
moved from /proc/acpi/battery to /sys/class/power_supply.  It is only
available from the former place if the kernel has been built with the
ACPI_PROCFS_POWER option, which is deprecated:

,----[ /usr/src/linux-2.6.26.1/drivers/acpi/Kconfig ]
| config ACPI_PROCFS_POWER
| 	bool "Deprecated power /proc/acpi directories"
| 	depends on PROC_FS
| 	default y
| 	---help---
| 	  For backwards compatibility, this option allows
|           deprecated power /proc/acpi/ directories to exist, even when
|           they have been replaced by functions in /sys.
|           The deprecated directories (and their replacements) include:
| 	  /proc/acpi/battery/* (/sys/class/power_supply/*)
| 	  /proc/acpi/ac_adapter/* (sys/class/power_supply/*)
| 	  This option has no effect on /proc/acpi/ directories
| 	  and functions, which do not yet exist in /sys
| 
| 	  Say N to delete power /proc/acpi/ directories that have moved to /sys/
| 
`----

At least Debian ships kernels that are built without ACPI_PROCFS_POWER,
so battery.el will not work with them.

Sven



[-- Attachment #3: Type: message/rfc822, Size: 964 bytes --]

From: Chong Yidong <cyd@stupidchicken.com>
To: 643-done@emacsbugs.donarmstrong.com
Subject: Re: 23.0.60; battery.el needs deprecated ACPI_PROCFS_POWER interface
Date: Fri, 08 Aug 2008 13:49:56 -0400
Message-ID: <87projiegr.fsf@stupidchicken.com>

I've checked in the patch.

bug#660: marked as done (mode line does not display battery status)

[Emacs bug Tracking System]

[-- Attachment #1: Type: text/plain, Size: 870 bytes --]


Your message dated Fri, 08 Aug 2008 13:49:56 -0400
with message-id <87projiegr.fsf@stupidchicken.com>
and subject line Re: 23.0.60; battery.el needs deprecated ACPI_PROCFS_POWER interface
has caused the Emacs bug report #643,
regarding mode line does not display battery status
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact don@donarmstrong.com
immediately.)


-- 
643: http://emacsbugs.donarmstrong.com/cgi-bin/bugreport.cgi?bug=643
Emacs Bug Tracking System
Contact don@donarmstrong.com with problems

[-- Attachment #2: Type: message/rfc822, Size: 8232 bytes --]

From: Emre Sahin <mail@emresahin.net>
To: bug-gnu-emacs@gnu.org
Subject: mode line does not display battery status
Date: Wed, 06 Aug 2008 22:59:29 +0300
Message-ID: <87r691ewym.fsf@leonardo.iesahin.net>


Please write in English if possible, because the Emacs maintainers
usually do not have translators to read other languages for them.

Your bug report will be posted to the bug-gnu-emacs@gnu.org mailing list,
and to the gnu.emacs.bug news group.

Please describe exactly what actions triggered the bug
and the precise symptoms of the bug:

Before upgrading my kernel, display-battery-mode was showing the
battery status in my mode line. With current kernel it doesn't show. 

battery.el requires /proc/acpi/battery to detect acpi support but
newer kernels does not have this file and battery-status-function
variable becomes nil. I asked this to Debian Testing mailing list and
they replied that aforementioned file does not exist in newer kernels
and battery.el should be upgraded. 

If Emacs crashed, and you have the Emacs process in the gdb debugger,
please include the output from the following gdb commands:
    `bt full' and `xbacktrace'.
If you would like to further debug the crash, please read the file
/usr/share/emacs/22.2/etc/DEBUG for instructions.


In GNU Emacs 22.2.1 (i486-pc-linux-gnu, X toolkit, Xaw3d scroll bars)
 of 2008-07-25 on raven, modified by Debian
Windowing system distributor `The X.Org Foundation', version 11.0.10402000
configured using `configure  '--build=i486-linux-gnu' '--host=i486-linux-gnu' '--prefix=/usr' '--sharedstatedir=/var/lib' '--libexecdir=/usr/lib' '--localstatedir=/var/lib' '--infodir=/usr/share/info' '--mandir=/usr/share/man' '--with-pop=yes' '--enable-locallisppath=/etc/emacs22:/etc/emacs:/usr/local/share/emacs/22.2/site-lisp:/usr/local/share/emacs/site-lisp:/usr/share/emacs/22.2/site-lisp:/usr/share/emacs/site-lisp:/usr/share/emacs/22.2/leim' '--with-x=yes' '--with-x-toolkit=athena' '--with-toolkit-scroll-bars' 'build_alias=i486-linux-gnu' 'host_alias=i486-linux-gnu' 'CFLAGS=-DDEBIAN -g -O2' 'LDFLAGS=-g' 'CPPFLAGS=''

Important settings:
  value of $LC_ALL: nil
  value of $LC_COLLATE: nil
  value of $LC_CTYPE: nil
  value of $LC_MESSAGES: nil
  value of $LC_MONETARY: nil
  value of $LC_NUMERIC: nil
  value of $LC_TIME: nil
  value of $LANG: en_US.UTF-8
  locale-coding-system: utf-8
  default-enable-multibyte-characters: t

Major mode: Group

Minor modes in effect:
  gnus-undo-mode: t
  desktop-save-mode: t
  display-battery-mode: t
  display-time-mode: t
  jabber-activity-mode: t
  jabber-mode-line-mode: t
  tooltip-mode: t
  mouse-wheel-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  blink-cursor-mode: t
  unify-8859-on-encoding-mode: t
  utf-translate-cjk-mode: t
  auto-compression-mode: t
  line-number-mode: t

Recent input:
<down> <down> <down> <down> <down> <down> <down> <down> 
<down> <down> <down> <down> <down> <down> <up> <return> 
<next> <prior> <C-tab> q g <next> <prior> <up> <up> 
<up> <up> <up> <up> <return> 1 0 0 <return> / o 1 0 
0 <return> <next> <next> <next> <next> <next> <down> 
<down> <down> <down> <return> <down-mouse-4> <mouse-4> 
<double-down-mouse-4> <double-mouse-4> <triple-down-mouse-4> 
<triple-mouse-4> <C-tab> q <up> <up> <up> <up> <up> 
<up> <up> <up> <up> <up> <up> <up> <up> <up> <up> <up> 
<up> <up> <up> <up> <up> <up> <up> <up> <return> <return> 
<next> <prior> <down> <down> <up> <up> <up> <up> <up> 
<up> <up> <up> <up> <up> <up> <up> <up> <up> <C-tab> 
<C-tab> <C-tab> q <next> <prior> <up> <up> <up> <up> 
<up> <up> <up> <up> <up> <down> <down> <down> <return> 
1 0 0 <return> M-> <up> <up> <up> <up> <right> <up> 
<up> <up> <up> <up> g C-x k <return> q n <C-tab> q 
<up> <down> <up> g <up> <up> <up> <up> <up> <up> <up> 
<up> <up> <up> <up> <up> <up> <up> <up> <up> <up> <up> 
<up> <up> <up> <up> <up> <up> <down> <return> <return> 
<next> <next> <next> <next> <next> <next> <next> <next> 
<next> <next> <prior> <next> <prior> <next> <next> 
<prior> <next> <next> <next> <next> <next> <next> <prior> 
<next> <next> <next> <prior> <next> <next> <prior> 
<next> <next> <next> <next> <next> <next> <next> <next> 
<prior> <next> <next> <next> <prior> <next> <next> 
<next> <next> <next> <next> <next> <next> <next> <C-tab> 
u q <return> c y <next> <next> <next> <prior> <prior> 
<prior> <next> <prior> <up> <up> <up> <up> <up> <up> 
<return> 2 0 <return> <down> <down> <down> <down> <down> 
<down> <down> <down> <down> <down> <down> <down> <down> 
<down> <down> <down> <down> <down> <down> <down> <up> 
<up> <up> <up> <up> <up> <up> <up> <up> <up> <up> <up> 
<up> <up> <up> <up> q M-x r e p <tab> o <tab> r <tab> 
b <tab> <return>

Recent messages:
Fetching headers for nntp+news.gmane.org:gmane.linux.debian.user.testing...done
Scoring...done
Generating summary...done
(No changes need to be saved)
Saving /home/iesahin/.newsrc.eld...
Saving file /home/iesahin/.newsrc.eld...
Wrote /home/iesahin/.newsrc.eld
Saving /home/iesahin/.newsrc.eld...done
Making completion list... [2 times]
Loading emacsbug...done

-- 
I. Emre Sahin                   @ Ankara, Turkey 
Software Developer & Consultant @ http://ydyazilim.com
Ph.D. Candidate       		@ http://cs.bilkent.edu.tr
Blogger, Writer, Poet  		@ http://emresahin.net
Maybe hears the phone  		@ +90 532 261 8985
Maybe gtalk (or jabber)   	@ i.emre.sahin@gmail.com







[-- Attachment #3: Type: message/rfc822, Size: 964 bytes --]

From: Chong Yidong <cyd@stupidchicken.com>
To: 643-done@emacsbugs.donarmstrong.com
Subject: Re: 23.0.60; battery.el needs deprecated ACPI_PROCFS_POWER interface
Date: Fri, 08 Aug 2008 13:49:56 -0400
Message-ID: <87projiegr.fsf@stupidchicken.com>

I've checked in the patch.