📝 fanquake opened a pull request: "contrib: update fixed seeds"
(https://github.com/bitcoin/bitcoin/pull/33283)
Update the fixed seeds pre 30 branch off.

💬 hebasto commented on pull request "Release: 30.0 translations update":
(https://github.com/bitcoin/bitcoin/pull/33275#issuecomment-3245511525)
> Sure, see https://github.com/maflcko/b-c-gui-translations-review/tree/99bf41c4e5ab74eee8c248b240e8e940e47e09ec/reviews

Many thanks, @maflcko! This is indeed very useful.

I discarded only a few suggestions for the Ukrainian translation:

1.
```
<source>Copy &amp;raw transaction</source>
<translation>Копіювати &amp;всю транзакцію</translation>

ERR
The term "raw transaction" in the Bitcoin context refers specifically to the transaction in its raw (serialized) he
...

💬 glozow commented on pull request "p2p: never check tx rejections by txid":
(https://github.com/bitcoin/bitcoin/pull/33066#issuecomment-3245518101)
Closing for now. We can revisit this cleanup later.

✅ glozow closed a pull request: "p2p: never check tx rejections by txid"
(https://github.com/bitcoin/bitcoin/pull/33066)

✅ glozow closed a pull request: "[NO MERGE] BIP331 Ancestor Package Relay"
(https://github.com/bitcoin/bitcoin/pull/27742)

💬 glozow commented on pull request "[NO MERGE] BIP331 Ancestor Package Relay":
(https://github.com/bitcoin/bitcoin/pull/27742#issuecomment-3245588079)
Ok Drahty I'll take the hint :joy:.

Most of the code here is in master now - the Orphan Resolution Module is part of `TxDownloadManager` (#30110, #31397, etc.) and the `TxOrphanage` eviction changes (#31829, etc.) replace the token bucket protection stuff here. One difference is that we didn’t add the `TxRequestTracker orphan_request_tracker` for vanilla orphan resolution, but that is probably needed for ancestor package relay.

Parts 2 and 3 are relatively smaller, but still need the mempo
...

💬 ellenkampguus commented on issue "Old wallet support (Berkeley 4.8)":
(https://github.com/bitcoin/bitcoin/issues/33273#issuecomment-3245607749)
I want to reopen the discussion as I am starting to doubt if Bitcoin Core is still catering to the Bitcoin user's needs.

💬 sipa commented on pull request "Add functional test for IPC interface":
(https://github.com/bitcoin/bitcoin/pull/33201#issuecomment-3245620914)
Rebased, and cherry-picked @Sjors' commit.

💬 pinheadmz commented on issue "Old wallet support (Berkeley 4.8)":
(https://github.com/bitcoin/bitcoin/issues/33273#issuecomment-3245622643)
What "user need" do you feel is not being addressed? Legacy wallets can be converted to the new improved format. This has been at least a [five-year project](https://github.com/bitcoin/bitcoin/issues/20160)

🤔 danielabrozzoni reviewed a pull request: "rpc, logging: add backgroundvalidation to getblockchaininfo"
(https://github.com/bitcoin/bitcoin/pull/33259#pullrequestreview-3176862036)
Concept ACK

I did a very rough first pass and the code looks good to me. As others have said, I would like to see a functional test testing the newly introduced fields when the background validation is in progress

💬 hebasto commented on pull request "cmake: fatal error when PIE not supported":
(https://github.com/bitcoin/bitcoin/pull/33282#issuecomment-3245626058)
cc @purpleKarrot @theuni

💬 hebasto commented on pull request "clang-tidy: disable clang-analyzer UndefinedBinaryOperatorResult warning":
(https://github.com/bitcoin/bitcoin/pull/33281#issuecomment-3245657952)
Can it be localized into the `src/ipc/capnp/.clang-tidy` file?

💬 hebasto commented on pull request "cmake: make missing Python interpreter behaviour more explicit":
(https://github.com/bitcoin/bitcoin/pull/33278#issuecomment-3245677855)
cc @purpleKarrot

💬 m3dwards commented on pull request "ci: Migrate CI to hosted Cirrus Runners":
(https://github.com/bitcoin/bitcoin/pull/32989#discussion_r2316358241)
Lint job looks at GPG keys on commits and AFAIK rarely fails. It's not like a "normal" lint that checks code style and therefore often fails.

💬 marcofleon commented on issue "build: clang-16 build broken on Debian Bookworm":
(https://github.com/bitcoin/bitcoin/issues/33279#issuecomment-3245714093)
Just got the same error with Clang 19. Installing the default Capn Proto packages (0.9.2) won't work it seems. I upgraded to 1.1.0 by building capnp from source and bitcoin builds now. It even builds an extra 1%...

<img width="1459" height="324" alt="Image" src="https://github.com/user-attachments/assets/19cca262-77e1-4984-ad19-ceb84c0c573d" />

💬 m3dwards commented on pull request "ci: Migrate CI to hosted Cirrus Runners":
(https://github.com/bitcoin/bitcoin/pull/32989#issuecomment-3245714292)
ACK 3c5da69a232ba1cfb935012aa53e57002efe0d77

💬 ryanofsky commented on pull request "clang-tidy: disable clang-analyzer UndefinedBinaryOperatorResult warning":
(https://github.com/bitcoin/bitcoin/pull/33281#issuecomment-3245742642)
> Can it be localized into the `src/ipc/capnp/.clang-tidy` file?

That seems like a good idea. But even the current version of this PR (c157a1bc8388fecec85f61286ae928721aa8d78a) doesn't seem to be working right now. I wonder if it is because the files are generated files not inside the source directory? I'm not sure what change might make sense to fix this.

Current tidy job output is:

- https://cirrus-ci.com/task/5720235140972544
- https://api.cirrus-ci.com/v1/task/5720235140972544/logs
...

📝 ryanofsky converted_to_draft a pull request: "clang-tidy: disable clang-analyzer UndefinedBinaryOperatorResult warning"
(https://github.com/bitcoin/bitcoin/pull/33281)
Disable `clang-analyzer-core.UndefinedBinaryOperatorResult` warning because it produces a false positive in a Cap'n Proto header:

```c++
/usr/include/kj/async-inl.h:609:37: warning: The left operand of '+' is a garbage value [clang-analyzer-core.UndefinedBinaryOperatorResult]
609 | return *(void**)(*(char**)obj + voff);
```

This was reported by fanquake in https://github.com/bitcoin/bitcoin/issues/33256 and was previously addressed in https://github.com/bitcoin-core/libmultiproc
...

💬 ryanofsky commented on pull request "clang-tidy: disable clang-analyzer UndefinedBinaryOperatorResult warning":
(https://github.com/bitcoin/bitcoin/pull/33281#issuecomment-3245745115)
Marking as draft because the change does not seem to have any effect right now.

💬 maflcko commented on pull request "ci: Migrate CI to hosted Cirrus Runners":
(https://github.com/bitcoin/bitcoin/pull/32989#discussion_r2316389311)
> It's not like a "normal" lint that checks code style and therefore often fails.

It is, and it fails occasionally: https://github.com/bitcoin/bitcoin/pulls?q=is%3Apr+%22task+lint%3A+https%3A%2F%2Fgithub.com%2Fbitcoin%2Fbitcoin%2Fruns%2F%22+is%3Aclosed

No opinion on the dep graph. The benefit of putting it in `needs` is that the other tasks are skipped (as they are possibly not needed). The downside of putting it in `needs` is that the other tasks are skipped (as the failure is intended/ig
...