<div dir="auto"><div>Awesome, that's great news! Thanks!<br><br><div class="gmail_quote"><div dir="ltr">On Fri, Jan 4, 2019, 4:32 AM Serge Guelton via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon, Dec 10, 2018 at 10:08:04AM +0100, Serge Guelton via llvm-dev wrote:<br>
> On Fri, Nov 30, 2018 at 03:25:10PM +0100, Serge Guelton via llvm-dev wrote:<br>
> > On Thu, Feb 01, 2018 at 04:47:03PM -0500, James Y Knight via llvm-dev wrote:<br>
> > > That said, I do think it could make sense to prepare llvm for the world in<br>
> > > which "python" is python3 on some systems. So, I'd propose the following:<br>
> > > 1. Change all #! lines to say "#!/usr/bin/env python2.7" instead of "#!/usr/bin<br>
> > > /env python", if they only work with 2.7.<br>
> > > 2. If someone feels motivated, and if it doesn't make the code obtuse, port<br>
> > > scripts to work with either version -- and for such scripts, change the #! line<br>
> > > to say "#!/usr/bin/env python".<br>
> > <br>
> > <br>
> > I've started the work:<br>
> > <br>
> >     <a href="https://reviews.llvm.org/D55121" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D55121</a><br>
> > <br>
> > More patch may come if there's an interest in that.<br>
> <br>
> Most if not all Python scripts in the clang/ repo have been ported and are waiting for reviews.<br>
> I've split them in independant patches to ease the review works:<br>
> <br>
> <a href="https://reviews.llvm.org/D55259" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D55259</a><br>
> <a href="https://reviews.llvm.org/D55258" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D55258</a><br>
> <a href="https://reviews.llvm.org/D55213" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D55213</a><br>
> <a href="https://reviews.llvm.org/D55208" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D55208</a><br>
> <a href="https://reviews.llvm.org/D55205" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D55205</a><br>
> <a href="https://reviews.llvm.org/D55204" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D55204</a><br>
> <a href="https://reviews.llvm.org/D55202" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D55202</a><br>
> <a href="https://reviews.llvm.org/D55200" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D55200</a><br>
> <a href="https://reviews.llvm.org/D55199" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D55199</a><br>
> <a href="https://reviews.llvm.org/D55197" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D55197</a><br>
> <a href="https://reviews.llvm.org/D55196" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D55196</a><br>
> <a href="https://reviews.llvm.org/D55193" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D55193</a><br>
> <a href="https://reviews.llvm.org/D55194" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D55194</a><br>
> <br>
> Have fun with these!<br>
<br>
Hi folks,<br>
<br>
It is my great pleasure to announce that most-if-not-all Python scripts from llvm,<br>
clang and test-suite have been updated to work (hopefully) the same way<br>
on Python2 and Python3.<br>
<br>
As a consequence, all shebangs now points to python, and not python2 or<br>
python3, and the documentation should now refer to python, whith no<br>
specified version.<br>
<br>
So what's next? Maybe some buildbot could be updated to specifically<br>
build with a given Python version, if it's not already the case.<br>
<br>
I'll also added an herald rule to review commited python scripts, to<br>
ensure the compatible property remains true over time.<br>
<br>
Oh, and I *really* would like to thanks Michael Platings for the tremendous<br>
reviewing job he's been doing on this patch set o/<br>
<br>
++<br>
Serge<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank" rel="noreferrer">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div></div></div>