raspiblitz icon indicating copy to clipboard operation
raspiblitz copied to clipboard

add knots option

Open Retropex opened this issue 9 months ago • 3 comments

  • [x] That its based against the dev branch - not the default release branch.

This is not ready to merge, its to start receiving advice on what should I add/remove.

Close #4678

Retropex avatar Mar 10 '25 14:03 Retropex

Hi thanks for the contribution. I am open to have alternative implementations to Bitcoin Core with RaspiBlitz as long as they are Interface compliant and dont need any changes on the rest of the code - as mentioned in #4678

So as far I can see @Retropex you are choosing to install Knots as an additional app - we now just have to look into all the details so that it can act as an in-place replacement for Bitcoin Core when activated. Its a bit like hotswapping an engine of a car, so there are some more things to care about.

On the top of my head we would need to check for the following:

  • Does it use the same config file as Bitcoin Core ... because the user might altready made lots auf personal changes there he wants to see replected also with Bitcoin Knots?
  • Does it take all the blockchain data, indexes, etc already build with Core or is there some reset/rebuilds needed?
  • Does this all works also if users witch back to Bitcoin Core? Are additional Knots config values just get ignored by Core or does it need cleaning?

rootzoll avatar Mar 12 '25 11:03 rootzoll

The answer to the two first questions is yes, in fact knots have the exact same binary bitcoind so once retrieved and extracted my script stop bitcoin core delete the binary on put the of knots instead. so there is not a single difference for the others apps.

Regarding the specific settings of knots I don't know how core handle it, let me check.

Retropex avatar Mar 12 '25 12:03 Retropex

Ok just checked and core ignore knots settings 👍

Retropex avatar Mar 12 '25 12:03 Retropex

Suggestion: Perhaps add an echo section or example_bitcoin.conf with the additional Knots-specific options.

I've done something similar for nix-bitcoin https://github.com/steepdawn974/nix-bitcoin/commit/3f369f934eaf1bf62db9bbc9701d1ca264214429#diff-fdb5fc87f6d3166e0005897fbbf92421aff1fc346287914dcf528f779a6b8b2dR289

steepdawn974 avatar Apr 30 '25 09:04 steepdawn974

Based on ongoing discussions around https://github.com/bitcoin/bitcoin/pull/32359, this one is needed more than ever! cc: @rootzoll, @Retropex

xcarlitos avatar Apr 30 '25 20:04 xcarlitos

Please implement this ASAP given how the Bitcoin Core dev team have become bad actors lately that want Bitcoin to act like SCoins

satwise avatar May 03 '25 22:05 satwise

Hope this can be the top priority and implement as soon as possible. I've tried to do it manually, it works, but somehow making some issues on the LND. I'm still beginner at this, but I want to move to knots sooner with clean setup. Thanks for your contribution @Retropex @rootzoll . Can't wait for this release.

medigital-id avatar May 04 '25 23:05 medigital-id

Merging this PR for (internal) release candidate testing .. created issue #4996 with tasks still on my todo list regarding this feature.

rootzoll avatar May 14 '25 00:05 rootzoll