Updated seeder README and main.cpp help message to reflect the -m option requirement.
Details
Details
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Branch
- SeederHelp
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 5050 Build 8163: Bitcoin ABC Buildbot (legacy) Build 8162: arc lint + arc unit
Event Timeline
Comment Actions
It's also unclear that the change actually reflects accurately the parameters requirements as if fDNS is not set, then the email is not required, which clearly indicate that in some situations, it is indeed optional.
Comment Actions
Sorry. I should have been more precise in my test method. I have updated it to reflect the commands I actually used to test this. The second command without the -m option failed with the following error message:
Supporting whitelisted filters: 0x1,0x5,0x11,0x15 No e-mail address set. Please use -m.
The third line succeeded.