💬 casey commented on issue "RFC: Formal description of the RPC API":
(https://github.com/bitcoin/bitcoin/issues/29912#issuecomment-2477233215)
I had no idea this issue existed! Thanks @0xB10C for sharing and for the ping.

My motivation was the current situation with Rust Bitcoin Core JSON RPC client crates, which is not great. There's one which is mostly complete, [rust-bitcoincore-rpc](https://github.com/rust-bitcoin/rust-bitcoincore-rpc), but which is now unmaintained, and a successor project [rust-bitcoind-json-rpc](https://github.com/rust-bitcoin/rust-bitcoind-json-rpc/), which is incomplete, but looks like it's a little more pr
...

💬 sr-gi commented on pull request "p2p: Fill reconciliation sets (Erlay) attempt 2":
(https://github.com/bitcoin/bitcoin/pull/30116#discussion_r1842776153)
I misunderstood the use of ForceRelay here. Will amend it so it applied to both

💬 maflcko commented on issue "RFC: Formal description of the RPC API":
(https://github.com/bitcoin/bitcoin/issues/29912#issuecomment-2477250414)
> * `RPCArg::m_names` has this comment: `can contain multiple aliases separated by | for named request arguments`. As a result, the generated JSON for an RPCArg has `names` key, whose value is a list of names. However, I could not find an instance of this being used in the code.

Are you sure? I checked for this in your generated json and found it https://github.com/casey/bitcoin/blob/f2f32b6cc44ef88e4c57e5b0a75935aa912e1862/api.json#L6669-L6674



> * I think that using JSON Schema would
...

💬 casey commented on issue "RFC: Formal description of the RPC API":
(https://github.com/bitcoin/bitcoin/issues/29912#issuecomment-2477278009)
> Are you sure? I checked for this in your generated json and found it https://github.com/casey/bitcoin/blob/f2f32b6cc44ef88e4c57e5b0a75935aa912e1862/api.json#L6669-L6674

Oh nice! I overlooked that, I was looking at the codebase, not the generated schema.

> > * I think that using JSON Schema would be ideal
>
> I agree, for the reasons given in [#29912 (comment)](https://github.com/bitcoin/bitcoin/issues/29912#issuecomment-2418963490). If the current specs don't fit within JSON Schema, i
...

💬 sr-gi commented on pull request "p2p: Fill reconciliation sets (Erlay) attempt 2":
(https://github.com/bitcoin/bitcoin/pull/30116#discussion_r1842808065)
Sorry, you're right. I've been looking at this being conditional to `m_txreconciliation->TryRemovingFromSet(peer_id, wtxid`, which is what happens with the descendants, but here it is unconditional.

It should be parsed only if `TryRemovingFromSet` succeeds.

💬 sr-gi commented on pull request "p2p: Fill reconciliation sets (Erlay) attempt 2":
(https://github.com/bitcoin/bitcoin/pull/30116#discussion_r1842810444)
Not sure I follow how the first comment is an issue, but I do agree with the second. This needs to be conditional to `TryRemovingFromSet` succeeding.

💬 0xB10C commented on issue "Tracepoint Interface Tracking Issue":
(https://github.com/bitcoin/bitcoin/issues/31274#issuecomment-2477300406)
> > We could internalize the relevant macro parts of systemtap's sys/sdt.h for the Linux tracepoints. This would allow us to drop the external dependency on systemtap
>
> This still makes sense to me, it's just a few macros which haven't changed in years and are super unlikely to change significantly, because they insert ELF annotations (according to a well-known spec) and NOP instructions.
>
> Dropping a guix dependency would be good!

I just found out about https://github.com/libbpf/us
...

💬 sr-gi commented on pull request "p2p: Fill reconciliation sets (Erlay) attempt 2":
(https://github.com/bitcoin/bitcoin/pull/30116#discussion_r1842841578)
That is fair, I'll change it to bool to be explicit

💬 sr-gi commented on pull request "p2p: Fill reconciliation sets (Erlay) attempt 2":
(https://github.com/bitcoin/bitcoin/pull/30116#discussion_r1842867828)
Done

💬 sr-gi commented on pull request "p2p: Fill reconciliation sets (Erlay) attempt 2":
(https://github.com/bitcoin/bitcoin/pull/30116#discussion_r1842870647)
Mave the move dependant on `TryRemovingFromSet`

💬 sr-gi commented on pull request "p2p: Fill reconciliation sets (Erlay) attempt 2":
(https://github.com/bitcoin/bitcoin/pull/30116#discussion_r1842873615)
The whole thing from L151-L160?

💬 sr-gi commented on pull request "p2p: Fill reconciliation sets (Erlay) attempt 2":
(https://github.com/bitcoin/bitcoin/pull/30116#discussion_r1842875471)
I'm guessing this is related to https://github.com/bitcoin/bitcoin/pull/30116#discussion_r1842873615, in which case I'll drop that

📝 furszy opened a pull request: "test: add global time to place exec tests within the same dir"
(https://github.com/bitcoin/bitcoin/pull/31291)
Solving https://github.com/bitcoin/bitcoin/pull/31000#discussion_r1836348991.

Modifies the unit/bench/fuzz tests default datadir path.

Groups all tests executed within each binary call under
a single directory prefixed by the current time.

Replicating the function test framework behavior.

Switching from:
```tmp/test_common bitcoin/test_name/current_time```
To:
```tmp/test_common bitcoin/current_time/test_name```

👍 TheCharlatan approved a pull request: "guix: scope pkg-config to Linux only"
(https://github.com/bitcoin/bitcoin/pull/31276#pullrequestreview-2437211248)
ACK bcd82b13f4649e57d7d106856aab7b2a6296d728

💬 achow101 commented on pull request "psbt: MuSig2 Fields":
(https://github.com/bitcoin/bitcoin/pull/31247#discussion_r1842895148)
If i need to retouch.

💬 sr-gi commented on pull request "p2p: Fill reconciliation sets (Erlay) attempt 2":
(https://github.com/bitcoin/bitcoin/pull/30116#discussion_r1842897353)
I changed `ReconSetSize` to return a tuple, and updated this to read:

```
"Now the set contains %i reconcilable transactions (plus %i delayed transactions).\n",
```

💬 achow101 commented on pull request "wallet: Remove IsMine from migration code":
(https://github.com/bitcoin/bitcoin/pull/30328#issuecomment-2477425377)
> Commit message in [d5d994c](https://github.com/bitcoin/bitcoin/commit/d5d994c02bb54db395da457724ec45539f1c10a8) incorrectly states: "This reverts commit [bbb1d51](https://github.com/bitcoin/bitcoin/commit/bbb1d51e1240da61db2ca1036f9ec91fd2f36f2d)."
>
> I believe that commit ended up being merged as [b231f4d](https://github.com/bitcoin/bitcoin/commit/b231f4d556876ae70305e8710e31d53525ded8ae).

Indeed, fixed.

💬 furszy commented on pull request "bench: add support for custom data directory":
(https://github.com/bitcoin/bitcoin/pull/31000#discussion_r1842899557)
Done in #31291

💬 achow101 commented on pull request "[refactor] Cleanup BlockAssembler mempool usage":
(https://github.com/bitcoin/bitcoin/pull/28843#issuecomment-2477429522)
ACK 192dac1d3370edd579db235d69c034726f37c8da

📝 casey opened a pull request: "Add `contrib/justfile` containing useful development workflow commands."
(https://github.com/bitcoin/bitcoin/pull/31292)
Add `contrib/justfile` containing useful development workflow commands.

Just recipes can be run by symlinking `contrib/justfile` into the repository root:

ln -s contrib/justfile justfile

And running:

just RECIPE

From any subdirectory.

Also add `/justfile` to `.gitignore`, to ignore the symlink into the repository root.

`just` is command runner with make-like syntax. It is not a build system, and only serves as convenient way of saving and running commands. It is avai
...