Bug 160025

Summary:	battery plasmoid uses deprecated /proc/acpi files
Product:	[Plasma] plasma4	Reporter:	David <StormByte>
Component:	widget-battery	Assignee:	Plasma Bugs List <plasma-bugs>
Status:	RESOLVED NOT A BUG
Severity:	wishlist
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed In:

Description David 2008-03-28 17:14:45 UTC

Version:           svn (using Devel)
Installed from:    Compiled sources
OS:                Linux

I want to post this bug not as a real bug, but as a warning.

I realized that battery plasmoid uses the deprecated /proc/acpi files to get its informations about battery. As I can see in config screen of kernel, this is considered deprecated (at least in 2.6.24).

So I suggest starting to be prepared for future /sys interface (/sys/bus/acpi/) if possible in case this dissapears from kernel's code not having this program useless.

What about it? If you already know about this issue, I'm sorry, I just wanted to let you know

Comment 1 Pino Toscano 2008-03-28 20:20:51 UTC

> I realized that battery plasmoid uses the deprecated /proc/acpi files

Show us the code, please, as I see it using a plasma dataengine using Solid only (no opening of files in /proc). And Solid in turn uses HAL to get the information from the system.

Comment 2 Kevin Ottens 2008-03-28 20:29:46 UTC

And HAL indeed still uses /proc/acpi files for those information, but parts read /sys already (hence the "I see my batterie twice" bug). But don't worry, they're slowly getting in order to fix that.

Closing as INVALID, it's out of our code.

Comment 3 David 2008-03-29 04:18:12 UTC

OK, I can't show you the affecting code, but what I can say is that if I disable deprecated /proc/acpi functions in kernel, I'll get a battery plasmoid unusable (it says N/A in battery charge left, and so on)...