[PATCH] D51220: [clang-tidy] run-clang-tidy fails using python 3.7

Andi via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 19 04:55:20 PDT 2018


This revision was automatically updated to reflect the committed changes.
Closed by commit rL342540: [clang-tidy] run-clang-tidy.py - fails using python 3.7 (authored by Abpostelnicu, committed by ).
Herald added a subscriber: llvm-commits.

Changed prior to commit:
  https://reviews.llvm.org/D51220?vs=166095&id=166096#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D51220

Files:
  clang-tools-extra/trunk/clang-tidy/tool/run-clang-tidy.py


Index: clang-tools-extra/trunk/clang-tidy/tool/run-clang-tidy.py
===================================================================
--- clang-tools-extra/trunk/clang-tidy/tool/run-clang-tidy.py
+++ clang-tools-extra/trunk/clang-tidy/tool/run-clang-tidy.py
@@ -167,9 +167,9 @@
     if proc.returncode != 0:
       failed_files.append(name)
     with lock:
-      sys.stdout.write(' '.join(invocation) + '\n' + output + '\n')
-      if err > 0:
-        sys.stderr.write(err + '\n')
+      sys.stdout.write(' '.join(invocation) + '\n' + output.decode('utf-8') + '\n')
+      if len(err) > 0:
+        sys.stderr.write(err.decode('utf-8') + '\n')
     queue.task_done()
 
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D51220.166096.patch
Type: text/x-patch
Size: 672 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180919/ef3023fd/attachment.bin>


More information about the cfe-commits mailing list