You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "Todd Lipcon (Code Review)" <ge...@cloudera.org> on 2016/03/23 22:49:56 UTC

[kudu-CR] KUDU-1336 (take 2). Add a simple preflight check for thirdparty build

Hello Adar Dembo,

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

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

to review the following change.

Change subject: KUDU-1336 (take 2). Add a simple preflight check for thirdparty build
......................................................................

KUDU-1336 (take 2). Add a simple preflight check for thirdparty build

We've had a lot of users hit common build problems like not using a new enough
C++11 compiler on RHEL6. This commit adds a simple set of pre-flight checks
that will hopefully make it easier for new users to build Kudu.

I tested this by running an ubuntu:14.04 docker image, running preflight.py and
installing whatever it asked me to until it passed. This seemed to cover all
of the packages mentioned in the install docs.

I also verified that this passed on a RHEL6 system with devtoolset.

This is a second attempt at committing this patch after the first attempt
was reverted. This revision differs by not requiring 'gdb', since it isn't
actually necessary to build Kudu, despite being recommended in the docs.

Original Change-Id: I93ee4165bd560f9cd3f03877bd3011decc7e1a6f
Reviewed-on: http://gerrit.cloudera.org:8080/2450

Change-Id: If55e6baed692dc1fb3e4c5953506631423666060
---
M thirdparty/build-thirdparty.sh
A thirdparty/preflight.py
2 files changed, 158 insertions(+), 0 deletions(-)


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

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