📝 hebasto locked a pull request: "."
(https://github.com/bitcoin/bitcoin/pull/31831)
<!--
*** 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
-->

<!--
Please provide clear motivation for your patch and explain how it improves
Bitcoin Core user experience or Bitcoin Core developer experience
significantly:

* Any test improvements or new tests that improv
...

💬 hebasto commented on pull request "Updated MacOS icon to more closely fit Apple's design standards":
(https://github.com/bitcoin-core/gui/pull/852#issuecomment-2647219126)
> The .icns file was compiled using Apple's own "iconutil" cli program.

What are steps to reproduce the new `src/qt/res/icons/bitcoin.icns` file?

⚠️ embetrix opened an issue: "Wallets not automatically loaded"
(https://github.com/bitcoin/bitcoin/issues/31832)
### Is there an existing issue for this?

- [x] I have searched the existing issues

### Current behaviour

after starting `bitcoind` the wallets are not automatically loaded

### Expected behaviour

thee wallets should be automatically loaded

### Steps to reproduce

My bitcoin.conf
```
pid=/run/bitcoin/bitcoind.pid
nodebuglogfile=1
datadir=/var/bitcoin
loglevel=info
proxy=127.0.0.1:9050
testnet4=1

[testnet4]
walletdir=/var/wallet

server=1
rpcuser=satobox
rpcport=8332
rpcconnect=127.0.0.1
rp
...

💬 Sjors commented on pull request "multiprocess: Lock CapnpProtocol::m_loop with mutex":
(https://github.com/bitcoin/bitcoin/pull/31815#issuecomment-2647256950)
Thanks. I dropped this from #30975 and rebased on your latest #31741 which contains https://github.com/chaincodelabs/libmultiprocess/pull/159.

💬 maflcko commented on issue "Wallets not automatically loaded":
(https://github.com/bitcoin/bitcoin/issues/31832#issuecomment-2647313705)
If you want to have a wallet loaded on startup, you'll have to set `load_on_startup` to `True`.

To learn more about the `loadwallet` RPC, you can try `help loadwallet`.

💬 maflcko commented on issue "Wallets not automatically loaded":
(https://github.com/bitcoin/bitcoin/issues/31832#issuecomment-2647314964)
Usually the issue tracker is used to track technical issues related to the Bitcoin Core code base.

General bitcoin questions and/or support requests are best directed to the [Bitcoin StackExchange](https://bitcoin.stackexchange.com) or the `#bitcoin` IRC channel on Libera Chat, or one of the Bitcoin subreddits, or any other place that you feel is well suited.

👍 vasild approved a pull request: "multiprocess: Add libmultiprocess git subtree"
(https://github.com/bitcoin/bitcoin/pull/31741#pullrequestreview-2605030796)
ACK e7743aa5df6387964c23f0629debf8c3cbb8ed4c

💬 vasild commented on pull request "multiprocess: Add libmultiprocess git subtree":
(https://github.com/bitcoin/bitcoin/pull/31741#discussion_r1948609163)
nit: `$(1)_file_name)` -> `$(1)_file_name`

Thanks for the comment. That line is still ~!@*&~(!@*&~&_ in my eyes. I wonder if it can be split into shorter and more easily understandable `define foo` functions...

💬 vasild commented on pull request "multiprocess: Add libmultiprocess git subtree":
(https://github.com/bitcoin/bitcoin/pull/31741#discussion_r1948608064)
nit: ` | head -n1` is unnecessary

💬 willcl-ark commented on issue "Wallets not automatically loaded":
(https://github.com/bitcoin/bitcoin/issues/31832#issuecomment-2647327456)
In addition to @maflcko's suggestion, another way to explicitly load wallets at startup is to use the `-wallet=<path>` startup option in your configuration (multiple times in your case, for multiple wallets).

💬 maflcko commented on pull request "contrib: fix `test_deterministic_coverage.sh` script for out-of-tree builds":
(https://github.com/bitcoin/bitcoin/pull/31588#issuecomment-2647372194)
lgtm ACK 459683bcfc70b77b3dae2e70c88cf5c6a007442e

Only change is a small mktemp cleanup.

I still haven't tested this (I am using a modified script locally anyway), but the code changes look harmless and reasonable. As for the scope, I wonder how much effort should be spent on this script, given that:

* No one(?) is using it right now unmodified, or has been in the past years?
* It is written in bash, causing compat issues with the ancient version of bash shipped on macOS, as well as th
...

💬 vasild commented on pull request "Split CConnman":
(https://github.com/bitcoin/bitcoin/pull/30988#issuecomment-2647374260)
It is better to post such suggestions not in the main thread of the PR but as a comment to some line of code, even if that is a random, unrelated line. That way replies will be grouped together with the questions instead of being scattered around in the main PR thread. In the main thread it is easier to forget to reply to some questions. As comments to some line of code, they can be tracked and eventually "resolved" to collapse them and reduce the main PR thread noise.

> #### WinSock
>
> W
...

💬 vasild commented on pull request "Split CConnman":
(https://github.com/bitcoin/bitcoin/pull/30988#issuecomment-2647374943)
It is better to post such suggestions not in the main thread of the PR but as a comment to some line of code, even if that is a random, unrelated line. That way replies will be grouped together with the questions instead of being scattered around in the main PR thread. In the main thread it is easier to forget to reply to some questions. As comments to some line of code, they can be tracked and eventually "resolved" to collapse them and reduce the main PR thread noise.

> #### WinSock
>
> W
...

💬 Sjors commented on pull request "refactor: Use std::span over Span":
(https://github.com/bitcoin/bitcoin/pull/31519#issuecomment-2647388325)
Concept ACK. I'll need to rebase some things, but I'm favor of getting that over with.

💬 hebasto commented on pull request "multiprocess: Add libmultiprocess git subtree":
(https://github.com/bitcoin/bitcoin/pull/31741#discussion_r1948687873)
1b3613505deb4baeacc3c4c2fc91ccca1a8ecec5

Usually, external dependencies follow the internal ones.

💬 vasild commented on pull request "Split CConnman":
(https://github.com/bitcoin/bitcoin/pull/30988#discussion_r1948692321)
Done, will be in next push.

💬 hebasto commented on pull request "multiprocess: Add libmultiprocess git subtree":
(https://github.com/bitcoin/bitcoin/pull/31741#issuecomment-2647421895)
> > Could the `CAPNP_EXECUTABLE`, `CAPNPC_CXX_EXECUTABLE` and all `CapnProto_*` CMake variables be marked as advanced to avoid cluttering the CMake cache view when inspected by the user?
>
> Sure, done in latest push.

The `CapnProto_*` CMake variables are still lingering:
```
$ cmake -B build -DENABLE_IPC=ON -L 2>&1 | grep CapnProto_
CapnProto_capnp-json_IMPORTED_LOCATION:FILEPATH=/usr/lib/x86_64-linux-gnu/libcapnp-json-1.0.1.so
CapnProto_capnp-rpc_IMPORTED_LOCATION:FILEPATH=/usr/lib/x
...

💬 vasild commented on pull request "Split CConnman":
(https://github.com/bitcoin/bitcoin/pull/30988#discussion_r1948716391)
By default trying to pass `std::span<std::byte>` to a function that takes `std::span<uint8_t>` gives:

```
no known conversion from 'span<std::byte>' to 'span<uint8_t>'
```

If I really insist then I can convert it like:

```cpp
void f(std::span<uint8_t> s);
...
std::span<std::byte> a;
f(std::span<uint8_t>{reinterpret_cast<uint8_t*>(a.data()), a.size()});
```

I find it cleaner to use `uint8_t` because it is already used in the existent code (`ReceiveMsgBytes()`) and avoid such fo
...

💬 hebasto commented on pull request "depends: Use `CC_FOR_BUILD` for `config.guess `":
(https://github.com/bitcoin/bitcoin/pull/29963#issuecomment-2647448250)
@theuni

> > However, switching the CC variable context from Makefile to the shell environment breaks expectations
>
> Arguably that's because this wasn't intended to be supported :)
>
> I suppose this fix is reasonable, though supporting env vars like this seems quite brittle.

There are testing environments, such as OSS-Fuzz, that rely on a project's build system to support environment variables.

💬 hebasto commented on pull request "depends: Make default `host` and `build` comparable":
(https://github.com/bitcoin/bitcoin/pull/30584#issuecomment-2647452355)
Friendly ping @laanwj @josibake @theuni ;)