Skip to content

Conversation

fanquake
Copy link
Member

hebasto and others added 2 commits July 14, 2025 13:36
When using CMake policies 3.14 and below, the `export(PACKAGE)` command
by default populates the user package registry, which is stored outside
the build tree. Setting the `CMAKE_EXPORT_NO_PACKAGE_REGISTRY` variable
disables this side effect.

In CMake 3.15 and later, this behavior is disabled by default, and the
variable has no effect.

Github-Pull: bitcoin#32943
Rebased-From: 44f3bae
@fanquake fanquake added this to the 28.3 milestone Jul 14, 2025
@DrahtBot
Copy link
Contributor

DrahtBot commented Jul 14, 2025

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Code Coverage & Benchmarks

For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32969.

Reviews

See the guideline for information on the review process.

Type Reviewers
ACK marcofleon

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

@fanquake fanquake marked this pull request as ready for review July 23, 2025 13:35
@fanquake fanquake requested a review from marcofleon July 23, 2025 13:47
Copy link
Contributor

@marcofleon marcofleon left a comment

Choose a reason for hiding this comment

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

ACK a9a71b8

Built and ran tests on 28.x branch locally. Skipped --legacy-wallet tests so lgtm 😌

@fanquake fanquake merged commit a6d0159 into bitcoin:28.x Jul 23, 2025
15 of 16 checks passed
@fanquake fanquake deleted the more_more_28_backports branch July 23, 2025 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants