👍 willcl-ark approved a pull request: "[28.x] 28.2rc1"
(https://github.com/bitcoin/bitcoin/pull/32480#pullrequestreview-2844686716)
ACK 186e3f1fb65bb693154c0b63b690bea1d279f089

`git grep v28` doesn't show any un-bumped versions.

✅ pinheadmz closed an issue: "TSan warning with legacy wallet on macos ("too long mutex cycle found")"
(https://github.com/bitcoin/bitcoin/issues/31986)

💬 pinheadmz commented on issue "TSan warning with legacy wallet on macos ("too long mutex cycle found")":
(https://github.com/bitcoin/bitcoin/issues/31986#issuecomment-2884764494)
Fixed by removing legacy wallet https://github.com/bitcoin/bitcoin/pull/28710

👋 instagibbs's pull request is ready for review: "test: add MAX_DISCONNECTED_TX_POOL_BYTES coverage"
(https://github.com/bitcoin/bitcoin/pull/32516)

💬 pinheadmz commented on issue "Running out of memory on a 2GB box - Initializing chainstate Chainstate [ibd] @ height -1 (null)":
(https://github.com/bitcoin/bitcoin/issues/31573#issuecomment-2884794681)
Any updates on this issue? I'll also point out that 2 GB is the [bare minimum](https://bitcoin.org/en/full-node#minimum-requirements).

💬 hodlinator commented on pull request "qa: Verify clean shutdown on startup failure":
(https://github.com/bitcoin/bitcoin/pull/30660#issuecomment-2884807855)
@ryanofsky & @ismaelsadeeq some of your remaining comments are addressed and ready for review in follow-up PR #32509. (Happy to get review from others as well of course).

💬 pinheadmz commented on issue "getblockchaininfo `verificationprogress` never reaches 1.0":
(https://github.com/bitcoin/bitcoin/issues/31127#issuecomment-2884833770)
This is a FAQ but is always closed as not planned. In my opinion, `progress` is only useful the first day you start a node to get some idea how long it'll take to be useable. I believe `lnd` returns `synced: true` whenever the tip timestamp is within two hours of now. I don't think bitcoind should "round up" the progress to 1 for UX reasons, the user or api consumer should decide on their own how risky their use case is it has been more than 0 seconds since receiving a new block.

🤔 furszy reviewed a pull request: "wallet: init, don't error out when loading legacy wallets"
(https://github.com/bitcoin/bitcoin/pull/32449#pullrequestreview-2844762782)
Updated per feedback. Additional test coverage included.

💬 pinheadmz commented on issue "Distribute darknet node addresses via DNS seeds using AAAA records":
(https://github.com/bitcoin/bitcoin/issues/31062#issuecomment-2884851261)
Do we need to hack DNS records for this? We have [BIP155](https://github.com/bitcoin/bips/blob/master/bip-0155.mediawiki) addrv2 messages to relay onion peers, all you need is one bitcoin peer to collect those.

The hard coded seeds list also provides onion, CJDNS, and i2p addresses.

💬 jonatack commented on issue "getblockchaininfo `verificationprogress` never reaches 1.0":
(https://github.com/bitcoin/bitcoin/issues/31127#issuecomment-2884853355)
I think it's also useful when your node comes back online after a shutdown or an outage (I have -getinfo along with -netinfo running as live `watch` dashboards, along with the debug log).

> the first day you start a node

One day to sync 😍 last 3 times I synced a node, it took between 10 days and 5 weeks depending on internet bandwidth + reliability.

That said, I agree this pull wasn't the best possible approach.

💬 pablomartin4btc commented on pull request "wallet: init, don't error out when loading legacy wallets":
(https://github.com/bitcoin/bitcoin/pull/32449#discussion_r2091843907)
I think similar to this one "assert_start_raises_init_error"... no?

✅ pinheadmz closed an issue: "Blockchain fully synced, but `bitcoin-cli -getinfo` shows `verification progress: 99.9999%`"
(https://github.com/bitcoin/bitcoin/issues/28847)

💬 pinheadmz commented on issue "Blockchain fully synced, but `bitcoin-cli -getinfo` shows `verification progress: 99.9999%`":
(https://github.com/bitcoin/bitcoin/issues/28847#issuecomment-2884884698)
Closing as duplicate of #31127 also #30293 and many others

💬 pablomartin4btc commented on pull request "wallet: init, don't error out when loading legacy wallets":
(https://github.com/bitcoin/bitcoin/pull/32449#issuecomment-2884886063)
> Updated per feedback. Additional test coverage included.

Is it worth it to verify that, previous to this PR, the node with the legacy wallet in its `settings.json` was failing to restart (using `assert_start_raises_init_error`)?

💬 sipsorcery commented on pull request "cmake: Add application manifests when cross-compiling for Windows":
(https://github.com/bitcoin/bitcoin/pull/32396#issuecomment-2884887096)
re-tACK 8f4fed7ec70093e2535423d63e9f9dd400c378ac.

✅ pinheadmz closed an issue: "Bitcoin ignores datadir and blocksdir parameter in .conf"
(https://github.com/bitcoin/bitcoin/issues/27246)

💬 pinheadmz commented on issue "Bitcoin ignores datadir and blocksdir parameter in .conf":
(https://github.com/bitcoin/bitcoin/issues/27246#issuecomment-2884894957)
closed by #27302

💬 pinheadmz commented on issue "listunspent, fundrawtransaction, getwalletinfo locks wallet for any other operation":
(https://github.com/bitcoin/bitcoin/issues/27002#issuecomment-2884897418)
> Yes, this is it! Version of bitcoind from this [#26008](https://github.com/bitcoin/bitcoin/pull/26008) pull request works just fine:

Can this issue be closed?

💬 hodlinator commented on pull request "multiprocess: Add bitcoin wrapper executable":
(https://github.com/bitcoin/bitcoin/pull/31375#discussion_r2091860732)
Git approach is command + subcommand:
```
₿ git sdfsdf
git: 'sdfsdf' is not a git command. See 'git --help'.

₿ git remote sdfsdf
error: unknown subcommand: `sdfsdf'
```

💬 gituser commented on issue "listunspent, fundrawtransaction, getwalletinfo locks wallet for any other operation":
(https://github.com/bitcoin/bitcoin/issues/27002#issuecomment-2884901552)
@pinheadmz no, this is still an actual bug for me.

e.g. `fundrawtransaction` blocks the wallet for me if it executes for a long time and also blocks all other open wallets for that time.