[llvm-dev] Migrate utils/ Python 2 scripts to Python 3

Serge Guelton via llvm-dev llvm-dev at lists.llvm.org
Mon Dec 10 01:08:04 PST 2018


On Fri, Nov 30, 2018 at 03:25:10PM +0100, Serge Guelton via llvm-dev wrote:
> On Thu, Feb 01, 2018 at 04:47:03PM -0500, James Y Knight via llvm-dev wrote:
> > That said, I do think it could make sense to prepare llvm for the world in
> > which "python" is python3 on some systems. So, I'd propose the following:
> > 1. Change all #! lines to say "#!/usr/bin/env python2.7" instead of "#!/usr/bin
> > /env python", if they only work with 2.7.
> > 2. If someone feels motivated, and if it doesn't make the code obtuse, port
> > scripts to work with either version -- and for such scripts, change the #! line
> > to say "#!/usr/bin/env python".
> 
> 
> I've started the work:
> 
>     https://reviews.llvm.org/D55121
> 
> More patch may come if there's an interest in that.

Most if not all Python scripts in the clang/ repo have been ported and are waiting for reviews.
I've split them in independant patches to ease the review works:

https://reviews.llvm.org/D55259
https://reviews.llvm.org/D55258
https://reviews.llvm.org/D55213
https://reviews.llvm.org/D55208
https://reviews.llvm.org/D55205
https://reviews.llvm.org/D55204
https://reviews.llvm.org/D55202
https://reviews.llvm.org/D55200
https://reviews.llvm.org/D55199
https://reviews.llvm.org/D55197
https://reviews.llvm.org/D55196
https://reviews.llvm.org/D55193
https://reviews.llvm.org/D55194

Have fun with these!


More information about the llvm-dev mailing list