Skip to content

Commit 2aaf8db

Browse files
authored
Merge pull request ceph#51573 from adamemerson/wip-boost-the-boost
build: Bump boost to 1.82 Reviewed-by: Casey Bodley <[email protected]>
2 parents 9ddcf45 + 244c5eb commit 2aaf8db

File tree

7 files changed

+24
-19
lines changed

7 files changed

+24
-19
lines changed

CMakeLists.txt

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -661,7 +661,7 @@ if(WITH_SYSTEM_BOOST)
661661
if(BOOST_ROOT AND CMAKE_LIBRARY_ARCHITECTURE)
662662
set(BOOST_LIBRARYDIR "${BOOST_ROOT}/lib/${CMAKE_LIBRARY_ARCHITECTURE}")
663663
endif()
664-
find_package(Boost 1.79 COMPONENTS ${BOOST_COMPONENTS} REQUIRED)
664+
find_package(Boost 1.82 COMPONENTS ${BOOST_COMPONENTS} REQUIRED)
665665
if(NOT ENABLE_SHARED)
666666
set_property(TARGET Boost::iostreams APPEND PROPERTY
667667
INTERFACE_LINK_LIBRARIES ZLIB::ZLIB)
@@ -675,11 +675,16 @@ else()
675675
set(BOOST_J ${DEFAULT_BOOST_J} CACHE STRING "max jobs for Boost build") # override w/-DBOOST_J=<n>
676676
set(Boost_USE_STATIC_LIBS ON)
677677
include(BuildBoost)
678-
build_boost(1.79
678+
build_boost(1.82
679679
COMPONENTS ${BOOST_COMPONENTS} ${BOOST_HEADER_COMPONENTS})
680680
endif()
681681
include_directories(BEFORE SYSTEM ${Boost_INCLUDE_DIRS})
682682

683+
if(Boost_VERSION VERSION_EQUAL 1.81 OR Boost_VERSION VERSION_EQUAL 1.82)
684+
# This is a workaround for https://github.com/boostorg/phoenix/issues/111
685+
add_compile_options($<$<COMPILE_LANGUAGE:CXX>:-DBOOST_PHOENIX_STL_TUPLE_H_>)
686+
endif()
687+
683688
# dashboard angular2 frontend
684689
option(WITH_MGR_DASHBOARD_FRONTEND "Build the mgr/dashboard frontend using `npm`" ON)
685690
option(WITH_SYSTEM_NPM "Assume that dashboard build tools already installed through packages" OFF)

cmake/modules/BuildBoost.cmake

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -149,14 +149,14 @@ function(do_build_boost root_dir version)
149149
check_boost_version("${PROJECT_SOURCE_DIR}/src/boost" ${version})
150150
set(source_dir
151151
SOURCE_DIR "${PROJECT_SOURCE_DIR}/src/boost")
152-
elseif(version VERSION_GREATER 1.79)
152+
elseif(version VERSION_GREATER 1.82)
153153
message(FATAL_ERROR "Unknown BOOST_REQUESTED_VERSION: ${version}")
154154
else()
155155
message(STATUS "boost will be downloaded...")
156156
# NOTE: If you change this version number make sure the package is available
157157
# at the three URLs below (may involve uploading to download.ceph.com)
158-
set(boost_version 1.79.0)
159-
set(boost_sha256 475d589d51a7f8b3ba2ba4eda022b170e562ca3b760ee922c146b6c65856ef39)
158+
set(boost_version 1.82.0)
159+
set(boost_sha256 a6e1ab9b0860e6a2881dd7b21fe9f737a095e5f33a3a874afc6a345228597ee6)
160160
string(REPLACE "." "_" boost_version_underscore ${boost_version} )
161161
string(JOIN " " boost_url
162162
https://boostorg.jfrog.io/artifactory/main/release/${boost_version}/source/boost_${boost_version_underscore}.tar.bz2

cmake/modules/FindBoost.cmake

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1653,7 +1653,7 @@ else()
16531653
# _Boost_COMPONENT_HEADERS. See the instructions at the top of
16541654
# _Boost_COMPONENT_DEPENDENCIES.
16551655
set(_Boost_KNOWN_VERSIONS ${Boost_ADDITIONAL_VERSIONS}
1656-
"1.80.0" "1.80" "1.79.0" "1.79"
1656+
"1.82.0" "1.82" "1.81.0" "1.81" "1.80.0" "1.80" "1.79.0" "1.79"
16571657
"1.78.0" "1.78" "1.77.0" "1.77" "1.76.0" "1.76" "1.75.0" "1.75" "1.74.0" "1.74"
16581658
"1.73.0" "1.73" "1.72.0" "1.72" "1.71.0" "1.71" "1.70.0" "1.70" "1.69.0" "1.69"
16591659
"1.68.0" "1.68" "1.67.0" "1.67" "1.66.0" "1.66" "1.65.1" "1.65.0" "1.65"

install-deps.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -148,7 +148,7 @@ function install_pkg_on_ubuntu {
148148

149149
function install_boost_on_ubuntu {
150150
ci_debug "Running install_boost_on_ubuntu() in install-deps.sh"
151-
local ver=1.79
151+
local ver=1.82
152152
local installed_ver=$(apt -qq list --installed ceph-libboost*-dev 2>/dev/null |
153153
grep -e 'libboost[0-9].[0-9]\+-dev' |
154154
cut -d' ' -f2 |
@@ -163,7 +163,7 @@ function install_boost_on_ubuntu {
163163
fi
164164
local codename=$1
165165
local project=libboost
166-
local sha1=892ab89e76b91b505ffbf083f6fb7f2a666d4132
166+
local sha1=2804368f5b807ba8334b0ccfeb8af191edeb996f
167167
install_pkg_on_ubuntu \
168168
$project \
169169
$sha1 \

make-dist

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -191,8 +191,8 @@ ln -s . $outfile
191191
tar cvf $outfile.version.tar $outfile/src/.git_version $outfile/ceph.spec
192192
# NOTE: If you change this version number make sure the package is available
193193
# at the three URLs referenced below (may involve uploading to download.ceph.com)
194-
boost_version=1.79.0
195-
download_boost $boost_version 475d589d51a7f8b3ba2ba4eda022b170e562ca3b760ee922c146b6c65856ef39 \
194+
boost_version=1.82.0
195+
download_boost $boost_version a6e1ab9b0860e6a2881dd7b21fe9f737a095e5f33a3a874afc6a345228597ee6 \
196196
https://boostorg.jfrog.io/artifactory/main/release/$boost_version/source \
197197
https://download.ceph.com/qa
198198
download_liburing 0.7 8e2842cfe947f3a443af301bdd6d034455536c38a455c7a700d0c1ad165a7543 \

src/rgw/rgw_asio_client.cc

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -39,11 +39,11 @@ int ClientIO::init_env(CephContext *cct)
3939
const auto& value = header->value();
4040

4141
if (field == beast::http::field::content_length) {
42-
env.set("CONTENT_LENGTH", value.to_string());
42+
env.set("CONTENT_LENGTH", std::string(value));
4343
continue;
4444
}
4545
if (field == beast::http::field::content_type) {
46-
env.set("CONTENT_TYPE", value.to_string());
46+
env.set("CONTENT_TYPE", std::string(value));
4747
continue;
4848
}
4949

@@ -62,26 +62,26 @@ int ClientIO::init_env(CephContext *cct)
6262
}
6363
*dest = '\0';
6464

65-
env.set(buf, value.to_string());
65+
env.set(buf, std::string(value));
6666
}
6767

6868
int major = request.version() / 10;
6969
int minor = request.version() % 10;
7070
env.set("HTTP_VERSION", std::to_string(major) + '.' + std::to_string(minor));
7171

72-
env.set("REQUEST_METHOD", request.method_string().to_string());
72+
env.set("REQUEST_METHOD", std::string(request.method_string()));
7373

7474
// split uri from query
7575
auto uri = request.target();
7676
auto pos = uri.find('?');
7777
if (pos != uri.npos) {
7878
auto query = uri.substr(pos + 1);
79-
env.set("QUERY_STRING", query.to_string());
79+
env.set("QUERY_STRING", std::string(query));
8080
uri = uri.substr(0, pos);
8181
}
82-
env.set("SCRIPT_URI", uri.to_string());
82+
env.set("SCRIPT_URI", std::string(uri));
8383

84-
env.set("REQUEST_URI", request.target().to_string());
84+
env.set("REQUEST_URI", std::string(request.target()));
8585

8686
char port_buf[16];
8787
snprintf(port_buf, sizeof(port_buf), "%d", local_endpoint.port());

win32_deps_build.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,8 @@ sslDir="${depsToolsetDir}/openssl"
2020
sslSrcDir="${depsSrcDir}/openssl"
2121

2222
# For now, we'll keep the version number within the file path when not using git.
23-
boostUrl="https://boostorg.jfrog.io/artifactory/main/release/1.79.0/source/boost_1_79_0.tar.gz"
24-
boostSrcDir="${depsSrcDir}/boost_1_79_0"
23+
boostUrl="https://boostorg.jfrog.io/artifactory/main/release/1.82.0/source/boost_1_82_0.tar.gz"
24+
boostSrcDir="${depsSrcDir}/boost_1_82_0"
2525
boostDir="${depsToolsetDir}/boost"
2626
zlibDir="${depsToolsetDir}/zlib"
2727
zlibSrcDir="${depsSrcDir}/zlib"

0 commit comments

Comments
 (0)