Bug 509754 - -n ${GOVERNOR} typo in /etc/init.d/cpuspeed
Summary: -n ${GOVERNOR} typo in /etc/init.d/cpuspeed
Keywords:
Status: CLOSED DUPLICATE of bug 505837
Alias: None
Product: Fedora
Classification: Fedora
Component: cpuspeed
Version: 11
Hardware: All
OS: Linux
low
low
Target Milestone: ---
Assignee: Jarod Wilson
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-07-05 19:44 UTC by Maciej Żenczykowski
Modified: 2009-07-06 17:58 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2009-07-06 13:51:38 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Maciej Żenczykowski 2009-07-05 19:44:52 UTC
My box has an EIST capable cpu, but missing ACPI tables, hence frequency scaling doesn't work.

In /etc/init.d/cpuspeed the line:
        if [ -d ${cpu0freqd} -a -n ${GOVERNOR} ]; then
should be:
        if [ -d ${cpu0freqd} -a -n "${GOVERNOR}" ]; then
to prevent errors about ] misparsing when GOVERNOR is empty.

# rpm -q --whatprovides /etc/init.d/cpuspeed 
cpuspeed-1.5-8.fc11.x86_64

However, I believe this has actually been fixed in a non-pushed to updates version of cpuspeed.

At least:

# rpm -q --whatprovides /etc/init.d/cpuspeed 
cpuspeed-1.5-10.fc11.x86_64

doesn't seem to have this issue.

Comment 1 Jarod Wilson 2009-07-06 13:51:38 UTC
cpuspeed-1.5-10.fc11 was pushed to the updates repo last Wednesday.

*** This bug has been marked as a duplicate of bug 505837 ***

Comment 2 Maciej Żenczykowski 2009-07-06 17:13:49 UTC
# yum list extras
Loaded plugins: dellsysidplugin2, refresh-packagekit
Extra Packages
cpuspeed.x86_64                       1:1.5-10.fc11                    installed
...

So it doesn't seem to be live yet (I installed manually from koji).

Comment 3 Jarod Wilson 2009-07-06 17:26:09 UTC
You have stale repo data or something.

$ yum list cpuspeed
Loaded plugins: refresh-packagekit
Installed Packages
cpuspeed.x86_64                                                          1:1.5-9.fc11                                                           @updates
Available Packages
cpuspeed.x86_64                                                          1:1.5-10.fc11                                                          updates

Comment 4 Maciej Żenczykowski 2009-07-06 17:58:36 UTC
Yeah, it does seem that "rm -rf /var/cache/yum/*" has revealed a whole bunch of updates.

Sorry, for the bother.


Note You need to log in before you can comment on or make changes to this bug.