Bitcoin Core Github
45 subscribers
113K links
Download Telegram
💬 TheCharlatan commented on pull request "RFC: coins: warn on oversized `-dbcache`":
(https://github.com/bitcoin/bitcoin/pull/33333#issuecomment-3265143719)
Concept ACK
🚀 fanquake merged a pull request: "Update libmultiprocess subtree to improve build and logs"
(https://github.com/bitcoin/bitcoin/pull/33322)
fanquake closed an issue: "build: indefinite mpgen hang on Ubuntu 22.04"
(https://github.com/bitcoin/bitcoin/issues/33176)
💬 fanquake commented on issue "build: indefinite mpgen hang on Ubuntu 22.04":
(https://github.com/bitcoin/bitcoin/issues/33176#issuecomment-3265327251)
Closing this with #33322. If there are new / more issues, lets open new issues.
💬 hebasto commented on pull request "clang-tidy: Disable `UndefinedBinaryOperatorResult` check in `src/ipc`":
(https://github.com/bitcoin/bitcoin/pull/33312#issuecomment-3265331404)
> Is it possible to move this into the capnp subdirectory to further narrow the scope of this exception?

The most recent CI log shows false-positive `clang-analyzer-core.UndefinedBinaryOperatorResult` warnings in the following files:
- `/home/admin/actions-runner/_work/_temp/build/src/ipc/capnp/init.capnp.proxy-server.c++`
- `/home/admin/actions-runner/_work/_temp/build/src/ipc/capnp/echo.capnp.proxy-server.c++`
- `/home/admin/actions-runner/_work/_temp/build/src/ipc/capnp/mining.capnp.pro
...
💬 vasild commented on pull request "Broadcast own transactions only via short-lived Tor or I2P connections":
(https://github.com/bitcoin/bitcoin/pull/29415#discussion_r2329682120)
Changed locally, will be in the next push, thanks!
🚀 fanquake merged a pull request: "contrib: update fixed seeds"
(https://github.com/bitcoin/bitcoin/pull/33283)
fanquake closed an issue: "build: clang build broken on Debian Bookworm"
(https://github.com/bitcoin/bitcoin/issues/33279)
💬 fanquake commented on issue "build: clang build broken on Debian Bookworm":
(https://github.com/bitcoin/bitcoin/issues/33279#issuecomment-3265439112)
Going to close this for now, based on https://github.com/bitcoin-core/libmultiprocess/pull/205 (which was pulled in via #33322). We can followup further if any similar issues surface while testing `30.x`.
💬 ryanofsky commented on issue "build: indefinite mpgen hang on Ubuntu 22.04":
(https://github.com/bitcoin/bitcoin/issues/33176#issuecomment-3265458641)
Makes sense to close. Since #33241 which includes https://github.com/bitcoin-core/libmultiprocess/pull/194, the build hang using old versions of GCC with v0.8.0 and v0.8.1 was fixed, but a cmake error was added warning about a CVE in v0.8.0, which means that Ubuntu 22.04 triggers this error. Ubuntu security team evaluated the CVE and opted not to fix it based on the severity and fact that is fix is in a library, so could cause other packages to need to be recompiled. #33322 improves the situatio
...
💬 fanquake commented on pull request "ci: reduce runner sizes on various jobs":
(https://github.com/bitcoin/bitcoin/pull/33319#issuecomment-3265467783)
cc @maflcko
💬 deyvid61 commented on pull request "Release: 30.0 translations update":
(https://github.com/bitcoin/bitcoin/pull/33275#discussion_r2329727735)
__
💬 TheCharlatan commented on pull request "Interfaces: Expose UTXO Snapshot Loading and Add Progress Notifications":
(https://github.com/bitcoin/bitcoin/pull/33117#issuecomment-3265507664)
Approach ACK
💬 jesterhodl commented on pull request "Release: 30.0 translations update":
(https://github.com/bitcoin/bitcoin/pull/33275#issuecomment-3265513185)
Hey @hebasto
I'm curious why the 30.x resources have a lot of untranslated labels. I know I completed 29.x
Could it be that they have been copied from 29.x to 30.x before I finished?

In screenshot below, the first one is a 29.x resource, the lower one is 30.x
<img width="1152" height="478" alt="image" src="https://github.com/user-attachments/assets/d2603008-d4af-4ab7-a444-4a47431a5802" />

Aside from the brand new labels, I can sync them up manually in Transifex, but just wanted to ask
...
💬 ryanofsky commented on pull request "multiprocess: Don't require bitcoin -m argument when IPC options are used":
(https://github.com/bitcoin/bitcoin/pull/33229#issuecomment-3265524908)
> Looks like Windows CI is still unhappy:
>
> ```
> File "D:\a\bitcoin\bitcoin/test/functional/tool_bitcoin.py", line 62, in run_test
> self.test_args([], [], expect_exe="bitcoind")
> ```
>
> Maybe @hebasto has an idea?
>
> I would be nice to get this in v30 so we don't have to teach people to use `-m` only for that to become unnecessary in the next release. But it's not a huge deal.

Yes I'd like to include / backport it in v30.

The code
...
💬 vasild commented on pull request "Broadcast own transactions only via short-lived Tor or I2P connections":
(https://github.com/bitcoin/bitcoin/pull/29415#issuecomment-3265576319)
> I tried sending a tx with fee rate 0.1 sat/vbyte

Added a note to "Further extensions" in PR description (see at the bottom) to consider checking the FEEFILTER message and retrying sooner than 10 minutes if the transaction is below their threshold.

> I finally received the tx back from another peer after 13 hours and 90 connection attempts.

I wonder how this compares to the current send-to-all-connected-peers approach. If it took 90 peers to find one to accept the transaction, and if w
...
💬 vasild commented on pull request "ci: detect outbound internet traffic generated while running tests":
(https://github.com/bitcoin/bitcoin/pull/31349#issuecomment-3265599978)
`778675ac71...151edfaf78`: rebase due to conflicts
📝 TheCharlatan opened a pull request: "net: Add interrupt to pcp retry loop"
(https://github.com/bitcoin/bitcoin/pull/33338)
Without this interrupt bitcoind takes a long time to exit if requested to do so after a failed pcp lookup on startup.
💬 TheCharlatan commented on pull request "net: Add interrupt to pcp retry loop":
(https://github.com/bitcoin/bitcoin/pull/33338#issuecomment-3265618371)
Logs:

<details>
<summary>Master logs when mapping fails (Click to expand)</summary>

```
2025-09-08T10:02:52Z msghand thread start
2025-09-08T10:02:52Z Imported mempool transactions from file: 3 succeeded, 0 failed, 0 expired, 0 already there, 0 waiting for initial broadcast
2025-09-08T10:02:52Z initload thread exit
2025-09-08T10:02:52Z [net:warning] pcp: Could not receive response: Connection refused (111)
2025-09-08T10:02:52Z [net] portmap: gateway [IPv6]: fe80::a6ce:daff:fe8f:12a0%
...
💬 hebasto commented on pull request "Release: 30.0 translations update":
(https://github.com/bitcoin/bitcoin/pull/33275#issuecomment-3265676083)
@jesterhodl
> Hey @hebasto I'm curious why the 30.x resources have a lot of untranslated labels. I know I completed 29.x Could it be that they have been copied from 29.x to 30.x before I finished?
>
> In screenshot below, the first one is a 29.x resource, the lower one is 30.x <img alt="image" width="1152" height="478" src="https://private-user-images.githubusercontent.com/103882255/486711123-d2603008-d4af-4ab7-a444-4a47431a5802.png?jwt=eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJnaXRod
...