💬 Jackwumingjie commented on issue "Method not found":
(https://github.com/bitcoin/bitcoin/issues/27617#issuecomment-1542172628)
My compile steps:
sudo mkdir /data/.btc && cd /data/.btc
sudo git clone https://github.com/bitcoin/bitcoin.git
cd bitcoin
sudo ./autogen.sh
sudo ./configure
sudo make
sudo make install

💬 kouloumos commented on issue "Method not found":
(https://github.com/bitcoin/bitcoin/issues/27617#issuecomment-1542174130)
You are probably missing the SQLite or BDB dependency to compile the wallet. When you run `./configure` make sure that summary (that's in your `config.log`) shows `with wallet = yes`.

If that's the case, you'll need to install the dependencies and then recompile following the steps you done before (`./configure`, `make`, etc.)

See also https://github.com/bitcoin/bitcoin/issues/26193.

💬 0xB10C commented on pull request "Improve performance of p2p inv to send queues":
(https://github.com/bitcoin/bitcoin/pull/27610#issuecomment-1542178451)
Concept ACK

💬 willcl-ark commented on pull request "Improve performance of p2p inv to send queues":
(https://github.com/bitcoin/bitcoin/pull/27610#issuecomment-1542181711)
ACK 5b34060

Currently running this cherry-picked on top of v24.0.1 on mainnet and seeing reduced resource usage.

👍 theStack approved a pull request: "test: added coverage to mining_basic.py"
(https://github.com/bitcoin/bitcoin/pull/27603#pullrequestreview-1420617170)
ACK a7b46a1feae60e38fe4bdcacf5034f44cae49222

💬 instagibbs commented on pull request "Improve performance of p2p inv to send queues":
(https://github.com/bitcoin/bitcoin/pull/27610#issuecomment-1542201540)
ACK https://github.com/bitcoin/bitcoin/pull/27610/commits/5b3406094f2679dfb3763de4414257268565b943

Significant reduction in CPU usage when influx of transactions is high and sustained. Allows an additional INV to trickle per additional 200 INV backlog.

💬 MarcoFalke commented on pull request "refactor, kernel: Decouple ArgsManager from blockstorage":
(https://github.com/bitcoin/bitcoin/pull/27125#issuecomment-1542220359)
trivial re-ACK 886a473fc48f2c7d67436b5d9ac5643cd007e27f 🐗

<details><summary>Show signature</summary>

Signature:

```
untrusted comment: signature from minisign secret key on empty file; verify via: minisign -Vm "${path_to_any_empty_file}" -P RWTRmVTMeKV5noAMqVlsMugDDCyyTSbA3Re5AkUrhvLVln0tSaFWglOw -x "${path_to_this_whole_four_line_signature_blob}"
RUTRmVTMeKV5npGrKx1nqXCw5zeVHdtdYURB/KlyA/LMFgpNCs+SkW9a8N95d+U4AP1RJMi+krxU1A3Yux4bpwZNLvVBKy0wLgM=
trusted comment: trivial re-ACK 886a
...

💬 sdaftuar commented on pull request "p2p: Avoid prematurely clearing download state for other peers":
(https://github.com/bitcoin/bitcoin/pull/27608#discussion_r1189926154)
Oops -- will fix in a followup. Thanks for catching.

💬 kouloumos commented on issue "rpc: Allow importing wallets by data instead of by filename":
(https://github.com/bitcoin/bitcoin/issues/27597#issuecomment-1542272946)
Is there a specific reason that you need to pass around the whole wallet? You might be able to do a work-around with descriptor wallets (default wallet type since [23.0](https://github.com/bitcoin/bitcoin/blob/master/doc/release-notes/release-notes-23.0.md#wallet)) using the `{list,import}descriptors` RPCs.

💬 fanquake commented on issue "Method not found":
(https://github.com/bitcoin/bitcoin/issues/27617#issuecomment-1542275484)
Ok. This isn't a bug, and can be fixed by reading the documentation. i.e https://github.com/bitcoin/bitcoin/blob/master/doc/build-unix.md. Closing for now.

✅ fanquake closed an issue: "Method not found"
(https://github.com/bitcoin/bitcoin/issues/27617)

💬 Zero-1729 commented on pull request "add ryanofsky to trusted-keys":
(https://github.com/bitcoin/bitcoin/pull/27604#issuecomment-1542296157)
Post merge ACK 59ebee3fb4181baf20fab263cf1b587ece1bd5e2

Verified key fingerprint checks out in my local keychain.

💬 jamesob commented on pull request "add ryanofsky to trusted-keys":
(https://github.com/bitcoin/bitcoin/pull/27604#issuecomment-1542307429)
ACK russ

💬 jamesob commented on pull request "add ryanofsky to trusted-keys":
(https://github.com/bitcoin/bitcoin/pull/27604#issuecomment-1542309525)

I'm also unable to verify the GPG key.

📝 MarcoFalke converted_to_draft a pull request: "ci: Run iwyu on all src files"
(https://github.com/bitcoin/bitcoin/pull/27571)
This makes it easier to look at the CI output of a file without having to manually add it first to the list.

🤔 instagibbs reviewed a pull request: "rpc: allow submitpackage to be called outside of regtest"
(https://github.com/bitcoin/bitcoin/pull/27609#pullrequestreview-1420792155)
concept ACK

💬 instagibbs commented on pull request "rpc: allow submitpackage to be called outside of regtest":
(https://github.com/bitcoin/bitcoin/pull/27609#discussion_r1190006006)
the string `packagerelay` doesn't exist yet in the codebase

💬 Jackwumingjie commented on issue "Method not found":
(https://github.com/bitcoin/bitcoin/issues/27617#issuecomment-1542318987)
@fanquake I have installed the sqlite, but other error display:
<img width="1078" alt="image" src="https://github.com/bitcoin/bitcoin/assets/39093109/a989acd1-9867-4dd4-b2c2-e1ebc9822660">

💬 MarcoFalke commented on issue "The `libbitcoinkernel` Project":
(https://github.com/bitcoin/bitcoin/issues/24303#issuecomment-1542335212)
Let's continue discussion in #27587?

✅ MarcoFalke closed an issue: "The `libbitcoinkernel` Project"
(https://github.com/bitcoin/bitcoin/issues/24303)