<div dir="ltr"><div dir="ltr">On Tue, Dec 17, 2019 at 5:12 AM Serge Guelton <<a href="mailto:sguelton@redhat.com">sguelton@redhat.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">At the beginning of the year, I've landed a large set of patches to support both Python 2 and Python3 in most Python scripts. Looks like I missed some of them :-)<div>At that time, backward portability with Python2 was still relevant, and I suspect it will still be the case for a few distributions that ship Python2 by default. That being said, Even RHEL8 uses Python3 by default, so at some point we may be able to drop the compatibility stuff.</div><div>Until then, I'd argue for maintaining compatibility as it's not a tremendous task.</div></div></blockquote><div><br></div><div>This is my feeling as well. In yesterday's instance, I lost more time to fixing bugs in the py3 detection logic on systems that don't have it than it took me to make the script just run fine with both Python 2 and 3.</div><div><br></div><div>On macOS, I think 10.15 is the first version that ships with python3, and that was released just 2 months ago.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Dec 17, 2019 at 10:54 AM James Henderson via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">I personally only use Python 3 reluctantly. I've yet to encounter a situation where I actually preferred Python 3. That being said, given the decision to retire Python 2.7 (*grumble* *grumble*), I'll probably be forced sometime in the new year to uninstall it by somebody in charge of security somewhere. I certainly don't see a personal need to have all scripts support Python 2, unless they are used in the build/test pipeline somewhere (i.e. get touched by a fresh check-all).<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 17 Dec 2019 at 05:31, Fangrui Song via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><a href="https://reviews.llvm.org/D71565" rel="noreferrer" target="_blank">https://reviews.llvm.org/D71565</a> intends to update llvm/utils/update_cc_test_checks.py to work with Python 2.<br>
<br>
In the original review, I suggested that we don't add Python 2<br>
compatibility for new features because Python 2.7 is retiring and some<br>
Linux distributions are even deprecating/removing Python 2 support. My<br>
feeling is:<br>
<br>
If some utilities do not support Python 2, we should probably not bother<br>
making them Python 2 compatible. Maintaining Python 2/3 compatibility<br>
may not worth the efforts. "utilities" include some command line tools<br>
under llvm/utils, which are not part of instructure like lit. What do<br>
people think?<br>
<br>
BTW, what's the Python 3 support status of build bots? Are there any<br>
running Python 3?<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div>
</blockquote></div></div>