💬 tapcrafter commented on pull request "init: Configure reachable networks before we start the RPC server":
(https://github.com/bitcoin/bitcoin/pull/32539#discussion_r2094164033)
I'm aware brevity is probably prioritized here. But I didn't recognize RFC4193 just from its number, so had to look it up. What about "RFC4193 style IPv6 ULA addresses are only allowed..."?

💬 yancyribbens commented on pull request "doc: Add missing top-level description to pruneblockchain RPC":
(https://github.com/bitcoin/bitcoin/pull/32333#issuecomment-2888498892)
cr ACK https://github.com/bitcoin/bitcoin/pull/32333/commits/135a0f0aa711b95c50aa4cbe0c38d82d647f1c8b

Nice, didn't know this rpc existed.

💬 yancyribbens commented on issue "Migrate from BTC/kvB to sat/vB on RPC and startup options":
(https://github.com/bitcoin/bitcoin/issues/32093#issuecomment-2888499988)
> Don't forget that fundrawtransaction accepts two different units, depending on snake case / camel case (!)

This seems like a bug that ought to be fixed separately?

💬 jonatack commented on issue "Migrate from BTC/kvB to sat/vB on RPC and startup options":
(https://github.com/bitcoin/bitcoin/issues/32093#issuecomment-2888506000)
> Don't forget that `fundrawtransaction` accepts two different units, depending on snake case / camel case (!)

See https://github.com/bitcoin/bitcoin/pull/20483.

💬 Ashkar776 commented on issue "Migrate from BTC/kvB to sat/vB on RPC and startup options":
(https://github.com/bitcoin/bitcoin/issues/32093#issuecomment-2888508710)
How to use

On Sat, 17 May, 2025, 9:38 pm Jon Atack, ***@***.***> wrote:

> *jonatack* left a comment (bitcoin/bitcoin#32093)
> <https://github.com/bitcoin/bitcoin/issues/32093#issuecomment-2888506000>
>
> Don't forget that fundrawtransaction accepts two different units,
> depending on snake case / camel case (!)
>
> See #20483 <https://github.com/bitcoin/bitcoin/pull/20483>.
>
> —
> Reply to this email directly, view it on GitHub
> <https://github.com/bitcoin/bitcoin/issues/32093#
...

💬 Ashkar776 commented on issue "Migrate from BTC/kvB to sat/vB on RPC and startup options":
(https://github.com/bitcoin/bitcoin/issues/32093#issuecomment-2888508833)
You have tutorial video step by step

On Sat, 17 May, 2025, 9:39 pm Ashkar Asku, ***@***.***>
wrote:

> How to use
>
> On Sat, 17 May, 2025, 9:38 pm Jon Atack, ***@***.***> wrote:
>
>> *jonatack* left a comment (bitcoin/bitcoin#32093)
>> <https://github.com/bitcoin/bitcoin/issues/32093#issuecomment-2888506000>
>>
>> Don't forget that fundrawtransaction accepts two different units,
>> depending on snake case / camel case (!)
>>
>> See #20483 <https://github.com/bitcoin/bitcoin/pull
...

💬 achow101 commented on pull request "guix: Notarize MacOS app bundle and codesign all MacOS and Windows binaries":
(https://github.com/bitcoin/bitcoin/pull/31407#issuecomment-2888528076)
Perhaps this should be backported to all current release branches?

💬 yancyribbens commented on pull request "rest: fetch spent transaction outputs by blockhash":
(https://github.com/bitcoin/bitcoin/pull/32540#issuecomment-2888533229)
Nice. I wonder about just adding a content-type like option so that the api endpoint is the same but you simply say if you want binary or json content returned? Maybe there's other endpoints that it could then be applied to as well..

🤔 rkrux reviewed a pull request: "wallet: Keep track of the wallet's own transaction outputs in memory"
(https://github.com/bitcoin/bitcoin/pull/27286#pullrequestreview-2848375525)
I have just started reviewing this PR - have gone through the PR description & commit messages, going through the diff at the moment.

The intent of the PR is quite cool! Due to the nature of the change, I decided to run benchmarks on my system and have shared below the best results of the lot I had (the results seem to be consistent across multiple runs).

### Observations
1. I can clearly see noticeable performance improvements in the `WalletBalance` benchmarks.
2. However, both `WalletA
...

🤔 i-am-yuvi reviewed a pull request: "test: Move `script_assets_tests` into its own suite"
(https://github.com/bitcoin/bitcoin/pull/31576#pullrequestreview-2848380496)
tACK 0859fc22f0baf00259cf964f9d39d36fba6cd291

- making skipped tests explicitly appear as 'Skipped' rather than silently omitting

```
Start 89: script_tests
Start 85: script_p2sh_tests
Start 87: script_segwit_tests
Start 88: script_standard_tests
Start 86: script_parse_tests
Start 84: script_assets_tests
1/6 Test #86: script_parse_tests ............... Passed 0.16 sec
2/6 Test #84: script_assets_tests ..............***Skipped 0.16 sec
3/6 Test #88: s
...

💬 hebasto commented on pull request "build: Use clang-cl to build on Windows natively":
(https://github.com/bitcoin/bitcoin/pull/31507#discussion_r2094220240)
> > Also, any reason to not fix the issues in leveldb, rather than adding more suppressions?
>
> I'll consider this option.

Please see https://github.com/bitcoin-core/leveldb-subtree/pull/53.

💬 hebasto commented on issue "Option to use dark theme for Windows":
(https://github.com/bitcoin-core/gui/issues/378#issuecomment-2888612967)
> There is too low contrast on the buttons in that pic

Feel free to open a fresh issue about that.

💬 Efikcoin commented on issue "SignatureCreator should supply auxiliary data argument for additional bip340 signature security":
(https://github.com/bitcoin/bitcoin/issues/31883#issuecomment-2888616578)
Efikcoin-mining pool stake
Reward💰

💬 andrewtoth commented on pull request "contrib: add xor-blocks tool to obfuscate blocks directory":
(https://github.com/bitcoin/bitcoin/pull/32451#issuecomment-2888622694)
Thank you @l0rinc @Sjors @laanwj @LarryRuane @1440000bytes @Kixunil @maflcko @yancyribbens for your reviews and suggestions!

The latest version changes the following:
- data is XOR'd 16 bytes at a time using the doubled key as a `u128`. This is done using pointer casting and unsafe dereferencing, so we don't have to copy the data back and forth when converting `[u8; 16]` to `u128`.
- A custom datadir option is now passed as `-datadir=`, similar to `bitcoind`.
- Files are read 16 bytes at
...

💬 andrewtoth commented on pull request "contrib: add xor-blocks tool to obfuscate blocks directory":
(https://github.com/bitcoin/bitcoin/pull/32451#discussion_r2094230217)
Done.

💬 andrewtoth commented on pull request "contrib: add xor-blocks tool to obfuscate blocks directory":
(https://github.com/bitcoin/bitcoin/pull/32451#discussion_r2094230354)
Replaced all logs of xor with obfuscation.

💬 andrewtoth commented on pull request "contrib: add xor-blocks tool to obfuscate blocks directory":
(https://github.com/bitcoin/bitcoin/pull/32451#discussion_r2094231982)
Done.

💬 andrewtoth commented on pull request "contrib: add xor-blocks tool to obfuscate blocks directory":
(https://github.com/bitcoin/bitcoin/pull/32451#discussion_r2094232317)
I use `BufWriter` now and call `flush`. That should call `sync_data` under the hood I presume?

💬 andrewtoth commented on pull request "contrib: add xor-blocks tool to obfuscate blocks directory":
(https://github.com/bitcoin/bitcoin/pull/32451#discussion_r2094232454)
Done. I use `BufReader` and read 16 bytes at a time. Memory usage is much lower. Speed doesn't seem to be impacted.

💬 andrewtoth commented on pull request "contrib: add xor-blocks tool to obfuscate blocks directory":
(https://github.com/bitcoin/bitcoin/pull/32451#discussion_r2094232466)
Done.