[PATCH] D82767: clang-format: Explicitly use python3

serge via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 9 09:15:30 PDT 2020


serge-sans-paille added a comment.

In D82767#2132903 <https://reviews.llvm.org/D82767#2132903>, @MyDeveloperDay wrote:

> We may not be consistent across all of LLVM
>
>   $ find . -name '*.py' -print -exec /usr/bin/head -2 {} \; | grep "#!" | sort | uniq -c
>         6 #! /usr/bin/env python
>         2 #! /usr/bin/env python3
>         2 #! /usr/bin/python
>         1 #!/bin/env python
>       133 #!/usr/bin/env python
>        13 #!/usr/bin/env python3
>        49 #!/usr/bin/python
>


My understanding is that explicitly requiring python3 may make sense if the script is not backward-compatible with python2, while requiring python means the version is not important.
At the end-of-year, we should be able to harmonize shebangs to #!/usr/bin/env python3 or #!/usr/bin/env python


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82767/new/

https://reviews.llvm.org/D82767





More information about the cfe-commits mailing list