Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 27344 - Bug in sys-apps/speedfreq init-script autodetection
Summary: Bug in sys-apps/speedfreq init-script autodetection
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: SpanKY
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-08-26 01:32 UTC by Christian Axelsson
Modified: 2004-01-28 16:39 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Fixed init-script (speedfreq.rc,977 bytes, text/plain)
2003-08-26 01:33 UTC, Christian Axelsson
Details
Fixes bad comparasion in battery.sh (battery.sh.patch,284 bytes, patch)
2003-08-26 06:55 UTC, Christian Axelsson
Details | Diff
Fixes bad comments in the conf (speedfreq.conf.patch,879 bytes, patch)
2003-08-26 06:56 UTC, Christian Axelsson
Details | Diff
Fixes autodetection of battery status. (speedfreq.diff,522 bytes, patch)
2004-01-28 14:00 UTC, Ulrich Müller
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Christian Axelsson 2003-08-26 01:32:12 UTC
There is a misstake in the sys-apps/speedfreq autodetection of batterystatus

Reproducible: Always
Steps to Reproduce:
1. Set SPEEDFREQ_SPEED="auto" is /etc/conf.d/speedfreq
2. Unplug AC power
3. restart speedfreqd via the initscript
4. run speedfreq -m
Actual Results:  
Step 4 shows 'performance'

Expected Results:  
It should be powersave
Comment 1 Christian Axelsson 2003-08-26 01:33:45 UTC
Created attachment 16625 [details]
Fixed init-script
Comment 2 Christian Axelsson 2003-08-26 06:54:09 UTC
Found two other bugs
Will attach files (messy to make two other bug reports as this one isnt solved yet).
Comment 3 Christian Axelsson 2003-08-26 06:55:37 UTC
Created attachment 16637 [details, diff]
Fixes bad comparasion in battery.sh
Comment 4 Christian Axelsson 2003-08-26 06:56:33 UTC
Created attachment 16638 [details, diff]
Fixes bad comments in the conf
Comment 5 SpanKY gentoo-dev 2003-08-28 08:57:57 UTC
the init script you posted is pretty much like the one in cvs ...

ive added the patch for battery.sh ...

and the comments in the conf were fixed with bug 27404

thanks ;)
Comment 6 Ulrich Müller gentoo-dev 2004-01-28 13:50:55 UTC
In the speedfreq-0.5 as of today this bug is present again. Reproduce exactly in the same way as above.

It looks like the mentioned line in /etc/init.d/speedfreq (v 1.2 2003/11/18 07:08:02 vapier) was changed again and does not work anymore.

I include a proposed patch as attachment.
Comment 7 Ulrich Müller gentoo-dev 2004-01-28 14:00:53 UTC
Created attachment 24559 [details, diff]
Fixes autodetection of battery status.
Comment 8 SpanKY gentoo-dev 2004-01-28 14:33:10 UTC
ultrich: the bug you describe and the bug this was about are completely different

please file a new bug ;)
Comment 9 Ulrich Müller gentoo-dev 2004-01-28 16:39:28 UTC
Same symptomps, same way to reproduce. Filed as #39709.
Please don't mark it as duplicate now. ;-)