🚀 fanquake merged a pull request: "depends: systemtap 5.3"
(https://github.com/bitcoin/bitcoin/pull/33373)
(https://github.com/bitcoin/bitcoin/pull/33373)
💬 pinheadmz commented on pull request "Remove unnecessary casts when calling socket operations":
(https://github.com/bitcoin/bitcoin/pull/33378#discussion_r2351871877)
right thanks!
(https://github.com/bitcoin/bitcoin/pull/33378#discussion_r2351871877)
right thanks!
💬 hodlinator commented on pull request "log: always print initial signature verification state":
(https://github.com/bitcoin/bitcoin/pull/33336#discussion_r2351892330)
Found a way to test 2):
```python
# nodes[4]
self.start_node(4, extra_args=["-assumevalid=" + block102.hash_hex])
second_chain_tip = int(self.nodes[4].getbestblockhash(), 16)
second_chain_time = self.nodes[4].getblock(self.nodes[4].getbestblockhash())['time'] + 1
second_chain_height = self.nodes[4].getblock(self.nodes[4].getbestblockhash())['height'] + 1
second_chain = []
for _ in range(2100):
block = create_block(sec
...
(https://github.com/bitcoin/bitcoin/pull/33336#discussion_r2351892330)
Found a way to test 2):
```python
# nodes[4]
self.start_node(4, extra_args=["-assumevalid=" + block102.hash_hex])
second_chain_tip = int(self.nodes[4].getbestblockhash(), 16)
second_chain_time = self.nodes[4].getblock(self.nodes[4].getbestblockhash())['time'] + 1
second_chain_height = self.nodes[4].getblock(self.nodes[4].getbestblockhash())['height'] + 1
second_chain = []
for _ in range(2100):
block = create_block(sec
...
📝 mstampfer opened a pull request: "contrib: Add zsh completion scripts"
(https://github.com/bitcoin/bitcoin/pull/33402)
Adds zsh completion support using the compdef system for:
- bitcoin-cli with RPC command completion and context-sensitive arguments
- bitcoin-tx with option/command completion and file path handling
- bitcoind with configuration option completion
These complement the existing bash completion scripts and provide
the same functionality for zsh users.
(https://github.com/bitcoin/bitcoin/pull/33402)
Adds zsh completion support using the compdef system for:
- bitcoin-cli with RPC command completion and context-sensitive arguments
- bitcoin-tx with option/command completion and file path handling
- bitcoind with configuration option completion
These complement the existing bash completion scripts and provide
the same functionality for zsh users.
💬 mstampfer commented on pull request "contrib: Add zsh completion scripts":
(https://github.com/bitcoin/bitcoin/pull/33402#issuecomment-3297709907)
This PR adds zsh completion scripts for Bitcoin Core tools, converting the existing bash completions to work with zsh's compdef system. This Improves user experience for zsh users by addressing the need for tab completion support for zsh which doesn't rely on the existing bash completion scripts.
The implementation maintains the same functionality as the bash versions while using zsh-native completion functions.
I'm happy to address any concerns or make requested changes.
(https://github.com/bitcoin/bitcoin/pull/33402#issuecomment-3297709907)
This PR adds zsh completion scripts for Bitcoin Core tools, converting the existing bash completions to work with zsh's compdef system. This Improves user experience for zsh users by addressing the need for tab completion support for zsh which doesn't rely on the existing bash completion scripts.
The implementation maintains the same functionality as the bash versions while using zsh-native completion functions.
I'm happy to address any concerns or make requested changes.
🤔 marcofleon reviewed a pull request: "test: Add submitblock test in interface_ipc"
(https://github.com/bitcoin/bitcoin/pull/33380#pullrequestreview-3229240434)
code review ACK 0a26731c4cc182e887ce959cdd301227cdc752d7
(https://github.com/bitcoin/bitcoin/pull/33380#pullrequestreview-3229240434)
code review ACK 0a26731c4cc182e887ce959cdd301227cdc752d7
✅ fanquake closed a pull request: "util/strencodings: guard SAFE_CHARS index and pre-reserve result"
(https://github.com/bitcoin/bitcoin/pull/33396)
(https://github.com/bitcoin/bitcoin/pull/33396)
💬 fanquake commented on pull request "util/strencodings: guard SAFE_CHARS index and pre-reserve result":
(https://github.com/bitcoin/bitcoin/pull/33396#issuecomment-3297809297)
Thanks, however we can leave this code as-is.
(https://github.com/bitcoin/bitcoin/pull/33396#issuecomment-3297809297)
Thanks, however we can leave this code as-is.
✅ fanquake closed a pull request: "refactor: Fix typo and correct template parameter inconsistency"
(https://github.com/bitcoin/bitcoin/pull/33394)
(https://github.com/bitcoin/bitcoin/pull/33394)
💬 fanquake commented on pull request "refactor: Fix typo and correct template parameter inconsistency":
(https://github.com/bitcoin/bitcoin/pull/33394#issuecomment-3297824742)
Thanks, however I think this code can be left as-is for now.
(https://github.com/bitcoin/bitcoin/pull/33394#issuecomment-3297824742)
Thanks, however I think this code can be left as-is for now.
💬 TheCharlatan commented on pull request "log: always print initial signature verification state":
(https://github.com/bitcoin/bitcoin/pull/33336#discussion_r2352051546)
Are the enums really only added for logging? That seems a bit heavy. How about moving this `if` block up and making it a lambda that gets called directly from your `if else` branches? afaict that could remove the need for them, which judged by the comments left here so far, seem to confuse people a bit with their communicated intent.
(https://github.com/bitcoin/bitcoin/pull/33336#discussion_r2352051546)
Are the enums really only added for logging? That seems a bit heavy. How about moving this `if` block up and making it a lambda that gets called directly from your `if else` branches? afaict that could remove the need for them, which judged by the comments left here so far, seem to confuse people a bit with their communicated intent.
💬 TheCharlatan commented on pull request "log: always print initial signature verification state":
(https://github.com/bitcoin/bitcoin/pull/33336#discussion_r2351814133)
I'm a bit confused what the condition in this test is supposed to trigger if all we are testing are invalid args. What is the rationale here?
(https://github.com/bitcoin/bitcoin/pull/33336#discussion_r2351814133)
I'm a bit confused what the condition in this test is supposed to trigger if all we are testing are invalid args. What is the rationale here?
👍 hodlinator approved a pull request: "Remove unnecessary casts when calling socket operations"
(https://github.com/bitcoin/bitcoin/pull/33378#pullrequestreview-3229478311)
ACK 67f632b6deb8b4aa190c458b71d2bc8c793626d5
(PR description still ends with "Opening as a draft for now to test CI on supported platforms.")
(https://github.com/bitcoin/bitcoin/pull/33378#pullrequestreview-3229478311)
ACK 67f632b6deb8b4aa190c458b71d2bc8c793626d5
(PR description still ends with "Opening as a draft for now to test CI on supported platforms.")
📝 willcl-ark opened a pull request: "WIP: Backport Cirrus runners to 29.x"
(https://github.com/bitcoin/bitcoin/pull/33403)
Backports #32989 to the 29.x branch
(https://github.com/bitcoin/bitcoin/pull/33403)
Backports #32989 to the 29.x branch
💬 hodlinator commented on pull request "coins: warn on oversized `-dbcache`":
(https://github.com/bitcoin/bitcoin/pull/33333#discussion_r2352176441)
Yeah, now that you use `InitWarning()` adjacently, it's clearer why we would expect true.
(https://github.com/bitcoin/bitcoin/pull/33333#discussion_r2352176441)
Yeah, now that you use `InitWarning()` adjacently, it's clearer why we would expect true.
👍 hodlinator approved a pull request: "coins: warn on oversized `-dbcache`"
(https://github.com/bitcoin/bitcoin/pull/33333#pullrequestreview-3229526417)
ACK 13fa0d0a433722f294854001b0561c079db12dbc
Very helpful to warn users against unintentionally making their system start swapping insanely.
(https://github.com/bitcoin/bitcoin/pull/33333#pullrequestreview-3229526417)
ACK 13fa0d0a433722f294854001b0561c079db12dbc
Very helpful to warn users against unintentionally making their system start swapping insanely.
💬 fanquake commented on issue "Default ASmap file path is not used unless -asmap is set":
(https://github.com/bitcoin/bitcoin/issues/33386#issuecomment-3298250998)
@fjahr
(https://github.com/bitcoin/bitcoin/issues/33386#issuecomment-3298250998)
@fjahr
⚠️ mstampfer opened an issue: "Add zsh completion scripts for Bitcoin Core command-line tools"
(https://github.com/bitcoin/bitcoin/issues/33404)
### Please describe the feature you'd like to see added.
Bitcoin Core currently provides bash completion scripts for `bitcoin-cli`, `bitcoin-tx`, and `bitcoind` in `contrib/completions/bash/`, and similarly, fish completion scripts are available in `contrib/completions/fish/`, but there are no equivalent completion scripts for zsh users. Many developers and users have adopted zsh as their default shell (especially Mac OSX users and with tools like Oh My Zsh), and they cannot benefit from tab co
...
(https://github.com/bitcoin/bitcoin/issues/33404)
### Please describe the feature you'd like to see added.
Bitcoin Core currently provides bash completion scripts for `bitcoin-cli`, `bitcoin-tx`, and `bitcoind` in `contrib/completions/bash/`, and similarly, fish completion scripts are available in `contrib/completions/fish/`, but there are no equivalent completion scripts for zsh users. Many developers and users have adopted zsh as their default shell (especially Mac OSX users and with tools like Oh My Zsh), and they cannot benefit from tab co
...
⚠️ deyvid61 opened an issue: "Gj"
(https://github.com/bitcoin/bitcoin/issues/33405)
../ test.rc1/ 28-Jul-2025 19:01 - test.rc2/ 28-Aug-2025 17:38 - SHA256SUMS 04-Sep-2025 17:36 2956 SHA256SUMS.asc 04-Sep-2025 17:36 9968 SHA256SUMS.ots 04-Sep-2025 17:37 422 bitcoin-29.1-aarch64-linux-gnu-debug.tar.gz 04-Sep-2025 17:36 464841495 bitcoin-29.1-aarch64-linux-gnu.tar.gz 04-Sep-2025 17:36 47955246 bitcoin-29.1-arm-linux-gnueabihf-debug.tar.gz 04-Sep-2025 17:36 467016379 bitcoin-29.1-arm-linux-gnueabihf.tar.gz 04-Sep-2025 17:36 44510860 bitcoin-29.1-arm64-apple-darwin-codesigning.tar.
...
(https://github.com/bitcoin/bitcoin/issues/33405)
../ test.rc1/ 28-Jul-2025 19:01 - test.rc2/ 28-Aug-2025 17:38 - SHA256SUMS 04-Sep-2025 17:36 2956 SHA256SUMS.asc 04-Sep-2025 17:36 9968 SHA256SUMS.ots 04-Sep-2025 17:37 422 bitcoin-29.1-aarch64-linux-gnu-debug.tar.gz 04-Sep-2025 17:36 464841495 bitcoin-29.1-aarch64-linux-gnu.tar.gz 04-Sep-2025 17:36 47955246 bitcoin-29.1-arm-linux-gnueabihf-debug.tar.gz 04-Sep-2025 17:36 467016379 bitcoin-29.1-arm-linux-gnueabihf.tar.gz 04-Sep-2025 17:36 44510860 bitcoin-29.1-arm64-apple-darwin-codesigning.tar.
...
💬 deyvid61 commented on issue "Gj":
(https://github.com/bitcoin/bitcoin/issues/33405#issuecomment-3298347665)
Deyvid61
(https://github.com/bitcoin/bitcoin/issues/33405#issuecomment-3298347665)
Deyvid61