ProgressBarStack assumes TTY

Bug #55166 reported by Rich Landers
2
Affects Status Importance Assigned to Milestone
Bazaar
Fix Released
Medium
John A Meinel

Bug Description

Using bzr under emacs TERM is set to 'dumb'. Yet, TTYProgressBar seems to be used (lots of lines of output separated by ^M).

Looks like ProgressBarStack should use ProgressBar, not TTYProgress bar, to take advantage of the terminal detection.

This hack works for me (untested otherwise):

progress.py:96:
        #self._klass = klass or TTYProgressBar
        self._klass = klass or ProgressBar

Fedora Core 5, installed from extras. "bzr --version" is 0.8.2

Revision history for this message
John A Meinel (jameinel) wrote :

Already fixed.
Along with an env var "BZR_PROGRESS_BAR" which can be set to 'none', 'tty', or 'dots' to force the progress bar to be of a particular type.

Changed in bzr:
importance: Untriaged → Medium
status: Unconfirmed → Confirmed
Revision history for this message
John A Meinel (jameinel) wrote :

forgot to close the bug

Changed in bzr:
assignee: nobody → jameinel
status: Confirmed → 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.