💬 Sjors commented on pull request "guix: update time-machine to 5cb84f2013c5b1e48a7d0e617032266f1e6059e2":
(https://github.com/bitcoin/bitcoin/pull/33185#issuecomment-3227126023)
I tried if f474673fed789257b2a9e9e5aabe6e734128b00a fixes #32759. It doesn't, but that's probably not expected since it just removes an implicit dependency.

To update the time machine, I had to manually `guix download` one item to build with `--no-substitutes`:
- `guix download https://downloads.sourceforge.net/project/boost/boost/1.83.0/boost_1_83_0.tar.bz2`

Still in the process of updating the time machine x86_64 which is taking quite long. Will publish hashes when that's done and also
...

💬 hodlinator commented on pull request "clang-format: align brace-after-struct and *-class formatting":
(https://github.com/bitcoin/bitcoin/pull/32813#discussion_r2303151367)
Why change the C++ Coding Style? I got used to the current one, which encourages `struct`s for POD as a separate thing.

📝 maflcko opened a pull request: "test: Use extra_port() helper in feature_bind_extra.py"
(https://github.com/bitcoin/bitcoin/pull/33260)
This is a refactor for self-validating and self-documenting code.

Currently, the test assumes that extra ports are available and just increments them without checking. However, this may not be the case when the test is modified to use more ports. In this case, the tests may fail intermittently and the failure is hard to debug.

Fix this confusion, by calling `p2p_port` each time. This ensures the required `assert n <= MAX_NODES` is checked each time.

Closes https://github.com/bitcoin/bit
...

🚀 fanquake merged a pull request: "[29.x] backport #33212"
(https://github.com/bitcoin/bitcoin/pull/33251)

📝 fanquake opened a pull request: "ci: return to using dash in CentOS job"
(https://github.com/bitcoin/bitcoin/pull/33261)
`dash` is available again: https://bugzilla.redhat.com/show_bug.cgi?id=2335416.

💬 maflcko commented on issue "tracing: test `interface_usdt_net.py` fails due to garbage message type in `net:outbound_message` tracepoint":
(https://github.com/bitcoin/bitcoin/issues/33246#issuecomment-3227642428)
I can reproduce via `-DAPPEND_CXXFLAGS='-O1 -g2'`, but it passes via `-DAPPEND_CXXFLAGS='-O1 -fno-inline -g2'`, so I wonder if this is a compiler error, or if the error is something else.

👍 maflcko approved a pull request: "ci: return to using dash in CentOS job"
(https://github.com/bitcoin/bitcoin/pull/33261#pullrequestreview-3159280776)
lgtm. Either shell should be fine here, for the purpose to test a non-bash shell.

💬 maflcko commented on pull request "ci: return to using dash in CentOS job":
(https://github.com/bitcoin/bitcoin/pull/33261#discussion_r2303567489)
nit: No need to advertise it here. This will only create a conflict with the GHA rewrite?

💬 maflcko commented on pull request "ci: return to using dash in CentOS job":
(https://github.com/bitcoin/bitcoin/pull/33261#discussion_r2303567806)
nit: remove ksh?

💬 maflcko commented on pull request "ci: return to using dash in CentOS job":
(https://github.com/bitcoin/bitcoin/pull/33261#issuecomment-3227827179)
lgtm ACK 509ffea40abbc706ef8b8fc449b7de8677fc5096

💬 Sjors commented on pull request "Add bitcoin-{node,gui} to release binaries for IPC":
(https://github.com/bitcoin/bitcoin/pull/31802#issuecomment-3227873440)
The sidecar Template Provider application now has its own repo: https://github.com/Sjors/sv2-tp

I suspect that the SRI team will have a Rust alternative soon(tm) as well.

🤔 janb84 reviewed a pull request: "ci: return to using dash in CentOS job"
(https://github.com/bitcoin/bitcoin/pull/33261#pullrequestreview-3160028763)
crACK 509ffea40abbc706ef8b8fc449b7de8677fc5096

PR Partially reverts faaabfaea768deb7767c489d32fd2097fd180872 because https://bugzilla.redhat.com/show_bug.cgi?id=2335416 is fixed.

- code-review ✅
- history-review ✅

💬 l0rinc commented on pull request "clang-format: align brace-after-struct and *-class formatting":
(https://github.com/bitcoin/bitcoin/pull/32813#discussion_r2304364072)
We can define it that way as well (though I don't agree with the distinction vs classes), but not specifying it directly results in different versions formatting structs differently, so we should define it explicitly

💬 jsarenik commented on issue "Package Relay Project Tracking":
(https://github.com/bitcoin/bitcoin/issues/27463#issuecomment-3228916881)
Let me note here that I noticed the zero (tested with LN Anchor) outputs are not being cleared in a read-only descriptor wallet even after they've been spent by a fee-paying transaction in a package.

💬 instagibbs commented on issue "Package Relay Project Tracking":
(https://github.com/bitcoin/bitcoin/issues/27463#issuecomment-3228924450)
@jsarenik could you open a new issue?

🤔 l0rinc reviewed a pull request: "test: Use extra_port() helper in feature_bind_extra.py"
(https://github.com/bitcoin/bitcoin/pull/33260#pullrequestreview-3160705049)
Concept ACK, please see my alternative suggestion which would get rid of variable reuse and would solve the port iteration in a more portable way

💬 l0rinc commented on pull request "test: Use extra_port() helper in feature_bind_extra.py":
(https://github.com/bitcoin/bitcoin/pull/33260#discussion_r2304600873)
I find these reassignments confusing - especially if it contains two ports now.
I think we can get rid of that by using lambdas here, which can provide two separate ports as inputs and the loop would generate the new ports

💬 l0rinc commented on pull request "test: Use extra_port() helper in feature_bind_extra.py":
(https://github.com/bitcoin/bitcoin/pull/33260#discussion_r2304594656)
@w0xlt was also modifying these in a different PR, I also considered a similar solution, but suggested using an iterator and lambdas instead https://github.com/bitcoin/bitcoin/pull/33231#discussion_r2296873010

📝 zkpepe opened a pull request: "fix: Merkle root calculation"
(https://github.com/bitcoin/bitcoin/pull/33262)
dropped the extra `SHA256D64` call that was messing up the result.
now it just builds the next level with `Hash(a, b)` like in Bitcoin Core, and the Merkle root comes out right.

✅ fanquake closed a pull request: "fix: Merkle root calculation"
(https://github.com/bitcoin/bitcoin/pull/33262)