byobu

byobu battery status shows only one battery

Bug #693560 reported by Philip Muškovac
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
byobu
Fix Released
Wishlist
Unassigned
byobu (Ubuntu)
Fix Released
Wishlist
Unassigned

Bug Description

Natty with byobu 3.16-0ubuntu1

usr/lib/byobu/battery has support for multiple batteries, but the actual code is:

for bat in /proc/acpi/battery/*; do

        </snip>

        printf "$bcolor%s$(color -)$color|%s|$(color -) " "$percent" "$sign"
        break
done

Thanks to the 'break' at the end it will abort after showing the first battery ignoring any other batteries.

Related branches

lp:byobu
lp:ubuntu/natty/byobu
Revision history for this message
Dustin Kirkland  (kirkland) wrote :

Correct. That is by design.

I can add support for defining the particular battery you want as BATTERY= in ~/.byobu/statusrc.

Changed in byobu:
status: New → Triaged
importance: Undecided → Wishlist
Dustin Kirkland  (kirkland)
Changed in byobu:
status: Triaged → Fix Committed
Dustin Kirkland  (kirkland)
Changed in byobu (Ubuntu):
status: New → Fix Committed
importance: Undecided → Wishlist
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
Dustin Kirkland  (kirkland)
Changed in byobu:
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.