[MERGE] Obey --quiet on select builtins (fixes 239523)
Aaron Bentley
aaron at aaronbentley.com
Wed Aug 27 04:26:12 BST 2008
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Neil Martinsen-Burrell wrote:
> Bug #239523 notes that bzr tag --quiet blah is not quiet. The reason is
> that many of the commands in builtins.py use self.outf.write() to give
> user feedback and take no account of the global trace.is_quiet() state.
> In trying to fix this, I've tried to get away from every command having
> to check is_quiet when they want to give feedback. To do that I added a
> method on Command objects called mention() that checks is_quiet
> appropriately before it writes to self.outf.
is_quiet should only silence optional text.
optional text should only be emitted to stderr
We already have trace.note for this. I have already asked you to use
trace.note. You don't provide any justification for not using it.
Your previous effort was almost perfect-- in fact, I approved it. I
don't understand why you're rewriting from scratch here.
bb:reject
Aaron
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD8DBQFItMlT0F+nu1YWqI0RAufrAJwL+yl6mDjpsHHGMsC2p/XkHhDUxgCdEbSn
LT9y645lm3wZIiPsJ50Hc8g=
=Fn5q
-----END PGP SIGNATURE-----
More information about the bazaar
mailing list