💬 TheCharlatan commented on pull request "kernel: Introduce initial C header API":
(https://github.com/bitcoin/bitcoin/pull/30595#issuecomment-2741688322)
Updated 2dc27e2860b97c2bffa5f18706917b21858e5594 -> 9fc6accf89ed001f70e107a8e9936f6dc3a35f41 ([kernelApi_30](https://github.com/TheCharlatan/bitcoin/tree/kernelApi_30) -> [kernelApi_31](https://github.com/TheCharlatan/bitcoin/tree/kernelApi_31), [compare](https://github.com/TheCharlatan/bitcoin/compare/kernelApi_30..kernelApi_31))

* Addressed @stickies-v's [comment](https://github.com/bitcoin/bitcoin/pull/30595#discussion_r2000864918), fixed naming in docstring for `kernel_LoggingConnection`.
...

💬 TheCharlatan commented on pull request "kernel: Introduce initial C header API":
(https://github.com/bitcoin/bitcoin/pull/30595#discussion_r2006477122)
This should just mirror the internal code at the moment, but I agree that it is not really useful to split this up. Will see if I can consolidate this.

💬 hebasto commented on pull request "build: Switch to Qt 6":
(https://github.com/bitcoin/bitcoin/pull/30997#discussion_r2006504759)
It seems there is an issue with older CMake versions. I was able to reproduce it on Ubuntu `x86_64` while compiling natively:
```
$ cmake --version
cmake version 3.22.6

CMake suite maintained and supported by Kitware (kitware.com/cmake).
$ gmake -C depends -j 16 qt
<snip>
Building qt...
ninja: error: 'qtbase/src/platformsupport/input/CMakeFiles/InputSupportPrivate.dir/cmake_pch.hxx.gch', needed by 'qtbase/src/platformsupport/input/CMakeFiles/InputSupportPrivate.dir/InputSupportPrivate_
...

⚠️ yellowred opened an issue: "Add a script for Signet or Regtest to unlock fee estimator"
(https://github.com/bitcoin/bitcoin/issues/32105)
### Please describe the feature you'd like to see added.

Currently, the fee estimator won't work in a private signet or regtest network, because there are usually not enough transactions being added. This forces clients to hardcode a default fee or have some custom logic for these networks.
The proposal is to add a script to /contrib/signet that would simulate a transactions volume just enough to allow `estimatesmartfee` to work.
It would be useful to have it in the bitcoin core repo, because:

...

💬 TheCharlatan commented on pull request "kernel: Introduce initial C header API":
(https://github.com/bitcoin/bitcoin/pull/30595#issuecomment-2741785465)
Rebased 9fc6accf89ed001f70e107a8e9936f6dc3a35f41 -> 29f05b91cf8a479e403b0322afeb5ff1133da221 ([kernelApi_31](https://github.com/TheCharlatan/bitcoin/tree/kernelApi_31) -> [kernelApi_32](https://github.com/TheCharlatan/bitcoin/tree/kernelApi_32), [compare](https://github.com/TheCharlatan/bitcoin/compare/kernelApi_31..kernelApi_32))

* Fixed silent merge conflict with #31519

💬 hebasto commented on pull request "build: Switch to Qt 6":
(https://github.com/bitcoin/bitcoin/pull/30997#discussion_r2006536581)
The problem occurs with CMake versions older than 3.25.0.

💬 hebasto commented on pull request "build: Switch to Qt 6":
(https://github.com/bitcoin/bitcoin/pull/30997#discussion_r2006546699)
Here is an upstream issue [fixed](https://gitlab.kitware.com/cmake/cmake/-/merge_requests/7759) in CMake 3.25.0 – https://gitlab.kitware.com/cmake/cmake/-/issues/23464

💬 willcl-ark commented on issue "Source code mapping for debugger has changed since cmake":
(https://github.com/bitcoin/bitcoin/issues/31204#issuecomment-2741816753)
This seems to work for me on Linux without any `.lldbinit` file:

```bash
will@ubuntu in …/src/core/bitcoin on  master [$] via △ v3.31.6 : 🐍 (core)
$ just clean build-depends
depends build running
copying packages: boost libevent qt expat libxcb xcb_proto libXau xproto freetype fontconfig libxkbcommon libxcb_util libxcb_util_render libxcb_util_keysyms libxcb_util_image libxcb_util_wm qrencode bdb sqlite systemtap zeromq
to: /home/will/src/core/bitcoin/depends/x86_64-pc-linux-gnu
To build Bitco
...

💬 willcl-ark commented on pull request "ci: run test-each-commit on merge to master":
(https://github.com/bitcoin/bitcoin/pull/32103#issuecomment-2741873830)
> > I think what you want is to checkout each old commit and then merge it (inside the CI task) with master. Not sure how to achieve it otherwise. Maybe you can share your steps to reproduce with `git`?
>
> To clarify, my suggestion would be to modify the `git rebase --exec 'run_test' base` into `git rebase --exec 'git log -1 && git merge --no-commit master && echo run_test && git merge --abort' base`. However, I haven't tested this.

Would it not in that case be simpler to checkout the PR
...

💬 jimhashhq commented on issue "Add a `indexesdir` option to hold the indexes directory":
(https://github.com/bitcoin/bitcoin/issues/32099#issuecomment-2741897166)
Do you have `txindex` enabled or disabled?

When `-txindex` is _disabled_, the (internal/primary SSD) index storage requirements were only 2% of total blockchain size I thought.. Running instead _with_ `-txindex` would indeed bump internal/primary storage up to around 10-12%, which may be what you're seeing..

Don't know if it would help your situation, but have you tried running w/o `-txindex`?

⚠️ Davidoff2778 opened an issue: "Bitcoin Core Config Generator"
(https://github.com/bitcoin/bitcoin/issues/32106)
https://jlopp.github.io/bitcoin-core-config-generator/#config=eyJjaGFpbiI6eyJ0ZXN0bmV0IjoxLCJ0ZXN0bmV0NCI6MSwicmVndGVzdCI6MSwic2lnbmV0IjoxfSwiY29yZSI6eyJibG9ja3Nvbmx5IjoxLCJjb2luc3RhdHNpbmRleCI6MSwiZGFlbW9uIjoxLCJkYWVtb253YWl0IjoxLCJhbGxvd2lnbm9yZWRjb25mIjoxLCJwZXJzaXN0bWVtcG9vbHYxIjoxLCJ0eGluZGV4IjoxfSwid2FsbGV0Ijp7ImFkZHJlc3N0eXBlIjoibGVnYWN5IiwiZGlzYWJsZXdhbGxldCI6MSwidW5zYWZlc3FsaXRlc3luYyI6MSwid2FsbGV0Y3Jvc3NjaGFpbiI6MX19

✅ pinheadmz closed an issue: "Bitcoin Core Config Generator"
(https://github.com/bitcoin/bitcoin/issues/32106)

💬 pinheadmz commented on issue "Bitcoin Core Config Generator":
(https://github.com/bitcoin/bitcoin/issues/32106#issuecomment-2741933825)
Whatever your issue is I think you need to open it in this repo: https://github.com/jlopp/bitcoin-core-config-generator/

🚀 fanquake merged a pull request: "fuzz: Use immediate task runner to increase fuzz stability"
(https://github.com/bitcoin/bitcoin/pull/31841)

💬 pinheadmz commented on issue "b-msghand invoked oom-killer: Master (v28.99) crashing during IBD":
(https://github.com/bitcoin/bitcoin/issues/31561#issuecomment-2741951050)
I don't think you *can* prune with txindex! I added a 1 TB volume -- this was the mainnet server for the BOSS2025 program, specifically the RPC scavenger hunt challenge.

There were several crashes, I can't remember the height they started at but it was long before 800000.

The conf file just had `txindex=1` but I must have turned on `debug=1` on the command line when the crashes started

💬 fanquake commented on pull request "refactor: Enforces Txid and Wtxid types in RelayTransaction":
(https://github.com/bitcoin/bitcoin/pull/32104#issuecomment-2742089261)
cc @marcofleon & #31001.

💬 sr-gi commented on pull request "refactor: Enforces Txid and Wtxid types in RelayTransaction":
(https://github.com/bitcoin/bitcoin/pull/32104#issuecomment-2742096552)
Oh, I didn't realize Marco was already working on this. I saw this when updating the Erlay PR and thought it may be good to standalone fix. I'm happy to close it if you're planning to get it addressed @marcofleon

⚠️ bonczis opened an issue: "A"
(https://github.com/bitcoin/bitcoin/issues/32107)
### Motivation

A

### Possible solution

_No response_

### Useful Skills

* Compiling Bitcoin Core from source
* Running the C++ unit tests and the Python functional tests
* ...


### Guidance for new contributors

Want to work on this issue?

For guidance on contributing, please read [CONTRIBUTING.md](https://github.com/bitcoin/bitcoin/blob/master/CONTRIBUTING.md) before opening your pull request.

✅ fanquake closed an issue: "A"
(https://github.com/bitcoin/bitcoin/issues/32107)

:lock: fanquake locked an issue: "A"
(https://github.com/bitcoin/bitcoin/issues/32107)

💬 TheCharlatan commented on pull request "fuzz: Use immediate task runner to increase fuzz stability":
(https://github.com/bitcoin/bitcoin/pull/31841#issuecomment-2742339375)
Post-merge ACK