HomePhabricator

[CMAKE] Fix missing inclusion when libbitcoinconsensus is disabled

Description

[CMAKE] Fix missing inclusion when libbitcoinconsensus is disabled

Summary:
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.

Test Plan:

cmake -GNinja .. -DBUILD_LIBBITCOINCONSENSUS=OFF
ninja install

Reviewers: #bitcoin_abc, deadalnix, jasonbcox

Reviewed By: #bitcoin_abc, jasonbcox

Differential Revision: https://reviews.bitcoinabc.org/D4341

Details

Provenance
FabienAuthored on Oct 31 2019, 16:59
nakihitoPushed on Oct 31 2019, 19:02
Reviewer
Restricted Project
Differential Revision
D4341: [CMAKE] Fix missing inclusion when libbitcoinconsensus is disabled
Parents
rSTAGINGb10b542d4863: Update qt/transactionview.cpp with Qt5 connect syntax
Branches
Unknown
Tags
Unknown
References
tag: phabricator/base/13849, tag: phabricator/base/13837