Page MenuHomePhabricator

Rename some seeder options
AbandonedPublic

Authored by nakihito on Oct 7 2019, 23:28.

Details

Reviewers
deadalnix
Fabien
jasonbcox
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Summary

The names for some of the long-form names of seeder options don't make
much sense or are not very descriptive. This change should make it a
little more clear what each option is.

Depends on D4223

Test Plan
make
./bitcoin-seeder -host=seeder.bitframe.org -nameserver=dev.bitframe.org -email=info@bitframe.org -testnet

Output should look something like:

Supporting whitelisted filters: 0x1,0x5,0x11,0x15
Using testnet.
Loading dnsseed.dat...done
Starting 4 DNS threads for  on ` (port 53).......done
Starting seeder...done
Starting 96 crawler threads...2019-10-07T23:26:28Z connect() to [2001:67c:130c:254::46]:8333 failed: Cannot assign requested address (99)
...

Diff Detail

Repository
rABC Bitcoin ABC
Branch
UpdateSeederOptions
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 7711
Build 13461: Bitcoin ABC Buildbot (legacy)
Build 13460: arc lint + arc unit

Event Timeline

Owners added a reviewer: Restricted Owners Package.Oct 7 2019, 23:28
nakihito edited the test plan for this revision. (Show Details)

Reverted changes to -proxyipvX.

nakihito requested review of this revision.Oct 8 2019, 00:08
deadalnix requested changes to this revision.Oct 8 2019, 15:20

Randomly changing parameters without updating the doc or even writing release note is indeed the shortest path toward success.

This revision now requires changes to proceed.Oct 8 2019, 15:20
jasonbcox requested changes to this revision.Oct 9 2019, 00:20

+1 to release notes

src/seeder/main.cpp
357 ↗(On Diff #13396)

s/mbox/email/g on this line would have caught the other use of mbox