Paths

Table of Contentst

Differential D5761

[CMAKE] Use a cmake template for config.ini
ClosedPublic
Actions

Authored by Fabien on Apr 17 2020, 14:27.

Details

Reviewers

deadalnix

Group Reviewers

Restricted Project

Commits

rSTAGING6141580111ce: [CMAKE] Use a cmake template for config.ini
rABC6141580111ce: [CMAKE] Use a cmake template for config.ini

Summary

This simplifies a lot the generation of the config.ini file with
cmake.

Test Plan

cmake -GNinja .. -DCMAKE_TOOLCHAIN_FILE=../cmake/platforms/Win64.cmake

Check the extension is set to .exe

cmake -GNinja ..

Check all the options but the fuzzer and bitcoind are ON

cmake -GNinja .. -DBUILD_BITCOIN_CLI=OFF -DBUILD_BITCOIN_WALLET=OFF

-DBUILD_BITCOIN_ZMQ=OFF
Check all the options are OFF

With Clang:

cmake -GNinja .. -DENABLE_SANITIZERS=fuzzer

Check the fuzzer is ON

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Apr 17 2020, 14:27

Herald added a reviewer: Restricted Project. · View Herald TranscriptApr 17 2020, 14:27

deadalnix accepted this revision.Apr 17 2020, 14:38

This revision is now accepted and ready to land.Apr 17 2020, 14:38

Build Bitcoin-ABC / Diffs / Diff Testing (build-diff) started.

Harbormaster completed remote builds in B10275: Diff 18906.Apr 17 2020, 15:39

Build Bitcoin-ABC / Diffs / Diff Testing (build-diff) passed.

Closed by commit rABC6141580111ce: [CMAKE] Use a cmake template for config.ini (authored by Fabien). · Explain WhyApr 17 2020, 17:58

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC6141580111ce: [CMAKE] Use a cmake template for config.ini.

Fabien added a commit: rSTAGING6141580111ce: [CMAKE] Use a cmake template for config.ini.May 5 2020, 19:01

Revision Contents
Changeset List

Path

Size

test/

CMakeLists.txt

24 lines

config.ini.cmake.in

22 lines

Diff 18918

View Options

test/CMakeLists.txt