<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class=""><br class=""><div><br class=""><blockquote type="cite" class=""><div class="">On Jan 29, 2021, at 3:18 PM, Stella Laurenzo via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" class="">llvm-dev@lists.llvm.org</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="auto" class="">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></div></blockquote><div><br class=""></div><div>This makes sense to me.</div><br class=""><blockquote type="cite" class=""><div class=""><br class=""><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" class="">llvm-dev@lists.llvm.org</a>> wrote:<br class=""></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" class="">
<div class=""><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 class=""></u></p></div></div></blockquote></div></div></blockquote><div>Pasting the used language in the docs to aid the discussion.</div><div><br class=""></div><div>Getting Started with the LLVM System —> Requirements —> Software</div><div><a href="https://llvm.org/docs/GettingStarted.html#software" class="">https://llvm.org/docs/GettingStarted.html#software</a></div><div><br class=""></div></div><blockquote style="margin: 0 0 0 40px; border: none; padding: 0px;" class=""><div><div><div>Compiling LLVM requires that you have several software packages installed. The</div></div></div><div><div><div>table below lists those required packages. The Package column is the usual name</div></div></div><div><div><div>for the software package that LLVM depends on. The Version column provides</div></div></div><div><div><div>“known to work” versions of the package. The Notes column describes how LLVM</div></div></div><div><div><div>uses the package and provides other details.</div></div></div></blockquote><div><div><br class=""></div><div>So the packages themselves are required, but the exact versions are “known to work”. Previously, I didn’t read the text carefully enough and misrepresented its meaning (causing the trouble that triggered this thread). I am voting for removing the “known to work” phrase from the docs since I don’t think the distinction between package and versioned-package is useful.</div><div><br class=""></div><div><font color="#000000" class="">Note that we already have a precedent for requiring the stated version with CMake. </font><span style="color: rgb(0, 0, 0);" class="">Most CMakeLists.txt start with `cmake_minimum_required(VERSION 3.13.4)`.</span></div><div><br class=""></div><div>As for the exact version (3.5 or 3.6) I have no strong preference, so I would follow Stella’s recommendation.</div><div><br class=""></div><blockquote type="cite" class=""><div class=""><div class="gmail_quote"><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" class=""><div class=""><p class="MsoNormal"><u class=""></u></p><p class="MsoNormal"><u class=""></u> <u class=""></u></p>
<div class="">
<div style="border:none;border-top:solid #e1e1e1 1.0pt;padding:3.0pt 0in 0in 0in" class=""><p class="MsoNormal"><b class="">From:</b> llvm-dev <<a href="mailto:llvm-dev-bounces@lists.llvm.org" rel="noreferrer noreferrer" target="_blank" class="">llvm-dev-bounces@lists.llvm.org</a>> <b class="">On Behalf Of
</b>Chris Tetreault via llvm-dev<br class="">
<b class="">Sent:</b> Friday, January 29, 2021 12:18 PM<br class="">
<b class="">To:</b> <a href="mailto:llvm-dev@lists.llvm.org" rel="noreferrer noreferrer" target="_blank" class="">llvm-dev@lists.llvm.org</a><br class="">
<b class="">Subject:</b> [EXT] [llvm-dev] Python version requirement<u class=""></u><u class=""></u></p>
</div>
</div><p class="MsoNormal"><u class=""></u> <u class=""></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" class="">https://reviews.llvm.org/D95635</a>) On
<a href="https://llvm.org/docs/GettingStarted.html#software" rel="noreferrer noreferrer" target="_blank" class="">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 class=""></u><u class=""></u></p><p class="MsoNormal"><u class=""></u> <u class=""></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 class=""></u><u class=""></u></p><p class="MsoNormal"><u class=""></u> <u class=""></u></p><p class="MsoNormal">Thanks,<u class=""></u><u class=""></u></p><p class="MsoNormal"> Christopher Tetreault<u class=""></u><u class=""></u></p>
</div>
</div>
_______________________________________________<br class="">
LLVM Developers mailing list<br class="">
<a href="mailto:llvm-dev@lists.llvm.org" rel="noreferrer noreferrer" target="_blank" class="">llvm-dev@lists.llvm.org</a><br class="">
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer noreferrer noreferrer" target="_blank" class="">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br class="">
</blockquote></div>
_______________________________________________<br class="">LLVM Developers mailing list<br class=""><a href="mailto:llvm-dev@lists.llvm.org" class="">llvm-dev@lists.llvm.org</a><br class="">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev<br class=""></div></blockquote></div><br class=""></body></html>