[LLVMdev] [RFC] Raise minimum required CMake version to 3.0

Chris Bieneman beanz at apple.com
Tue Feb 24 09:41:52 PST 2015


> On Feb 24, 2015, at 9:32 AM, Tobias Grosser <tobias at grosser.es> wrote:
> 
> On 24.02.2015 17:45, Tobias Grosser wrote:
>> On 09.02.2015 20:12, Chris Bieneman wrote:
>>> It came up on another thread that our current minimum required CMake
>>> version 2.8.8, has some bugs that cause issues when building with MSVC
>>> + Ninja, and one potential solution was to raise the minimum required
>>> version of CMake.
>>> 
>>> CMake 3.0 is now 6 months old and CMake 3.1 has been released. I would
>>> like to propose moving our minimum required CMake version to 3.0.
>>> 
>>> I’ve attached patches to enforce the change in case anyone wants to
>>> test it out.
>>> 
>>> Thoughts/comments/concerns/objections?
>> 
>> Hi Chris,
>> 
>> this update broke my cmake LLVM+Polly buildbot (to my knowledge most
>> other bots use autoconf). I reverted this temporarily to avoid the
>> buildbot noise (this comes a little late, as I was traveling the last
>> days).
>> 
>> The buildbot is based on the latest debian stable (wheezy), which comes
>> with cmake 2.8.9. Is 2.8.9 enough to fix the bug?
>> 
>> Also, could we just limit the cmake version on windows builds? In the
>> end on linux cmake 2.8.9 works great.
> 
> Rafael asked if updating the bot is not an option. It indeed is an option, but requires cmake to be manually downloaded and installed instead of using the available packages.

Building CMake from source is really easy:

(1) git clone git://cmake.org/cmake.git <git://cmake.org/cmake.git> &&  cd cmake
(2) ./bootstrap --prefix=$PWD/bootstrap
(3) mkdir build && cd build
(4) ../bootstrap/bin/cmake -G Ninja ..
(5) ninja install

Since you already have CMake you can probably skip bootstrapping in step 2.

> 
> This is OK for one bot, but it will add complications for everybody who uses debian and wants to install LLVM. It also will make it more complicated to move to a fully cmake based build as all such debian based buildbots would require a manual installation of cmake (I have 12 more of these).

I really don’t think we should hold back moving to newer CMake versions because people don’t want to update their bots. The OS X and Windows bots don’t come with CMake at all, and people managing those bots aren’t complaining about having to update their bots.

> 
> Before doing so I wanted to understand if it is indeed intended to require debian-stable users to *manually* install and manage cmake or if there is not a lower overhead version. Can this bug not be fixed for windows without complicating live for debian users?

I don’t really care that much about this specific bug because I’m not a Windows user, but we could probably put some hacks into our CMake configs to work around it. My big point here is that we shouldn’t have to. We shouldn’t be stuck on a 2 year old version of a tool just because some version of Linux hasn’t updated.

-Chris

> 
> Cheers,
> Tobias

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20150224/b6eed1b3/attachment.html>


More information about the llvm-dev mailing list