Todd Lipcon has submitted this change and it was merged. 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 Reviewed-on: http://gerrit.cloudera.org:8080/2615 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> --- M thirdparty/build-thirdparty.sh A thirdparty/preflight.py 2 files changed, 158 insertions(+), 0 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/2615 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: If55e6baed692dc1fb3e4c5953506631423666060 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
