still using tty progressbar when we don't have a tty

Bug #51107 reported by John A Meinel
2
Affects Status Importance Assigned to Milestone
Bazaar
Fix Released
Undecided
John A Meinel

Bug Description

When we upgraded to nested progress bars, we lost the ability to create a dots progress bar instead of a tty.
This patch fixes this, but needs a test case.
=== modified file 'bzrlib/progress.py'
--- bzrlib/progress.py 2006-06-21 02:33:40 +0000
+++ bzrlib/progress.py 2006-06-27 13:12:33 +0000
@@ -93,7 +93,7 @@
         self._show_count = show_count
         self._to_messages_file = to_messages_file
         self._stack = []
- self._klass = klass or TTYProgressBar
+ self._klass = klass or ProgressBar

     def top(self):
         if len(self._stack) != 0:

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

cleanup-progress branch.

Changed in bzr:
assignee: nobody → jameinel
status: Unconfirmed → Fix Committed
Revision history for this message
John A Meinel (jameinel) wrote :

Default is selected based on 'isatty'. Can be forced with env var BZR_PROGRESS_BAR

Changed in bzr:
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.