💬 pythcoiner commented on pull request "miniscript: account for all `StringType` variants in `Miniscriptdescriptor::ToString()`":
(https://github.com/bitcoin/bitcoin/pull/31734#discussion_r1934941378)
done

💬 pythcoiner commented on pull request "miniscript: account for all `StringType` variants in `Miniscriptdescriptor::ToString()`":
(https://github.com/bitcoin/bitcoin/pull/31734#discussion_r1934941489)
done

💬 pythcoiner commented on pull request "miniscript: account for all `StringType` variants in `Miniscriptdescriptor::ToString()`":
(https://github.com/bitcoin/bitcoin/pull/31734#discussion_r1934941602)
done

💬 pythcoiner commented on pull request "miniscript: account for all `StringType` variants in `Miniscriptdescriptor::ToString()`":
(https://github.com/bitcoin/bitcoin/pull/31734#issuecomment-2623406968)
Thanks for the review @hodlinator! comments addressed & added more details to the summary

💬 ryanofsky commented on pull request "multiprocess: Add bitcoin wrapper executable":
(https://github.com/bitcoin/bitcoin/pull/31375#issuecomment-2623415052)
Rebased 5e0196c8e0d1687bcf59a9569bb8b7268d1f195d -> 0e4ee158cadc3eb8f6af1b33440ae9a95fe19487 ([`pr/wrap.17`](https://github.com/ryanofsky/bitcoin/commits/pr/wrap.17) -> [`pr/wrap.18`](https://github.com/ryanofsky/bitcoin/commits/pr/wrap.18), [compare](https://github.com/ryanofsky/bitcoin/compare/pr/wrap.17-rebase..pr/wrap.18)) to fix conflict

💬 ryanofsky commented on pull request "cmake: Move internal binaries from bin/ to libexec/":
(https://github.com/bitcoin/bitcoin/pull/31679#issuecomment-2623417011)
Updated 2a952710661a1990ed1c24b0ebd16de8ac0df87c -> 47a872236e070814ad74922d3f8a653e1c6af968 ([`pr/libexec.1`](https://github.com/ryanofsky/bitcoin/commits/pr/libexec.1) -> [`pr/libexec.2`](https://github.com/ryanofsky/bitcoin/commits/pr/libexec.2), [compare](https://github.com/ryanofsky/bitcoin/compare/pr/libexec.1..pr/libexec.2)) to fix CI error

💬 Eunovo commented on pull request "Benchmark Chainstate::ConnectBlock duration":
(https://github.com/bitcoin/bitcoin/pull/31689#discussion_r1934978868)
> so why not create the `CTxOut` vector directly when we are creating the `scriptpubkeys` vector and then pass the `CTxOut` vector in as an argument?

I updated the PR to do that.

> I also think a comment explicitly mentioning we are creating an same number input, same number output transaction would be worth adding.

I also added a comment stating this

💬 Eunovo commented on pull request "Benchmark Chainstate::ConnectBlock duration":
(https://github.com/bitcoin/bitcoin/pull/31689#issuecomment-2623477551)
I've updated the code to take a reference to the underlying test_setup object following @TheCharlatan and @josibake reviews. I've also added more comments explaining how the transactions in the test block are created.

💬 Eunovo commented on pull request "Benchmark Chainstate::ConnectBlock duration":
(https://github.com/bitcoin/bitcoin/pull/31689#issuecomment-2623504472)
> My first guess is the extra time is coming from hashing the transaction multiple times for the different signature digest algorithms? Just a guess; will need to dig into the code more to understand.

I'm pretty sure this is the case after looking in `interpreter.cpp`. I added a comment explaining this

👍 i-am-yuvi approved a pull request: "test: fixes p2p_ibd_txrelay wait time"
(https://github.com/bitcoin/bitcoin/pull/31759#pullrequestreview-2582919327)
ACK 1973a9e4f1dfba57051135d6e1bca80979de3879

Good catch!

```
p2p_ibd_txrelay.py | ✓ Passed | 2 s

ALL | ✓ Passed | 2 s (accumulated)
Runtime: 2 s

```

👍 i-am-yuvi approved a pull request: "test: deduplicates p2p_tx_download constants"
(https://github.com/bitcoin/bitcoin/pull/31758#pullrequestreview-2582964859)
ACK efeaf2e0c1f0de0d7c3819eb9beaefee2ddf0710

```
TEST | STATUS | DURATION

p2p_tx_download.py | ✓ Passed | 33 s

ALL | ✓ Passed | 33 s (accumulated)
```

👍 i-am-yuvi approved a pull request: "test: make sure we are on sync with a peer before checking if they have sent a message"
(https://github.com/bitcoin/bitcoin/pull/31760#pullrequestreview-2583021496)
ACK 3f4b104b1b7e1b87c0be8e395e02b6ae3c5d7b08

💬 Sjors commented on pull request "rpc: have getblocktemplate mintime account for timewarp":
(https://github.com/bitcoin/bitcoin/pull/31600#issuecomment-2623772947)
@fjahr the new rule only applies to blocks at `height % 2016 == 0`, so I'm confused about the heights you found.

The change here only affects what miners put in their blocks, it doesn't change the consensus check. So it shouldn't be dangerous in any case.

💬 Sjors commented on pull request "ci: build multiprocess on most jobs":
(https://github.com/bitcoin/bitcoin/pull/30975#issuecomment-2623801480)
The "test each commit" job jails on 36db25f0bcaab6af9358faaad9214d5b77a71cb2 which is part of #31741:

```
Start 5: mptest
Unable to find executable: /home/runner/work/bitcoin/bitcoin/build/src/ipc/libmultiprocess/test/mptest
```

The ASan + LSan job also fails over missing mptest.

The macOS 14 native no depends job fails with https://github.com/chaincodelabs/libmultiprocess/issues/138.

The tidy job found several issues:

```
[13:27:31.569] /ci_container_base/src/ipc/libmultipr
...

💬 Sjors commented on pull request "multiprocess: Add libmultiprocess git subtree":
(https://github.com/bitcoin/bitcoin/pull/31741#issuecomment-2623803509)
I found several issue while unleashing the CI on these changes: https://github.com/bitcoin/bitcoin/pull/30975#issuecomment-2623801480

Let me know which ones you want to address here, and which can be dealt with later.

💬 Sjors commented on issue "RFC: multiprocess binaries in 29.0":
(https://github.com/bitcoin/bitcoin/issues/31756#issuecomment-2623824284)
> First of all why is the default position that we should ship a new binary on short notice right before feature freeze

#30975 was opened in October and from the start had a clear intention of shipping these binaries in a release. This goes back to several discussions in the months before that, including [at core dev](https://btctranscripts.com/bitcoin-core-dev-tech/2024-10/multiprocess-binaries) (the minutes aren't precise, so don't take them as gospel).

So it wasn't short notice. However, ov
...

💬 hodlinator commented on issue "intermittent issue in p2p_orphan_handling.py":
(https://github.com/bitcoin/bitcoin/issues/31700#issuecomment-2623867669)
Another case: https://github.com/hodlinator/bitcoin/actions/runs/13042105883/job/36386032546#step:7:15916

📝 maximevtush opened a pull request: "Update LICENSE"
(https://github.com/bitcoin/bitcoin/pull/31761)
Updated the copyright year to 2025 in the LICENSE file

✅ fanquake closed a pull request: "Update LICENSE"
(https://github.com/bitcoin/bitcoin/pull/31761)

📝 fanquake locked a pull request: "Update LICENSE"
(https://github.com/bitcoin/bitcoin/pull/31761)
Updated the copyright year to 2025 in the LICENSE file