[PATCH] D37661: [sancov] coverage-report-server.py: ServerHandler(): open file as UTF8

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 12 11:52:29 PDT 2017


lebedev.ri added a comment.

In https://reviews.llvm.org/D37661#868351, @kcc wrote:

> can we just stick to python2?
>  I am not sure for how long this code will survive (I hope to migrate to clang's SourceBasedCodeCoverage) and don't want to spend time improving it.


Right now that script hardcodes python3:

  #!/usr/bin/env python3

IMHO python2 is dead, and really really, should not be used for anything new.


Repository:
  rL LLVM

https://reviews.llvm.org/D37661





More information about the llvm-commits mailing list