Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Bitcoin Core 23.0 and 24.0.1 #37

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

johnzweng
Copy link

  • Adds Bitcoin Core version 23.0
  • Adds Bitcoin Core version 24.0

Dockerfiles based on existing Bitcoin Core version 22.0.
Both images built and tested locally on a x86_64 machine.

@andrewtoth
Copy link

Bitcoin 24.0 was not announced and should not be released as a docker image. https://github.com/bitcoin/bitcoin/blob/master/doc/release-notes/release-notes-24.0.md
This should be updated to use Bitcoin 24.0.1.

@johnzweng
Copy link
Author

Sorry, you are totally right. I overlooked this on my side. Will change it to 24.0.1.

Signed-off-by: Johannes Zweng <johannes@zweng.at>
@johnzweng johnzweng force-pushed the add-bitcoin-core-v23-and-v24 branch from 6bc14fa to 453791e Compare December 20, 2022 06:27
@johnzweng
Copy link
Author

johnzweng commented Dec 20, 2022

Done. ✅

I changed 24.0 to 24.0.1 reverified signatures from SHA256SUMS.asc (but you should not trust me and do the same yourself), built it on my machine and ran it. Seems to work. 🙂

@johnzweng johnzweng changed the title Add Bitcoin core 23.0 and 24.0 Add Bitcoin core 23.0 and 24.0.1 Dec 20, 2022
@johnzweng johnzweng changed the title Add Bitcoin core 23.0 and 24.0.1 Add Bitcoin Core 23.0 and 24.0.1 Dec 20, 2022
CONFIG_PREFIX=$'regtest=1\n[regtest]'
elif [[ "${BITCOIN_NETWORK}" == "testnet" ]]; then
CONFIG_PREFIX=$'testnet=1\n[test]'
elif [[ "${BITCOIN_NETWORK}" == "mainnet" ]]; then

Choose a reason for hiding this comment

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

Starting bitcoind with -mainnet=1 errors with Invalid parameter -mainnet=1. Was this supposed to be signet?

Choose a reason for hiding this comment

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

Specifying it in the config file like this, it gives a warning:
Ignoring unknown configuration value mainnet

Copy link
Author

Choose a reason for hiding this comment

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

@andrewtoth thanks for your advice! You are totally right. I added the option for signet.

For mainnet I still add [main] to the bitcoin.conf (but don't add mainnet=1 anymore).

…ainnet

Signed-off-by: Johannes Zweng <johannes@zweng.at>
@johnzweng
Copy link
Author

Hi @NicolasDorier ! 🙂

Just was wondering if this repo is still active, but then discovered the repo under btcpayserver: https://github.com/btcpayserver/dockerfile-deps

I assume this is the new place where current Docker image builds live. I will target potential future PRs there.

Maybe you could add a notice to this repo which points to the new one. 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants