Skip to content

Conversation

@Fuzzbawls
Copy link
Collaborator

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.

@Fuzzbawls Fuzzbawls self-assigned this Dec 24, 2020
@Fuzzbawls Fuzzbawls added this to the Future milestone Dec 30, 2020
@Fuzzbawls Fuzzbawls changed the title [Build] Don't compile PoW mining RPC commands by default [WIP][Build] Don't compile PoW mining RPC commands by default Dec 30, 2020
@Fuzzbawls Fuzzbawls marked this pull request as draft December 30, 2020 22:44
@Fuzzbawls Fuzzbawls force-pushed the 2020_rpc-no-compile-mining-commands branch from 6fb62fd to 624fdff Compare January 17, 2021 07:31
@Fuzzbawls Fuzzbawls marked this pull request as ready for review January 17, 2021 07:32
@random-zebra random-zebra modified the milestones: Future, 5.1.0 Jan 17, 2021
@Fuzzbawls Fuzzbawls changed the title [WIP][Build] Don't compile PoW mining RPC commands by default [Build] Don't compile PoW mining RPC commands by default Jan 17, 2021
@Fuzzbawls Fuzzbawls modified the milestones: Future, 5.1.0 Jan 20, 2021
random-zebra
random-zebra previously approved these changes Jan 22, 2021
Copy link

@random-zebra random-zebra left a 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).

@random-zebra random-zebra added the Needs Release Notes Placeholder tag for anything needing mention in the "Notable Changes" section of release notes label Jan 22, 2021
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.
@Fuzzbawls Fuzzbawls force-pushed the 2020_rpc-no-compile-mining-commands branch from 624fdff to 1a5d903 Compare January 28, 2021 02:04
@Fuzzbawls Fuzzbawls removed the Needs Release Notes Placeholder tag for anything needing mention in the "Notable Changes" section of release notes label Jan 28, 2021
@Fuzzbawls
Copy link
Collaborator Author

rebased and added a note in the release notes template. Not going to add anything to the build-*.md files as this is an edge use case and configure --help provides all the documentation necessary.

Copy link

@random-zebra random-zebra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

re-utACK 1a5d903

Copy link

@furszy furszy left a 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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants