💬 brunoerg commented on pull request "fuzz: wallet: add target for spkm migration":
(https://github.com/bitcoin/bitcoin/pull/29694#discussion_r2071773924)
Yes, these other things are also not necessary when using in memory db. I'm gonna address it.

💬 mzumsande commented on pull request "cli: add -usefile option":
(https://github.com/bitcoin/bitcoin/pull/32399#issuecomment-2847489011)
Since this will only change the test framework now, I'll just add it to #32290 - no extra PR needed anymore.

✅ mzumsande closed a pull request: "cli: add -usefile option"
(https://github.com/bitcoin/bitcoin/pull/32399)

📝 fanquake opened a pull request: "build: replace header checks with `__has_include`"
(https://github.com/bitcoin/bitcoin/pull/32405)
Replace the checks in CMake, with the equivalent functionality provided by the standard library (since C++17).
See https://en.cppreference.com/w/cpp/preprocessor/include.

Guix Build:
```bash
02f63c78b657e08dfc2a14b910ba28bdd00e67726093ff680e8eb8b4fd3787ea guix-build-e7fdd31ee0ce/output/aarch64-linux-gnu/SHA256SUMS.part
fbb9e2abd636f5d125141d2fe60350bb0678ae830f66e7047ae0b47451a63b81 guix-build-e7fdd31ee0ce/output/aarch64-linux-gnu/bitcoin-e7fdd31ee0ce-aarch64-linux-gnu-debug.tar.gz
855
...

💬 brunoerg commented on pull request "fuzz: wallet: add target for spkm migration":
(https://github.com/bitcoin/bitcoin/pull/29694#discussion_r2071800794)
Why? the for after `// Unload the wallets` are basically playing with `created_wallets` which is used only if the db is not in memory. Same for the lines after `// Verify that there is no dangling wallet` which needs `ptr_wallet`.

📝 instagibbs opened a pull request: "policy: uncap datacarrier by default"
(https://github.com/bitcoin/bitcoin/pull/32406)
Retains the `-datacarrier*` args, marks them as deprecated, and does not require another argument for multiple OP_RETURN outputs.

If a user has set `-datacarriersize` the value is "budgeted" across all seen OP_RETURN output scriptPubKeys. In other words the total script bytes stays the same, but can be spread across any number of outputs. This is done to not introduce an additional argument to support multiple outputs.

I do not advise people use the option with custom arguments and it is m
...

🤔 BrandonOdiwuor reviewed a pull request: "Wallet: Fix Non-Ranged Descriptors with Range [0,0] Trigger Unexpected Wallet Errors in AddWalletDescriptor"
(https://github.com/bitcoin/bitcoin/pull/32344#pullrequestreview-2812344598)
Code Review ACK fa0993462b3bc8db40c1a6ea2e7f5fd4a22353a6

💬 l0rinc commented on pull request "validation: write chainstate to disk every hour":
(https://github.com/bitcoin/bitcoin/pull/30611#issuecomment-2847561360)
Post-merge good news - the periodic flushing doesn't seem to introduce any slowdown for reindex-chainstate:

<details>
<summary>reindex-chainstate</summary>

```bash
COMMITS="68ac9f116c0228a277f18f60ba2278b56356e6ac 58e7798f7e85ecda97e32eec29c190e5ac958dc0"; \
STOP_HEIGHT=888888; DBCACHE=45000; \
CC=gcc; CXX=g++; \
BASE_DIR="/mnt/my_storage"; DATA_DIR="$BASE_DIR/BitcoinData"; LOG_DIR="$BASE_DIR/logs"; \
(echo ""; for c in $COMMITS; do git fetch origin $c -q && git log -1 --pretty=forma
...

💬 Sjors commented on pull request "Shuffle depends instructions and recommend modern make for macOS":
(https://github.com/bitcoin/bitcoin/pull/32086#discussion_r2071824993)
Added, cc @vasild

💬 Sjors commented on pull request "Shuffle depends instructions and recommend modern make for macOS":
(https://github.com/bitcoin/bitcoin/pull/32086#issuecomment-2847576668)
Rebased just in case and added a commit to switch FreeBSD recommendation to `gmake`.

📝 rebroad converted_to_draft a pull request: "Enhanced Traffic Graph Widget with Multi-timeframe Support and Data Persistence"
(https://github.com/bitcoin-core/gui/pull/866)
This PR improves the network traffic graph widget in the Debug window to provide:
1. Multiple time range options (from 5 minutes to 28 days)
2. Data persistence between sessions
3. Interactive visualization features including tooltips and logarithmic scaling

## Motivation
The existing network traffic graph has limited utility with its fixed time range and lack of historical data preservation. This enhancement allows users to:
- Analyze network traffic patterns over different timeframes

...

🤔 hebasto reviewed a pull request: "build: replace header checks with `__has_include`"
(https://github.com/bitcoin/bitcoin/pull/32405#pullrequestreview-2812387753)
Concept ACK. This improves code locality.

🤔 marcofleon reviewed a pull request: "fuzz: Remove unused TimeoutExpired catch in fuzz runner"
(https://github.com/bitcoin/bitcoin/pull/32388#pullrequestreview-2812454130)
crACK fa4804009ceba96926edd7f55ea22442ebdc665d

💬 l0rinc commented on pull request "util: explicitly close all AutoFiles that have been written":
(https://github.com/bitcoin/bitcoin/pull/29307#discussion_r2071924816)
`m_was_written = true`, since `AutoFile::write_buffer` also dirties (which is the term we use in other cases, if you think it describes the behavior better, consider renaming)

👍 hebasto approved a pull request: "Shuffle depends instructions and recommend modern make for macOS"
(https://github.com/bitcoin/bitcoin/pull/32086#pullrequestreview-2812537498)
re-ACK 7238f242d78f3d71834764031d7904d19525bab3.

💬 l0rinc commented on pull request "util: explicitly close all AutoFiles that have been written":
(https://github.com/bitcoin/bitcoin/pull/29307#discussion_r2071926224)
I find returning error codes for boolean values a lot more confusing.

🤔 mabu44 reviewed a pull request: "test: remove Boost SIGCHLD workaround."
(https://github.com/bitcoin/bitcoin/pull/32403#pullrequestreview-2812561636)
ACK 3add6ab9adcd722d57c6d488581358ae9b377f1a

<details>
<summary>Testing procedure</summary>

Reviewed the changes and compiled the code, then run
```bash
./build_dev/bin/test_bitcoin --log_level=all --run_test=util_tests/test_LockDirectory
```
The output of the test looks good to me.
</details>

📝 AlnetharyT opened a pull request: "النظاري (#1)"
(https://github.com/bitcoin/bitcoin/pull/32407)
* versionbits: Use std::array instead of C-style arrays

* versionbits: Remove params from AbstractThresholdConditionChecker

For an abstract class, specifying parameters in detail serves no point; and for the concrete implementation, changing the consensus parameters between invocations doesn't make sense. So simplify the class by removing the consensus params from the method arguments, and just make it a member variable in the concrete object where needed. This also allows dropping dummy p
...

✅ fanquake closed a pull request: "النظاري (#1)"
(https://github.com/bitcoin/bitcoin/pull/32407)

💬 monlovesmango commented on pull request "tests: Added progress tracker when running fuzz test_runner.py":
(https://github.com/bitcoin/bitcoin/pull/32354#discussion_r2071988479)
The summary at the end should probably be removed now?