D4229 introduced the BUILD_LIBBITCOINCONSENSUS switch, and moved the
include(InstallationHelper) directive conditionally. This will prevent
installing other targets when BUILD_LIBBITCOINCONSENSUS is disabled.
This diff fixes the issue by moving the directive to avoid conditional
inclusion.
Details
Details
- Reviewers
deadalnix jasonbcox - Group Reviewers
Restricted Project - Commits
- rSTAGINGfc49fe70e9bf: [CMAKE] Fix missing inclusion when libbitcoinconsensus is disabled
rABCfc49fe70e9bf: [CMAKE] Fix missing inclusion when libbitcoinconsensus is disabled
cmake -GNinja .. -DBUILD_LIBBITCOINCONSENSUS=OFF ninja install
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Branch
- cmake_fix_include
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 7937 Build 13877: Bitcoin ABC Buildbot (legacy) Build 13876: arc lint + arc unit