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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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 3478

View Options

cmake/modules/TestSuite.cmake

[cmake] Fix test run command lineClosedPublicActions