[Pm-utils] Re: pm-utils' 55battery: WTF?

[Olivier Blin]

David Zeuthen <david at fubar.dk> writes:

> On Tue, 2007-03-13 at 19:35 +0100, Stefan Seyfried wrote:
>> 		dbus-send --print-reply --system --reply-timeout=2000 \
>> 			--dest=org.freedesktop.Hal $x \
>> 			org.freedesktop.Hal.Device.Rescan string:$x
>
> [...]
>
>> so what does this monster do? Why is this needed? It stealy me half a second
>> of resume time, so it'd better be worth it ;-)
>> 
>> If somebody can convince me that it is really needed, then it needs at least
>> a comment telling why this is the case.
>
> It tells HAL to rescan the battery - in case someone changed the battery
> we want the new serial number etc... if I recall Peter's reasoning
> correctly. I think this check is better done in HAL than in pm-utils; it
> doesn't make sense to make pm-utils call into HAL at all - Peter, is
> this fine with you? 

But what if pm-hibernate is run directly from command line and not
from hal-system-power-hibernate-linux ?
Or do you forget about command line users in pm-utils?

-- 
Olivier Blin - Mandriva