💬 glozow commented on pull request "random: move VerifyRNDRRS above InitHardwareRand":
(https://github.com/bitcoin/bitcoin/pull/31902#issuecomment-2669472019)
Closing since #31908 was merged.

💬 glozow commented on issue "GetRandBytes() Hangs on Samsung Galaxy S25 and OnePlus 13":
(https://github.com/bitcoin/bitcoin/issues/31817#issuecomment-2669479367)
Has not been fixed since #31908 reverted #31826, but also out of scope - see https://github.com/bitcoin/bitcoin/pull/31908#issuecomment-2669309907. I believe this should stay closed.

🤔 pablomartin4btc reviewed a pull request: "doc: update translation generation cmake example"
(https://github.com/bitcoin/bitcoin/pull/31731#pullrequestreview-2627689858)
ACK 79eb14003f42e1329fe3cdc17a42023c8b2d14b9

If we want to use the `dev-mode` preset perhaps we need to build `/depends` with `MULTIPROCESS=1`, as the preset has `"WITH_MULTIPROCESS": "ON",`. Please consider #31899 as it fixes the existent problem with multiprocess built.

We should add `-DWITH_USDT=OFF` (fails on mac otherwise) as @hebasto [confirmed](https://github.com/bitcoin/bitcoin/pull/31731#discussion_r1935753074) it won't affect the translation.

Just for reference, translation g
...

📝 Lynn-Matini opened a pull request: "Testing Matini"
(https://github.com/bitcoin/bitcoin/pull/31909)
Currently testing the PR assignment. No issue ID yet😊

💬 pinheadmz commented on pull request "guix: Notarize MacOS app bundle and codesign all MacOS and Windows binaries":
(https://github.com/bitcoin/bitcoin/pull/31407#issuecomment-2669518794)
I'm having an issue codesigning:

```

--> ./detached-sig-create.sh <...>p12 <...>p8 <...UUID...>

WARNING: Part of the file was not parsed: 37428 bytes
Enter the passphrase for /Volumes/safe2/bitcoin-core-csk/Archive/cert@bitcoincorecodesigning.org.p12:
Enter the passphrase for /Volumes/safe2/bitcoin-core-csk/app_store_connect_api_key/AuthKey_5ZTN3N6A87.p8:
WARNING: Part of the file was not parsed: 37428 bytes
Code signature created
WARNING: Part of the file was not parsed: 37428 byt
...

✅ theuni closed a pull request: "Testing Matini"
(https://github.com/bitcoin/bitcoin/pull/31909)

💬 theuni commented on pull request "Testing Matini":
(https://github.com/bitcoin/bitcoin/pull/31909#issuecomment-2669539171)
Please don't clog our c-i/review pipelines with senseless PRs.

💬 mzumsande commented on pull request "wallet: fix crash on double block disconnection":
(https://github.com/bitcoin/bitcoin/pull/31757#discussion_r1962235520)
"Ensure the tx is still abandoned and the wallet has no balance" sounds too positive to me - the test comment should say clearly that this is a bug in master and the wallet is showing the wrong balance because it incorrectly treats a tx which is in the current chain as abandoned (even if it doesn't crash anymore).

💬 sipa commented on pull request "Add waitNext() to BlockTemplate interface":
(https://github.com/bitcoin/bitcoin/pull/31283#discussion_r1962248389)
What is the rationale for this creating and returning a *new* `BlockTemplate` object, as opposed to replacing the current (`*this`) one? As far as I can see the current one becomes useless, as its `m_block_template` will be outdated, so future calls to `waitNext` for it will return instantly anyway.

💬 l0rinc commented on pull request "doc: update translation generation cmake example":
(https://github.com/bitcoin/bitcoin/pull/31731#discussion_r1962279607)
Added back `-DWITH_USDT=OFF`

💬 l0rinc commented on pull request "doc: update translation generation cmake example":
(https://github.com/bitcoin/bitcoin/pull/31731#issuecomment-2669606517)
Thanks @pablomartin4btc for the review, I've added back `-DWITH_USDT=OFF`

💬 l0rinc commented on pull request "refactor: modernize outdated trait patterns using helper aliases (C++14/C++17)":
(https://github.com/bitcoin/bitcoin/pull/31904#discussion_r1962284588)
Pushed https://github.com/llvm/llvm-project/pull/127811 and reverted the file here, thanks!

💬 ryanofsky commented on pull request "Add waitNext() to BlockTemplate interface":
(https://github.com/bitcoin/bitcoin/pull/31283#discussion_r1962302827)
> As far as I can see the current one becomes useless, as its `m_block_template` will be outdated, so future calls to `waitNext` for it will return instantly anyway.

This should not be the case. This returns a pointer to a new block template so miners are able to mine with the old one while calling this.

💬 sipa commented on pull request "Add waitNext() to BlockTemplate interface":
(https://github.com/bitcoin/bitcoin/pull/31283#discussion_r1962305664)
Sure, while calling `waitNext`, but after it returns, isn't it the case that the current `BlockTemplate` becomes useless?

💬 ryanofsky commented on pull request "Add waitNext() to BlockTemplate interface":
(https://github.com/bitcoin/bitcoin/pull/31283#discussion_r1962322579)
> Sure, while calling `waitNext`, but after it returns, isn't it the case that the current `BlockTemplate` becomes useless?

It's not useless if submitSolution can still be called. This should be the case as long as the tip hasn't changed and miners are still mining with this template. If the tip has changed or miners are no longer mining with this template, the client can discard it.

💬 achow101 commented on pull request "guix: Notarize MacOS app bundle and codesign all MacOS and Windows binaries":
(https://github.com/bitcoin/bitcoin/pull/31407#issuecomment-2669709312)
> I'm having an issue codesigning:

Fixed in https://github.com/achow101/signapple/commit/0c087854fd2822ab63512cbfa3b55706239b27c2. Note that this only affects code signers so I will hold off on updating signapple in guix for now.

💬 sipa commented on pull request "Add waitNext() to BlockTemplate interface":
(https://github.com/bitcoin/bitcoin/pull/31283#discussion_r1962357905)
Oh, great, that answers my question.

💬 pinheadmz commented on pull request "guix: Notarize MacOS app bundle and codesign all MacOS and Windows binaries":
(https://github.com/bitcoin/bitcoin/pull/31407#issuecomment-2669745876)
Updated detached codesigs:

https://github.com/pinheadmz/bitcoin-detached-sigs/tree/achow101-macos-notarization-e181bda


## all.SHA256SUMS
```

09160970e038df29d6c2ae3da69adf77ac496f65e114e3bbb56272465b9c230a bitcoin-e181bda061ca-aarch64-linux-gnu-debug.tar.gz
f94c5f7734858dfc77717849934112cceef44a80b06fb1c21f41a78733092c6a bitcoin-e181bda061ca-aarch64-linux-gnu.tar.gz
44dcd5864d1c529c431f5e80966d4efe74a14330182460ac945852bc67ad7e84 bitcoin-e181bda061ca-arm-linux-gnueabihf-debug.ta
...

💬 pinheadmz commented on pull request "guix: Notarize MacOS app bundle and codesign all MacOS and Windows binaries":
(https://github.com/bitcoin/bitcoin/pull/31407#issuecomment-2669764500)
Reviewers can also try downloading these signed binaries to see what their browsers think ... ?

https://www.dropbox.com/scl/fo/zpz3gjtzz66gtejnqkrfs/AIWbAU_Y3pY_O93cy9Q6vds?rlkey=oucgyfg4sm5p9pa2c0gum2kxa&dl=0

🚀 fanquake merged a pull request: "cmake: Exclude generated sources from translation"
(https://github.com/bitcoin/bitcoin/pull/31899)