]> git.notmuchmail.org Git - notmuch/commit
nmbug-status: Decode Popen output using the user's locale
authorW. Trevor King <wking@tremily.us>
Mon, 10 Feb 2014 18:40:24 +0000 (10:40 -0800)
committerDavid Bremner <david@tethera.net>
Tue, 11 Feb 2014 01:04:55 +0000 (21:04 -0400)
commitfd29d3f4fba4c5c88244b13de443c016d23da17b
treea8261f99195453084a3b793274ef8a86b9eeb763
parentee6b5c372f724d2352d155736b81d2d46130c929
nmbug-status: Decode Popen output using the user's locale

Avoid:

  $ ./nmbug-status --list-views
  Traceback (most recent call last):
    File "./nmbug-status", line 47, in <module>
      'cat-file', 'blob', sha1+':status-config.json'],
  TypeError: can't concat bytes to str

by explicitly converting the byte-stream read from Popen into a
Unicode string.  On Python 2, this conversion is str -> unicode; on
Python 3 it is bytes -> str.

_ENCODING is derived from the user's locale (or system default) in an
attempt to match Git's output encoding.  It may be more robust to skip
the encoding/decoding by using a Python wrapper like pygit2 [1] for
Git access.  That's a fairly heavy dependency though, and using the
locale will probably work.

[1]: http://www.pygit2.org/
devel/nmbug/nmbug-status