🚀 fanquake merged a pull request: "wallet: Cleanup accidental encryption keys in watchonly wallets"
(https://github.com/bitcoin/bitcoin/pull/28724)

💬 Sjors commented on pull request "Add multiprocess binaries to release build (except Windows)":
(https://github.com/bitcoin/bitcoin/pull/30975#issuecomment-2582967477)
Rebased to see if nothing new breaks. Clarified the PR description to indicate Windows support (probably) won't make it into this PR.

As described in the TODO, before marking this as ready review, there's one UndefinedBehaviorSanitizer left to resolve upstream in libmultiprocess. I might also wait for #31375 and #31161 to land first (added to description).

💬 0xB10C commented on pull request "mining: bugfix: Fix duplicate coinbase tx weight reservation":
(https://github.com/bitcoin/bitcoin/pull/31384#issuecomment-2582976824)
Concept ACK

⚠️ Lin-web-AI opened an issue: "比特币"
(https://github.com/bitcoin/bitcoin/issues/31637)
********@@********

✅ pinheadmz closed an issue: "比特币"
(https://github.com/bitcoin/bitcoin/issues/31637)

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

✅ maflcko closed a pull request: "fix: grammatical errors"
(https://github.com/bitcoin/bitcoin/pull/31636)

💬 maflcko commented on pull request "fix: grammatical errors":
(https://github.com/bitcoin/bitcoin/pull/31636#issuecomment-2583000205)
historic release notes are archived and should not be modified

💬 fanquake commented on pull request "psbt: add non-default sighash types to PSBTs and unify sighash type match checking":
(https://github.com/bitcoin/bitcoin/pull/31622#issuecomment-2583006911)
https://github.com/bitcoin/bitcoin/actions/runs/12700499877/job/35403281996?pr=31622#step:12:8987:
```bash
test 2025-01-10T00:28:24.738000Z TestFramework (ERROR): Assertion failed
Traceback (most recent call last):
File "D:\a\bitcoin\bitcoin\build\test\functional\test_framework\test_framework.py", line 135, in main
self.run_test()
File "D:\
...

💬 fanquake commented on pull request "cmake: Set top-level target output locations":
(https://github.com/bitcoin/bitcoin/pull/31161#issuecomment-2583015317)
https://cirrus-ci.com/task/6223844022681600?logs=ci#L2898:
```bash
[08:40:37.725] + LD_LIBRARY_PATH=/ci_container_base/depends/i686-pc-linux-gnu/lib
[08:40:37.725] + test/functional/test_runner.py --ci -j10 --tmpdirprefix /ci_container_base/ci/scratch/test_runner/ --ansi --combinedlogslen=99999999 --timeout-factor=40 --v2transport --quiet --failfast
[08:40:38.317] 2025-01-10T13:40:38.202000Z TestFramework (INFO): PRNG seed is: 5569585375096173377
[08:40:38.317] 2025-01-10T13:40:38.206000Z T
...

💬 Sjors commented on pull request "cmake: Set top-level target output locations":
(https://github.com/bitcoin/bitcoin/pull/31161#issuecomment-2583073982)
Concept ACK

Lightly tested 8c44a947b501c8417454527637bd3adb3ce2ff4e by building on macOS 15.2 (M4). I like how the documentation is changed with a scripted-diff. The build system changes in the first commit look reasonable to me at first glance, but did not review thoroughly.

@ryanofsky wrote:

> Possibly, it might make sense to build bitcoin-chainstate.exe and bitcoinkernel.dll to libexec/ instead of bin/.

I like that as well. If we add them to the release later, they won't clutter `
...

💬 mzumsande commented on issue "Potential crash (assert) rescanning wallet":
(https://github.com/bitcoin/bitcoin/issues/31474#issuecomment-2583089022)
Up to you of course, but yes, if this fixed the problem for you it'd help to say it there - thanks!

💬 fanquake commented on pull request "depends: add *FLAGS to gen_id":
(https://github.com/bitcoin/bitcoin/pull/31125#issuecomment-2583098454)
Guix Build:
```bash
3922712c31b158aefa34a0700c02dc0c2c02c318002ead81cc7c9836c34c7b6a guix-build-01df180bfb82/output/aarch64-linux-gnu/SHA256SUMS.part
600a22f791e838113d1b0401f0b038a5cf7e28638e6eb80cd80f39885dd2c63b guix-build-01df180bfb82/output/aarch64-linux-gnu/bitcoin-01df180bfb82-aarch64-linux-gnu-debug.tar.gz
e9a43319a6f6358940ff265be574d067604ce4f31d7c1e5be0c143c4407892ef guix-build-01df180bfb82/output/aarch64-linux-gnu/bitcoin-01df180bfb82-aarch64-linux-gnu.tar.gz
a049be1301297687
...

💬 theuni commented on pull request "cmake: Set top-level target output locations":
(https://github.com/bitcoin/bitcoin/pull/31161#issuecomment-2583175364)
> * It would be good if multiprocess binaries `bitcoin-node` and `bitcoin-gui` could be built in libexec/ instead of than bin/

This makes sense to me if we're going to go down the route of the wrapper executable, but I don't think we should do that until users aren't actually supposed to be launching them directly.

IMO it's not a problem to move them after a release cycle or two if the expected user behavior changes, if nothing else that's actually a good indication that their behavior _sh
...

💬 fanquake commented on pull request "build: enable libc++ and libstdc++ hardening":
(https://github.com/bitcoin/bitcoin/pull/31424#discussion_r1910658510)
Not sure. This just seems to be removing an option (that otherwise doesn't conflict with the other changes)?

💬 fanquake commented on pull request "build: enable libc++ and libstdc++ hardening":
(https://github.com/bitcoin/bitcoin/pull/31424#discussion_r1910658664)
> so we use _GLIBCXX_ASSERTIONS in both Debug and non-Debug builds.

How did you test the performance impacts of this? For example master:
```bash
./master/src/bench/bench_bitcoin -filter=LinearizeOptimallyExample11
| ns/op | op/s | err% | ins/op | cyc/op | IPC | bra/op | miss% | total | benchmark
|--------------------:|--------------------:|--------:|----------------:|----------------:|-------:|---------------:|--------:|-
...

💬 darosior commented on pull request "Miner: never create a template which exploits the timewarp bug":
(https://github.com/bitcoin/bitcoin/pull/31376#issuecomment-2583326352)
Yes. I intend to rebase on top of it once it's merged.

💬 darosior commented on pull request "Miner: never create a template which exploits the timewarp bug":
(https://github.com/bitcoin/bitcoin/pull/31376#issuecomment-2583327852)
Actually, this and #31600 can go in either order.

💬 hebasto commented on pull request "depends: add *FLAGS to gen_id":
(https://github.com/bitcoin/bitcoin/pull/31125#issuecomment-2583331131)
My Guix build:
```
aarch64
3922712c31b158aefa34a0700c02dc0c2c02c318002ead81cc7c9836c34c7b6a guix-build-01df180bfb82/output/aarch64-linux-gnu/SHA256SUMS.part
600a22f791e838113d1b0401f0b038a5cf7e28638e6eb80cd80f39885dd2c63b guix-build-01df180bfb82/output/aarch64-linux-gnu/bitcoin-01df180bfb82-aarch64-linux-gnu-debug.tar.gz
e9a43319a6f6358940ff265be574d067604ce4f31d7c1e5be0c143c4407892ef guix-build-01df180bfb82/output/aarch64-linux-gnu/bitcoin-01df180bfb82-aarch64-linux-gnu.tar.gz
a049be13
...

💬 Sjors commented on pull request "Add multiprocess binaries to release build (except Windows)":
(https://github.com/bitcoin/bitcoin/pull/30975#issuecomment-2583384638)
Pushed a commit to test https://github.com/chaincodelabs/libmultiprocess/pull/121