👍 tdb3 approved a pull request: "chainparams: Handle Testnet4 in GetNetworkForMagic"
(https://github.com/bitcoin/bitcoin/pull/30625#pullrequestreview-2231690134)
cr ACK b0ec8716bf27335686471e0ae4c6a34f9a08f33c

💬 theStack commented on pull request "contrib: add tool to convert compact-serialized UTXO set to SQLite database":
(https://github.com/bitcoin/bitcoin/pull/27432#issuecomment-2282299449)
Rebased on master, with the following changes:
* adapted to the new snapshot format (VERSION=2, see #30598)
* extended the netmagic table with Testnet4 (see #29775)
* extended the test to also create a pay-to-anchor UTXO (see #30352)
* print out the supposed snapshot height after processing all coins, by tracking the maximum height (see https://github.com/bitcoin/bitcoin/pull/30598#issuecomment-2276350922)

💬 allanperlee commented on issue "use MiniWallet in functional test `rpc_signrawtransactionwithkey.py` for funding txs":
(https://github.com/bitcoin/bitcoin/issues/30600#issuecomment-2282304362)
Hi @theStack, I could soon start this issue

⚠️ sahil5d opened an issue: "Evaluate Unappreciated Evidence that Satoshi Nakamoto was Elon Musk"
(https://github.com/bitcoin/bitcoin/issues/30627)
### Please describe the feature you'd like to see added.

Add to the knowledge of Satoshi Nakamoto the following information:

## Unappreciated Evidence that Satoshi Nakamoto was Elon Musk

### An IP address in Los Angeles

On Satoshi: "This is not a TOR exit node which implies that this is the IP address used by Satoshi on 2009–01–10 and he was in Van Nuys [Los Angeles] on this day."
 https://whoissatoshi.wordpress.com/2016/02/20/satoshi-in-california/

On Musk: "The jet is registered
...

✅ achow101 closed an issue: "Evaluate Unappreciated Evidence that Satoshi Nakamoto was Elon Musk"
(https://github.com/bitcoin/bitcoin/issues/30627)

:lock: achow101 locked an issue: "Evaluate Unappreciated Evidence that Satoshi Nakamoto was Elon Musk"
(https://github.com/bitcoin/bitcoin/issues/30627)

⚠️ VadymK666 opened an issue: "Чудовий початок роботи 👌💪"
(https://github.com/bitcoin/bitcoin/issues/30628)
### Motivation

GitHub автоматически создает ссылки, если допустимые URL-адреса записываются в комментарий. Дополнительные сведения см. в разделе

### Possible solution

Если для репозитория настроены пользовательские автоматические ссылки, то ссылки на внешние ресурсы, такие как проблема JIRA или тикет Zendesk, конвертируются в сокращенные ссылки. Чтобы узнать, какие автоматические ссылки доступны в репозитории, обратитесь к кому-то, у кого есть разрешения администратора в отношении репозитория
...

✅ maflcko closed an issue: "Чудовий початок роботи 👌💪"
(https://github.com/bitcoin/bitcoin/issues/30628)

:lock: fanquake locked an issue: "Чудовий початок роботи 👌💪"
(https://github.com/bitcoin/bitcoin/issues/30628)

👍 maflcko approved a pull request: "chainparams: Handle Testnet4 in GetNetworkForMagic"
(https://github.com/bitcoin/bitcoin/pull/30625#pullrequestreview-2231783071)
review ACK b0ec8716bf27335686471e0ae4c6a34f9a08f33c

Left a style-nit to avoid a copy and to make the code shorter, but only if you re-touch.

💬 maflcko commented on pull request "chainparams: Handle Testnet4 in GetNetworkForMagic":
(https://github.com/bitcoin/bitcoin/pull/30625#discussion_r1712953968)
style-nit: I know you copied this from the surrounding code, but creating a copy is not needed to compare two const references, and if the size of the second range in `std::equal` is different, the result is either wrong or undefined behavior (UB/uninitialized read). See also the notes section (https://en.cppreference.com/w/cpp/algorithm/equal#Notes), which says "... When comparing entire containers for equality, operator== for the corresponding container are usually preferred".

So that'd be:
...

💬 paplorinc commented on pull request "build: Introduce CMake-based build system":
(https://github.com/bitcoin/bitcoin/pull/30454#discussion_r1712955667)
> The cmake -S src/bench -B build command fails.

It seems to me [they're using `ninja`](https://github.com/bitcoin-cash-node/bitcoin-cash-node/blob/master/doc/benchmarking.md?plain=1#L8) for these tasks, i.e. [`ninja bench-bitcoin`](https://github.com/bitcoin-cash-node/bitcoin-cash-node/blob/master/doc/ninja_targets.md?plain=1#L251) instead of raw cmake.

💬 hebasto commented on pull request "build: Introduce CMake-based build system":
(https://github.com/bitcoin/bitcoin/pull/30454#discussion_r1712956905)
> > The cmake -S src/bench -B build command fails.
>
> It seems to me [they're using `ninja`](https://github.com/bitcoin-cash-node/bitcoin-cash-node/blob/master/doc/benchmarking.md?plain=1#L8) for these tasks, i.e. [`ninja bench-bitcoin`](https://github.com/bitcoin-cash-node/bitcoin-cash-node/blob/master/doc/ninja_targets.md?plain=1#L251) instead of raw cmake.

It seems this discussion has gone off topic :)

💬 paplorinc commented on pull request "build: Introduce CMake-based build system":
(https://github.com/bitcoin/bitcoin/pull/30454#discussion_r1712957036)
Didn't realize that, please resolve it in that case

👍 rkrux approved a pull request: "BlockAssembler: return selected packages vsize and feerate"
(https://github.com/bitcoin/bitcoin/pull/30391#pullrequestreview-2231801330)
tACK [f581657](https://github.com/bitcoin/bitcoin/commit/f581657b3a1acd0fc57dc9461d3a7e0835e17a24)

`make, make check` are successful. Left a suggestion.
Thanks for keeping the PR short and focussed.

💬 rkrux commented on pull request "BlockAssembler: return selected packages vsize and feerate":
(https://github.com/bitcoin/bitcoin/pull/30391#discussion_r1712971476)
IMO, depending upon finding **only** the feeRate in the vector of tuples gets rid of some level of robustness in the verification here because it ignores the possibility of duplicate feeRates, and makes the tester/author wary of writing the test in a way that ensures there are no duplicates.
How about **also** checking for the packageSize as well?

💬 fjahr commented on pull request "wallet: Fix listwalletdir listing of migrated default wallets and generated backup files":
(https://github.com/bitcoin/bitcoin/pull/30265#issuecomment-2282736669)
Code review ACK 6b2dcba07670f04f32c0dc3a2c86fd805c85f12d

Also verified manually that the added tests do cover the change of behavior.

💬 Sjors commented on pull request "validation: assumeutxo params mainnet":
(https://github.com/bitcoin/bitcoin/pull/28553#issuecomment-2282740748)
```
2024-08-10T19:39:49Z [snapshot] computing UTXO stats for background chainstate to validate snapshot - this could take a few minutes
Killed
```

I think this happened because I used a RAM drive for pruned sync and it ran out of space (a Guix build I ran at the same time failed too). Would be nice to fail more gracefully, but I don't think it's related to the specific snapshot here.

💬 paplorinc commented on pull request "build: Introduce CMake-based build system":
(https://github.com/bitcoin/bitcoin/pull/30454#discussion_r1712974890)
How much rewrite do we need to do here?
I have pushed https://github.com/hebasto/bitcoin/pull/322 to synchronize on the style, let me know how you imagine it.

💬 m3dwards commented on pull request "contrib: fix check-deps.sh to check for weak symbols":
(https://github.com/bitcoin/bitcoin/pull/30415#issuecomment-2282747080)
ACK 2bca4af323c5b505689c5203e6ddba8becac5dc8

Tested on Debian x86.

With the suppression disabled I got:

```shell
Error: libbitcoin_consensus_a-pubkey.o depends on libbitcoin_util_a-strencodings.o symbol 'std::optional<std::vector<unsigned char, std::allocator<unsigned char> > > TryParseHex<unsigned char>(std::basic_string_view<char, std::char_traits<char> >)'
```