<div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Mar 25, 2020 at 5:01 PM Tom Stellard 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:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 03/25/2020 06:20 AM, Louis Dionne wrote:<br>
> <br>
> <br>
>> On Mar 25, 2020, at 00:47, Tom Stellard <<a href="mailto:tstellar@redhat.com" target="_blank">tstellar@redhat.com</a>> wrote:<br>
>><br>
>> On 03/24/2020 09:00 PM, Petr Hosek via llvm-dev wrote:<br>
>>> In October, there was a discussion about updating CMake to 3.15: <a href="http://lists.llvm.org/pipermail/llvm-dev/2019-October/136295.html" rel="noreferrer" target="_blank">http://lists.llvm.org/pipermail/llvm-dev/2019-October/136295.html</a>. No decision was made, but maybe we should revisit that proposal? If we're going to require a newer version of CMake for some subprojects, I'd prefer to bump the minimum CMake version for all of LLVM.<br>
> <br>
> My personal opinion is that there's a tendency to view all subprojects under the LLVM umbrella as a single, monolithic project. That leads to the desire to make decisions for the whole project, which is often difficult, as opposed to making the right decision for each subproject, which is often easier. This results on subprojects being blocked from doing the right thing for them, like we've seen happen for pre-commit CI. But that's a much larger (non-technical) discussion than the scope of a simple CMake version bump.<br>
> <br>
> Let's try to bump CMake for all of LLVM and see how that goes.<br>
> <br>
>> Yes, I agree we should bump the version for all of LLVM, but I don't<br>
>> think we should bump the version without a long-term cmake usage plan.<br>
>> e.g. something like: After every release branch, we bump the cmake version<br>
>> to whatever version of cmake is X months old.<br>
>><br>
>> I think the concern that this was our one chance to bump the CMake version<br>
>> led to the choice of 3.15 as the next version, which would be too new for some Linux distros.<br>
>> I think if we had a planned upgrade path, it would be easier for those of us that<br>
>> want something really new to settle on a release that is a little bit older.<br>
> <br>
> Ok, how about the following policy:<br>
> <br>
>   After every release branch, we bump the CMake version to whatever version of CMake is 12 months old.<br>
> <br>
> This is simple, straightforward, and it gives a full year of old CMakes being supported. If we did this right now, this would take us to CMake 3.14.0, released around March 14th, 2019 (<a href="https://github.com/Kitware/CMake/releases/tag/v3.14.0" rel="noreferrer" target="_blank">https://github.com/Kitware/CMake/releases/tag/v3.14.0</a>). I believe the expectation should be that recent CMakes are upgraded using some package manager or download from the site -- we can't really expect the CMake version to be the one provided by the system, because that is either non-existent or very old on most Linux distributions AFAICT. Fortunately, installing a new CMake is incredibly easy.<br>
> <br>
> Is everybody OK with the above policy? What would be the preferred place to document it?<br>
> <br>
<br>
12 months is fine with me.<br>
<br>
I'm not sure the best place to document the policy.  Some suggestions are here:<br>
<a href="https://llvm.org/docs/CMake.html" rel="noreferrer" target="_blank">https://llvm.org/docs/CMake.html</a> or in the Programmer's manual.<br></blockquote><div><br></div><div>I think the relevant metric here needs to be "which version is available on current LTS distros", and not "how old is the release". The current cmake version I have on Ubuntu 18.04 is cmake 3.10.2, so I personally would be happy with a bump to that version, but not something newer. People using other distributions probably have other considerations. If someone wants to perform a cmake version bump, I believe the first step would be to gather what the cmake version availability across distros looks like right now, so an informed decision can be made.<br></div><div><br></div><div>Regards,<br></div><div>Nikita</div></div></div>