🤔 promag reviewed a pull request: "qt6, test: Handle deprecated code"
(https://github.com/bitcoin-core/gui/pull/839#pullrequestreview-2346817911)
Code review ACK 5625840c11db2065a1c8d8de3babb6466eda04d4.

💬 JayBitron commented on issue "Unable to cross compile on linux for macos (28.x branch)":
(https://github.com/bitcoin/bitcoin/issues/31027#issuecomment-2392482422)
I'm using version 20 it returns `/usr/bin/ld: unrecognised emulation mode: llvm`
when I create symbolic links to version 16 it returns `/usr/bin/ld: unrecognised emulation mode: acosx_version_min`
It's is different error on each version

💬 JayBitron commented on issue "Unable to cross compile on linux for macos (28.x branch)":
(https://github.com/bitcoin/bitcoin/issues/31027#issuecomment-2392503162)
debian unstable (sid) run on clang-20 doesn't need to downgrade it

💬 JayBitron commented on issue "Unable to cross compile on linux for macos (28.x branch)":
(https://github.com/bitcoin/bitcoin/issues/31027#issuecomment-2392513451)
I just freshly installed a linux and went the steps as you said still getting this error:
```
root@localhost:~/.build/bitcoin# make -C depends HOST=x86_64-apple-darwin -j32
make: Entering directory '/root/.build/bitcoin/depends'
Fetching boost_1_81_0.tar.gz from https://archives.boost.io/release/1.81.0/source/
% Total % Received % Xferd Average Speed Time Time Time Current
Dload Upload Total Spent Left Speed
100 133M 100 133M
...

💬 JayBitron commented on issue "Unable to cross compile on linux for macos (28.x branch)":
(https://github.com/bitcoin/bitcoin/issues/31027#issuecomment-2392515334)
I know now bitcoin 28.x macos builds are broken officially, in 27.x all tools were built in, about 800MB down was taken down.

👍 tdb3 approved a pull request: "ci: set a ctest test timeout of 1200 (20 minutes)"
(https://github.com/bitcoin/bitcoin/pull/31026#pullrequestreview-2346849054)
ACK 93dda4c70254750a5b5c0e7f7d2d89032519b999

Looks like a good way to cut off abnormally running unit tests.
As a sanity check, confirmed that `ctest` returns non-zero when the timeout is hit (e.g. `ctest --timeout 10 --test-dir build`).

💬 JayBitron commented on issue "Unable to cross compile on linux for macos (28.x branch)":
(https://github.com/bitcoin/bitcoin/issues/31027#issuecomment-2392519954)
Yes, I see `native_llvm.mk` was removed from 28.x

⚠️ glozow pinned an issue: "Release Schedule for 29.0"
(https://github.com/bitcoin/bitcoin/issues/31029)
Here is a proposed release schedule for `v29.0`, the next major release of Bitcoin Core. The dates are set to target a release in early April.

## 2025-02-06 :construction:
- Open Transifex translations for `v29.0`
- Soft translation string freeze (no large or non-critical string changes until release)
- Finalize and close translations for `v27.0`

## 2025-02-20 :construction:
- Feature freeze (bug fixes only until release)
- Translation string freeze (no more source language changes un
...

🤔 theStack reviewed a pull request: "refactor: TxDownloadManager + fuzzing"
(https://github.com/bitcoin/bitcoin/pull/30110#pullrequestreview-2342669240)
Left some more review comments below; these are mostly pedantic nits that can be done in a follow-up, but also one (minor?) behavior change.

💬 theStack commented on pull request "refactor: TxDownloadManager + fuzzing":
(https://github.com/bitcoin/bitcoin/pull/30110#discussion_r1786893269)
never-used-nit
```suggestion
```

💬 theStack commented on pull request "refactor: TxDownloadManager + fuzzing":
(https://github.com/bitcoin/bitcoin/pull/30110#discussion_r1784925614)
```suggestion
for (NodeId nodeid = 0; nodeid < NUM_PEERS; ++nodeid) {
```

💬 theStack commented on pull request "refactor: TxDownloadManager + fuzzing":
(https://github.com/bitcoin/bitcoin/pull/30110#discussion_r1784927176)
```suggestion
for (NodeId nodeid = 0; nodeid < NUM_PEERS; ++nodeid) {
```

💬 theStack commented on pull request "refactor: TxDownloadManager + fuzzing":
(https://github.com/bitcoin/bitcoin/pull/30110#discussion_r1786893532)
never-used-nit
```suggestion
```

💬 theStack commented on pull request "refactor: TxDownloadManager + fuzzing":
(https://github.com/bitcoin/bitcoin/pull/30110#discussion_r1784926619)
```suggestion
Assert(package_to_validate.m_senders.back() < NUM_PEERS);
```

💬 theStack commented on pull request "refactor: TxDownloadManager + fuzzing":
(https://github.com/bitcoin/bitcoin/pull/30110#discussion_r1784339629)
for this fuzz test it probably doesn't matter if one more output than specified is added, but...
```suggestion
for (size_t o = 0; o < num_outputs; ++o) tx.vout.emplace_back(CENT, P2WSH_OP_TRUE);
```

💬 theStack commented on pull request "refactor: TxDownloadManager + fuzzing":
(https://github.com/bitcoin/bitcoin/pull/30110#discussion_r1786932357)
nit: that comment is duplicated on the call-site

💬 theStack commented on pull request "refactor: TxDownloadManager + fuzzing":
(https://github.com/bitcoin/bitcoin/pull/30110#discussion_r1786912728)
nit (these would be moved again in the later commit 4b8cb00fcb429630f717954b5f1c397ce54a8475 ["[refactor] make AlreadyHaveTx and Find1P1CPackage private to TxDownloadImpl"], so might be easier to just fix in a follow-up)
```suggestion
* - m_lazy_recent_rejects
* - m_lazy_recent_rejects_reconsiderable (if include_reconsiderable = true)
* - m_lazy_recent_confirmed_transactions
```

💬 theStack commented on pull request "refactor: TxDownloadManager + fuzzing":
(https://github.com/bitcoin/bitcoin/pull/30110#discussion_r1787010637)
Missed this on previous review rounds, but this commit seems to change behaviour, as the <100k memusage condition for txs to be added to the compact-block-extra-txn cache (originally introduced in #9499, commits https://github.com/bitcoin/bitcoin/pull/9499/commits/7f8c8cab1e537809848088d3bfaa13ecca7ac8cc / https://github.com/bitcoin/bitcoin/pull/9499/commits/863edb45b9841c3058e883015c7576e6f69e3cc6) is now also applied to orphan transactions.

📝 AgusR7 opened a pull request: "test: use context managers and add file existence checks in feature_fee_estimation.py"
(https://github.com/bitcoin/bitcoin/pull/31030)
update feature_fee_estimation.py test script to improve resource management and exception safety:

Use context managers for file operations: Replaced all instances of file handling using "open()" with "with open()" statements to ensure files are properly closed after their operations, even if exceptions occur.

Add file existence checks: Before performing file operations like os.utime() and os.remove(), added checks using os.path.exists() to prevent errors when files are missing.

📝 dollarparity opened a pull request: "doc: clarify 'filename' argument in 'loadwallet' RPC"
(https://github.com/bitcoin/bitcoin/pull/31031)
Update the 'loadwallet' RPC help text to specify that the 'filename' should be provided as a path relative to the wallets directory. Add examples showing how to load a wallet from a different directory.

<!--
*** 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
-->

<!--
Ple
...