📝 fanquake locked a pull request: "doc: Correct capitalization in release-process.md"
(https://github.com/bitcoin/bitcoin/pull/31971)
This PR corrects the capitalization of "if necessary" in the `doc/release-process.md` file.

The original sentence had "If necessary," with a capital "I". This has been corrected to "if necessary," with a lowercase "i" to maintain consistency.

This change improves the formatting and consistency of the documentation within the release process documentation file.

📝 fanquake locked a pull request: "."
(https://github.com/bitcoin/bitcoin/pull/31934)

💬 romanz commented on pull request "contrib: add tool to convert compact-serialized UTXO set to SQLite database":
(https://github.com/bitcoin/bitcoin/pull/27432#issuecomment-2692696000)
In order to reduce conversion time, I have rewritten the above tool in Rust :)
https://github.com/romanz/utxo-to-parquet

It converts the serialized UTXO set into a Parquet file, allowing efficient querying.
For example, retrieving https://mempool.space/address/1BitcoinEaterAddressDontSendf59kuE unspent coins using DuckDB takes <1s :)
```sql
D SELECT txid, vout, amount, height FROM '/home/user/tmp/mainnet-886001-utxo.parquet'
WHERE script = from_hex('76a914759d6677091e973b9e9d99f19c68fb
...

💬 RomainGratier commented on issue "Bitcoin Core MacOS - Possible Miner Infection - Bitcoin Core MacOS - Possível Infecção por Minerador":
(https://github.com/bitcoin/bitcoin/issues/31970#issuecomment-2692744621)
+1 on the last comment, and if I can give some context around security discussions around Bitcoin core (and malware detection), you can take a look at this thread: https://bitcoin.stackexchange.com/questions/98797/when-i-try-to-download-bitcoin-core-on-macos-it-says-apple-cannot-check-it-for

In any case, if you’re ever in doubt, always download from the official site and verify the digital signatures provided. Guessing that you trust Bitcoin Core source code.

🤔 Prabhat1308 reviewed a pull request: "qa: clarify and document assumeutxo tests"
(https://github.com/bitcoin/bitcoin/pull/31907#pullrequestreview-2652739998)
tACK [`dedc2d9`](https://github.com/bitcoin/bitcoin/pull/31907/commits/dedc2d9cecccb866230c07112d00288237d12eaa)

- tested the amount with compression and varint encoding to match .
- On the case of implementing varint serialization and compression logic , it would be a great addition to bring in the logic to test_framework (or in any other PR , no preference to bring it in this one).

💬 Prabhat1308 commented on pull request "qa: clarify and document assumeutxo tests":
(https://github.com/bitcoin/bitcoin/pull/31907#discussion_r1976641187)
as the above comment has already asked , I can't seem to understand why `VARINT(0)` was added considering we have already exceeded the max amount in the first amount written itself.

💬 davidrobinsonau commented on pull request "Added rescan option for import descriptors":
(https://github.com/bitcoin/bitcoin/pull/31668#discussion_r1976743171)
What do you suggest as the return value for the function when never is passed? So that this can be easily checked against for rescan? it can't be 0 as "0 can be specified to scan the entire blockchain." -1, null?

static int64_t GetImportTimestamp(const UniValue& data, int64_t now)
```
{
if (data.exists("timestamp")) {
const UniValue& timestamp = data["timestamp"];
if (timestamp.isNum()) {
return timestamp.getInt<int64_t>();
} else if (timestamp.i
...

💬 Crypt-iQ commented on issue "p2p: lingering entries in `mapBlockSource`":
(https://github.com/bitcoin/bitcoin/issues/29410#issuecomment-2693020111)
I left some details out of both the original PR and this issue and I think the missing information is needed to come to a solution:
- Node A is currently at height `h` and receives a `BLOCK` message from an _honest_ peer B for a stale block at height `h`. There are a couple ways that a `BLOCK` message could be sent instead of a `CMPCTBLOCK` message. For example, the compact-block slots could all be taken by an attacker or maybe there was a [collision for short ids](https://github.com/bitcoin/bit
...

💬 rkrux commented on pull request "Add assumeutxo chainparams to release-process.md":
(https://github.com/bitcoin/bitcoin/pull/31940#discussion_r1977046435)
Yes, I see the same issue. This is as per the code: https://github.com/bitcoin/bitcoin/blob/master/src/rpc/blockchain.cpp#L1019-L1021

Following commands work for me instead:
```
bitcoincli gettxoutsetinfo
bitcoincli gettxoutsetinfo muhash <block-height>
bitcoincli gettxoutsetinfo none <block-height>
```

👍 rkrux approved a pull request: "Add mainnet assumeutxo param at height 880,000"
(https://github.com/bitcoin/bitcoin/pull/31969#pullrequestreview-2653545385)
Concept ACK 14f16748557faf57cf4b0f4c91c162592557434c

I have not tested this because I run a pruned node and my block's prune height is more than the 880k.

In the PR description, the last command should pass `true` instead.

> Once the snapshot is loaded:
> ```
> bitcoin-cli setnetworkactive true
> ```
----------

> I'll upload a torrent later.

Any idea why don't we upload snapshots in [bitcoincore.org/bin](https://bitcoincore.org/bin/) as well?

💬 rkrux commented on pull request "rpc: add cli examples, update docs":
(https://github.com/bitcoin/bitcoin/pull/31958#discussion_r1977187442)
I tend to agree, updated.

💬 rkrux commented on pull request "rpc: add cli examples, update docs":
(https://github.com/bitcoin/bitcoin/pull/31958#issuecomment-2693800784)
Updated example output of `walletcreatefundedpsbt` RPC:

```
Examples:

Create a PSBT with automatically picked inputs that sends 0.5 BTC to an address and has a fee rate of 2 sat/vB:
> bitcoin-cli walletcreatefundedpsbt "[]" "[{\"bc1q09vm5lfy0j5reeulh4x5752q25uqqvz34hufdl\":0.5}]" 0 "{\"add_inputs\":true,\"fee_rate\":2}"

Create the same PSBT as the above one instead using named arguments:
> bitcoin-cli -named walletcreatefundedpsbt outputs="[{\"bc1q09vm5lfy0j5reeulh4x5752q25uqqvz34huf
...

💬 mprenditore commented on pull request "Added rescan option for import descriptors":
(https://github.com/bitcoin/bitcoin/pull/31668#discussion_r1977231841)
@davidrobinsonau I think both could be good, but `-1` seems the best option here

💬 hebasto commented on pull request "ci: Test cross-built Windows executables on Windows natively":
(https://github.com/bitcoin/bitcoin/pull/31176#issuecomment-2694007427)
Rebased to refresh the CI.

💬 Sjors commented on pull request "Add assumeutxo chainparams to release-process.md":
(https://github.com/bitcoin/bitcoin/pull/31940#issuecomment-2694168087)
Concept ACK

Every major release seems plenty, same as assumevalid.

👋 saikiran57's pull request is ready for review: "Added rescan option for import descriptors"
(https://github.com/bitcoin/bitcoin/pull/31668)

💬 saikiran57 commented on pull request "Added rescan option for import descriptors":
(https://github.com/bitcoin/bitcoin/pull/31668#discussion_r1977417409)
Updated the code

💬 Sjors commented on pull request "Add mainnet assumeutxo param at height 880,000":
(https://github.com/bitcoin/bitcoin/pull/31969#issuecomment-2694189118)
Added torrent to the description. @achow101, if you agree we should host it, can you add it to bitcoincore.org and give me the link? I'll add it as a binary seed.

💬 laanwj commented on pull request "seeds: update makeseeds regex and DNS fixed seeds":
(https://github.com/bitcoin/bitcoin/pull/31960#discussion_r1977456821)
The regexp doesn't match the entire string until the end, so "0.19.0.1" and such already match. This change will only make it that "0.19.1" no longer matches.

💬 laanwj commented on pull request "seeds: update makeseeds regex and DNS fixed seeds":
(https://github.com/bitcoin/bitcoin/pull/31960#discussion_r1977459441)
We shouldn't have `|` at the end of the last clause, as this will make it match the empty string too (so effectively everything starting with `Satoshi:` matches).