💬 w0xlt commented on pull request "wallet: Add `exportwatchonlywallet` RPC to export a watchonly version of a wallet":
(https://github.com/bitcoin/bitcoin/pull/32489#discussion_r2376758113)
Can `desc.cache` be empty here ?

💬 glozow commented on pull request "[28.x] More backports":
(https://github.com/bitcoin/bitcoin/pull/33415#discussion_r2376784754)
Is there a configure flag I need? I'm just running gen-manpages.py after building.

💬 instagibbs commented on pull request "Cluster mempool implementation":
(https://github.com/bitcoin/bitcoin/pull/28676#discussion_r2376795377)
pendantic: it's redundant *when an RBF is encountered*, but not otherwise and the result is cached so pretty cheap to throw around

📝 glozow opened a pull request: "[28.x] backports + 28.3rc1"
(https://github.com/bitcoin/bitcoin/pull/33476)
Built on top of #33415

Includes backports of
- #33106
- #30125
- #30948
- #30784

💬 glozow commented on pull request "[28.x] More backports":
(https://github.com/bitcoin/bitcoin/pull/33415#issuecomment-3330324239)
Opened #33476 to follow

💬 fanquake commented on pull request "build: Move CMAKE_SKIP_INSTALL_RPATH from CMake to Guix script":
(https://github.com/bitcoin/bitcoin/pull/33470#discussion_r2376849213)
Can this just be `CMAKE_SKIP_RPATH`? (also move to `CONFIGFLAGS`)

💬 fanquake commented on pull request "[28.x] backports + 28.3rc1":
(https://github.com/bitcoin/bitcoin/pull/33476#discussion_r2376868135)
Leftover from rebase?

💬 maflcko commented on pull request "validation: fetch block inputs on parallel threads >10% faster IBD":
(https://github.com/bitcoin/bitcoin/pull/31132#discussion_r2376962309)
seems a bit odd to have the number of nodes in a test influence whether or not the test has to be edited to remove or add `-par=1` everywhere. Would it not be easier to just globally set `-par=2` for all funtional tests?


```diff
diff --git a/test/functional/test_framework/util.py b/test/functional/test_framework/util.py
index e5a5938f07..42bb213dd3 100644
--- a/test/functional/test_framework/util.py
+++ b/test/functional/test_framework/util.py
@@ -459,6 +459,7 @@ def write_config(conf
...

💬 andrewtoth commented on pull request "validation: fetch block inputs on parallel threads >10% faster IBD":
(https://github.com/bitcoin/bitcoin/pull/31132#discussion_r2377071137)
Yes, I wondered if that would be more invasive to other tests though.

📝 fjahr opened a pull request: "Rollback for dumptxoutset without invalidating blocks"
(https://github.com/bitcoin/bitcoin/pull/33477)
This is an alternative approach to implement `dumptxoutset` with rollback that was discussed a few times. It does not rely on `invalidateblock` and `reconsiderblock` and instead creates a temporary copy of the coins DB, modifies this copy by rolling back as many blocks as necessary and then creating the dump from this temp copy DB. See also https://github.com/bitcoin/bitcoin/pull/29553#issuecomment-1978480989, https://github.com/bitcoin/bitcoin/issues/32817#issuecomment-3012406102 and #29565 dis
...

💬 fjahr commented on pull request "rpc: Fix dumptxoutset rollback with competing forks":
(https://github.com/bitcoin/bitcoin/pull/33444#issuecomment-3330754815)
> Regarding the test failure - let me investigate and fix that. Would you prefer I:

I have opened #33477 now, as you can see there I am describing that there are some trade-offs between the approach and master which this PR would then be applied to. Let's see what reviewers think about those trade-offs. You should keep it open and ideally fix the test. If the CI is failing that usually signals to reviewers that the PR isn't ready for consideration since the code will need to change anyway. So
...

💬 fjahr commented on pull request "Rollback for dumptxoutset without invalidating blocks":
(https://github.com/bitcoin/bitcoin/pull/33477#issuecomment-3330760975)
cc @Sjors since you were asking for this approach a few times :)

💬 ryanofsky commented on issue "`bitcoin-node` is unkillable after mining IPC connection is established":
(https://github.com/bitcoin/bitcoin/issues/33463#issuecomment-3330765943)
Have been trying to reproduce this bug in python by calling waitTipChanged followed by a shutdown but so far no luck.

With no `await asyncio.sleep` after `waitTipChanged` and before `stop_node` I get a "peer disconnected" error. With `await asyncio.sleep(0)` I get a "broken pipe" error, and with longer sleeps I see a nullopt BlockRef returned from waitTipChanged().

These results are all expected, and I didn't see a way to make the node hang on shutdown. Changes made to interface_ipc.py are sh
...

🤔 mzumsande reviewed a pull request: "cli: Handle arguments that can be either JSON or string"
(https://github.com/bitcoin/bitcoin/pull/33230#pullrequestreview-3264798788)
Code Review ACK df67bb6fd84c393eaf00f19074085ee080546bd3

Possible additional changes or follow-ups:
- Release note (so that users know that this is possible now)
- Adjust doc examples (https://github.com/bitcoin/bitcoin/pull/33230#issuecomment-3216227775)

I am not really a fan of multi-type args that require workarounds like this - e.g. I see no reason `getblockstats` would support a height while `getblock` doesn't - but that ship has sailed.

💬 plebhash commented on issue "`bitcoin-node` is unkillable after mining IPC connection is established":
(https://github.com/bitcoin/bitcoin/issues/33463#issuecomment-3330793682)
still haven't found the time for the stack trace, busy with Sv2 release... apologies for that

but it's pretty much gone on my side

so I'm happy if we close this, or I can also share stack trace later

💬 w0xlt commented on pull request "wallet: Add `exportwatchonlywallet` RPC to export a watchonly version of a wallet":
(https://github.com/bitcoin/bitcoin/pull/32489#discussion_r2377142018)
The assertion below (same as found in `wallet_migration.py`) fails.
Is this expected ?

```suggestion
# Assert no rescan occurs when restoring the exported watch-only wallet.
with self.online.assert_debug_log(unexpected_msgs=["Rescanning"]):
self.online.restorewallet(export_name, res["exported_file"])
```

💬 alfredopalhares commented on pull request "docs: Undeprecate datacarrier and datacarriersize configuration options":
(https://github.com/bitcoin/bitcoin/pull/33453#issuecomment-3330855441)
Concept ACK

Come to your senses devs, the users have spoken.

💬 151henry151 commented on pull request "build: Move CMAKE_SKIP_INSTALL_RPATH from CMake to Guix script":
(https://github.com/bitcoin/bitcoin/pull/33470#discussion_r2377208338)
I think I've made the correct adjustments -- can you check [dd5c517](https://github.com/bitcoin/bitcoin/commit/dd5c517757f97b68f7eb07628222c958b47f742b) and let me know if this is what you had in mind?

🤔 BrunoSampaioDev reviewed a pull request: "rpc: addpeeraddress: throw on invalid IP"
(https://github.com/bitcoin/bitcoin/pull/33430#pullrequestreview-3265078475)
LGTM

💬 l0rinc commented on pull request "log: reduce excessive "rolling back/forward" messages during block replay":
(https://github.com/bitcoin/bitcoin/pull/33443#discussion_r2377590716)
Good point, thanks, fixed.