<div dir="ltr">Hi Chris,<div><br></div><div>I agree with your logic, and will make the version change when I land it.</div><div><br></div><div>Cheers,</div><div><br></div><div>James</div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, 27 Oct 2015 at 16:58 Chris Bieneman via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@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">beanz accepted this revision.<br>
beanz added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
LGTM. One comment below, more for posterity than anything else.<br>
<br>
<br>
================<br>
Comment at: CMakeLists.txt:2<br>
@@ +1,3 @@<br>
+cmake_minimum_required(VERSION 2.8.12.2)<br>
+<br>
+# The test-suite is designed to be built in release mode anyway and<br>
----------------<br>
jmolloy wrote:<br>
> My unscientific straw poll of LLVM users inside ARM says that 2.8.12.2 is still the minimum we can expect - Ubuntu 14.10 LTS still ships this, I'm afraid, and as it's the latest LTS I'm not sure it's old enough to deprecate.<br>
The minimum version is dictated per-project, there is no global expectation. There is more burden involved in raising a version, than creating a new project with a higher minimum requirement, which is why I asked that we consider moving this up (to avoid having to move it up later).<br>
<br>
We don't have any requirement or expectation that we will support the version of tools in an LTS Linux release. We have had this conversation numerous times, and it is not considered unreasonable that we might in the future request that Linux users behave like all the other users of LLVM and acquire their own copies of CMake (neither OS X nor Windows ship CMake or package managers). The reason LLVM and Clang haven't raised their minimum versions is that we haven't had compelling reasons to justify the change.<br>
<br>
I believe that starting a new build system free of version-based checks is compelling reason to start this new project with a higher minimum version. Ideally I would want this to require at least CMake 3.2 which has been out for 6 months or so. That said, my personal preference for trying to push forward our CMake version isn't motivation enough to argue about landing this patch.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D14046" rel="noreferrer" target="_blank">http://reviews.llvm.org/D14046</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>