# HG changeset patch # User Denis Laxalde # Date 1528278975 -7200 # Node ID 4d8268ef1fb713f793508d73fef19cb40986adba # Parent aa999699e5047da7512ececd8ead42a2f1b15e4b [py3] Decode subprocess stdout in toolsutils.show_diffs() On Python 3, subprocess' pipes are bytes stream so we must decode them before printing. Without this change, diff of configuration files (e.g. all-in-one.conf as displayed upon instance upgrade) will rendered as a single line on Python 3. diff -r aa999699e504 -r 4d8268ef1fb7 cubicweb/toolsutils.py --- a/cubicweb/toolsutils.py Thu May 03 16:47:51 2018 +0200 +++ b/cubicweb/toolsutils.py Wed Jun 06 11:56:15 2018 +0200 @@ -96,7 +96,7 @@ """ import shutil pipe = subprocess.Popen(['diff', '-u', appl_file, ref_file], stdout=subprocess.PIPE) - diffs = pipe.stdout.read() + diffs = pipe.stdout.read().decode('utf-8') if diffs: if askconfirm: print()