Differential D1273

[cmake] Fix test run command line
ClosedPublic
Actions

Authored by deadalnix on Apr 10 2018, 21:04.

Details

Reviewers

schancel

Group Reviewers

Restricted Project

Commits

rSTAGING2f62da3f6848: [cmake] Fix test run command line
rABC2f62da3f6848: [cmake] Fix test run command line

Summary

Useless arguement are passed to test, which is ok most of the time as they ignore it, but sometime, it's not.

Test Plan

Change the command line for ctest to pass the -V flag.

ninja check

Ensure no superfluous flags are passed.

Diff Detail

Repository

rABC Bitcoin ABC

Branch

cmaketest

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 2287
Build 2712: Bitcoin ABC Buildbot (legacy)
Build 2711: arc lint + arc unit

Event Timeline

deadalnix created this revision.Apr 10 2018, 21:04

Herald added a reviewer: Restricted Project. · View Herald TranscriptApr 10 2018, 21:04

Build Bitcoin-ABC :: Staging :: Bitcoin-ABC Staging started.

Build Bitcoin-ABC :: Staging :: Bitcoin-ABC Staging passed.

Harbormaster completed remote builds in B2287: Diff 3468.Apr 10 2018, 21:28

schancel accepted this revision.Apr 11 2018, 14:16

This revision is now accepted and ready to land.Apr 11 2018, 14:16

Closed by commit rABC2f62da3f6848: [cmake] Fix test run command line (authored by deadalnix). · Explain WhyApr 11 2018, 23:07

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

cmake/

modules/

TestSuite.cmake

2 lines

Diff 3468

View Options

cmake/modules/TestSuite.cmake

[cmake] Fix test run command lineClosedPublicActions