Page MenuHomePhabricator

[cmake] link test runners by default
ClosedPublic

Authored by deadalnix on Dec 1 2019, 02:19.

Details

Summary

This is not an expensive operation, is properly tracked by dependencies and ensure the presence of these file is predictable.

Test Plan
rm test/functional/test_runner.py
ninja

Check test/functional/test_runner.py was linked.

Diff Detail

Repository
rABC Bitcoin ABC
Branch
cmakewoes
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 8364
Build 14747: Default Diff Build & Tests
Build 14746: arc lint + arc unit

Event Timeline

map the sanitization with cmake doc ( https://cmake.org/cmake/help/v3.10/policy/CMP0037.html ) and reduce verbosity.

Fabien added a subscriber: Fabien.

Thanks, this one was on my list :)

This revision is now accepted and ready to land.Dec 2 2019, 08:47
This revision was automatically updated to reflect the committed changes.