Skip to content

Conversation

@ksykulev
Copy link
Contributor

@ksykulev ksykulev commented Jul 7, 2025

#8593

  • Read the CONTRIBUTING.md guide at the root of the repo.
  • Ensure the code is formatted building the format_check target.
    If it is not, then move the committed files to the git staging area,
    build the format target to format them, and then re-commit.
    More information is available on the wiki.
  • Ensure your PR contains a single logical change.
  • Ensure your PR contains tests for the changes you're submitting.
  • Describe your changes with as much detail as you can.
  • Link any issues this PR is related to.
  • Remove the text above.

thirdparty_boost_scope_exit
thirdparty_boost_circular_buffer
thirdparty_boost_describe
thirdparty_boost_scope
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Linking errors due to missing headers.

)

# Add all boost library include paths for cross-library dependencies
file(GLOB boost_lib_dirs "${BOOST_ROOT}/libs/*/include")
Copy link
Contributor Author

@ksykulev ksykulev Jul 7, 2025

Choose a reason for hiding this comment

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

boost 1.88.0 uses b2 to create the unified headers, getting around this by including everything.

else()
set_property(SOURCE "${library_root}/src/fcontext.cpp" PROPERTY COMPILE_FLAGS "-std=c++17")
endif()
endif()
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not finding the standard library? 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I wonder if this is fixed by #8640
@lucasmrod

@ksykulev ksykulev force-pushed the upgrade-boost branch 11 times, most recently from 25dbe6a to 59be982 Compare July 8, 2025 00:48
@ksykulev ksykulev force-pushed the upgrade-boost branch 2 times, most recently from 52ebebc to eb5880c Compare July 8, 2025 11:46
@ksykulev
Copy link
Contributor Author

Blocked by work defined in this issue: fleetdm/fleet#34108

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.

1 participant