should delete global is_quiet/is_verbose

Bug #656694 reported by Martin Pool
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Bazaar
Confirmed
Medium
Martin Pool
Breezy
Triaged
Medium
Unassigned

Bug Description

In 2.3b1 bzr has a _verbosity_level, is_quiet and is_verbose functions in trace.py. These cause test isolation issues (eg in landing of lp:~mbp/bzr/scripts) and they're not consistently respected. Commands that want a -v or -q option typically want to take some more actions based on that rather than just having trace output suppressed. It would perhaps be perhaps more reasonable to have a verbosity level in the ui object.

Martin Pool (mbp)
tags: added: tech-debt
removed: debt
Martin Pool (mbp)
Changed in bzr:
assignee: nobody → Martin Pool (mbp)
status: Confirmed → In Progress
Martin Pool (mbp)
Changed in bzr:
status: In Progress → Confirmed
Jelmer Vernooij (jelmer)
Changed in brz:
importance: Undecided → Medium
milestone: none → 3.0.0
Jelmer Vernooij (jelmer)
Changed in brz:
status: New → Triaged
Jelmer Vernooij (jelmer)
Changed in brz:
milestone: 3.0.0 → 3.1.0
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.