Page MenuHomePhabricator
Differential D809

Build the seeder using cmake/ninja
ClosedPublic
Actions

Authored by deadalnix on Dec 15 2017, 01:45.

Details

Reviewers
schancel
Group Reviewers
Restricted Project
Commits
rSTAGINGd5f371e548ee: Build the seeder using cmake/ninja
rABCd5f371e548ee: Build the seeder using cmake/ninja
Summary

As per title. Do not work on windows as the seeder's code is not protable.

Test Plan
cmake -GNinja ..
ninja

Then run the seeder and checks it works.

Diff Detail

Repository
rABC Bitcoin ABC
Branch
cmakeseeder
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 1377
Build 1377: arc lint + arc unit

Event Timeline

deadalnix created this revision.Dec 15 2017, 01:45
deadalnix updated this revision to Diff 2138.Dec 15 2017, 14:17

Actually use the BUILD_SEEDER option to build the seeder or not.

Harbormaster completed remote builds in B1382: Diff 2138.Dec 15 2017, 14:17
deadalnix updated this revision to Diff 2140.Dec 15 2017, 16:57

Better option name

Harbormaster completed remote builds in B1384: Diff 2140.Dec 15 2017, 16:57
schancel accepted this revision.Dec 15 2017, 18:59
This revision is now accepted and ready to land.Dec 15 2017, 18:59
Closed by commit rABCd5f371e548ee: Build the seeder using cmake/ninja (authored by deadalnix). · Explain WhyDec 16 2017, 00:05
This revision was automatically updated to reflect the committed changes.
Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 16 2017, 00:05

Revision Contents
Changeset List

PathSize
src/
9 lines
crypto/
6 lines
seeder/
14 lines

Diff 2132

View Options

src/CMakeLists.txt

Loading...
View Options

src/crypto/CMakeLists.txt

Loading...
View Options

src/seeder/CMakeLists.txt

Loading...