diff mbox

[3/3] pwclient: Fix Python 3 encoding of received strings

Message ID 1481333552-2294-4-git-send-email-thomas.monjalon@6wind.com
State Superseded
Headers show

Commit Message

Thomas Monjalon Dec. 10, 2016, 1:32 a.m. UTC 
The conversion encode("utf-8") makes a byte stream which is
poorly printed with Python 3.
However this encoding is required for Popen.communicate() but must be
done after str.join() which applies to a real string.

Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
---
 patchwork/bin/pwclient | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

Comments

Stephen Finucane Dec. 12, 2016, 12:04 p.m. UTC | #1 
On Sat, 2016-12-10 at 02:32 +0100, Thomas Monjalon wrote:
> The conversion encode("utf-8") makes a byte stream which is
> poorly printed with Python 3.
> However this encoding is required for Popen.communicate() but must be
> done after str.join() which applies to a real string.
> 
> Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>

Tested by both Sean and I.

Reviewed-by: Stephen Finucane <stephen@that.guru>
diff mbox

Patch

diff --git a/patchwork/bin/pwclient b/patchwork/bin/pwclient
index 42a7fd0..5a9e93c 100755
--- a/patchwork/bin/pwclient
+++ b/patchwork/bin/pwclient
@@ -269,7 +269,7 @@  def action_check_info(rpc, check_id):
     print(s)
     print('-' * len(s))
     for key, value in sorted(check.items()):
-        print("- %- 14s: %s" % (key, unicode(value).encode("utf-8")))
+        print("- %- 14s: %s" % (key, unicode(value)))
 
 
 def action_check_create(rpc, patch_id, context, state, url, description):
@@ -293,7 +293,7 @@  def action_info(rpc, patch_id):
     print(s)
     print('-' * len(s))
     for key, value in sorted(patch.items()):
-        print("- %- 14s: %s" % (key, unicode(value).encode("utf-8")))
+        print("- %- 14s: %s" % (key, unicode(value)))
 
 
 def action_get(rpc, patch_id):
@@ -311,7 +311,7 @@  def action_get(rpc, patch_id):
         i += 1
 
     with open(fname, 'w') as f:
-        f.write(unicode(s).encode("utf-8"))
+        f.write(unicode(s))
         print('Saved patch to %s' % fname)
 
 
@@ -748,15 +748,15 @@  def main():
             for patch_id in non_empty(h, patch_ids):
                 s = rpc.patch_get_mbox(patch_id)
                 if len(s) > 0:
-                    i.append(unicode(s).encode("utf-8"))
+                    i.append(unicode(s))
             if len(i) > 0:
-                pager.communicate(input="\n".join(i))
+                pager.communicate(input="\n".join(i).encode("utf-8"))
             pager.stdin.close()
         else:
             for patch_id in non_empty(h, patch_ids):
                 s = rpc.patch_get_mbox(patch_id)
                 if len(s) > 0:
-                    print(unicode(s).encode("utf-8"))
+                    print(unicode(s))
 
     elif action == 'info':
         for patch_id in non_empty(h, patch_ids):