You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "Matthew Jacobs (Code Review)" <ge...@cloudera.org> on 2016/03/17 19:57:44 UTC

[Toolchain-CR] IMPALA-3191: toolchain components should use toolchain cmake

Hello Casey Ching, Tim Armstrong,

I'd like you to reexamine a change.  Please visit

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

to look at the new patch set (#3).

Change subject: IMPALA-3191: toolchain components should use toolchain cmake
......................................................................

IMPALA-3191: toolchain components should use toolchain cmake

The toolchain doesn't build on sles12. It appears to be a
cmake bug [1] though it hasn't been fixed yet upstream.

The cmake bug description mentions this workaround patching
cmake to avoid calling POSIX select(), and we should be
using the toolchain cmake to build toolchain components.

This also reverts the previous attempt to mitigate the issue
by avoiding bootstrapping cmake in parallel. That seemed to
help avoid the issue during bootstrapping, but the hang
could still occur later as well.

1: https://cmake.org/Bug/view.php?id=15873

Change-Id: I8a31098de42b4d5cb93c12bb2e054eadb8b8c72a
---
M buildall.sh
M init.sh
M source/avro/build.sh
M source/cmake/build.sh
A source/cmake/cmake-3.2.3-patches/0001-IMPALA-3191.patch
M source/googletest/build.sh
M source/gtest/build.sh
M source/llvm/build-source-tarball.sh
M source/llvm/build-trunk.sh
M source/lz4/build.sh
M source/zlib/build.sh
11 files changed, 37 insertions(+), 22 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Toolchain refs/changes/67/2567/3
-- 
To view, visit http://gerrit.cloudera.org:8080/2567
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I8a31098de42b4d5cb93c12bb2e054eadb8b8c72a
Gerrit-PatchSet: 3
Gerrit-Project: Toolchain
Gerrit-Branch: master
Gerrit-Owner: Matthew Jacobs <mj...@cloudera.com>
Gerrit-Reviewer: Casey Ching <ca...@cloudera.com>
Gerrit-Reviewer: Matthew Jacobs <mj...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>