💬 ajtowns commented on pull request "refactor / kernel: Move non-gArgs chainparams functionality to kernel":
(https://github.com/bitcoin/bitcoin/pull/26177#discussion_r1108469843)
Seems more ergonomic to write:

```c++
CChainParams::SigNetOptions ReadSigNetArgs(const ArgsManager& args)
{
...
```

but maybe this is just deliberately keeping with the same pattern as per https://github.com/bitcoin/bitcoin/pull/26883/files#r1068247937

💬 ajtowns commented on pull request "refactor / kernel: Move non-gArgs chainparams functionality to kernel":
(https://github.com/bitcoin/bitcoin/pull/26177#discussion_r1108499523)
Rename to `GetBuriedDeployment()` ? This function should be part of deploymentinfo.cpp, shouldn't it?

💬 ajtowns commented on pull request "refactor / kernel: Move non-gArgs chainparams functionality to kernel":
(https://github.com/bitcoin/bitcoin/pull/26177#discussion_r1108500764)
`name` is really `deployment_id` (`dep` for short)?

💬 ajtowns commented on pull request "refactor / kernel: Move non-gArgs chainparams functionality to kernel":
(https://github.com/bitcoin/bitcoin/pull/26177#discussion_r1108508687)
Using optionals for signet but not regtest makes sense to me.

🚀 MarcoFalke merged a pull request: "test: previous releases: add v24.0.1"
(https://github.com/bitcoin/bitcoin/pull/26586)

🚀 MarcoFalke merged a pull request: "doc: remove mention of "proper signing key""
(https://github.com/bitcoin/bitcoin/pull/27107)

🚀 MarcoFalke merged a pull request: "test: add coverage for unparsable `-maxuploadtarget`"
(https://github.com/bitcoin/bitcoin/pull/26714)

⚠️ yanagawak opened an issue: "Why is Bitcoin Core structured as a wallet? I know that it began as a wallet, but is it possible to make it like a GUI block explorer instead?"
(https://github.com/bitcoin/bitcoin/issues/27110)
I'm always frustrated when I open up Bitcoin Core, and then all I get is an empty wallet GUI. I see no information about the block-chain itself. When I want to see on-chain data, I have to go to centralized websites instead.

I would like to see Bitcoin Core be restructured as a GUI block explorer. Delete all the wallet features. Bitcoin Core is a really, really bad wallet, to be honest. Didn't Core developer Luke Dash Junior just lose his entire stash of 300 coins because he was using a inter
...

✅ fanquake closed an issue: "Why is Bitcoin Core structured as a wallet? I know that it began as a wallet, but is it possible to make it like a GUI block explorer instead?"
(https://github.com/bitcoin/bitcoin/issues/27110)

💬 MarcoFalke commented on issue "Why is Bitcoin Core structured as a wallet? I know that it began as a wallet, but is it possible to make it like a GUI block explorer instead?":
(https://github.com/bitcoin/bitcoin/issues/27110#issuecomment-1433194111)
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.

🚀 fanquake merged a pull request: "doc: FreeBSD build doc updates to reflect removal of install_db4.sh"
(https://github.com/bitcoin/bitcoin/pull/26773)

💬 yancyribbens commented on pull request "refactor: Move coin_control variable to test setup section":
(https://github.com/bitcoin/bitcoin/pull/26154#issuecomment-1433228883)
@achow101

> Yes, it could be a problem if the test is removed, but it's unlikely that the test will be removed, and if it is removed, the line can be just as easily moved at that time.

I did not mean to imply that a test might be removed. I was only pointing out that this kind of coding practice is confusing because the variable definition doesn't belong _only_ to the test.

> While this isn't incorrect, it's not meaningfully helpful. It doesn't really add anything useful to the codeba
...

💬 TheCharlatan commented on pull request "refactor / kernel: Move non-gArgs chainparams functionality to kernel":
(https://github.com/bitcoin/bitcoin/pull/26177#discussion_r1108602508)
Yes, also keeping it the same as in: https://github.com/bitcoin/bitcoin/pull/25862/files#diff-4e268aeb074a176689aace31957a889c1f39c909e6678ca7be8707bda1a7be56R11

💬 instagibbs commented on pull request "[22.x] Backports":
(https://github.com/bitcoin/bitcoin/pull/26927#issuecomment-1433230124)
ACK https://github.com/bitcoin/bitcoin/pull/26927/commits/ea584a617c6853eb1f9740600cd9db75d77948eb

💬 prusnak commented on pull request "doc: remove mention of "proper signing key"":
(https://github.com/bitcoin/bitcoin/pull/27107#issuecomment-1433230441)
ACK 304ae6d

💬 TheCharlatan commented on pull request "refactor / kernel: Move non-gArgs chainparams functionality to kernel":
(https://github.com/bitcoin/bitcoin/pull/26177#discussion_r1108608610)
I see, I'll use `dep`, because that's what is used elsewhere in the code.

💬 Willtech commented on issue "Setting `onlynet=onion` still makes some IPv4 connections.":
(https://github.com/bitcoin/bitcoin/issues/12344#issuecomment-1433251695)
I only sometimes look just for interest and occasionally find this issue unless network has been updated. It is not rare it just happens when it does.

💬 fanquake commented on pull request "test/BIP324: functional tests for v2 P2P encryption":
(https://github.com/bitcoin/bitcoin/pull/24748#issuecomment-1433257883)
Moved to draft given it's based on multiple other PRs.

📝 fanquake converted_to_draft a pull request: "test/BIP324: functional tests for v2 P2P encryption"
(https://github.com/bitcoin/bitcoin/pull/24748)
This PR introduces support for v2 P2P encryption(BIP 324) in the existing functional test framework and adds functional tests for the same.

It's built on top of:
* #24545
* #24005

The first 2 commits help the CI pass since the secp256k1 subtree directory is touched by the parent PR 24545.
The next 2 commits bring in the mentioned parent PRs. They'll all be removed when the parent PRs get merged.

### commits overview
1. Introducing cryptographic constructs(HKDF, ECDH, ChaCha20, Poly
...

💬 instagibbs commented on pull request "[23.x] Backports":
(https://github.com/bitcoin/bitcoin/pull/26921#issuecomment-1433258574)
Only extra commit/change is af862661654966d5de614755ab9bd1b5913e0959

ACK https://github.com/bitcoin/bitcoin/pull/26921/commits/52376d9217060ce84e992e374d5dc2beae40bb06