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
PaperSword has quit [Quit: PaperSword]
conman has quit [Remote host closed the connection]
jon_atack has joined #bitcoin-core-dev
jonatack has quit [Ping timeout: 240 seconds]
jonatack has joined #bitcoin-core-dev
jon_atack has quit [Ping timeout: 252 seconds]
jerryf has quit [Ping timeout: 240 seconds]
jerryf has joined #bitcoin-core-dev
jon_atack has joined #bitcoin-core-dev
jonatack has quit [Ping timeout: 255 seconds]
_durandal has joined #bitcoin-core-dev
conman has joined #bitcoin-core-dev
durandal_ has quit [Ping timeout: 240 seconds]
zeropoint has quit [Quit: leaving]
jonatack has joined #bitcoin-core-dev
jon_atack has quit [Ping timeout: 252 seconds]
l0rinc has quit [Quit: l0rinc]
<bitcoin-git> [bitcoin] jderibamar opened pull request #33174: Add startup log message for initialization confirmation (master...improve-test-docs) https://github.com/bitcoin/bitcoin/pull/33174
Guest1 has joined #bitcoin-core-dev
Guest1 has quit [Quit: Client closed]
joetor5 has joined #bitcoin-core-dev
bitdex has joined #bitcoin-core-dev
twistedline has quit []
bitcoinlover has joined #bitcoin-core-dev
twistedline has joined #bitcoin-core-dev
jon_atack has joined #bitcoin-core-dev
jonatack has quit [Ping timeout: 255 seconds]
jon_atack has quit [Ping timeout: 252 seconds]
joetor5 has quit [Quit: joetor5]
l0rinc has joined #bitcoin-core-dev
cmirror has quit [Remote host closed the connection]
cmirror has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] Ataraxia009 closed pull request #33126: Removing Bitcoin core text where unnecessary (master...multi-client-support) https://github.com/bitcoin/bitcoin/pull/33126
<bitcoin-git> [bitcoin] Ataraxia009 closed pull request #33127: Adding alert for failure to prevent dead-end user crash (master...launch-crash-failure) https://github.com/bitcoin/bitcoin/pull/33127
bitcoinlover has quit [Ping timeout: 260 seconds]
Christoph_ has joined #bitcoin-core-dev
l0rinc has quit [Quit: l0rinc]
BlueMattMtrxBot has quit [Server closed connection]
BlueMattMtrxBot has joined #bitcoin-core-dev
kevkevin has quit [Remote host closed the connection]
Guest19 has joined #bitcoin-core-dev
WizJin_ has joined #bitcoin-core-dev
WizJin_ has quit [Remote host closed the connection]
WizJin_ has joined #bitcoin-core-dev
Guest19 has quit [Client Quit]
WizJin_ has quit [Client Quit]
kevkevin has joined #bitcoin-core-dev
Guest95 has joined #bitcoin-core-dev
Guest95 has left #bitcoin-core-dev [#bitcoin-core-dev]
kevkevin has quit [Ping timeout: 260 seconds]
Guest67 has joined #bitcoin-core-dev
Guest30 has joined #bitcoin-core-dev
Guest67 has quit [Client Quit]
WizJin has joined #bitcoin-core-dev
Guest30 has quit [Quit: Client closed]
kevkevin has joined #bitcoin-core-dev
kevkevin has quit [Ping timeout: 255 seconds]
<bitcoin-git> [bitcoin] rejected-l opened pull request #33175: ci: bump actions/checkout to v5 (master...master) https://github.com/bitcoin/bitcoin/pull/33175
<bitcoin-git> [bitcoin] fanquake closed pull request #33175: ci: bump actions/checkout to v5 (master...master) https://github.com/bitcoin/bitcoin/pull/33175
<fanquake> must be the new drive-by trend
f321x has joined #bitcoin-core-dev
f321x has quit [Remote host closed the connection]
f321x has joined #bitcoin-core-dev
Christoph_ has quit [Quit: Christoph_]
dviola has quit [Quit: WeeChat 4.7.0]
robszarka has joined #bitcoin-core-dev
szarka has quit [Ping timeout: 260 seconds]
bitcoinlover has joined #bitcoin-core-dev
kevkevin has joined #bitcoin-core-dev
Guyver2 has joined #bitcoin-core-dev
Christoph_ has joined #bitcoin-core-dev
kevkevin has quit [Ping timeout: 276 seconds]
robszarka has quit [Quit: Leaving]
szarka has joined #bitcoin-core-dev
kevkevin has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake closed pull request #33174: Add startup log message for initialization confirmation (master...improve-test-docs) https://github.com/bitcoin/bitcoin/pull/33174
Guyver2 has left #bitcoin-core-dev [Closing Window]
laanwj has quit [Server closed connection]
laanwj has joined #bitcoin-core-dev
shwouchk has quit [Read error: Connection reset by peer]
shwouchk has joined #bitcoin-core-dev
jerryf has quit [Remote host closed the connection]
jerryf has joined #bitcoin-core-dev
kevkevin has quit [Ping timeout: 260 seconds]
kevkevin has joined #bitcoin-core-dev
uasf has quit [Server closed connection]
uasf has joined #bitcoin-core-dev
joetor5 has joined #bitcoin-core-dev
aleggg has joined #bitcoin-core-dev
jonatack has joined #bitcoin-core-dev
warren has quit [Server closed connection]
warren has joined #bitcoin-core-dev
jespada has joined #bitcoin-core-dev
jon_atack has joined #bitcoin-core-dev
jonatack has quit [Ping timeout: 252 seconds]
jonatack has joined #bitcoin-core-dev
jon_atack has quit [Ping timeout: 252 seconds]
<bitcoin-git> [bitcoin] fanquake opened pull request #33178: guix: increase maximum allowed (runtime) GCC to 7 (master...sym_check_gcc_7) https://github.com/bitcoin/bitcoin/pull/33178
virtu has quit [Ping timeout: 260 seconds]
virtu has joined #bitcoin-core-dev
theStack has quit [Server closed connection]
joetor5 has quit [Quit: joetor5]
<sipa> hebasto: awesome!
<hebasto> gh glitch?
<sipa> the PRs are still there if you go to #12345
<corebot> https://github.com/bitcoin/bitcoin/issues/12345 | Asymmetric encryption stubs. by Rutkouski · Pull Request #12345 · bitcoin/bitcoin · GitHub
<sipa> just not listed
<sipa> in the overview
<glozow> finally, we did it guys, we're free!
<bitcoin-git> [bitcoin] glozow pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/273e600e65c2...ec484bd5ce43
<bitcoin-git> bitcoin/master db32280 willcl-ark: util: detect and warn when using exFAT on macOS
<bitcoin-git> bitcoin/master ec484bd merge-script: Merge bitcoin/bitcoin#31453: util: detect and warn when using exFAT on Mac...
<bitcoin-git> [bitcoin] glozow merged pull request #31453: util: detect and warn when using exFAT on MacOS (master...macos-exfat) https://github.com/bitcoin/bitcoin/pull/31453
Guest25 has joined #bitcoin-core-dev
Guest25 has quit [Client Quit]
jon_atack has joined #bitcoin-core-dev
jonatack has quit [Ping timeout: 245 seconds]
janb84 has joined #bitcoin-core-dev
jonatack has joined #bitcoin-core-dev
jon_atack has quit [Ping timeout: 260 seconds]
<bitcoin-git> [bitcoin] Sjors opened pull request #33179: doc: update wallet build instruction (master...2025/08/wallet) https://github.com/bitcoin/bitcoin/pull/33179
jon_atack has joined #bitcoin-core-dev
jonatack has quit [Ping timeout: 260 seconds]
jonatack has joined #bitcoin-core-dev
jon_atack has quit [Ping timeout: 255 seconds]
zeropoint has joined #bitcoin-core-dev
jon_atack has joined #bitcoin-core-dev
visiteur has joined #bitcoin-core-dev
zeropoint has quit [Ping timeout: 248 seconds]
jonatack has quit [Ping timeout: 248 seconds]
l0rinc has joined #bitcoin-core-dev
jonatack has joined #bitcoin-core-dev
jon_atack has quit [Ping timeout: 248 seconds]
bugs_ has joined #bitcoin-core-dev
visiteur has quit [K-Lined]
LUKASZICHO has joined #bitcoin-core-dev
l0rinc has quit [Quit: l0rinc]
LUKASZICHO has quit [Client Quit]
l0rinc has joined #bitcoin-core-dev
Christoph_ has quit [Ping timeout: 260 seconds]
bitcoinlover has quit [Ping timeout: 252 seconds]
Christoph_ has joined #bitcoin-core-dev
ghost43_ has joined #bitcoin-core-dev
ghost43 has quit [Ping timeout: 240 seconds]
stringintech has joined #bitcoin-core-dev
jespada has quit [Ping timeout: 248 seconds]
l0rinc has quit [Quit: l0rinc]
<Sjors[m]1> I'd like to propose #31802 for the v30 milestone, if only to discuss if it needs to be punted to the next release again.
<corebot> https://github.com/bitcoin/bitcoin/issues/31802 | Add bitcoin-{node,gui} to release binaries for IPC by Sjors · Pull Request #31802 · bitcoin/bitcoin · GitHub
jespada has joined #bitcoin-core-dev
purpleKarrot has joined #bitcoin-core-dev
stringintech has quit [Changing host]
stringintech has joined #bitcoin-core-dev
stringintech has quit [Quit: Client closed]
theStack has joined #bitcoin-core-dev
stringintech has joined #bitcoin-core-dev
jon_atack has joined #bitcoin-core-dev
jonatack has quit [Ping timeout: 252 seconds]
jespada has quit [Ping timeout: 252 seconds]
jespada has joined #bitcoin-core-dev
josie has quit [Server closed connection]
josie has joined #bitcoin-core-dev
<stringintech> Reviewing IBD staller detection logic; the condition for setting the staller variable in FindNextBlocks() differs from the condition for setting m_stalling_since for that staller after detection. The first condition checks if we've reached the end of the IBD window and found no blocks to download from the peer, but the second checks if there are
<stringintech> any in-flight requests from that peer (which could include block requests from earlier). I wondered whether this difference is intentional. (I can think of a scenario where no in-flight requests are resolved from any peer and we wouldn't trigger the timeout for any peer until the in-flight requests are cleared)
stringintech10 has joined #bitcoin-core-dev
f321x has quit [Quit: f321x]
stringintech10 has quit [Client Quit]
stringintech73 has joined #bitcoin-core-dev
stringintech has quit [Ping timeout: 250 seconds]
stringintech73 has quit [Client Quit]
stringintech has joined #bitcoin-core-dev
jonatack has joined #bitcoin-core-dev
<sipa> stringintech: hmm
zeropoint has joined #bitcoin-core-dev
jon_atack has quit [Ping timeout: 276 seconds]
<sipa> i'm probably the author of that code, but i don't recall that being intentional
<sipa> it's also possible it has been changed since i wrote it
<stringintech> sipa: Thanks! Wouldn't it make more sense if the second one would check for vToDownload.empty() instead of state.vBlocksInFlight.empty() since we could have requested a block from this peer but we could not because of the staller?
WizJin_ has joined #bitcoin-core-dev
WizJin has quit [Ping timeout: 260 seconds]
stringintech has quit [Ping timeout: 250 seconds]
stringintech has joined #bitcoin-core-dev
stringintech has quit [Ping timeout: 250 seconds]
jon_atack has joined #bitcoin-core-dev
jonatack has quit [Ping timeout: 252 seconds]
bugs_ has quit [Quit: Leaving]
l0rinc has joined #bitcoin-core-dev
___nick___ has joined #bitcoin-core-dev
___nick___ has quit [Client Quit]
<bitcoin-git> [bitcoin] achow101 pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/ec484bd5ce43...73972d56176a
<bitcoin-git> bitcoin/master 0173788 Ryan Ofsky: wallet: Translate [default wallet] string in progress messages
<bitcoin-git> bitcoin/master db225ce Ryan Ofsky: wallet, refactor: Replace GetDisplayName() with LogName()
<bitcoin-git> bitcoin/master 73972d5 Ava Chow: Merge bitcoin/bitcoin#31296: wallet: Translate [default wallet] string in ...
<bitcoin-git> [bitcoin] achow101 merged pull request #31296: wallet: Translate [default wallet] string in progress messages (master...pr/dtran) https://github.com/bitcoin/bitcoin/pull/31296
___nick___ has joined #bitcoin-core-dev
l0rinc has quit [Quit: l0rinc]
l0rinc has joined #bitcoin-core-dev
Christoph_ has quit [Quit: Christoph_]
jonatack has joined #bitcoin-core-dev
jon_atack has quit [Ping timeout: 272 seconds]
jespada_ has joined #bitcoin-core-dev
jespada has quit [Ping timeout: 276 seconds]
___nick___ has quit [Ping timeout: 252 seconds]
bitdex has quit [Ping timeout: 240 seconds]
l0rinc has quit [Quit: l0rinc]
VonNaturAustreVe has joined #bitcoin-core-dev
bitdex has joined #bitcoin-core-dev
zeropoint has quit [Quit: leaving]
jon_atack has joined #bitcoin-core-dev
jonatack has quit [Ping timeout: 272 seconds]
purpleKarrot has quit [Quit: purpleKarrot]
purpleKarrot has joined #bitcoin-core-dev
BGL has quit [Ping timeout: 248 seconds]
l0rinc has joined #bitcoin-core-dev
jonatack has joined #bitcoin-core-dev
jon_atack has quit [Ping timeout: 252 seconds]
jon_atack has joined #bitcoin-core-dev
jonatack has quit [Ping timeout: 252 seconds]
<bitcoin-git> [bitcoin] achow101 pushed 4 commits to master: https://github.com/bitcoin/bitcoin/compare/73972d56176a...dadf15f88cba
<bitcoin-git> bitcoin/master 266dd0e Anthony Towns: net_processing: drop MaybePunishNodeForTx
<bitcoin-git> bitcoin/master b29ae9e Anthony Towns: validation: only check input scripts once
<bitcoin-git> bitcoin/master 876dbdf Anthony Towns: tests: drop expect_disconnect behaviour for tx relay
<bitcoin-git> [bitcoin] achow101 merged pull request #33050: net, validation: don't punish peers for consensus-invalid txs (master...202507-nopunishtx) https://github.com/bitcoin/bitcoin/pull/33050
PaperSword has joined #bitcoin-core-dev
PaperSword has quit [Client Quit]
joetor5 has joined #bitcoin-core-dev
jerryf has quit [Ping timeout: 240 seconds]
ghost43 has joined #bitcoin-core-dev
ghost43_ has quit [Ping timeout: 240 seconds]
jerryf has joined #bitcoin-core-dev
nymius has joined #bitcoin-core-dev
BGL has joined #bitcoin-core-dev
l0rinc has quit [Quit: l0rinc]
flag has quit [Ping timeout: 265 seconds]
flag has joined #bitcoin-core-dev
l0rinc has joined #bitcoin-core-dev
purpleKarrot has quit [Quit: purpleKarrot]
purpleKarrot has joined #bitcoin-core-dev
aleggg has quit [Ping timeout: 260 seconds]
jonatack has joined #bitcoin-core-dev
jon_atack has quit [Ping timeout: 252 seconds]
jon_atack has joined #bitcoin-core-dev
jonatack has quit [Ping timeout: 252 seconds]
l0rinc has quit [Quit: l0rinc]
nymius has quit [Quit: nymius]
jonatack has joined #bitcoin-core-dev
jon_atack has quit [Ping timeout: 248 seconds]
VonNaturAustreVe has quit [Ping timeout: 255 seconds]
<lightlike> stringintech: (not online but maybe you'll see this in the logs) I think the idea behind the stalling logic is that we are in a stalling situation when a peer that could do work for us has to become idle because we can't assign it any.
<lightlike> We don't require that its assigned queue of 16 blocks must be full all of the time. So as long as state.vBlocksInFlight is non-empty, we are not in a stalling case yet, because the peer has still work scheduled and isn't idle.
<lightlike> I think the suggested alternative would also be possible (require all queues to be at the MAX_BLOCKS_IN_TRANSIT_PER_PEER limit), it would be stricter, resulting in earlier (and therefore more) stalling disconnections. I don't think though that any of the two approaches is inconsistent.
VonNaturAustreVe has joined #bitcoin-core-dev
VonNaturAustreVe has quit [Ping timeout: 255 seconds]
l0rinc has joined #bitcoin-core-dev