[MERGE][#151844] Unicode-safe output from ``bzr info``

Martin Pool mbp at canonical.com
Mon Oct 15 03:10:11 BST 2007


Martin Pool has voted tweak.
Status is now: Conditionally approved
Comment:
Thankyou for addressing this.

You could be more specific in the news entry about just what's being 
fixed -- that unrepresentable characters will be replaced.

          self.assertNotEqual(-1, txt.find(msg.encode('ascii', 
'replace')))
+
+    def test_info(self):
+        bzr = self.run_bzr_decode
+        bzr(['branch', u'.', self.info['directory']])
+        bzr(['info', self.info['directory']])
+        bzr(['info', self.info['directory']], encoding='ascii')

We've been removing uses of bzr= like this - it does make the tests 
shorter but less explicit.

Where does self.info[] get set anyhow?


+    def test_info(self):
+        bzr = self.run_bzr_decode
+        bzr(['branch', u'.', self.info['directory']])
+        bzr(['info', self.info['directory']])
+        bzr(['info', self.info['directory']], encoding='ascii')


For details, see: 
http://bundlebuggy.aaronbentley.com/request/%3C5288a560710120333t50ad35eagccd51987e20adea5%40mail.gmail.com%3E



More information about the bazaar mailing list