You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "Adar Dembo (Code Review)" <ge...@ip-10-146-233-104.ec2.internal> on 2016/01/07 09:00:02 UTC

[kudu-CR] cmake: relax version requirement

Hello Todd Lipcon,

I'd like you to do a code review.  Please visit

    http://gerrit.cloudera.org:8080/1733

to review the following change.

Change subject: cmake: relax version requirement
......................................................................

cmake: relax version requirement

In terms of raw feature set, cmake 3.2.0 is sufficient, and relaxing the
version requirement a bit allows cmake from Ubuntu 15.10 to be used.

Going forward let's use a new policy that is forgiving while still safe:
1. Require the oldest cmake that still satisfies Kudu's desired feature set.
2. Provide a cmake in thirdparty of the same version or newer.

Change-Id: I661f9d725abf8a31c82b308ceb7e6ef49a0551b4
---
M CMakeLists.txt
1 file changed, 6 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/33/1733/1
-- 
To view, visit http://gerrit.cloudera.org:8080/1733
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I661f9d725abf8a31c82b308ceb7e6ef49a0551b4
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <to...@cloudera.com>