Skip to content

cmake: Add BOOST_TEST_HEADERS_ONLY configuration variable #445

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Apr 17, 2025

Conversation

hebasto
Copy link
Contributor

@hebasto hebasto commented Apr 16, 2025

This PR allows the build to be configured to install only the Boost.Test headers required for using the headers-only variant of the Unit Test Framework.

The README.md file at https://github.com/boostorg/cmake should also be updated to mention this new Boost.Test-specific configuration variable.

This change allows the build to be configured to install only the
Boost.Test headers required for using the headers-only variant of the
Unit Test Framework.

The `README.md` file at https://github.com/boostorg/cmake should also be
updated to mention this new Boost.Test-specific configuration variable.

Co-authored-by: Cory Fields <cory-nospam-@coryfields.com>
@mborland
Copy link
Member

Hi, @hebasto. This seems like a reasonable change to me. I have #446 cycling to fix up the CI system since 20.04 is being removed from Github Actions. Once the CI system is back to green, I'll pull it into this branch for you.

* Move ubuntu 20.04 runs into containers

* Fix syntax error
@mborland mborland merged commit 097e978 into boostorg:develop Apr 17, 2025
40 checks passed
hebasto added a commit to hebasto/boost.cmake that referenced this pull request Apr 17, 2025
@hebasto hebasto deleted the 250416-headers-only branch April 17, 2025 13:09
@hebasto
Copy link
Contributor Author

hebasto commented Apr 17, 2025

The README.md file at https://github.com/boostorg/cmake should also be updated to mention this new Boost.Test-specific configuration variable.

Done in boostorg/cmake#82.

pdimov added a commit to boostorg/cmake that referenced this pull request Apr 17, 2025
fanquake added a commit to bitcoin/bitcoin that referenced this pull request Jun 26, 2025
…stem

6c2538d depends: Bump boost to 1.88.0 and use new CMake buildsystem (Cory Fields)

Pull request description:

  Originally #30434.

  This has a few advantages over the old method of simply copying headers:
  - Installs proper CMake files which can be picked up by our buildsystem
  - Only installs necessary headers, not all of Boost

  Pulls in upstreamed boostorg/test#445.

ACKs for top commit:
  willcl-ark:
    tACK 6c2538d
  hebasto:
    re-ACK 6c2538d, only rebased since my previous [review](#32665 (review)).

Tree-SHA512: fc3fce77b21c8ea500370841f44f1cc87e0bb09cdde55f75d2f90853cb06a6f8c73ac6ca9ca3e91a879e9f95dd59aa40254c1b04e7a168c52fa1b31cc5b7f537
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.

2 participants