Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5818 to look at the new patch set (#7). Change subject: [thirdparty] Make Boost a regular dependency ...................................................................... [thirdparty] Make Boost a regular dependency Previously our use of boost was header-only. However the new timestamp type makes extensive use of boost's date_time and, as such, we need to compile at least libboostdate_time. Since boost uses its own build system this required tinkering with jamfiles to make sure that the library is correctly built under tsan. Finally this patch also addresses a couple of other issues: - Adds "thirdparty/installed/common/include" to the include dirs explicitly (we were doing this indirectly when we added the boost include dir before). - Since FindBoost.cmake now explicitly sets BOOST_INCLUDE_DIR to point to wherever boost was installed, this removes the logic that makes sure we didn't pick up boost from the system path. Change-Id: I277c4fda15575e271c426735552762884cb28c43 --- M CMakeLists.txt A cmake_modules/FindKuduBoost.cmake M thirdparty/build-definitions.sh M thirdparty/build-thirdparty.sh 4 files changed, 119 insertions(+), 28 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/18/5818/7 -- To view, visit http://gerrit.cloudera.org:8080/5818 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I277c4fda15575e271c426735552762884cb28c43 Gerrit-PatchSet: 7 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Todd Lipcon <t...@apache.org>