✅ hebasto closed a pull request: "Intro: Never change the prune checkbox after the user has touched it"
(https://github.com/bitcoin-core/gui/pull/658)

💬 jamesob commented on pull request "p2p: Log addresses of stalling peers":
(https://github.com/bitcoin/bitcoin/pull/27761#issuecomment-1564603683)
github ACK https://github.com/bitcoin/bitcoin/commit/fb02a3cd1a105bdf60ca39e1858e77685be88976

🚀 fanquake merged a pull request: "p2p: Log addresses of stalling peers"
(https://github.com/bitcoin/bitcoin/pull/27761)

🤔 hebasto reviewed a pull request: "ci: Add missing set -e to 01_base_install.sh"
(https://github.com/bitcoin/bitcoin/pull/27739#pullrequestreview-1446580353)
> Otherwise errors are silently ignored

Should we do the same in `ci/test/00_setup_env.sh`?

https://cirrus-ci.com/task/5206141060251648:
```
./ci/test/00_setup_env.sh: line 33: /ci_base_install/depends/config.guess: No such file or directory
```

💬 hebasto commented on pull request "ci: Add missing set -e to 01_base_install.sh":
(https://github.com/bitcoin/bitcoin/pull/27739#discussion_r1207044005)
Mention `-x` option in the PR description?

🤔 brunoerg reviewed a pull request: "fuzz: Change LIMIT_TO_MESSAGE_TYPE from a compile-time to a run-time setting"
(https://github.com/bitcoin/bitcoin/pull/27766#pullrequestreview-1446589212)
Concept ACK

💬 brunoerg commented on pull request "p2p: cleanup `LookupIntern`, `Lookup` and `LookupHost`":
(https://github.com/bitcoin/bitcoin/pull/26261#discussion_r1207055971)
No, going to change it

💬 brunoerg commented on pull request "p2p: cleanup `LookupIntern`, `Lookup` and `LookupHost`":
(https://github.com/bitcoin/bitcoin/pull/26261#issuecomment-1564654141)
@stickies-v I'm addressing it at this moment, will push soon.

💬 hebasto commented on pull request "ci: Add missing set -e to 01_base_install.sh":
(https://github.com/bitcoin/bitcoin/pull/27739#discussion_r1207074805)
`FILE_ENV="./ci/test/00_setup_env_mac_native_arm64.sh" ./ci/test_run_all.sh` fails locally for me now with the following error:
```
ERROR: invalid tag "": repository name must have at least one component
```

Also https://github.com/bitcoin/bitcoin/blob/fa0d36ffb5c8348598998eeb3ea6a3ee93bd6350/ci/test/04_install.sh#L37

💬 brunoerg commented on pull request "p2p: cleanup `LookupIntern`, `Lookup` and `LookupHost`":
(https://github.com/bitcoin/bitcoin/pull/26261#discussion_r1207080551)
The alternative to keep `BOOST_REQUIRE` is creating one more variable but I don't think it's worth, would have to do it in many places. Using `Assert` could also be an alternative, is there any harm of using it? Perhaps import `util/check` just for it?

💬 brunoerg commented on pull request "p2p: cleanup `LookupIntern`, `Lookup` and `LookupHost`":
(https://github.com/bitcoin/bitcoin/pull/26261#discussion_r1207083634)
Since it's a non-blocker I'm pushing addressing all lastest nits, except this one, to not delay and cause futher rebases conflicts.

💬 brunoerg commented on pull request "p2p: cleanup `LookupIntern`, `Lookup` and `LookupHost`":
(https://github.com/bitcoin/bitcoin/pull/26261#issuecomment-1564679459)
Force-pushed addressing:

- https://github.com/bitcoin/bitcoin/pull/26261#discussion_r1201199915
- https://github.com/bitcoin/bitcoin/pull/26261#discussion_r1177587552

Maybe done with nits, and we can have it merged?

👍 stickies-v approved a pull request: "p2p: cleanup `LookupIntern`, `Lookup` and `LookupHost`"
(https://github.com/bitcoin/bitcoin/pull/26261#pullrequestreview-1446643228)
re-ACK 5c832c3820253affc65c0ed490e26e5b0a4d5c9b - just addressing two nits, no other changes

<details>
<summary><code>git range-diff 8b59231641845f71df37e163bf5b8157fb197d05 4eee95e57bb6f773bcaeb405bca949f158a62134 5c832c3820253affc65c0ed490e26e5b0a4d5c9b</code></summary>

```
1: c0cf3d70e = 1: 5c1774a56 p2p, refactor: return `std::vector<CNetAddr>` in `LookupIntern`
2: 94b5edbf6 = 2: 7799eb125 p2p, refactor: return `std::vector<CNetAddr>` in `LookupHost`
3: 617bc481e ! 3: 34bcdfc
...

📝 mmienko opened a pull request: "update osx build docs"
(https://github.com/bitcoin/bitcoin/pull/27769)
- Add Jetbrains IDE dir to gitignore
- Document how to verify Homebrew during OSX build setup

While the project builds for osx M2, developers might not have envrionments properly setup (I didn't becuase I think I installed Homebrew from a guide rather than official site).
Hopefully improved documentation can mitigate future mishaps. Inspired by https://github.com/bitcoin/bitcoin/issues/25556.

💬 glozow commented on pull request "wallet: when a block is disconnected, update transactions that are no longer conflicted":
(https://github.com/bitcoin/bitcoin/pull/27145#issuecomment-1564716873)
> I've fixed a bug which resulted in the conflicted tx not being marked as inactive if the conflicting tx was not in mapTxSpends.

> If mapTxSpends.count(outpoint) == 1, the outpoint was spent only once, so the wallet shouldn't contain any conflicting tx.

I interpreted the bug as = we wouldn't mark a conflicted tx inactive if the block transaction was not in `mapTxSpends` (which would mean count == 1). Normally we'd expect them both to be there (count > 1).

💬 hebasto commented on pull request "update osx build docs":
(https://github.com/bitcoin/bitcoin/pull/27769#discussion_r1207132244)
This should be done in developer's personal environment, not in the repo.

💬 mmienko commented on pull request "update osx build docs":
(https://github.com/bitcoin/bitcoin/pull/27769#discussion_r1207134303)
Ah yeah, there is a global gitignore. Thanks, will revert.

💬 glozow commented on pull request "p2p: Stop relaying non-mempool txs":
(https://github.com/bitcoin/bitcoin/pull/27625#issuecomment-1564726392)
concept ACK

🤔 glozow reviewed a pull request: "wallet: when a block is disconnected, update transactions that are no longer conflicted"
(https://github.com/bitcoin/bitcoin/pull/27145#pullrequestreview-1446696893)
ACK 89df7987c2f1eea42454c2b0efc31a924fbfd3a8

💬 kevkevinpal commented on pull request "rpc: remove deprecated "warning" field from {create,load,restore,unload}wallet":
(https://github.com/bitcoin/bitcoin/pull/27757#issuecomment-1564752809)
ACK [d78f9a7](https://github.com/bitcoin/bitcoin/pull/27757/commits/d78f9a7ed7acf5e2610f0b108123bae1a376f819)

compiled and ran the code below and got the following response
```
--> ./src/bitcoin-cli -regtest -named createwallet wallet_name=oops2 descriptors=false passphrase=""
{
"name": "oops2",
"warnings": [
"Empty string given as passphrase, wallet will not be encrypted.",
"Wallet created successfully. The legacy wallet type is being deprecated and support for creating an
...