💬 fjahr commented on pull request "test: Fix race condition in IPC interface block progation test":
(https://github.com/bitcoin/bitcoin/pull/33880#discussion_r2543746081)
Fixed

💬 hebasto commented on pull request "guix: build `bitcoin-qt` with static libxcb & utils":
(https://github.com/bitcoin/bitcoin/pull/33537#discussion_r2543752271)
It would be helpful if someone could report this upstream.

💬 fanquake commented on pull request "depends: update xcb-util packages to latest versions":
(https://github.com/bitcoin/bitcoin/pull/33851#discussion_r2543759301)
That required adding new dependencies, which I didn't want to do here. It's the same reason libxkbcommon isn't updated further, as that would require a new build tool/system (meson), and new dependencies.

💬 fjahr commented on pull request "refactor, docs: Embedded ASMap [2/3]: Refactor asmap internals and add documentation":
(https://github.com/bitcoin/bitcoin/pull/33878#discussion_r2543764811)
Ah, I see, thanks for clarifying! Done.

💬 achow101 commented on pull request "wallet: don't consider unconfirmed TRUC coins with ancestors":
(https://github.com/bitcoin/bitcoin/pull/33528#issuecomment-3554911366)
ACK dcd42d6d8f160ae8bc12c152099a6e6473658e30

✅ waketraindev closed a pull request: "net: Decouple `CConnman::GetAddresses` from `CNode`"
(https://github.com/bitcoin/bitcoin/pull/33900)

💬 waketraindev commented on pull request "net: Decouple `CConnman::GetAddresses` from `CNode`":
(https://github.com/bitcoin/bitcoin/pull/33900#issuecomment-3554921486)
Closed. Going to let Net Split WG handle this one :) thanks for the comments.

👍 brunoerg approved a pull request: "test: add `-alertnotify` test for large work invalid chain warning"
(https://github.com/bitcoin/bitcoin/pull/33893#pullrequestreview-3485040821)
reACK c5b301f6ac0d3f882cc55397e3fb0cf95058704b

💬 hebasto commented on pull request "depends: update xcb-util packages to latest versions":
(https://github.com/bitcoin/bitcoin/pull/33851#discussion_r2543816478)
> That required adding new dependencies...

Such as?

FWIW, [this](https://github.com/hebasto/bitcoin/commits/pr33851/1119/) branch seem to build fine.

💬 fjahr commented on pull request "build: Embedded ASMap [3/3]: Build binary dump header file":
(https://github.com/bitcoin/bitcoin/pull/28792#discussion_r2543817367)
> I can't remember where we landed on making this is a build option. Has anyone asked for the ability to turn this off at build time, as opposed to just at runtime (if it becomes the default)

We didn't really land anywhere. My last response on it was [here](https://github.com/bitcoin/bitcoin/pull/28792#issuecomment-2962598663) and then the discussion didn't continue, I think. Similar to the default location discussion I am not going to die on this hill, I just want to give users the options t
...

💬 achow101 commented on pull request "wallet: remove redundant sighash calculation in Musig2 signing flow":
(https://github.com/bitcoin/bitcoin/pull/33665#issuecomment-3554963168)
> The original intent of this refactor to reduce seeing duplicate code

Considering how much extra is being changed to avoid the duplication, I don't really think this is worth it.

> but it should also help in improving performance of the wallet signing operations.

I don't think that's the case. The MuSig2 functions that are all recomputing the same sighash are never actually doing so in the same call to `SignMuSig2`. `CreateMuSig2AggregateSig` exits early when there aren't enough pubnon
...

💬 D33r-Gee commented on pull request "Interfaces: Expose UTXO Snapshot Loading and Add Progress Notifications":
(https://github.com/bitcoin/bitcoin/pull/33117#issuecomment-3555056928)
with [d0c32c2](https://github.com/bitcoin/bitcoin/commit/d0c32c28e59d3076100bf4c12d94f8b2d4060943) fixed RPC command issue that surfaced after rebase

💬 fjahr commented on pull request "build: Embedded ASMap [3/3]: Build binary dump header file":
(https://github.com/bitcoin/bitcoin/pull/28792#discussion_r2543875825)
I think this is just poor wording on my part, I don't mean a completely new process there, I mean the process that is described in this document can be run again at any time so that there is a new map created that can be used for a release. If that answers your question then I will amend that part of the document to be more clear.

I will still give short answer to the questions in case above doesn't resolve them. As mentioned previously here: https://github.com/bitcoin/bitcoin/pull/28792/file
...

💬 achow101 commented on pull request "wallet: Expand MuSig test coverage and follow-ups":
(https://github.com/bitcoin/bitcoin/pull/33636#issuecomment-3555082242)
Concept ACK

💬 fjahr commented on pull request "build: Embedded ASMap [3/3]: Build binary dump header file":
(https://github.com/bitcoin/bitcoin/pull/28792#issuecomment-3555104094)
Rebased since https://github.com/bitcoin/bitcoin/pull/33026 was merged, of course still based on https://github.com/bitcoin/bitcoin/pull/33878.

💬 achow101 commented on pull request "wallet: warn against accidental unsafe older() import":
(https://github.com/bitcoin/bitcoin/pull/33135#issuecomment-3555111778)
ACK fb72cc33be570d12fc6b76146fc89047e58f5aaf

💬 achow101 commented on pull request "wallet: warn against accidental unsafe older() import":
(https://github.com/bitcoin/bitcoin/pull/33135#issuecomment-3555127585)
Silent merge conflict?

```
Running 2 test cases...
./test/descriptor_tests.cpp(1272): fatal error: in "descriptor_tests/descriptor_older_warnings": critical check !descs.empty() has failed

*** 1 failure is detected in the test module "Bitcoin Core Test Suite"
```

💬 achow101 commented on pull request "kernel: handle null or empty directories in implementation":
(https://github.com/bitcoin/bitcoin/pull/33867#issuecomment-3555163702)
ACK 6657bcbdb4d0359c1843ca31fb3670c7c0c260d5

💬 achow101 commented on pull request "clang-format: Set InsertNewlineAtEOF: true":
(https://github.com/bitcoin/bitcoin/pull/33896#issuecomment-3555168238)
ACK fa1bf6818f0910f997e5235a197ff51f2e18780d

🚀 achow101 merged a pull request: "kernel: handle null or empty directories in implementation"
(https://github.com/bitcoin/bitcoin/pull/33867)