achow101 changed the topic of #bitcoin-core-dev to: Bitcoin Core development discussion and commit log | Feel free to watch, but please take commentary and usage questions to #bitcoin | Channel logs: http://www.erisian.com.au/bitcoin-core-dev/, http://gnusha.org/bitcoin-core-dev/ | Weekly Meeting Thursday @ 16:00 UTC | Meeting topics http://gnusha.org/bitcoin-core-dev/proposedmeetingtopics.txt
shiza has quit [Server closed connection]
shiza has joined #bitcoin-core-dev
spynx has joined #bitcoin-core-dev
spynxic has quit [Remote host closed the connection]
katsu has quit [Server closed connection]
katsu has joined #bitcoin-core-dev
w0xlt has joined #bitcoin-core-dev
Artea has quit [Server closed connection]
<achow101> l0rinc: added for now
Artea has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] achow101 pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/c808dfbbdcea...76de2f8e55f3
<bitcoin-git> bitcoin/master fa90d44 MarcoFalke: test: Fix intermittent issues in feature_assumevalid.py
<bitcoin-git> bitcoin/master 76de2f8 Ava Chow: Merge bitcoin/bitcoin#34571: test: Fix intermittent issues in feature_assu...
<bitcoin-git> [bitcoin] achow101 merged pull request #34571: test: Fix intermittent issues in feature_assumevalid.py (master...2602-test-av-int-fail) https://github.com/bitcoin/bitcoin/pull/34571
<bitcoin-git> [bitcoin] achow101 pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/76de2f8e55f3...96bec216ec34
<bitcoin-git> bitcoin/master afea2af ANAVHEOBA: net: reduce log level for PCP/NAT-PMP NOT_AUTHORIZED failures
<bitcoin-git> bitcoin/master 96bec21 Ava Chow: Merge bitcoin/bitcoin#34549: net: reduce log level for PCP/NAT-PMP NOT_AUT...
<bitcoin-git> [bitcoin] achow101 merged pull request #34549: net: reduce log level for PCP/NAT-PMP NOT_AUTHORIZED failures (master...pcp-loglevel) https://github.com/bitcoin/bitcoin/pull/34549
johnzweng has quit [Server closed connection]
johnzweng has joined #bitcoin-core-dev
conman has quit [Quit: Konversation terminated!]
conman has joined #bitcoin-core-dev
spynx is now known as spynxic
szkl has quit [Server closed connection]
szkl has joined #bitcoin-core-dev
dergoegge has quit [Server closed connection]
dergoegge has joined #bitcoin-core-dev
fanquake has quit [Server closed connection]
fanquake has joined #bitcoin-core-dev
durandal__ has joined #bitcoin-core-dev
emcy__ has quit [Ping timeout: 252 seconds]
w0xlt has quit [Quit: w0xlt]
memset has quit [Remote host closed the connection]
memset has joined #bitcoin-core-dev
memset has quit [Remote host closed the connection]
memset has joined #bitcoin-core-dev
nervana21 has quit [Quit: Client closed]
ghost43 has quit [Remote host closed the connection]
ghost43_ has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] ajtowns opened pull request #34628: p2p: Replace per-peer transaction rate-limiting with global rate limits (master...202602-mempool-invtosend) https://github.com/bitcoin/bitcoin/pull/34628
<bitcoin-git> [bitcoin] ryanofsky pushed 7 commits to master: https://github.com/bitcoin/bitcoin/compare/96bec216ec34...ee2065fdeaca
<bitcoin-git> bitcoin/master 69b01af Andrew Toth: coins: add PeekCoin()
<bitcoin-git> bitcoin/master 67c0d17 Andrew Toth: coins: introduce CoinsViewOverlay
<bitcoin-git> bitcoin/master 73e99a5 Andrew Toth: coins: don't mutate main cache when connecting block
<bitcoin-git> [bitcoin] ryanofsky merged pull request #34165: coins: don't mutate main cache when connecting block (master...dont-mutate-cache) https://github.com/bitcoin/bitcoin/pull/34165
jerryf_ has joined #bitcoin-core-dev
BGL has quit [Ping timeout: 245 seconds]
michael____ has quit [Ping timeout: 245 seconds]
michael____ has joined #bitcoin-core-dev
jerryf has quit [Remote host closed the connection]
elichai2 has quit [Ping timeout: 252 seconds]
elichai2 has joined #bitcoin-core-dev
l0rinc has quit [Remote host closed the connection]
l0rinc has joined #bitcoin-core-dev
szarka has joined #bitcoin-core-dev
svanstaa has joined #bitcoin-core-dev
svanstaa_ has quit [Ping timeout: 264 seconds]
dzxzg has quit [Remote host closed the connection]
BGL has joined #bitcoin-core-dev
cmirror has quit [Remote host closed the connection]
cmirror has joined #bitcoin-core-dev
svanstaa_ has joined #bitcoin-core-dev
svanstaa has quit [Ping timeout: 276 seconds]
shesek has quit [Server closed connection]
shesek has joined #bitcoin-core-dev
ghost43_ has quit [Ping timeout: 252 seconds]
ghost43 has joined #bitcoin-core-dev
jerryf_ has quit [Remote host closed the connection]
ghost43 has quit [Remote host closed the connection]
jerryf has joined #bitcoin-core-dev
ghost43 has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] maflcko closed pull request #34533: wallet: resubmit transactions with private broadcast if enabled (master...wallet_rebroadcast_use_private) https://github.com/bitcoin/bitcoin/pull/34533
<bitcoin-git> [bitcoin] maflcko reopened pull request #34533: wallet: resubmit transactions with private broadcast if enabled (master...wallet_rebroadcast_use_private) https://github.com/bitcoin/bitcoin/pull/34533
<bitcoin-git> [bitcoin] Kushmanmb opened pull request #34630: linux-bu (master...master) https://github.com/bitcoin/bitcoin/pull/34630
<bitcoin-git> [bitcoin] DrahtBot closed pull request #34630: linux-bu (master...master) https://github.com/bitcoin/bitcoin/pull/34630
marcofleon has quit [Server closed connection]
marcofleon has joined #bitcoin-core-dev
ghost43 has quit [Remote host closed the connection]
ghost43 has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] sedited pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/ee2065fdeaca...1a54886b639a
<bitcoin-git> bitcoin/master 3d82ec5 sstone: Add a "tx output spender" index
<bitcoin-git> bitcoin/master 0b96b9c sstone: Minimize mempool lock, sync txo spender index only when and if needed
<bitcoin-git> bitcoin/master 1a54886 merge-script: Merge bitcoin/bitcoin#24539: Add a "tx output spender" index
<bitcoin-git> [bitcoin] sedited merged pull request #24539: Add a "tx output spender" index (master...add-txospender-index) https://github.com/bitcoin/bitcoin/pull/24539
timbo_xyz has joined #bitcoin-core-dev
l0rinc has quit [Quit: l0rinc]
Guyver2 has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] willcl-ark opened pull request #34633: Revert "ci: Treat SHA1 LLVM signing key as warning" (master...revert-llvm-signing) https://github.com/bitcoin/bitcoin/pull/34633
l0rinc has joined #bitcoin-core-dev
timbo_xy1 has joined #bitcoin-core-dev
timbo_xyz has quit [Remote host closed the connection]
janb84 has quit [Server closed connection]
janb84 has joined #bitcoin-core-dev
emzy has quit [Server closed connection]
emzy has joined #bitcoin-core-dev
jerryf_ has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/1a54886b639a...641a1954f7ae
<bitcoin-git> bitcoin/master 3574905 will: Revert "ci: Treat SHA1 LLVM signing key as warning"
<bitcoin-git> bitcoin/master 641a195 merge-script: Merge bitcoin/bitcoin#34633: Revert "ci: Treat SHA1 LLVM signing key as wa...
<bitcoin-git> [bitcoin] fanquake merged pull request #34633: Revert "ci: Treat SHA1 LLVM signing key as warning" (master...revert-llvm-signing) https://github.com/bitcoin/bitcoin/pull/34633
jerryf has quit [Ping timeout: 252 seconds]
jerryf_ has quit [Remote host closed the connection]
jerryf has joined #bitcoin-core-dev
emcy__ has joined #bitcoin-core-dev
durandal__ has quit [Ping timeout: 264 seconds]
<bitcoin-git> [bitcoin] sedited closed pull request #31382: kernel: Flush in ChainstateManager destructor (master...chainman_flush_destructor) https://github.com/bitcoin/bitcoin/pull/31382
<bitcoin-git> [bitcoin] sedited pushed 10 commits to master: https://github.com/bitcoin/bitcoin/compare/641a1954f7ae...cb3473a6804f
<bitcoin-git> bitcoin/master df53a3e Ryan Ofsky: rpc refactor: stop using deprecated getCoinbaseCommitment method
<bitcoin-git> bitcoin/master b970cdf Ryan Ofsky: test framework: expand expected_stderr, expected_ret_code options
<bitcoin-git> bitcoin/master ff995b5 Ryan Ofsky: ipc test: add workaround to block_reserved_weight exception test
<bitcoin-git> [bitcoin] sedited merged pull request #34568: mining: Break compatibility with existing IPC mining clients (master...pr/mbreak) https://github.com/bitcoin/bitcoin/pull/34568
<fjahr> FYI: the asmap-data repository has moved into the bitcoin-core org: https://github.com/bitcoin-core/asmap-data
<bitcoin-git> [gui] hebasto opened pull request #931: Release: Update src/qt/locale/bitcoin_en.xlf after string freeze (master...260220-tr) https://github.com/bitcoin-core/gui/pull/931
enochazariah has joined #bitcoin-core-dev
enochazariah has quit [Client Quit]
enochazariah has joined #bitcoin-core-dev
<enochazariah> I see PR #34184 addresses a template flood during IBD/catch-up. Conceptually, why does the node even generate/send templates during this phase? Shouldn't it suppress all mining activity until it is fully caught up to the network tip?
<corebot> https://github.com/bitcoin/bitcoin/issues/34184 | mining: add cooldown to createNewBlock() immediately after IBD by Sjors · Pull Request #34184 · bitcoin/bitcoin · GitHub
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/cb3473a6804f...a2fd558760ab
<bitcoin-git> bitcoin/master 79c934b Hennadii Stepanov: cmake: Fix NetBSD-specific workaround for Boost
<bitcoin-git> bitcoin/master a2fd558 merge-script: Merge bitcoin/bitcoin#34572: cmake: Fix NetBSD-specific workaround for Boo...
<bitcoin-git> [bitcoin] fanquake merged pull request #34572: cmake: Fix NetBSD-specific workaround for Boost (master...260212-netbsd-boost) https://github.com/bitcoin/bitcoin/pull/34572
<Sjors[m]1> enochazariah: it's similar to the getblocktemplate RPC, which you can also call before we're fully caught up. If you call it again immedidately after it returns you also get a flood.
<Sjors[m]1> But the IPC is designed to be push rather than poll based, so the problem is a bit more pronounced.
<Sjors[m]1> There's no independent mining process in the node. It generates a new block template when you ask for one.
<Sjors[m]1> #33421 is a step in that direction though
<corebot> https://github.com/bitcoin/bitcoin/issues/33421 | node: add `BlockTemplateCache` by ismaelsadeeq · Pull Request #33421 · bitcoin/bitcoin · GitHub
<enochazariah> OKay, that makes sense. RPC was 'pull' so the client set the pace, but IPC 'pushes' on every tip update, which exposes the flood during catchup. Makes sense why we need the cooldown until we get proper manager logic
<enochazariah> Thanks
adil has joined #bitcoin-core-dev
tarotfied has quit [Server closed connection]
tarotfied has joined #bitcoin-core-dev
adil has quit [Quit: adil]
<fanquake> Sjors[m]1: looking at #33777, what things are actual bugfixes/regressions, and should ship, as opposed to nice to have
<corebot> https://github.com/bitcoin/bitcoin/issues/33777 | Mining interface tracking issue · Issue #33777 · bitcoin/bitcoin · GitHub
<fanquake> It sucks if things haven't gotten review this cycle, but I'm not sure about listing a bunch of (conflicting) things for v31, on the day of feature freeze
<fanquake> Seems like this list should have appeared 2-3 months ago
<fanquake> I take it any of the fixes have already been confirmed as working from downstream?
<Sjors[m]1> The first 3 are a package that fix node crashes by connected IPC clients. It first adds a test, then fixes it in libmultiprocess and then pulls the fix in.
<Sjors[m]1> The last one fixes a flood of templates, as discussed above. It's been actively reviewed over the past month or two, but had to be based on #34568, which just got merged today.
<corebot> https://github.com/bitcoin/bitcoin/issues/34568 | mining: Break compatibility with existing IPC mining clients by ryanofsky · Pull Request #34568 · bitcoin/bitcoin · GitHub
<Sjors[m]1> So it's really just two bug fixes.
<fanquake> So the first changes are still blocked on a change that needs merging into bitcoin-core/libmultiprocess
<Sjors[m]1> The rest of the tracking issue is stuff that can easily wait for v32 or later
<Sjors[m]1> fanquake: yes
<Sjors[m]1> Clients can work around the issue and avoid crashing the node, but that doesn't seem ideal.
<Sjors[m]1> And I think https://github.com/bitcoin-core/libmultiprocess/pull/240 is in good shape.
<Sjors[m]1> Neither are breaking changes, so if they end up in v32.1 backport that's not the end of the world.
<Sjors[m]1> v31.1 I mean
<Sjors[m]1> Added clarification to the tracking issue.
Ara has quit [Server closed connection]
Ara has joined #bitcoin-core-dev
theStack has quit [Quit: theStack]
theStack has joined #bitcoin-core-dev
Guyver2 has quit [Quit: Going offline, see ya! (www.adiirc.com)]
jonatack has quit [Ping timeout: 252 seconds]
jonatack has joined #bitcoin-core-dev
jonatack has quit [Ping timeout: 269 seconds]
<bitcoin-git> [bitcoin] hodlinator closed pull request #31929: http: Make server shutdown more robust (master...2025/02/stop_http_robust) https://github.com/bitcoin/bitcoin/pull/31929
jonatack has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fjahr opened pull request #34635: rpc, index: txospenderindex follow-ups (master...2026-02-24539-follow-up) https://github.com/bitcoin/bitcoin/pull/34635
jonatack has quit [Ping timeout: 268 seconds]
brunoerg has joined #bitcoin-core-dev
maflcko has quit [Server closed connection]
maflcko has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 246 seconds]
infernix has quit [Server closed connection]
<bitcoin-git> [bitcoin] svanstaa opened pull request #34636: node: allocate index caches proportional to usage patterns (master...improve-index-cache-allocation) https://github.com/bitcoin/bitcoin/pull/34636
enochazariah has quit [Quit: Ping timeout (120 seconds)]
apollodorus has quit [Ping timeout: 264 seconds]
infernix has joined #bitcoin-core-dev
katsu has quit [Ping timeout: 264 seconds]
katsu has joined #bitcoin-core-dev
l0rinc has quit [Quit: l0rinc]
bugs_ has joined #bitcoin-core-dev
l0rinc has joined #bitcoin-core-dev
l0rinc has quit [Client Quit]
S3RK has quit [Ping timeout: 245 seconds]
l0rinc has joined #bitcoin-core-dev
S3RK has joined #bitcoin-core-dev
iSiUp has quit [Ping timeout: 252 seconds]
memset has quit [Remote host closed the connection]
memset has joined #bitcoin-core-dev
enochazariah has joined #bitcoin-core-dev
enochazariah has quit [Quit: Client closed]
Guyver2 has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
dodo has quit [Server closed connection]
<bitcoin-git> [bitcoin] janb84 opened pull request #34638: guix: Update storage requirement for HOSTS and added warning for Guix overhead (master...guix-build-update) https://github.com/bitcoin/bitcoin/pull/34638
dodo has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 252 seconds]
timbo_xy1 has quit [Quit: WeeChat 4.8.1]
memset has quit [Remote host closed the connection]
memset has joined #bitcoin-core-dev
enochazariah has joined #bitcoin-core-dev
memset has quit [Remote host closed the connection]
memset has joined #bitcoin-core-dev
enochazariah has quit [Quit: Ping timeout (120 seconds)]
Guest53 has joined #bitcoin-core-dev
Guyver2 has left #bitcoin-core-dev [Closing Window]
memset has quit [Remote host closed the connection]
memset has joined #bitcoin-core-dev
Guest53 has quit [Quit: Client closed]
<bitcoin-git> [bitcoin] l0rinc closed pull request #34435: refactor: use `_MiB`/`_GiB` consistently for byte conversions (master...util/byte-units-mib-gib) https://github.com/bitcoin/bitcoin/pull/34435
<bitcoin-git> [bitcoin] l0rinc reopened pull request #34435: refactor: use `_MiB`/`_GiB` consistently for byte conversions (master...util/byte-units-mib-gib) https://github.com/bitcoin/bitcoin/pull/34435
memset has quit [Remote host closed the connection]
memset_ has joined #bitcoin-core-dev
antanst712 has quit [Server closed connection]
antanst712 has joined #bitcoin-core-dev
memset_ has quit [Remote host closed the connection]
memset has joined #bitcoin-core-dev
timbo_xyz has joined #bitcoin-core-dev
<bitcoin-git> [qa-assets] maflcko pushed 1 commit to main: https://github.com/bitcoin-core/qa-assets/compare/195b736c7401...6afc2a87e0ba
<bitcoin-git> qa-assets/main 6afc2a8 MarcoFalke: initial coins_view_overlay minimal set
svanstaa_ has quit [Quit: https://quassel-irc.org - Chat comfortably. Anywhere.]
svanstaa has joined #bitcoin-core-dev
svanstaa has quit [Ping timeout: 265 seconds]
svanstaa has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] ryanofsky closed pull request #34284: ipc, test: Add tests for unclean disconnect and thread busy behavior (master...pr/ipc-testasync) https://github.com/bitcoin/bitcoin/pull/34284
<bitcoin-git> [bitcoin] l0rinc closed pull request #31644: leveldb: show non-default options during init (master...l0rinc/leveldb-options-logging) https://github.com/bitcoin/bitcoin/pull/31644
flooded has quit [Remote host closed the connection]
flooded has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
svanstaa has quit [Ping timeout: 264 seconds]
svanstaa_ has joined #bitcoin-core-dev
brunoerg has quit [Remote host closed the connection]
ghost43 has quit [Remote host closed the connection]
ghost43 has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Remote host closed the connection]
memset has quit [Remote host closed the connection]
memset has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] hebasto opened pull request #34639: iwyu: Remove some `pragma: export` and other improvements (master...260220-iwyu-pragma) https://github.com/bitcoin/bitcoin/pull/34639
brunoerg has quit [Remote host closed the connection]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 252 seconds]
brunoerg has joined #bitcoin-core-dev
memset has quit [Remote host closed the connection]
<bitcoin-git> [bitcoin] l0rinc closed pull request #34320: coins: remove redundant and confusing `CCoinsViewDB::HaveCoin` (master...l0rinc/remove-HaveCoin) https://github.com/bitcoin/bitcoin/pull/34320
<bitcoin-git> [bitcoin] l0rinc reopened pull request #34320: coins: remove redundant and confusing `CCoinsViewDB::HaveCoin` (master...l0rinc/remove-HaveCoin) https://github.com/bitcoin/bitcoin/pull/34320
<andrewtoth_> Is there a way to enable re-running failed jobs, instead of close->open? I know it's possible in other repos.
memset has joined #bitcoin-core-dev
<l0rinc> I've been fighting with rebases and fuzzers and CI all day...
<cfields> ryanofsky: can you help me understand conceptually how a shared_ptr works across ipc? Reading type-pointer.h isn't helping me much.
<cfields> ryanofsky: more specifically.. where/how is lifetime management (refcount) shared between the processes?
brunoerg has quit [Remote host closed the connection]
<sipa> andrewtoth_: you can ask here, some people have permission to do that
brunoerg has joined #bitcoin-core-dev
<hebasto> andrewtoth_: is anything needed to be re-run now?
l0rinc has quit [Quit: l0rinc]
<andrewtoth_> Nothing in particular for me right now, but if I have to reach out and ask it's easier to close->open.
<andrewtoth_> We can't give permission to do that to contributors?
<hebasto> I guess it's limited to people with write permission only
<hebasto> close-reopen uses a new master to rebase on, while re-run uses the old master
l0rinc has joined #bitcoin-core-dev
durandal__ has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] hebasto pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/a2fd558760ab...ce6898f9a803
<bitcoin-git> bitcoin/master 231dd04 will: build: define CMAKE_COMPILE_WARNING_AS_ERROR as a cache option
<bitcoin-git> bitcoin/master ce6898f Hennadii Stepanov: Merge bitcoin/bitcoin#34605: build: define CMAKE_COMPILE_WARNING_AS_ERROR ...
<bitcoin-git> [bitcoin] hebasto merged pull request #34605: build: define CMAKE_COMPILE_WARNING_AS_ERROR as a cache option (master...warning-as-error) https://github.com/bitcoin/bitcoin/pull/34605
emcy__ has quit [Ping timeout: 264 seconds]
durandal__ has quit [Ping timeout: 276 seconds]
bugs_ has quit [Ping timeout: 268 seconds]
brunoerg has quit [Remote host closed the connection]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 256 seconds]
durandal__ has joined #bitcoin-core-dev
Earnestly has quit [Ping timeout: 252 seconds]
brunoerg has joined #bitcoin-core-dev
Earnestly has joined #bitcoin-core-dev
aleggg has quit [Ping timeout: 264 seconds]
jonatack has joined #bitcoin-core-dev
brunoerg_ has joined #bitcoin-core-dev
brunoerg_ has quit [Remote host closed the connection]
brunoerg_ has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 276 seconds]
durandal_ has joined #bitcoin-core-dev
brunoerg_ has quit [Remote host closed the connection]
durandal__ has quit [Ping timeout: 264 seconds]
<dergoegge> andrewtoth_: github's permission system is not fine grained enough for that afaik
<achow101> rerunning workflows requires write permissions
memset has quit [Remote host closed the connection]
memset has joined #bitcoin-core-dev
Earnestly has quit [Read error: Connection reset by peer]
Earnestly has joined #bitcoin-core-dev