Hello Internal Jenkins,

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

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

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

Change subject: C++11: Simplify thirdparty TSAN build
......................................................................

C++11: Simplify thirdparty TSAN build

This commit addresses Adar's feedback on the C++11 patch. TSAN instrumented
dependencies are now always built on Linux and installed into an alternate
prefix. This commit will be squashed into the C++11 mega patch prior to
submission.

The thirdparty directory now includes three prefixes: thirdparty/installed for
tools and C-only libraries, thirdparty/installed-deps for C++ dependencies, and
thirdparty/installed-deps-tsan for C++ dependencies for TSAN builds. Keeping the
prefixes seperate allows us to not require rebuilding thirdparty when switching
to or from a TSAN instrumented build.

Change-Id: I5e9824db19383556da88053779bdde1e66cdfa44
---
M CMakeLists.txt
M README.adoc
M build-support/tools/kudu-lint/cmake_modules/FindLLVM.cmake
M cmake_modules/FindBitshuffle.cmake
M cmake_modules/FindCrcutil.cmake
M cmake_modules/FindCyrusSASL.cmake
M cmake_modules/FindGFlags.cmake
M cmake_modules/FindGLog.cmake
M cmake_modules/FindGMock.cmake
M cmake_modules/FindGPerf.cmake
M cmake_modules/FindKRPC.cmake
M cmake_modules/FindLibEv.cmake
M cmake_modules/FindLibUnwind.cmake
M cmake_modules/FindLz4.cmake
M cmake_modules/FindProtobuf.cmake
M cmake_modules/FindSnappy.cmake
M cmake_modules/FindSqueasel.cmake
M cmake_modules/FindVmem.cmake
M cmake_modules/FindZlib.cmake
M src/kudu/client/samples/CMakeLists.txt
M thirdparty/.gitignore
A thirdparty/build-definitions.sh
M thirdparty/build-if-necessary.sh
M thirdparty/build-thirdparty.sh
M thirdparty/download-thirdparty.sh
M thirdparty/vars.sh
26 files changed, 722 insertions(+), 1,082 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/63/1763/3
-- 
To view, visit http://gerrit.cloudera.org:8080/1763
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I5e9824db19383556da88053779bdde1e66cdfa44
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Internal Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>

Reply via email to