💬 maflcko commented on pull request "doc: add `/spenttxouts` to REST-interface.md":
(https://github.com/bitcoin/bitcoin/pull/32842#issuecomment-3021867622)
lgtm ACK dd99cedc0bfe7d7eee0f543bb27dab005c426c66

💬 maflcko commented on pull request "mempool: Avoid needless vtx iteration during IBD":
(https://github.com/bitcoin/bitcoin/pull/32827#discussion_r2176475573)
Yeah, not sure. It would be good to see some flamegraphs or IBD speedup, to see what matters?

💬 davidgumberg commented on pull request "wallet: Fix relative path backup during migration.":
(https://github.com/bitcoin/bitcoin/pull/32273#issuecomment-3022237757)
> Won't this turn `/home/user/arbitrarywalletname.db` into `/home/user/user_%d.legacy.bak` ? Seems incorrect?

> This PR currently breaks that. Although `test_direct_file` doesn't currently fail, adding a check for the wallet's name in the backup path does make it fail, e.g.
>
> [...]

@luke-jr @achow101
Thanks for catching this, I've added the test case @achow101 suggested for this in 5cd3e2794d78848e7040e.

----

@ryanofsky
> Following would seem to be a good fix:
>
> [...]

...

📝 bigspider opened a pull request: "doc: clarify that the "-j N" goes after the "--build build" part"
(https://github.com/bitcoin/bitcoin/pull/32846)
I was surprised that something like `cmake -j 4 --build build` doesn't work, so this might help others to not make the same mistake.

💬 Sjors commented on pull request "ipc: Handle unclean shutdowns better":
(https://github.com/bitcoin/bitcoin/pull/32345#discussion_r2176634257)
Maybe comment that the `!` is for gcc

💬 maflcko commented on pull request "doc: clarify that the "-j N" goes after the "--build build" part":
(https://github.com/bitcoin/bitcoin/pull/32846#issuecomment-3022289027)
lgtm ACK 69f3b2039147558f5c9b452016ab980930e85222

💬 Sjors commented on pull request "ipc: Handle unclean shutdowns better":
(https://github.com/bitcoin/bitcoin/pull/32345#discussion_r2176648523)
Ok, we'll find out, if CI passes in the latest https://github.com/Sjors/bitcoin/pull/90

🤔 BrandonOdiwuor reviewed a pull request: "wallet: Remove ISMINE_WATCHONLY and watchonly from RPCs"
(https://github.com/bitcoin/bitcoin/pull/32618#pullrequestreview-2974133938)
Tested ACK 003a3cdb29dcc1e3e1a53f8227de73389071fbd1

Confirmed that all watch-only related functionality has been removed from the wallet and relevant RPCs, in line with the deprecation of legacy wallets.

- Verified that watch only has been removed from balances and `getbalance` RPC.
- Tested that `include_watchonly` option and `iswatchonly` flag was removed from all RPCs including `getaddressinfo`, `fundrawtransaction`, `send`, `sendall`, `walletcreatefundedpsbt`, `listreceivedbyaddress`
...

🚀 fanquake merged a pull request: "doc: add `/spenttxouts` to REST-interface.md"
(https://github.com/bitcoin/bitcoin/pull/32842)

🚀 fanquake merged a pull request: "contrib: correct variable name in p2p_monitor.py"
(https://github.com/bitcoin/bitcoin/pull/32816)

💬 b-l-u-e commented on pull request "net: Fix Discover() not running when using -bind=0.0.0.0:port":
(https://github.com/bitcoin/bitcoin/pull/32757#discussion_r2176886961)
thank you for the suggestion.

💬 b-l-u-e commented on pull request "net: Fix Discover() not running when using -bind=0.0.0.0:port":
(https://github.com/bitcoin/bitcoin/pull/32757#discussion_r2176889079)
thank you for the suggestion..i will make changes

🤔 hebasto reviewed a pull request: "doc: clarify that the "-j N" goes after the "--build build" part"
(https://github.com/bitcoin/bitcoin/pull/32846#pullrequestreview-2974314528)
Concept ACK. It follows CMake [docs](https://cmake.org/cmake/help/latest/manual/cmake.1.html#build-a-project).

Why not apply such a change across all the docs?

See the output of `git grep -i 'Use "-j N"'` for relevant instances.

💬 fanquake commented on pull request "build, docs: Fix Boost-related issues on NetBSD":
(https://github.com/bitcoin/bitcoin/pull/32828#issuecomment-3022809958)
cc @purpleKarrot

💬 stickies-v commented on pull request "doc: clarify that the "-j N" goes after the "--build build" part":
(https://github.com/bitcoin/bitcoin/pull/32846#issuecomment-3022837089)
Concept ACK

> See the output of git grep -i 'Use "-j N"' for relevant instances.

Agreed with this. I also couldn't find any other patterns that should be updated here.

💬 Sjors commented on pull request "ipc: Handle unclean shutdowns better":
(https://github.com/bitcoin/bitcoin/pull/32345#discussion_r2176931877)
Narrator: it does

💬 fanquake commented on pull request "depends: fix libevent `_WIN32_WINNT` usage":
(https://github.com/bitcoin/bitcoin/pull/32837#issuecomment-3022864921)
> Any idea why the issue manifests only with mingw-w64 13.x?

It's due to this change: https://github.com/mingw-w64/mingw-w64/commit/6cfc1fd2ca2fdbbb8d6570084970bea2ef100d2e. The headers have started defining `NTDDI_VERSION` based on the value of `_WIN32_WINNT` (if the version was < Windows 10). Given that libevent was undefining our `_WIN32_WINNT`, and then redefining it to a value < Windows 10 (`0x0501`), `NTDDI_VERSION` is also defined to that value, which was leading to functions not being
...

💬 purpleKarrot commented on pull request "build, docs: Fix Boost-related issues on NetBSD":
(https://github.com/bitcoin/bitcoin/pull/32828#issuecomment-3022868889)
ACK 5a5ddbd78922236402df378c8588a7b0b3f83a13

I wonder why no other projects run into this issue.

🤔 BrandonOdiwuor reviewed a pull request: "rpc, doc: clarify watch-only wallets balances in RPCHelp"
(https://github.com/bitcoin/bitcoin/pull/32761#pullrequestreview-2974424721)
Code Review ACK cb44680db40c85ba338f2a054511cd68ba6e89ba after reviewing https://github.com/bitcoin/bitcoin/pull/32618

💬 Eunovo commented on pull request "wallet: Remove ISMINE_WATCHONLY and watchonly from RPCs":
(https://github.com/bitcoin/bitcoin/pull/32618#issuecomment-3022945375)
Consider cleaning up `wallet_balance.py` too, like this:
```diff
diff --git a/test/functional/wallet_balance.py b/test/functional/wallet_balance.py
index 4efc0169eb..ac0374c3fa 100755
--- a/test/functional/wallet_balance.py
+++ b/test/functional/wallet_balance.py
@@ -80,9 +80,9 @@ class WalletTest(BitcoinTestFramework):
assert_equal(self.nodes[0].getbalance("*"), 50)
assert_equal(self.nodes[0].getbalance("*", 1), 50)
assert_equal(self.nodes[0].getbalance(minco
...