Otavio Salvador wrote:
> Your patch looks ok but are you sure we don't have desktop machines
> with APM?

Possibly.. if you want to limit it to machines with a battery, add a
subtest like this:

if [ -e /proc/apm ] && [ "$(cut -d ' ' -f 7 /proc/apm)" != "-1%" ]; then

Note that this will fail to detect laptops with their batteries removed
or with very broken batteries that cannot report charge.

(BTW, you might want to cc -submitter if you want most submitters to
notice your questions on bugs.)

-- 
see shy jo

Attachment: signature.asc
Description: Digital signature

Reply via email to