Status line shows incorrect menu shortcut

Bug #490894 reported by agnul
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
byobu
Fix Released
Medium
Dustin Kirkland 
byobu (Ubuntu)
Fix Released
Medium
Dustin Kirkland 

Bug Description

The latest version of byobu (2.39-0ubuntu1~ppa1) always shows F9 as the menu shortcut, even if "screen-escape-sequences" keybindings are selected.

Things I've tried:

* removing the ~/.byobu and configuring different keybindings
* changing the keybindings to "Function keys" and back to "screen-escape-sequences"
* toggling the "menu" status notification off and on again
* stopping and restarting byobu after each of the changes above

byobu always shows F9 as the menu shortcut. If it's any help I'm running byobu on a fully patched hardy server while logged in remotely from a windows box using putty. The same happens on a laptop running 2.38 on karmic and gnome-terminal.

Related branches

cyba (marcus-kempe)
Changed in byobu:
status: New → Fix Released
status: Fix Released → New
Revision history for this message
Dustin Kirkland  (kirkland) wrote : Re: [Bug 490894] Re: Status line shows incorrect menu shortcut

Please paste the output of:

sh -x /usr/lib/byobu/menu

Revision history for this message
agnul (sucrabu) wrote :

Here it is:

+ [ = --detail ]
+ PKG=byobu
+ [ -r /home/agnul/.byobu/keybindings]
[: 1: missing ]
+ bindings=/home/agnul/.screenrc
+ grep -qs ^source.*screen-escape-keys$ /home/agnul/.screenrc
+ grep -qs ^source.*none$ /home/agnul/.screenrc
+ key=F9
+ gettext Menu
+ printf \005{= kw}%s:<\005{-}\005{=b kw}F9\005{= kw}> Menu
 {= kw}Menu:<{-}{=b kw}F9{= kw}>

Revision history for this message
agnul (sucrabu) wrote :

Looking at the /usr/lib/byobu/menu script I've been able to fix the problem just adding a space at line 22, changing

[ -r "$HOME/.$PKG/keybindings"]

to

[ -r "$HOME/.$PKG/keybindings" ]

Changed in byobu:
status: New → Fix Committed
importance: Undecided → Medium
assignee: nobody → Dustin Kirkland (kirkland)
Changed in byobu (Ubuntu):
status: New → Fix Committed
importance: Undecided → Medium
assignee: nobody → Dustin Kirkland (kirkland)
status: Fix Committed → In Progress
Changed in byobu (Ubuntu):
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (4.4 KiB)

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

---------------
byobu (2.40-0ubuntu1) lucid; urgency=low

  * bin/updates_available, statusrc: move the color coding out to the config
    file
  * byobu-config, byobu.1, po/*, profiles/common,
    profiles_generator/profile.skel, rpm/byobu.spec, statusrc:
    - add an apport hook, LP: #485380
  * bin/apport: improve the formatting of the detailed output to
    suggest installing apport if necessary
  * debian/control: suggest apport
  * bin/apport, bin/battery, bin/cpu_freq, bin/disk, bin/ec2_cost,
    bin/fan_speed, bin/hostname, bin/ip_address, bin/load_average,
    bin/logo, bin/mail, bin/mem_available, bin/mem_used, bin/menu,
    bin/network, bin/processes, bin/reboot_required, bin/release,
    bin/temp_c, bin/temp_f, bin/updates_available, bin/uptime, bin/users,
    bin/whoami, bin/wifi_quality statusrc: define a set of color variables,
    such that it's easier to make these theme-able
  * windows/common: suggest w3m rather than elinks
  * byobu-status, profiles/common, profiles_generator/profile.skel,
    statusrc: obviate the need for separate profiles per color; read the
    profile colors from a config file, and use a special "color" status
    item to dynamically render it
  * bin/menu: add a space to the -r test, LP: #490894
  * profiles_generator/profile.skel: use the reverse color for the current
    window highlighting; works better with odd color combinations, maximizes
    contrast
  * byobu-status: separate COLOR into FOREGROUND and BACKGROUND; more
    flexibility
  * byobu-config, byobu-select-profile: rework to independently select
    background color and foreground color, rather than profile; improve text
  * debian/install, debian/rules, profiles/common,
    profiles_generator/generate, profiles_generator/profile.skel,
    rpm/byobu.spec: get rid of the generated profiles; now, everything is
    handled by FOREGROUND and BACKGROUND colors, dynamically
  * byobu-janitor: clean up old ~/.byobu/profile symlinks; guess the user's
    colors and set them in ~/.byobu/colors; ensure that profile gets linked
    to the common one
  * byobu-select-profile, byobu-select-profile.1: deprecate the --set
    option, only support --background and --foreground now; update the
    documentation
  * debian/profile: establish symlinks for the color profiles to ensure
    smooth upgrades; all point to the common profile now
  * byobu-config, statusrc: enable whoami, hostname, ip_address by default;
    it seems I almost *always* turn these on, seem useful; re-disable if
    there are noisy complaints
  * byobu-janitor, statusrc: seed the ~/.byobu/status file with the
    default values, making it easier for users to customize themselves
  * bin/custom, byobu-config, byobu.1, profiles/common, rpm/byobu.spec,
    statusrc: add support for a custom status script indicator
  * byobu-config, byobu-select-profile: make color selection more readable;
    support ctrl-<space> escape sequence
  * bin/custom, byobu-status: support --detail in custom scripts
  * byobu-export, byobu-config:
    - comprehensively rewritten, basically put all of byobu's installed
      files in ~/.byobu, and use...

Read more...

Changed in byobu (Ubuntu):
status: Fix Committed → Fix Released
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.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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