💬 l0rinc commented on pull request "doc: add missing copyright headers":
(https://github.com/bitcoin/bitcoin/pull/31864#discussion_r1986151160)
Is there any reason for not updating these as well?
```suggestion
// file COPYING or https://opensource.org/license/MIT.
```

💬 l0rinc commented on pull request "doc: add missing copyright headers":
(https://github.com/bitcoin/bitcoin/pull/31864#discussion_r1986151662)
I see you've removed the `2024-present` from the latest push - is that a general direction from now on?

📝 hodlinator opened a pull request: "qa: Enable feature_init.py on Windows"
(https://github.com/bitcoin/bitcoin/pull/32021)
Windows has been skipped since feature_init.py was added in #23289. Possibly due to poorer support on older Python versions, or attempts to use `CTRL_C_EVENT` (which didn't work in my testing either) instead of `CTRL_BREAK_EVENT`.

💬 hodlinator commented on pull request "qa: Fix TxIndex race conditions":
(https://github.com/bitcoin/bitcoin/pull/32010#discussion_r1986159680)
As this thread prompted me to understand the beginning of the test better, I realized I could take a stab at making it work on Windows, and found a way: #32021.

💬 fjahr commented on pull request "qa: Fix TxIndex race conditions":
(https://github.com/bitcoin/bitcoin/pull/32010#discussion_r1986177762)
Right, I'm not passionate about the naming here. `sync_blocks` is syncing between two blockchains, `sync_txindex` is syncing between a blockchain and an index, so it's not wrong IMO. But `wait_for_txindex()` sounds fine, too. So up to you what you want to choose and if you want to change this at all.

💬 maflcko commented on pull request "rpc: Support v3 raw transactions creation":
(https://github.com/bitcoin/bitcoin/pull/31936#discussion_r1986292978)
not sure about default args. It would be better to explicitly enumerate the call sites and just put the default there, if needed.

💬 TheCharlatan commented on pull request "refactor: Improve assumeutxo state representation":
(https://github.com/bitcoin/bitcoin/pull/30214#discussion_r1986316731)
I've always been a bit puzzled of this check. Is it really required? If so, why don't we need to check it on every iteration of the do loop further down? What if it is actually reached, will validation on that chain just always be stuck in an unrecoverable state? Wouldn't a fatal error be more appropriate then?

💬 hebasto commented on issue "build: ccache doesn't hit across build dirs":
(https://github.com/bitcoin/bitcoin/issues/31994#issuecomment-2708846512)
## TL;DR

To enable ccache hits across different build directories on Linux or macOS, configure the ccache options as follows:
```
base_dir = <path_to_your_home_directory>
hash_dir = false
```
or via environment variables:
```
$ export CCACHE_BASEDIR=$HOME
$ export CCACHE_NOHASHDIR=1
```

## Detailed Solution

All following commands were executed on Ubuntu 24.04.2 LTS with [ccache 4.9.1](https://packages.ubuntu.com/noble/ccache) installed.

Consider building in two separate build directories wit
...

💬 hebasto commented on pull request "build: Enhance Ccache performance across worktrees and build trees":
(https://github.com/bitcoin/bitcoin/pull/30861#issuecomment-2708847639)
> > I'm wondering in which scenarios a user might wish to disable the `-fdebug-prefix-map` flag?
>
> I don't think there are any. Just mentioned this because `-fdebug-prefix-map` seems to be added conditionally in our build.

Specifically, which condition are you referring to?

💬 hebasto commented on issue "build: ccache doesn't hit across build dirs":
(https://github.com/bitcoin/bitcoin/issues/31994#issuecomment-2708848882)
Amending our docs, as demonstrated in https://github.com/davidgumberg/bitcoin/commit/1c6ae1043d59d01cbf52f353e50a63b0afa883c4, could be an alternative to https://github.com/bitcoin/bitcoin/pull/30861.

💬 maflcko commented on issue "Revisiting us self-hosting parts of our CI":
(https://github.com/bitcoin/bitcoin/issues/31965#issuecomment-2708856306)
Thanks for the survey and summary, as well as the additional input on this topic!

I don't have a strong opinion on this, and anything is probably fine, as long as it "works".

Just some random thoughts:

* The GHA runners that are included for public repositories are likely too weak to run all of the CI tasks in the maximum timeout (apart from the cache being too small as well, as mentioned above). So the goal that more of the CI tasks can run on forks trivially may not be achievable.
* The 32-
...

💬 dongwook-chan commented on pull request "rpc: Support v3 raw transactions creation":
(https://github.com/bitcoin/bitcoin/pull/31936#discussion_r1986324475)
@rkrux
Utilized existing `raw_multisig_transaction_legacy_tests` but with version 3 raw transaction. This seems to cover all the domain you specified. Could you pls take a look?
https://github.com/bitcoin/bitcoin/blob/1121cb7a91e0dfa4f247273bf89e2e1b47c82628/test/functional/rpc_rawtransaction.py#L96

💬 dongwook-chan commented on pull request "rpc: Support v3 raw transactions creation":
(https://github.com/bitcoin/bitcoin/pull/31936#discussion_r1986325419)
Added variable to denote min, max value of the version.
https://github.com/bitcoin/bitcoin/blob/1121cb7a91e0dfa4f247273bf89e2e1b47c82628/src/rpc/rawtransaction_util.cpp#L159-L160

💬 dongwook-chan commented on pull request "rpc: Support v3 raw transactions creation":
(https://github.com/bitcoin/bitcoin/pull/31936#discussion_r1986325898)
@glozow
Are you implying that we should use the variable instead of constant values in the test code?

💬 dongwook-chan commented on pull request "rpc: Support v3 raw transactions creation":
(https://github.com/bitcoin/bitcoin/pull/31936#discussion_r1986326221)
@luke-jr @adamandrews1
So should I remove the line? If I should, could you advise which code should be updated to avoid failure of the aforementioned validation?

💬 maflcko commented on pull request "qa: Enable feature_init.py on Windows":
(https://github.com/bitcoin/bitcoin/pull/32021#issuecomment-2708870591)
lgtm ACK 59c4930394cafc939eb396224b3d60d01ba0ce37

💬 maflcko commented on pull request "Docs: fix typos in documentation files":
(https://github.com/bitcoin/bitcoin/pull/32011#issuecomment-2708875507)
lgtm ACK 5601bab4f8b01fdef7a54c9e397d513217ab1c1f

💬 maflcko commented on pull request "random: Initialize variables in hardware RNG functions":
(https://github.com/bitcoin/bitcoin/pull/31912#issuecomment-2708878173)
The pull request title and commit message looks off. Also there seems to be an ongoing discussion in https://github.com/bitcoin/bitcoin/pull/31908#issuecomment-2687917758

📝 glozow converted_to_draft a pull request: "p2p: improve TxOrphanage denial of service bounds and increase -maxorphantxs"
(https://github.com/bitcoin/bitcoin/pull/31829)
This PR is part of the orphan resolution project, see #27463.

We want to limit the CPU work and memory used by `TxOrphanage` to avoid denial of service attacks. On master, this is achieved by limiting the number of transactions in this data structure to 100, and the weight of each transaction to 400KWu (the largest standard tx) [0]. We always allow new orphans, but if the addition causes us to exceed 100, we evict one randomly. This is dead simple, but has problems:
- It makes the orphanage
...

💬 fanquake commented on pull request "cmake: Check for `makensis` tool before using it":
(https://github.com/bitcoin/bitcoin/pull/32019#issuecomment-2708916206)
Now that `nsis` is required, the docs about installing `nsis` in `build-windows.md` should be updated.