Alexey Serbin has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/17699 )

Change subject: [cmake] Boost to use /dev/[u]random for UUID generation (take 2)
......................................................................

[cmake] Boost to use /dev/[u]random for UUID generation (take 2)

Since boost's UUID library is a header-only built dependency, it's not
enough adding -DBOOST_UUID_RANDOM_PROVIDER_FORCE_POSIX into b2's
cxxflags while building boost in Kudu's thirdparty.  This patch
addresses the issue, defining the BOOST_UUID_RANDOM_PROVIDER_FORCE_POSIX
macro for the preprocessor while building Kudu binaries themselves.
This is similar to the treatment of a similar macro for the boost's
date_time library (the latter is both a binary and a header dependency).

Maybe, the issue might be solved by solely adding the corresponding
flag into the b2's 'define' flag, but I decided to proceed the same
way as it is done for another macro for the boost library.

This is a follow-up to 27418145bc60ceb009c663626a88e57748a85c9f.

Change-Id: I5eedb644bd9f1cd7a48254580bc8307053ce6c9f
Reviewed-on: http://gerrit.cloudera.org:8080/17687
Reviewed-by: Attila Bukor <abu...@apache.org>
Tested-by: Alexey Serbin <aser...@cloudera.com>
(cherry picked from commit 35b5664f908cd1250c9f01e5dff77b653cfd12b7)
Reviewed-on: http://gerrit.cloudera.org:8080/17699
Reviewed-by: Bankim Bhavsar <ban...@cloudera.com>
Tested-by: Kudu Jenkins
---
M CMakeLists.txt
M thirdparty/build-definitions.sh
2 files changed, 9 insertions(+), 2 deletions(-)

Approvals:
  Bankim Bhavsar: Looks good to me, approved
  Kudu Jenkins: Verified

--
To view, visit http://gerrit.cloudera.org:8080/17699
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.15.x
Gerrit-MessageType: merged
Gerrit-Change-Id: I5eedb644bd9f1cd7a48254580bc8307053ce6c9f
Gerrit-Change-Number: 17699
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <abu...@apache.org>
Gerrit-Reviewer: Bankim Bhavsar <ban...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to