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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to