HomePhabricator

[CMAKE] Fix missing protobuf include directory

Authored by Fabien on May 13 2019, 08:01.

Description

[CMAKE] Fix missing protobuf include directory

Summary:
The Protobuf include directory has been mistakenly removed in D3016.

This patch reverts the previous wrong change.

Full explanation:
The include directories are automatically added by CMake when using
target_link_libraries on IMPORTED target, but this is not the case
here. IMPORTED targets have been added for protobuf in later CMake
versions, past our minimum required one.

Test Plan:

mkdir buildcmake && cd buildcmake
cmake -GNinja ..
ninja
./src/qt/bitcoin-qt # Sanity check

Reviewers: #bitcoin_abc, deadalnix

Reviewed By: #bitcoin_abc, deadalnix

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

Details

Committed
FabienMay 16 2019, 07:26
Pushed
FabienMay 16 2019, 07:26
Reviewer
Restricted Project
Differential Revision
D3030: [CMAKE] Fix missing protobuf include directory
Parents
rABCc7f22f815621: Fix more init bugs.
Branches
Unknown
Tags
Unknown