Ubuntu
byobu package

CPU temperature status does not work

Bug #696071 reported by أحمد المحمودي (Ahmed El-Mahmoudy)
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
byobu (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

Binary package hint: byobu

In byobu 3.17-0ubuntu1, the path /proc/acpi/ibm/thermal was added in the for loop to support a certain laptops, that resulted in a regression for other systems that do not have this path, and the temperature won't be displayed in the status bar.

The attached patch adds a check for the validity of the path to avoid breaking from the loop on a non-existing path.

Tags: patch

Related branches

lp:byobu
lp:ubuntu/natty/byobu
Revision history for this message
أحمد المحمودي (Ahmed El-Mahmoudy) (aelmahmoudy) wrote :
Brian Murray (brian-murray)
tags: added: patch
Revision history for this message
Dustin Kirkland  (kirkland) wrote :

Thanks!

Changed in byobu (Ubuntu):
status: New → Fix Committed
importance: Undecided → Medium
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package byobu - 3.20-0ubuntu1

---------------
byobu (3.20-0ubuntu1) natty; urgency=low

  * usr/lib/byobu/network: drop unnecessary unit=kbps
  * usr/bin/byobu: allow for -xS or the like, LP: #684926
  * usr/bin/byobu-select-session, usr/share/man/man1/byobu.1,
    usr/share/man/man1/byobu-select-session.1: support "hiding"
    sessions, by prepending a "." to the session name, LP: #599334
  * usr/share/man/man1/byobu-launcher.1: update documentation
  * usr/bin/byobu-launcher-install, usr/share/man/man1/byobu-launcher-
    install.1, usr/share/man/man1/byobu-launcher-uninstall.1,
    usr/share/man/man1/Makefile.am:
    - add an option for -n|--no-logout to byobu-launcher-install,
      document changes in new manpages, LP: #604931
  * usr/lib/byobu/cpu_temp: find the temp monitor, LP: #676080
  * usr/lib/byobu/battery, usr/share/man/man1/byobu.1: allow for
    overriding detected BATTERY, LP: #693560
  * usr/lib/byobu/cpu_temp: test that file exists, is readable, and has
    size > 0, LP: #696071
 -- Dustin Kirkland <email address hidden> Tue, 28 Dec 2010 00:05:13 -0600

Changed in byobu (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.