Status line shows incorrect menu shortcut

Reported by agnul on 2009-12-01
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
byobu
Medium
Dustin Kirkland 
byobu (Ubuntu)
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) on 2009-12-01
Changed in byobu:
status: New → Fix Released
status: Fix Released → New

Please paste the output of:

sh -x /usr/lib/byobu/menu

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}>

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
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  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers