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>