Hello Ashwani Raina, Kudu Jenkins, Abhishek Chennaka, Joe McDonnell,

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

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

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

Change subject: WIP [thirdparty] update boost up 1.91.0
......................................................................

WIP [thirdparty] update boost up 1.91.0

WIP:
  * Collect initial feedback
  * Explore an alternative approach creating the git archive: include
    only the modules that are used by Kudu and its thirdparty
    components.  That way the compressed source tarball is ~4.5x times
    smaller in size: 28MiB vs 127MiB

This is one of the preliminary updates in the context of KUDU-3788.
The main motivation behind this update is to use cmake for building
the Boost library instead of the built-in b2 tool.  With cmake-based
build now it's possible to stage the pre-built binaries into a custom
DESTDIR while still keeping pre-defined PREFIX which doesn't depend on
the location of the staging directory.

And of course, the new version comes with many improvements and updates.

In Boost 1.91.0, the BOOST_UUID_RANDOM_PROVIDER_FORCE_POSIX macro does
not exist anymore: platform-specific entropy providers were removed and
random generators switched to std::random_device since 1.86.0 [1].
As it turns out, none of the shenanigans with explicitly setting
compiler for --with-toolset are necessary when building with cmake
since it's smart enough to derive proper toolset from CC and CXX
variables.

[1] 
https://www.boost.org/doc/libs/latest/libs/uuid/doc/html/uuid.html#introduction

Change-Id: I4c9c2e9260b892c721ec2ba887c61b8aeef35888
---
M thirdparty/build-definitions.sh
M thirdparty/download-thirdparty.sh
M thirdparty/vars.sh
3 files changed, 63 insertions(+), 68 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/81/24581/2
--
To view, visit http://gerrit.cloudera.org:8080/24581
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I4c9c2e9260b892c721ec2ba887c61b8aeef35888
Gerrit-Change-Number: 24581
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Ashwani Raina <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to