📝 Ari4ka opened a pull request: "Create python-package.yml"
(https://github.com/bitcoin/bitcoin/pull/31968)
<!--
*** Please remove the following help text before submitting: ***

Pull requests without a rationale and clear improvement may be closed
immediately.

GUI-related pull requests should be opened against
https://github.com/bitcoin-core/gui
first. See CONTRIBUTING.md
-->

<!--
Please provide clear motivation for your patch and explain how it improves
Bitcoin Core user experience or Bitcoin Core developer experience
significantly:

* Any test improvements or new tests that improv
...

✅ fanquake closed a pull request: "Create python-package.yml"
(https://github.com/bitcoin/bitcoin/pull/31968)

💬 Prabhat1308 commented on pull request "contrib: Add deterministic-unittest-coverage":
(https://github.com/bitcoin/bitcoin/pull/31901#issuecomment-2692265876)
Concept ACK [`fa99c3b`](https://github.com/bitcoin/bitcoin/pull/31901/commits/fa99c3b544b631cfe34d52fb5e71636aedb1b423)
Fully agree with removing the outdated bash script.

I have only checked the `deterministic-unittest-coverage` and not the `deterministic-fuzz-coverage` on MacOS 15.3.1 with clang 19 . It has been giving me some flaky results . On the first run both `util_string_tests` or `util_tests` were giving me result as deterministic coverage.

<details>

<summary>First run</summa
...

💬 luke-jr commented on pull request "rpc: Support v3 raw transactions creation":
(https://github.com/bitcoin/bitcoin/pull/31936#discussion_r1976434270)
No, the issue isn't this one line of code, but the overall addition of the parameter

💬 hodlinator commented on pull request "miniscript refactor: Remove unique_ptr-indirection (#30866 follow-up)":
(https://github.com/bitcoin/bitcoin/pull/31713#discussion_r1976441195)
Thanks for having a look! `const` fields are the bane of making types movable, as in c7f1829b4bb3a579c65a99a62b42f64de946c0da of this very PR, so good to have another argument in that direction.

Still think there's some value in marking fields as `const` to collapse down the mental model of the code when moving is not necessary. Wish we had "const except on move".

But I've proven my point that it *can* now be made `const`. I'll remove it on the next push.

💬 luke-jr commented on pull request "rpc: add cli examples, update docs":
(https://github.com/bitcoin/bitcoin/pull/31958#discussion_r1976442528)
We shouldn't be encouraging this in the first place. Suggest replacing the example with yours.

📝 Sjors opened a pull request: "Add mainnet assumeutxo param at height 880,000"
(https://github.com/bitcoin/bitcoin/pull/31969)
#31940 suggests adding a snapshot at every major release.

This snapshot should be suitable for v29. I picked the most recent multiple of 10K blocks.

I'll upload a torrent later. You can generate the snapshot yourself:

```sh
bitcoin-cli -rpcclienttimeout=0 -named dumptxoutset utxo-880000.dat rollback=880000
```

And then load it on a fresh node during IBD with:

```
bitcoin-cli -rpcclienttimeout=0 loadtxoutset utxo-880000.dat
```

Note that it's more performant to turn off netw
...

💬 Sjors commented on pull request "Add assumeutxo chainparams to release-process.md":
(https://github.com/bitcoin/bitcoin/pull/31940#issuecomment-2692296552)
See #31969 for a potential v29 snapshot.

💬 yancyribbens commented on pull request "test: Add expected result assertions":
(https://github.com/bitcoin/bitcoin/pull/31656#issuecomment-2692332226)
@murchandamus sounds good. I added your suggestion to the test description and updated the commit message accordingly. Also rebased to make testing easier.

⚠️ RodrigoCamposDS opened an issue: "Bitcoin Core MacOS - Possible Miner Infection - Bitcoin Core MacOS - Possível Infecção por Minerador"
(https://github.com/bitcoin/bitcoin/issues/31970)
### Is there an existing issue for this?

- [x] I have searched the existing issues

### Current behaviour

🇺🇸

Subject: 🚨 Bitcoin Core for MacOS - Possible Miner Infection 🚨

Dear Bitcoin Core developers,

I downloaded the Bitcoin Core software directly from the official website (bitcoincore.org) for macOS, and my antivirus detected a possible malware embedded in the program.

Avast identified the file as MacOS:Miner-AR [PUP], and after checking on VirusTotal, several other antivirus progra
...

👍 hodlinator approved a pull request: "doc: Update documentation to include Clang/llvm based coverage report generation"
(https://github.com/bitcoin/bitcoin/pull/31933#pullrequestreview-2652570456)
ACK 1245e05325b41101eddc76ba9214d910489e35b6

- Good to include some comments on expected warnings (I was getting some too).
- Thanks for incorporating my latest feedback!
- Agree that functional/fuzz test instructions could either be a separate PR or left as an exercise to the reader.

💬 hodlinator commented on pull request "doc: Update documentation to include Clang/llvm based coverage report generation":
(https://github.com/bitcoin/bitcoin/pull/31933#discussion_r1976483663)
nit: Could shorten PR title. (Doesn't invalidate ACKs).
```diff
-doc: Update documentation to include Clang/llvm based coverage report generation
+doc: Add Clang/LLVM based coverage report generation
```

🤔 janb84 reviewed a pull request: "doc: Update documentation to include Clang/llvm based coverage report generation"
(https://github.com/bitcoin/bitcoin/pull/31933#pullrequestreview-2652573782)
re ACK [1245e05](https://github.com/bitcoin/bitcoin/commit/1245e05325b41101eddc76ba9214d910489e35b6)

The extra comments are informative of the possible warnings and their influence on the result.

💬 janb84 commented on pull request "doc: Update documentation to include Clang/llvm based coverage report generation":
(https://github.com/bitcoin/bitcoin/pull/31933#discussion_r1976485591)
Small nit, creating a blockquote note remark section is maybe more fitting:

> Note: The "counter mismatch" warning can be safely ignored, though it can be resolved by updating to Clang 19.\
> The warning occurs due to version mismatches but doesn't affect the coverage report generation.

💬 janb84 commented on pull request "test: Add expected result assertions":
(https://github.com/bitcoin/bitcoin/pull/31656#issuecomment-2692401596)
re ACK [a015b7e](https://github.com/bitcoin/bitcoin/commit/a015b7e13daacdfb6db0eada50563dec70c5afb2)

✅ RodrigoCamposDS closed an issue: "Bitcoin Core MacOS - Possible Miner Infection - Bitcoin Core MacOS - Possível Infecção por Minerador"
(https://github.com/bitcoin/bitcoin/issues/31970)

⚠️ RodrigoCamposDS reopened an issue: "Bitcoin Core MacOS - Possible Miner Infection - Bitcoin Core MacOS - Possível Infecção por Minerador"
(https://github.com/bitcoin/bitcoin/issues/31970)
### Is there an existing issue for this?

- [x] I have searched the existing issues

### Current behaviour

🇺🇸

Subject: 🚨 Bitcoin Core for MacOS - Possible Miner Infection 🚨

Dear Bitcoin Core developers,

I downloaded the Bitcoin Core software directly from the official website (bitcoincore.org) for macOS, and my antivirus detected a possible malware embedded in the program.

Avast identified the file as MacOS:Miner-AR [PUP], and after checking on VirusTotal, several other antivirus progra
...

🤔 hodlinator reviewed a pull request: "Add assumeutxo chainparams to release-process.md"
(https://github.com/bitcoin/bitcoin/pull/31940#pullrequestreview-2652580681)
Concept ACK

Should we also add a later step in the release process for the actual generation of the assumeutxo snapshot, maybe referencing *doc/assumeutxo.md*?

💬 hodlinator commented on pull request "Add assumeutxo chainparams to release-process.md":
(https://github.com/bitcoin/bitcoin/pull/31940#discussion_r1976492053)
Must be holding it wrong, but the `gettxoutsetinfo`-RPC isn't allowing me to specify both a `hash_type` and block height?

💬 willcl-ark commented on pull request "Add assumeutxo chainparams to release-process.md":
(https://github.com/bitcoin/bitcoin/pull/31940#issuecomment-2692422674)
We could do. Sjors has a tidy command in the linked PR.