Differential D6960

[CMAKE] Optionally install bitcoin-bench
ClosedPublic
Actions

Authored by Fabien on Jul 17 2020, 09:30.

Details

Reviewers

deadalnix

Group Reviewers

Restricted Project

Commits

rABC458e260c6dd4: [CMAKE] Optionally install bitcoin-bench

Summary

As per title.

Test Plan

Won't install bitcoin-bench:

ninja install

Will install bitcoin-bench:

cmake -DCOMPONENT=bitcoin-bench -P cmake_install.cmake

Diff Detail

Repository

rABC Bitcoin ABC

Branch

install_bench

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 12039
Build 23956: Default Diff Build & Tests	build-diff
Build 23955: Build without Wallet	build-without-wallet
Build 23954: Build with clang-10	build-clang-10
Build 23953: arc lint + arc unit

Event Timeline

Fabien created this revision.Jul 17 2020, 09:30

Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 17 2020, 09:30

Fabien requested review of this revision.Jul 17 2020, 09:30

Harbormaster completed remote builds in B12039: Diff 22327.Jul 17 2020, 09:41

deadalnix accepted this revision.Jul 17 2020, 12:01

This revision is now accepted and ready to land.Jul 17 2020, 12:01

Closed by commit rABC458e260c6dd4: [CMAKE] Optionally install bitcoin-bench (authored by Fabien). · Explain WhyJul 17 2020, 12:32

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC458e260c6dd4: [CMAKE] Optionally install bitcoin-bench.

Revision Contents
Changeset List

Path

Size

cmake/

modules/

InstallationHelper.cmake

4 lines

src/

bench/

CMakeLists.txt

3 lines

Diff 22327

View Options

cmake/modules/InstallationHelper.cmake