💬 ajtowns commented on pull request "net: Provide block templates to peers on request":
(https://github.com/bitcoin/bitcoin/pull/33191#issuecomment-3312726054)
Patchset to also request templates from outbound peers is at https://github.com/ajtowns/bitcoin/commits/202508-sendtemplate3/

💬 benthecarman commented on pull request "depends: static libxcb-cursor":
(https://github.com/bitcoin/bitcoin/pull/33434#issuecomment-3312740332)
Sorry, that was for v30rc1

built guix binary for this PR. This fixes so I don't need `libxcb-cursor0` installed to run the binary and is the same where no dark mode but also no windowing bug

💬 hebasto commented on pull request "depends: static libxcb-cursor":
(https://github.com/bitcoin/bitcoin/pull/33434#discussion_r2363468002)
> Haven't tested this, or looked for potential side effects (i.e #33432 claims that installing the dep breaks dark mode). Also related to #32097.

I’m still not sure I understand this fix. Perhaps @benthecarman could provide [more details](https://github.com/bitcoin/bitcoin/issues/33432#issuecomment-3311581711) about the reported bug?

💬 glozow commented on pull request "rpc: Always return per-wtxid entries in submitpackage tx-results":
(https://github.com/bitcoin/bitcoin/pull/33427#discussion_r2363493462)
I think CHECK_NONFATAL is fine. `Assume` might make it seem like this API quirk is more important than it actually is, and then we'd have to explain why it's ok to delete it later.

💬 benthecarman commented on issue "v30rc1 Weird GUI windowing behavior":
(https://github.com/bitcoin/bitcoin/issues/33432#issuecomment-3312812001)
compiled from source without `libxcb-cursor0`

```
2025-09-19T16:11:53Z Ignoring unknown configuration value mempoolfullrbf
2025-09-19T16:11:53Z Ignoring unknown configuration value signet.signetblocktime
2025-09-19T16:11:53Z Bitcoin Core version v30.0rc1 (release build)
2025-09-19T16:11:53Z Qt 6.4.2 (dynamic), plugin=wayland
2025-09-19T16:11:53Z No static plugins.
2025-09-19T16:11:53Z Style: fusion / QFusionStyle
2025-09-19T16:11:53Z System: Ubuntu 24.04.3 LTS, x86_64-little_endian-lp64
2025-09
...

👋 fanquake's pull request is ready for review: "depends: static libxcb-cursor"
(https://github.com/bitcoin/bitcoin/pull/33434)

💬 benthecarman commented on pull request "depends: static libxcb-cursor":
(https://github.com/bitcoin/bitcoin/pull/33434#discussion_r2363496059)
https://github.com/bitcoin/bitcoin/issues/33432#issuecomment-3312812001

🤔 glozow reviewed a pull request: "p2p: Increase tx relay rate"
(https://github.com/bitcoin/bitcoin/pull/28592#pullrequestreview-3245717908)
utACK b81f37031c8f2ccad9346f1b65ee0f8083c44796

💬 glozow commented on pull request "rpc: Always return per-wtxid entries in submitpackage tx-results":
(https://github.com/bitcoin/bitcoin/pull/33427#discussion_r2363502186)
is optional now false?

💬 hebasto commented on issue "v30rc1 Weird GUI windowing behavior":
(https://github.com/bitcoin/bitcoin/issues/33432#issuecomment-3312837619)
Thanks, @benthecarman!

From the provided logs, it appears that the reported bug occurs when building with the system’s Qt 6.4.2, rather than with the statically linked release binary.

The requirement to install `libxcb-cursor0` is already a known issue and is being tracked elsewhere.

💬 PiRK commented on pull request "test: Fix TestFlushBehavior when coin has non-zero dynamic memory usage":
(https://github.com/bitcoin/bitcoin/pull/33381#issuecomment-3312853835)
let's see if this works. I moved the problematic test line to its own test suite, where it does not affect other flush tests.

The failure I'm trying to solve seems intermittent. It happens about 75% of the time on master if I just add the scriptPubKey to MakeCoin.
After my commit I get no failure in 10 runs of `./build/bin/test_bitcoin --run_test=coins_tests`

💬 PiRK commented on pull request "test: Fix TestFlushBehavior when coin has non-zero dynamic memory usage":
(https://github.com/bitcoin/bitcoin/pull/33381#issuecomment-3312873728)
To clarify the rationale for this PR: the `SanityCheck` test added to `flush_all` in https://github.com/bitcoin/bitcoin/pull/28280/commits/a14edada8a051e280af6fedd5130be40247e2d7a does not have full coverage if `cachedCoinsUsage` is not exercised.

This PR ensure that the `cachedCoinsUsage` part of `SanityCheck` now actually does something useful for the `TestFlushBehavior` suite.

💬 john-moffett commented on pull request "rpc: Always return per-wtxid entries in submitpackage tx-results":
(https://github.com/bitcoin/bitcoin/pull/33427#discussion_r2363560078)
I'd think it stays optional, as successful (and partially successful) submissions won't have an error for every tx. See, eg, [this test](https://github.com/bitcoin/bitcoin/blob/edb871cba22a658714ce97e843ff84cfe1ba9ac4/test/functional/rpc_packages.py#L417), where only the child has an error.

💬 fanquake commented on pull request "ci: run s390x job":
(https://github.com/bitcoin/bitcoin/pull/33436#issuecomment-3312901010)
Thanks @willcl-ark. Pulled in (some version) of your change, and added a `NO_QT=1` for now.

💬 glozow commented on pull request "rpc: Always return per-wtxid entries in submitpackage tx-results":
(https://github.com/bitcoin/bitcoin/pull/33427#discussion_r2363629883)
Oh I did not read that carefully 🤦 yes.

🚀 fanquake merged a pull request: "p2p: Increase tx relay rate"
(https://github.com/bitcoin/bitcoin/pull/28592)

💬 fanquake commented on pull request "p2p: Increase tx relay rate":
(https://github.com/bitcoin/bitcoin/pull/28592#issuecomment-3312961173)
Backported to `v30.x` in #33424.

💬 fanquake commented on pull request "ci: run s390x job":
(https://github.com/bitcoin/bitcoin/pull/33436#issuecomment-3312993728)
Looks like this passed in `98m` (without Qt and nothing cached).

💬 marcofleon commented on pull request "fuzz: reduce iterations in slow targets":
(https://github.com/bitcoin/bitcoin/pull/33429#issuecomment-3313116463)
coverage:

- `txdownloadman` [master](https://marcofleon.github.io/coverage/txdownloadman/) [this PR](https://marcofleon.github.io/coverage/txdownloadman_33429/)
- `txdownloadman_impl` [master](https://marcofleon.github.io/coverage/txdownloadman_impl/) [this PR](https://marcofleon.github.io/coverage/txdownloadman_impl_33429/)
- `mini_miner` [master](https://marcofleon.github.io/coverage/mini_miner/) [this PR](https://marcofleon.github.io/coverage/mini_miner_33429/)
- `tx_pool_standard` [mas
...

🤔 enirox001 reviewed a pull request: "fuzz: reduce iterations in slow targets"
(https://github.com/bitcoin/bitcoin/pull/33429#pullrequestreview-3246274219)
Concept ACK 6a33970

While working on #32870, I considered reducing iterations to speed up fuzzing, but wasn’t sure it was feasible since that wasn’t the issue’s intended goal. However, I noticed that reducing iterations did speed up the MiniMiner fuzz test.

This seems like a good approach to solving the issue, pending the reimplementation of MiniMiner.