💬 jamesob commented on pull request "rpc: add getdescriptoractivity":
(https://github.com/bitcoin/bitcoin/pull/30708#discussion_r1817839632)
Fixed, thanks.

💬 jamesob commented on pull request "rpc: add getdescriptoractivity":
(https://github.com/bitcoin/bitcoin/pull/30708#discussion_r1817839644)
Done.

💬 jamesob commented on pull request "rpc: add getdescriptoractivity":
(https://github.com/bitcoin/bitcoin/pull/30708#issuecomment-2439552457)
Rebased.

👍 TheCharlatan approved a pull request: "build: Rename `PACKAGE_*` variables to `CLIENT_*`"
(https://github.com/bitcoin/bitcoin/pull/31042#pullrequestreview-2397236697)
ACK 36a3de0af9a9120aa98a07ca4abbb77ee6e58ef9

The last two commits could be squashed together though? I don't think that would complicate the scripted diff.

💬 hebasto commented on pull request "build: Rename `PACKAGE_*` variables to `CLIENT_*`":
(https://github.com/bitcoin/bitcoin/pull/31042#issuecomment-2439562411)
> The last two commits could be squashed together though? I don't think that would complicate the scripted diff.

Thanks! Squashed.

👍 TheCharlatan approved a pull request: "build: Rename `PACKAGE_*` variables to `CLIENT_*`"
(https://github.com/bitcoin/bitcoin/pull/31042#pullrequestreview-2397244727)
Re-ACK 2076a0863ce6e7fcd5a1f2c29c38c1305816e57d

👍 1440000bytes approved a pull request: "Drop miniupnp dependency"
(https://github.com/bitcoin/bitcoin/pull/31130#pullrequestreview-2397245584)
Code Review ACK https://github.com/bitcoin/bitcoin/pull/31130/commits/40e5f26a3ff77e50df808f6f850c617aec2df203

💬 andrewtoth commented on pull request "index: initial sync speedup, parallelize process":
(https://github.com/bitcoin/bitcoin/pull/26966#discussion_r1817847707)
This would make sense if we wanted to reuse this for CCheckQueue. Then, we could just loop and `ProcessTask` on the main thread once when we call `Wait`.

🤔 glozow reviewed a pull request: "functional test: Additional package evaluation coverage"
(https://github.com/bitcoin/bitcoin/pull/31152#pullrequestreview-2397255278)
reACK f32c34d0c3d4041a301822b27e88d6db4cbf631e

👍 instagibbs approved a pull request: "test: added test to assert TX decode rpc error on submitpackage rpc"
(https://github.com/bitcoin/bitcoin/pull/31139#pullrequestreview-2397254972)
ACK 1fab1d601e40edc7296cf3d60a7e917b0e6493b3

with minor suggestion

💬 instagibbs commented on pull request "test: added test to assert TX decode rpc error on submitpackage rpc":
(https://github.com/bitcoin/bitcoin/pull/31139#discussion_r1817851813)
```suggestion
assert txid_list[0] not in node.getrawmempool()
assert txid_list[1] not in node.getrawmempool()
```

🚀 glozow merged a pull request: "functional test: Additional package evaluation coverage"
(https://github.com/bitcoin/bitcoin/pull/31152)

💬 TheCharlatan commented on pull request "build, ci: Fix linking `bitcoin-chainstate.exe` to `bitcoinkernel.dll` on Windows":
(https://github.com/bitcoin/bitcoin/pull/31158#discussion_r1817854236)
I checked if this could also be a problem for other extern symbols that are currently not used in `bitcoin-chainstate`, but still part of the kernel library. For example in `src/validation.h` we have `extern const std::vector<std::string> CHECKLEVEL_DOC;`, and it looks like using it is indeed problematic too if it were used externally: https://github.com/TheCharlatan/bitcoin/actions/runs/11531552497/job/32102377337#step:10:425.
There are a bunch of other such cases, where I'm not entirely sure
...

📝 hebasto opened a pull request: "cmake: Set top-level target output locations"
(https://github.com/bitcoin/bitcoin/pull/31161)
This PR sets the target output locations to the `bin` and `lib` subdirectories within the build tree, creating a directory structure that mirrors that of the installed targets.

This approach is widely adopted by the large projects, such as [LLVM](https://github.com/llvm/llvm-project/blob/e146c1867e8decfd423034f63a3a863733e03f04/lldb/cmake/modules/LLDBStandalone.cmake#L128-L130):
```cmake
set(CMAKE_RUNTIME_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/bin)
set(CMAKE_LIBRARY_OUTPUT_DIRECTORY ${CMAKE_
...

⚠️ carnhofdaki opened an issue: "Stop at header"
(https://github.com/bitcoin/bitcoin/issues/31162)
### Please describe the feature you'd like to see added.

While preparing the little ter.gz file for https://github.com/fjahr/test_chain_init/issues/1 I stumbled upon the fact that one is unable to download just a couple of blocks in the beginning without synchronizing the full header chain.

I would be happy if the `stopatheight` option was respected also header-wise. This is a feature request.

Using 28.0 x86_64 Linux release binaries from bitcoincore.org

```
$ bitcoind -help -help-deb
...

👍 tdb3 approved a pull request: "rpc: add getdescriptoractivity"
(https://github.com/bitcoin/bitcoin/pull/30708#pullrequestreview-2397262411)
re ACK f383db76ec3aaa9391509c1d9cca763d11b6fe00

Changes include updating `scriptpubkey_hex` result type to `STR_HEX`, cleaner `Coin` usage, and adding a test for unconfirmed tx.

<details>
<summary>git range-diff d79ea809d28197b1b4e3748aa1715272b53601d0..1365ee8e9c7a20aa63bcddb1a6d5843c05ff9330 25dacae9c7feb31308271e2fd5a127c1fc230c2f..f383db76ec3aaa9391509c1d9cca763d11b6fe00</summary>

```diff
1: 1365ee8e9c7 ! 1: f383db76ec3 rpc: add getdescriptoractivity
@@ src/rpc/blockchain.c
...

💬 TheCharlatan commented on pull request "cmake: Set top-level target output locations":
(https://github.com/bitcoin/bitcoin/pull/31161#issuecomment-2439601361)
Concept ACK

💬 instagibbs commented on pull request "Ephemeral Dust":
(https://github.com/bitcoin/bitcoin/pull/30239#discussion_r1817860890)
I haven't looked lately but these harnesses should trigger package RBF behavior. I added the package mempool results checking to check the related invariants.

💬 instagibbs commented on pull request "Ephemeral Dust":
(https://github.com/bitcoin/bitcoin/pull/30239#discussion_r1817860896)
updated

💬 instagibbs commented on pull request "Ephemeral Dust":
(https://github.com/bitcoin/bitcoin/pull/30239#discussion_r1817860901)
yeah that's confusing, reworded