Quote from: jtoomim on February 01, 2018, 09:43:19 PM

Quote from: sawa on February 01, 2018, 07:11:13 PM

Those who wish can use the pruned blockchain (configuration examples for demons with a pruned blockchain in README.md)

The check_block_header function (which uses the getblockheader RPC method) is already compatible with pruned blockchains. It's only the getblock RPC that fails on pruned blockchains. The commit you made to change the initialization routine will allow people to use the wrong daemon at startup (e.g. Bitcoin ABC on the regular BTC network) and is dangerous.

For reference, I added support for pruned mode in this commit, with the fork protection and some renaming done here.

I did a more thorough rpc check https://github.com/ilsawa/p2pool-jtoomim/commit/e7d0a252c2401dabc63f29adfd4dbaf9a2f0efc0

You have it will not work with new wallets https://github.com/jtoomim/p2pool/blob/1mb_segwit/p2pool/bitcoin/networks/bitcoin.py#L18

Code:

not (yield bitcoind.rpc_getinfo ()) ['testnet']

I'm interested to hear what you think about this commit: https://github.com/ilsawa/p2pool-jtoomim/commit/342a7bbb9e3a13a2583722811751c5819c033fb3

And please add this commit to your repository:
https://github.com/ilsawa/p2pool-jtoomim/commit/e43f9e295d1c398b056db40bb1cf5736cab44fcd