F8 in tmux backend does not handle spaces correctly

Bug #1224352 reported by Chow Loong Jin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
byobu (Ubuntu)
Fix Released
Medium
Dustin Kirkland 

Bug Description

  affects ubuntu/byobu

When hitting F8 and renaming the window with a space inside, the command throws
an error and doesn't actually rename the window. "C-b ," from tmux's default
config works correctly though.

From tmux's default config:
bind-key , command-prompt -I #W "rename-window '%%'"

Byobu's config:
bind-key F8 command-prompt -p (rename-window) "rename-window %%"

I'd suggest either using tmux's version, or surrounding %% in the Byobu version with ''.

--
Kind regards,
Loong Jin

Changed in byobu (Ubuntu):
importance: Undecided → Medium
assignee: nobody → Dustin Kirkland (kirkland)
status: New → In Progress
Revision history for this message
Dustin Kirkland  (kirkland) wrote :

What version of Byobu are you running? I see that this is already fixed in trunk...

I did commit a small change, using the -I "#W" to show the current title.

Revision history for this message
Chow Loong Jin (hyperair) wrote : Re: [Bug 1224352] Re: F8 in tmux backend does not handle spaces correctly

On Thu, Sep 12, 2013 at 03:51:49PM -0000, Dustin Kirkland wrote:
> What version of Byobu are you running? I see that this is already fixed
> in trunk...

The one in raring:
byobu:
  Installed: 5.35-0ubuntu1
  Candidate: 5.35-0ubuntu1
  Version table:
 *** 5.35-0ubuntu1 0
        500 http://sg.archive.ubuntu.com/ubuntu/ raring/main amd64 Packages
        100 /var/lib/dpkg/status

> I did commit a small change, using the -I "#W" to show the current
> title.

Thanks.

--
Kind regards,
Loong Jin

Revision history for this message
Launchpad Janitor (janitor) wrote :

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

---------------
byobu (5.58-0ubuntu1) saucy; urgency=low

  * usr/share/byobu/keybindings/f-keys.tmux,
    usr/share/byobu/keybindings/f-keys.tmux.disable,
    usr/share/byobu/keybindings/Makefile.am,
    usr/share/byobu/keybindings/mouse.tmux.disable,
    usr/share/byobu/keybindings/mouse.tmux.enable,
    usr/share/doc/byobu/help.tmux.txt:
    - add support for a new keybinding, Alt-F12, to toggle on/off
      tmux mouse support
    - display a message when toggling either on or off mouse or
      F-keys support
  * usr/share/byobu/keybindings/f-keys.screen,
    usr/share/byobu/keybindings/f-keys.screen.disable,
    usr/share/byobu/keybindings/f-keys.tmux,
    usr/share/byobu/keybindings/f-keys.tmux.disable,
    usr/share/doc/byobu/help.tmux.txt: LP: #1218680
    - byobu-screen has long supported Ctrl-A-~ which saves your scrollback
      buffer to a file
    - byobu-tmux had never quite implemented this
    - Add a new keybinding to both byobu-screen and byobu-tmux that adds
      this support, Shift-F9
  * usr/share/byobu/keybindings/f-keys.tmux: LP: #1224352
    - show current window name when prompting for rename-window
 -- Dustin Kirkland <email address hidden> Tue, 03 Sep 2013 14:40:09 -0500

Changed in byobu (Ubuntu):
status: In Progress → 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.