[cmake] Use list append when adding path to CMAKE_MODULE_PATH
ClosedPublic
Actions

Authored by deadalnix on Jan 7 2020, 16:17.

Details

Reviewers

Group Reviewers

Restricted Project

Commits

rSTAGING5045d3bdcf8c: [cmake] Use list append when adding path to CMAKE_MODULE_PATH
rABC5045d3bdcf8c: [cmake] Use list append when adding path to CMAKE_MODULE_PATH

Summary

Using the right tools for the job.

Test Plan

cmake -GNinja ..
ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Branch

listappend

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 8765
Build 15512: Default Diff Build & Tests
Build 15511: arc lint + arc unit

Event Timeline

deadalnix created this revision.Jan 7 2020, 16:17

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 7 2020, 16:17

Build Bitcoin-ABC / Diffs / Diff Testing started.

deadalnix mentioned this in D4844: [CMAKE] Allow building secp256k1 as a standalone project.Jan 7 2020, 16:20

Harbormaster completed remote builds in B8765: Diff 15197.Jan 7 2020, 16:24

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Fabien accepted this revision.Jan 7 2020, 16:25

This revision is now accepted and ready to land.Jan 7 2020, 16:25

Closed by commit rABC5045d3bdcf8c: [cmake] Use list append when adding path to CMAKE_MODULE_PATH (authored by deadalnix). · Explain WhyJan 7 2020, 16:30

This revision was automatically updated to reflect the committed changes.

deadalnix added a commit: rABC5045d3bdcf8c: [cmake] Use list append when adding path to CMAKE_MODULE_PATH.

deadalnix added a commit: rSTAGING5045d3bdcf8c: [cmake] Use list append when adding path to CMAKE_MODULE_PATH.Apr 7 2020, 22:18

Revision Contents
Changeset List

Path

Size

CMakeLists.txt

5 lines

Diff 15197

View Options

[cmake] Use list append when adding path to CMAKE_MODULE_PATHClosedPublicActions