byobu doesn't set bash prompt properly

Bug #1280550 reported by Arminius Silvanus on 2014-02-15
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Dustin Kirkland 
byobu (Ubuntu)
Dustin Kirkland 

Bug Description

I normally use zsh, but occasionally I switch to bash. In that case, the bashrc in /usr/share/byobu/profiles/bashrc doesn't set my prompt like it should do.

I've done a bit of research on it - byobu's bashrc checks for the environment variable SHELL, which is set to the user's _preferred_ shell, not the currently running shell (see In my case, SHELL points to /usr/bin/zsh, regardless whether I'm running zsh or bash.

The correct way to check if bash is running is to check for the environment variable BASH. I've attached a patch that does just that.

Dustin Kirkland  (kirkland) wrote :

bzr commit -m '* usr/share/byobu/profiles/bashrc: LP: #1280550
  - test the $BASH variable, rather than $SHELL, to tell if we're
    running in a bash environment' --fixes 'lp:1280550'
Committing to: /home/local/media/source/byobu/byobu/
modified debian/changelog
modified usr/share/byobu/profiles/bashrc
Committed revision 2293.

Changed in byobu:
status: New → In Progress
importance: Undecided → Medium
assignee: nobody → Dustin Kirkland  (kirkland)
Changed in byobu (Ubuntu):
status: New → In Progress
importance: Undecided → Medium
assignee: nobody → Dustin Kirkland  (kirkland)
Changed in byobu:
status: In Progress → Fix Committed
Changed in byobu (Ubuntu):
status: In Progress → Fix Committed

The attachment "0001-Fixed-bash-env-var-check.patch" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

tags: added: patch
Changed in byobu:
status: Fix Committed → Fix Released
Launchpad Janitor (janitor) wrote :
Download full text (3.5 KiB)

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

byobu (5.74-0ubuntu1) trusty; urgency=low

  [ Dustin Kirkland ]
  * usr/lib/byobu/include/ LP: #1286367
    - merged in updated instance pricing script from
  * usr/share/byobu/profiles/tmux: LP: #1273685
    - remove double sourcing of local .tmux.conf
  * usr/lib/byobu/battery: LP: #1270451
    - add support for yet another batter type (Android)
    - thanks to Felix Krull for the initial patch
  * usr/share/byobu/profiles/tmux: LP: #1270551
    - enable ctrl-left and ctrl-right to move among words

  [ Chris Johnston ]
  * usr/share/man/man1/byobu-layout.1: LP: #1288738
    - fix manpage typo

  [ Mike Miller ]
  * usr/lib/byobu/logo: LP: #1278006
    - fix RHEL detection for logo

  [ Daniel Hahler ]
  * usr/bin/byobu-keybindings: LP: #1258093
    - fix toggling of f-keys
  * usr/share/byobu/profiles/tmux: LP: #1258534
    - ensure that we respect BYOBU_NO_TITLE in tmux

  [ Ryo ONODERA ]
  *, etc/profile.d/ => etc/profile.d/Z97-, usr/bin/byobu-config => usr/bin/,
    usr/bin/byobu-ctrl-a => usr/bin/, usr/bin/byobu-
    disable-prompt => usr/bin/, usr/bin/byobu-
    disable => usr/bin/, usr/bin/byobu-enable-prompt =>
    usr/bin/, usr/bin/byobu-enable =>
    usr/bin/, usr/bin/byobu-export => usr/bin/byobu-, usr/bin/byobu-janitor => usr/bin/,
    usr/bin/byobu-launcher-install => usr/bin/,
    usr/bin/byobu-launcher-uninstall => usr/bin/byobu-launcher-, usr/bin/byobu-launcher => usr/bin/,
    usr/bin/byobu-launch => usr/bin/, usr/bin/byobu-
    layout => usr/bin/, usr/bin/byobu-prompt =>
    usr/bin/, usr/bin/byobu-quiet => usr/bin/byobu-, usr/bin/byobu-reconnect-sockets => usr/bin/byobu-, usr/bin/byobu-select-backend => usr/bin/byobu-, usr/bin/byobu-select-profile => usr/bin/byobu-, usr/bin/byobu-select-session => usr/bin/byobu-, usr/bin/byobu-shell => usr/bin/,
    usr/bin/byobu-silent => usr/bin/, usr/bin/byobu-
    status-detail => usr/bin/, usr/bin/byobu-
    status => usr/bin/, usr/bin/byobu-ugraph =>
    usr/bin/, usr/bin/byobu-ulevel => usr/bin/byobu-, usr/bin/byobu => usr/bin/,
    usr/lib/byobu/include/ =>
    usr/lib/byobu/include/, usr/lib/byobu/include/dirs =>
    usr/lib/byobu/include/, usr/lib/byobu/include/toggle-utf8 =>
    usr/lib/byobu/include/, usr/share/byobu/tests/byobu-
    time-notifications => usr/share/byobu/tests/byobu-time-
    - use @prefix@ rather than hardcoding /usr, makes code
      more portable and easier to build with ./config --prefix

  [ Arminius Silvanus ]
  * usr/share/byobu/profiles/bashrc: LP: #1280550
    - te...


Changed in byobu (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers