💬 achow101 commented on pull request "wallet: Load database records in a particular order":
(https://github.com/bitcoin/bitcoin/pull/24914#discussion_r1183913027)
Done

🚀 fanquake merged a pull request: "ci: Use arm_container.dockerfile"
(https://github.com/bitcoin/bitcoin/pull/27562)

🤔 theStack reviewed a pull request: "wallet: Pass through transaction locktime and preset input sequences and scripts to CreateTransaction"
(https://github.com/bitcoin/bitcoin/pull/25273#pullrequestreview-1411318920)
Concept ACK

💬 theStack commented on pull request "wallet: Pass through transaction locktime and preset input sequences and scripts to CreateTransaction":
(https://github.com/bitcoin/bitcoin/pull/25273#discussion_r1183946012)
(in commit bf275f358a5e816cf9a8ab6e7dbd39f6dcdf80d7) nit: this could just be a one-liner using [std::optional<T>::value_or](https://en.cppreference.com/w/cpp/utility/optional/value_or)
```suggestion
return {m_script_sig.value_or(CScript{}), m_script_witness.value_or(CScriptWitness{})};
```

💬 theStack commented on pull request "wallet: Pass through transaction locktime and preset input sequences and scripts to CreateTransaction":
(https://github.com/bitcoin/bitcoin/pull/25273#discussion_r1183964527)
(in commit bf275f358a5e816cf9a8ab6e7dbd39f6dcdf80d7): nit: could avoid introducing temporary variables by directly assigning to script_sig/script_witness using std::tie
```suggestion
std::tie(script_sig, script_witness) = coin_control.GetScripts(coin->outpoint);
```

💬 Sjors commented on issue "iOS Deployment Target for RPC":
(https://github.com/bitcoin/bitcoin/issues/11720#issuecomment-1533428152)
To elaborate a bit: the idea is to have Rust (or even Swift??) bindings for the kernel. That's a much easier platform to build on than the full daemon, especially for iOs.

💬 achow101 commented on pull request "wallet: fix deadlock in bdb read write operation":
(https://github.com/bitcoin/bitcoin/pull/27556#issuecomment-1533474401)
ACK de202f860b3f79774d5235dec59a140fa5c82fa7

💬 achow101 commented on pull request "Implement Mini version of BlockAssembler to calculate mining scores":
(https://github.com/bitcoin/bitcoin/pull/27021#issuecomment-1533510937)
ACK 6b605b91c1faf2c7f7cc0c9d39b4fcfd66dc2965

💬 mzumsande commented on issue "Coinstats index corrupted after invalidateblock and clean shutdown":
(https://github.com/bitcoin/bitcoin/issues/27558#issuecomment-1533520155)
I haven't looked into the root cause yet, but some quick tests seem to suggest that this could be fixed by #25193 - at least I could reproduce it with master, but not with that branch.

💬 achow101 commented on pull request "wallet: Filter-out "send" addresses from `listreceivedby*`":
(https://github.com/bitcoin/bitcoin/pull/25973#issuecomment-1533521273)
> Why do we need to cross-reference with the address book if we already traversed the whole wallet's transactions outputs and filtered-out all the ones that are not belonging to the wallet?

To lookup the label.

***

There is a silent merge conflict with master.

💬 kevkevinpal commented on pull request "mempool / rpc: add getprioritisationmap, delete a mapDeltas entry when delta==0":
(https://github.com/bitcoin/bitcoin/pull/27501#issuecomment-1533528440)
Concept ACK

⚠️ Sataur opened an issue: "coreTORwin64.zip"
(https://github.com/bitcoin/bitcoin/issues/27567)
### Please describe the feature you'd like to see added.

There should be additional/optional bitcoin-24.0.1-[win](https://bitcoincore.org/en/download/)64.zip with TOR

### Is your feature related to a problem, if so please describe it.

Some users may be confused by that browser:
_So, you are trying to use the bundled Tor from Tor [Browser](https://twitter.com/spectre135/status/1284452766186909700) rather than a dedicated Tor daemon? This is not recommended._



### Describe the solution yo
...

💬 pinheadmz commented on issue "coreTORwin64.zip":
(https://github.com/bitcoin/bitcoin/issues/27567#issuecomment-1533560729)
This is unlikely to be fixed in Bitcoin Core, users can install and update Tor on their own and that is outside the responsibility of this project.

✅ pinheadmz closed an issue: "coreTORwin64.zip"
(https://github.com/bitcoin/bitcoin/issues/27567)

💬 pinheadmz commented on pull request "net: call getaddrinfo() in detachable thread to prevent stalling":
(https://github.com/bitcoin/bitcoin/pull/27557#issuecomment-1533568647)
Still waiting for CI to finish but I removed the need for internet access by looking up the name `"localhost"` instead. That is a non-numeric lookup that `getaddrinfo()` can do without the actual DNS. I think I also fixed the ipv6 test issue.

I'm interested in following the worker pool idea to make these requests async as opposed to new threads.

💬 brunoerg commented on pull request "rpc, p2p: allow `disconnectnode` with subnet":
(https://github.com/bitcoin/bitcoin/pull/26576#issuecomment-1533573510)
@josibake thank you, I just updated the description.

> so why not make the RPC expose the full functionality? I think I'd be more annoyed if I saw that DisconnectNode supported subnet in the code, but the RPC wouldn't let me use it.

> I think disconnecting from all ports on the same IP seems like a compelling enough use case for a very small change.

Yea, I only opened this one because the change would be really small, and since `DisconnectNode` supports subnet, it might worth.

💬 hebasto commented on pull request "build: Add CMake-based build system":
(https://github.com/bitcoin/bitcoin/pull/25797#issuecomment-1533601248)
Updated on top of the recent https://github.com/hebasto/bitcoin/pull/15.

CI [build](https://cirrus-ci.com/build/5395835672854528) is :green_circle:

Guix builds:
```
040eb5ee40cf57dcfe34caf8598cd856ac8585565c74386905feccbeb1868830 guix-build-7cc0a620fd74/output/aarch64-linux-gnu/SHA256SUMS.part
bb542a379b8c8ff54c645b4260ddf26265efcd0fc204fa09ce5eaf201f30ddb7 guix-build-7cc0a620fd74/output/aarch64-linux-gnu/bitcoin-7cc0a620fd74-aarch64-linux-gnu-debug.tar.gz
c8a307a90343c8553ff98e1720
...

💬 pinheadmz commented on issue "Add RPC to set transaction comment":
(https://github.com/bitcoin/bitcoin/issues/12594#issuecomment-1533608333)
@GSPP `txid` by definition is a unique identifier for a transaction. Comments can be added to new transactions in some RPCs like `sendtoaddress` and those comments are visible in the GUI. What do you need? The ability to edit a transaction comment in the GUI?

👍 furszy approved a pull request: "Introduce `MockableDatabase` for wallet unit tests"
(https://github.com/bitcoin/bitcoin/pull/26715#pullrequestreview-1411697889)
ACK 33e2b82

💬 furszy commented on pull request "wallet: Pass through transaction locktime and preset input sequences and scripts to CreateTransaction":
(https://github.com/bitcoin/bitcoin/pull/25273#discussion_r1181279440)
tiny nit:
just because I'm using this in another PR:
```c++
change_pos == -1 ? std::nullopt : std::make_optional(change_pos)
```