byobu -ls doesn't show sessions with custom sessionnames

Bug #630673 reported by Brian Parma
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
byobu
Fix Released
Low
Unassigned
byobu (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

if the user doesn't provide a session name with "-S sessionname", then byobu automatically sets "-S byobu". According to screen's manpage, this applies the sessionname "byobu" to -ls/-list, -r, (and -x?).

This is fine for having a single screen/byobu session open, but if the user has created multiple sessions with "byobu -S custom", than those sessions don't appear in "byobu -ls"/"byobu -list" and can't be attached with "byobu -r"/"byobu -x".

I don't know if this is the best way to fix this, but an easy way is just to add these 4 commands to the case statement at line 79:

PROFILE="-c $BYOBU_PREFIX/share/$PKG/profiles/byoburc"
NAME="-S $PKG"
# Zero out $NAME if user has specified a -S
for i in $@; do
    case $i in
        -S|-ls|-list|-x|-r) NAME= ;;
    esac
done

If there is only one session, screen -r and screen -x behave the same with or without the -S byobu specified, so this change doesn't change single-session behavior.

Related branches

Revision history for this message
Dustin Kirkland  (kirkland) wrote :

Subscribing Scott Moser, since he and I discussed this recently.

Revision history for this message
Dustin Kirkland  (kirkland) wrote :

So the easy workaround would be for you to prepend "byobu." onto your custom session name, like:

  byobu -S byobu.asdf

I suppose I could have byobu force this for you.

Revision history for this message
Brian Parma (bj0) wrote :

That would work as well, though it would be better if it was automatic, because I know I would forget.

This still would only show sessions created with byobu, though. If someone also used screen to create a session, byobu would still not see it.

Revision history for this message
Dustin Kirkland  (kirkland) wrote :

Okay, triaged/low.

I'll get this fixed soon.

Changed in byobu:
status: New → Triaged
importance: Undecided → Low
Changed in byobu:
status: Triaged → Fix Committed
Changed in byobu:
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (3.2 KiB)

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

---------------
byobu (3.7-0ubuntu1) natty; urgency=low

  [ Dustin Kirkland ]
  * usr/lib/byobu/ec2_cost, usr/share/byobu/ec2/rates.eu_ie,
    usr/share/byobu/ec2/rates.us_ca, usr/share/byobu/ec2/rates.us_va:
    - add ec2_cost support for t1.micro instances, LP: #669190
  * etc/byobu/statusrc, usr/bin/byobu-config, usr/lib/byobu/Makefile.am,
    usr/lib/byobu/notify_osd, usr/share/byobu/profiles/common,
    usr/share/man/man1/byobu.1:
    - add notify_osd support to byobu, LP: #669189
    - separate the perl into a script of its own
    - add per-pid notifier monitors, terminate when screen terminates
    - allow for disabling the notify_osd status; must handle in the perl
      script
    - use BYOBU_PREFIX for local installs
  * usr/bin/byobu-config, usr/bin/byobu-janitor, usr/bin/byobu-launcher-
    uninstall, usr/share/byobu/keybindings/f-keys:
    - use gsed, where available; should help with BSD issues running byobu,
      LP: #651993
  * README: add a note about gsed
  * usr/share/man/man1/byobu.1: add a synopsis section that mentions options
    simply passed through to screen, LP: #669566
  * usr/bin/byobu, usr/bin/byobu-janitor,
    usr/share/byobu/profiles/common: change location of $SSH_AUTH_SOCK symlink
    from /var/run to $HOME, as we can't be sure that the user's var/run
    dir will exist before screen starts, LP: #664059
  * etc/update-motd.d, etc/update-motd.d/55-window-manager,
    etc/update-motd.d/Makefile.am, Makefile.am, usr/bin/byobu-disable,
    usr/bin/byobu-enable:
    - add a helpful update-motd message and a convenient command to enable-and-
      launch byobu
  * usr/bin/byobu-select-session: enable byobu-select-session to list
    custom session names, LP: #630673
  * debian/templates, etc/byobu/statusrc, usr/bin/byobu, usr/bin/byobu-
    janitor, usr/bin/byobu-launcher, usr/bin/byobu-launcher-install,
    usr/bin/byobu-launcher-uninstall, usr/bin/byobu-select-profile,
    usr/bin/byobu-status, usr/lib/byobu/custom, usr/lib/byobu/disk_io,
    usr/lib/byobu/ec2_cost, usr/lib/byobu/hostname,
    usr/lib/byobu/ip_address, usr/lib/byobu/logo,
    usr/lib/byobu/mem_available, usr/lib/byobu/menu,
    usr/lib/byobu/network, usr/lib/byobu/.notify_osd,
    usr/lib/byobu/reboot_required, usr/lib/byobu/services,
    usr/lib/byobu/updates_available, usr/share/byobu/keybindings/f-keys,
    usr/share/byobu/keybindings/screen-escape-keys,
    usr/share/byobu/profiles/byoburc, usr/share/byobu/profiles/common,
    usr/share/man/man1/byobu.1:
    - xdg user dirs overhaul, LP: #553105
  * etc/byobu/statusrc, usr/bin/byobu-config, usr/lib/byobu/mail,
    usr/lib/byobu/Makefile.am, usr/lib/byobu/raid,
    usr/share/byobu/profiles/common, usr/share/man/man1/byobu.1:
    - add a raid notification, LP: #669191
  * usr/bin/byobu-status, usr/lib/byobu/notify_osd:
    - fix all detail printing
  * usr/bin/byobu:
    - fix listing of sessions, LP: #672023
  * usr/lib/byobu/services:
    - drop max instance count from NC service reporting; was inaccurate

  [ Marcus Asshauer ]
  * usr/bin/byobu-config: add directory option when creating new window,
    LP: #623033
 -- Dustin Kirkla...

Read more...

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