Page MenuHomePhabricator

[CMAKE] Don't install secp256k1 by default unless it is standalone
ClosedPublic

Authored by Fabien on Aug 24 2020, 10:42.

Details

Summary

It is no longer installed via ninja install but it is still possible
to install it with ninja install-secp256k1 as needed.

Test Plan

On Bitcoin ABC project, check it is no longer installed:

ninja install

And install it via:

ninja install-secp256k1

On secp256k1 standalone library, check the lib is installed as
expected.:

ninja install

Diff Detail

Repository
rABC Bitcoin ABC
Branch
cmake_secp256k1_install_exclude_from_all
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 12453
Build 25067: Build Diffbuild-clang-10 · build-clang-tidy · build-diff · build-without-wallet
Build 25066: arc lint + arc unit