<div dir="auto">I think we should enforce a minimum version and that should not be less than 3.6. <3.6 is eol, and bug fixes for 3.6 will continue through 2021. Obviously, we should make sure that bots are upgraded of course, I would think...</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jan 29, 2021, 12:28 PM Chris Tetreault via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank" rel="noreferrer">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">
<div lang="EN-US" link="#0563C1" vlink="#954F72" style="word-wrap:break-word">
<div>
<p class="MsoNormal">It was actually pointed out to me that the linked page just says “known to work”, not “minimum required”, so now I’m very confused as to what to do. I think two reasonable ways ahead are to just roll with it and require 3.6, or to revert
the recent patches that require it and to find the lowest version that works with the resulting code. Thoughts?<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<div style="border:none;border-top:solid #e1e1e1 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b>From:</b> llvm-dev <<a href="mailto:llvm-dev-bounces@lists.llvm.org" rel="noreferrer noreferrer" target="_blank">llvm-dev-bounces@lists.llvm.org</a>> <b>On Behalf Of
</b>Chris Tetreault via llvm-dev<br>
<b>Sent:</b> Friday, January 29, 2021 12:18 PM<br>
<b>To:</b> <a href="mailto:llvm-dev@lists.llvm.org" rel="noreferrer noreferrer" target="_blank">llvm-dev@lists.llvm.org</a><br>
<b>Subject:</b> [EXT] [llvm-dev] Python version requirement<u></u><u></u></p>
</div>
</div>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal"> I just pushed a patch that uses CMake to actually enforce the minimum version of Python 3.6. (<a href="https://reviews.llvm.org/D95635" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D95635</a>) On
<a href="https://llvm.org/docs/GettingStarted.html#software" rel="noreferrer noreferrer" target="_blank">https://llvm.org/docs/GettingStarted.html#software</a>, we document that Python 3.6 is the minimum version of python that is required for LLVM, and some patches have gone in to use Python 3.6 features.
As of a few days ago, if you try to run the test suite without python 3.6 or greater, you will get a syntax error in the test suite python code. However, I had to revert my patch because some of the build bots don’t actually have Python 3.6. Specifically,
I got a failure in the fuchsia-x86_64-linux bot that has Python 3.5.3. I have no way of knowing if any other infrastructure has this issue. (unless they email me to complain)<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal"> I think that Ideally we want CMake to enforce the version minimum, but we can’t do that until the build bots are all updated. I wanted to raise awareness of this issue so that hopefully we can get this sorted out soon.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">Thanks,<u></u><u></u></p>
<p class="MsoNormal"> Christopher Tetreault<u></u><u></u></p>
</div>
</div>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" rel="noreferrer noreferrer" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer noreferrer noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div>