Skip to content

Conversation

@shangabl
Copy link
Contributor

Motivation

Modifications

Change summary

  • Upgrade to Boost 1.81
  • Had to trail and error some minor changes to tests and logging to compile successfully.

Testing

Built locally, ran unit tests, connected to tunnel.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

set(Boost_USE_DEBUG_RUNTIME OFF)
#set_property(GLOBAL PROPERTY Boost_USE_MULTITHREADED ON)
set(BOOST_PKG_VERSION "1.79.0" CACHE STRING "")
find_package(Boost ${BOOST_PKG_VERSION} REQUIRED COMPONENTS system log log_setup thread program_options date_time filesystem)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can use EXACT to enforce a specific version (default behavior is greater than or equal to specified version) : find_package(Boost ${BOOST_PKG_VERSION} EXACT REQUIRED COMPONENTS

Also, maybe add asio and http to the list of COMPONENTS, or do that later in another PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It turns out asio and the other components we discussed with Marco are header only and can't be included here or it breaks.

Do we want EXACT here? Ideally we'd be as backwards & forwards compatible as possible.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Previously we did not enforce an exact version, so when people tried building the local proxy with boost 1.81 while we were targeting 1.79, they got a build error instead of a cmake validation error

@shangabl shangabl merged commit f460fdf into main Feb 3, 2023
@shangabl shangabl deleted the boost-1.81 branch February 3, 2023 01:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants