Make displayed thermal zone configurable

Bug #411108 reported by أحمد المحمودي (Ahmed El-Mahmoudy)
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
byobu (Ubuntu)
Fix Released
Low
Dustin Kirkland 

Bug Description

Binary package hint: byobu

I have a problem with the temp_c script, it displays the temperature of the
first thermal zone (for laptops with several thermal zones) actually the first
zone is not the one that I need to be displayed on status bar. So I've made a
patch for temp_c that makes it read a conf file from ~/.byobu, so a user can
set the thermal zone that he actually needs to see.

This idea can also be extended for battery.

Related branches

Revision history for this message
أحمد المحمودي (Ahmed El-Mahmoudy) (aelmahmoudy) wrote :
Revision history for this message
Dustin Kirkland  (kirkland) wrote :

Thanks for the report, I can confirm the issue as others have brought this up too.

I'm going to solve this slightly differently, though. There are several status scripts that can take overrides in separate files configured in .byoby/. I'm going to combine all of those into a single rc file where each can be set.

:-Dustin

Changed in byobu (Ubuntu):
status: New → Confirmed
importance: Undecided → Medium
importance: Medium → Low
assignee: nobody → Dustin Kirkland (kirkland)
Changed in byobu (Ubuntu):
status: Confirmed → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

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

---------------
byobu (2.26-0ubuntu1) karmic; urgency=low

  * bin/temp_c, bin/temp_f: fix temperature listing, LP: #411108
  * bin/disk, bin/logo, bin/network, bin/release, bin/temp_c, bin/temp_f,
    byobu-janitor: move config files to sourced config file, use
    janitor item to clean up
  * byobu-status: source, rather than exec the target status script,
    such that the script inherits the source status config variables;
    source statusrc (which contains overrides)
  * bin/disk: fix up reading of MONITORED_DISK
  * byobu.1: update documentation for new .byobu/statusrc file
  * bin/ip_address: allow interface override
  * bin/ip_address, bin/mem_used, bin/temp_c, bin/temp_f,
    bin/wifi_quality: performance improvements, reduce the number of
    forked shells by multiple sed-calls on one line by using a single
    sed call with multiple -e arguments (thanks for the suggestion smoser)
  * debian/control: bumping standards version
  * po/byobu-bg.po, po/byobu-de.po, po/byobu-en_GB.po, po/byobu-es.po,
    po/byobu-fr.po, po/byobu-hu.po, po/byobu-id.po, po/byobu-it.po,
    po/byobu-ja.po, po/byobu-la.po, po/byobu-nl.po, po/byobu-ru.po,
    po/byobu-sv.po, po/es.po, po/fr.po, po/ru.po: translations import
    from launchpad

 -- Dustin Kirkland <email address hidden> Mon, 17 Aug 2009 14:30:27 -0500

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.

Other bug subscribers

Remote bug watches

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