-
Notifications
You must be signed in to change notification settings - Fork 725
[Build] Don't compile PoW mining RPC commands by default #2105
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Build] Don't compile PoW mining RPC commands by default #2105
Conversation
6fb62fd to
624fdff
Compare
random-zebra
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK 624fdff09d7da6138d32b98fe4c32d15728aa157.
Maybe it's worth to document this new configure flag --enable-mining-rpc in the build-* notes, or somewhere else (at least in the release notes).
This adds a configure flag to skip compiling the PoW-related mining RPC commands, which aren't ever used outside of the initial PoW phase. The `generate` command is retained for regtest compatibility.
624fdff to
1a5d903
Compare
|
rebased and added a note in the release notes template. Not going to add anything to the |
random-zebra
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
re-utACK 1a5d903
furszy
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK 1a5d903 and merging
This adds a configure flag to skip compiling the PoW-related mining RPC
commands, which aren't ever used outside of the initial PoW phase.
The
generatecommand is retained for regtest compatibility.