Byobu fails to launch correctly with no arguments in nethack mode

Bug #568751 reported by Jason Conti
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
byobu
Fix Released
Low
Unassigned
byobu (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

The /usr/bin/byobu script, when launched with no arguments, checks if screen -wipe outputs "No Sockets found in ", but in nethack mode, the correct string is "This room is empty ". This results in things such as ~/.byobu/windows not getting sourced. Screen will load in nethack mode if ~/.nethackrc exists or $NETHACKOPTIONS. Including a quick patch to check for either of those and check for the appropriate string. May be a problem on a system where screen is compiled without the NETHACK flag. Perhaps both strings should always be tested?

Related branches

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

Thanks for the report and patch. This looks like a perfectly reasonable thing to include. It will be in the next release ;-)

Cheers.

Changed in byobu:
status: New → Triaged
importance: Undecided → Low
status: Triaged → Fix Committed
Revision history for this message
Dustin Kirkland  (kirkland) wrote :

Will be in 2.72.

Changed in byobu (Ubuntu):
status: New → Fix Committed
importance: Undecided → Low
Changed in byobu:
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

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

---------------
byobu (2.74-0ubuntu1) maverick; urgency=low

  * usr/lib/byobu/disk_io: fix regex to support RAID devices, LP: #572855
  * debian/postinst: make debconf question failsafe, LP: #569041
  * usr/bin/byobu, usr/share/byobu/profiles/byoburc: add support for
    different window sets, LP: #517796
  * usr/bin/byobu-select-session: use screen -ls, less overhead
  * usr/share/man/man1/byobu.1: document the different window sets

byobu (2.73-0ubuntu1) maverick; urgency=low

  [ Dustin Kirkland ]
  * usr/bin/byobu-reconnect-sockets: look for gpg sessions quietly
  * usr/bin/byobu-launcher: ensure that if user does not want a nested
    session, they still get a local shell

  [ Jon Bernard ]
  * etc/byobu/statusrc, usr/bin/byobu-config, usr/lib/byobu/rcs_cost,
    usr/share/byobu/profiles/common, usr/share/man/man1/byobu.1:
    - Rackspace cost estimation (based on the ec2 one)

byobu (2.72-0ubuntu1) maverick; urgency=low

  * usr/bin/byobu: support nethack screen mode, LP: #568751
  * usr/share/man/man1/byobu.1: document a sample custom script using
    custom colors, LP: #568967
  * usr/bin/byobu: LANG=C before grepping for text
  * usr/bin/byobu: handle unset term in tput 256 color check
  * bin/byobu-config, lib/byobu/disk_io, share/byobu/profiles/common,
    share/man/man1/byobu.1: added a disk throughput notification
  * usr/bin/byobu-janitor: add hyphenation to the regex to allow for
    disk_io
  * usr/lib/byobu/disk_io: improve MONITORED_DISK overriding
  * usr/lib/byobu/disk: minor simplification

byobu (2.71-0ubuntu1) maverick; urgency=low

  * usr/bin/byobu-launch, usr/bin/byobu-launcher-install,
    usr/bin/byobu-janitor: exit 0 after launching byobu; this is superior
    to previously reverted implementations where we exec'd byobu, since
    we'll only exit 0 if the byobu/screen session exits cleanly; this will
    keep the user from having to do the double-exit when detaching,
    LP: #568306

byobu (2.70-0ubuntu1) maverick; urgency=low

  * usr/bin/byobu-launcher: fix regression in last upload

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

  * usr/bin/byobu-launcher: prompt on nested screen connection, LP: #565398
  * debian/po/POTFILES.in, debian/po/de.po, debian/po/en_GB.po,
    debian/po/es.po, debian/po/pt.po, debian/po/ru.po,
    debian/po/templates.pot, po/bg.po, po/cs.po, po/de.po, po/es.po,
    po/fr.po, po/hu.po, po/is.po, po/it.po, po/la.po, po/lt.po, po/nl.po,
    po/pt.po, po/pt_BR.po, po/ru.po, po/sv.po, po/tr.po, po/uk.po:
    - translations merged, LP: #567309
  * bin/byobu-janitor, share/byobu/keybindings/f-keys,
    share/byobu/profiles/common, share/man/man1/byobu.1: add a keybinding,
    ctrl-a ~ to write the current window's buffer to a file, LP: #565399
  * debian/control, debian/screen-profiles.postinst, debian/preinst,
    debian/prerm, debian/rules, usr/bin/byobu-janitor:
    - remove more remnants of screen-profiles
  * debian/rules: drop color symlinks, no longer necessary, should be
    cleaned up by byobu-janitor now
 -- Dustin Kirkland <email address hidden> Wed, 28 Apr 2010 23:11:14 -0500

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