Move checkpoint data selection to chainparams #4801

pull jtimon wants to merge 2 commits into bitcoin:master from jtimon:chainparams changing 5 files +98 −74

jtimon commented at 8:09 pm on August 31, 2014: contributor

This way we avoid the calls to Params().NetworkID() in checkpoints.cpp. If at some point we’re going to read the chainparams from data files, we probably want to put the checkpoint data there as well
Separate data from checkpoints.cpp b260aade80
Move checkpoint data selection to chainparams 0781bde927
jtimon force-pushed on Aug 31, 2014
BitcoinPullTester commented at 9:01 pm on August 31, 2014: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/p4801_0781bde927f8cbf55693eb63ef4c6ac96886d49f/ for binaries and test log. This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.
jtimon commented at 10:28 am on September 2, 2014: contributor

Included in #4802, closing.
jtimon closed this on Sep 2, 2014
MarcoFalke locked this on Sep 8, 2021