Skip to content

Conversation

Sjors
Copy link
Member

@Sjors Sjors commented Sep 23, 2023

A small step towards dropping checkpoints entirely, make the option to disable them a bit easier to find.

bitcoind --help
...
  -checkpoints
       Enable rejection of any forks from the known historical chain until
       block 295000 (default: 1)

Before this PR this required bitcoind --help-debug.

A potential next step could be to change the default to 0 in a future release, then drop the checkpoints in the next release (#28524) and then drop all the checkpoint code one release later.

@DrahtBot
Copy link
Contributor

DrahtBot commented Sep 23, 2023

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Code Coverage

For detailed information about the code coverage, see the test coverage report.

Reviews

See the guideline for information on the review process.
A summary of reviews will appear here.

@fanquake
Copy link
Member

make the option to disable them a bit easier to find.

Not sure. If we aren't completely convinced about actually dropping them yet, I'm not sure that making the option more apparent to users is the right thing to do.

@Sjors Sjors marked this pull request as draft September 27, 2023 15:16
@Sjors
Copy link
Member Author

Sjors commented Sep 27, 2023

That makes sense. Marking this as draft until we feel confident enough to advertise this option. I've been running it for half a year :-)

@Sjors Sjors force-pushed the 2023/09/look-mom-no-checkpoints branch from 2c56ae4 to ffc604e Compare February 13, 2024 14:15
@fanquake
Copy link
Member

fanquake commented Mar 6, 2024

Closing for now. This is a simple change, but doesn't make sense yet, and can easily be reopened / redone once there is a clear path forward for #25725.

@fanquake fanquake closed this Mar 6, 2024
@bitcoin bitcoin locked and limited conversation to collaborators Mar 6, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants