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
hardtotell has quit [Quit: pz]
ghost43_ has joined #bitcoin-core-dev
ghost43 has quit [Ping timeout: 252 seconds]
Guest23 has joined #bitcoin-core-dev
Guest23 has quit [Client Quit]
Guest87 has joined #bitcoin-core-dev
Guest87 has quit [Client Quit]
hardtotell has joined #bitcoin-core-dev
memset has quit [Remote host closed the connection]
memset has joined #bitcoin-core-dev
Guest74 has joined #bitcoin-core-dev
memset has quit [Remote host closed the connection]
memset has joined #bitcoin-core-dev
Guest74 has quit [Client Quit]
dzxzg2 has quit [Remote host closed the connection]
Guest97 has joined #bitcoin-core-dev
memset has quit [Remote host closed the connection]
memset has joined #bitcoin-core-dev
Guest97 has quit [Client Quit]
Guest3 has joined #bitcoin-core-dev
Guest3 has quit [Client Quit]
mudsip has joined #bitcoin-core-dev
mudsip has quit [Client Quit]
bomb-on has quit [Quit: aллилѹіа!]
cmirror has quit [Remote host closed the connection]
cmirror has joined #bitcoin-core-dev
durandal_ has quit [Ping timeout: 240 seconds]
brunoerg has quit [Ping timeout: 245 seconds]
bitdex has joined #bitcoin-core-dev
durandal_ has joined #bitcoin-core-dev
itsme has joined #bitcoin-core-dev
itsme has quit [Quit: Client closed]
justache has quit [Read error: Connection reset by peer]
sliv3r__ has quit [Ping timeout: 240 seconds]
justache has joined #bitcoin-core-dev
sliv3r__ has joined #bitcoin-core-dev
ghost43_ has quit [Ping timeout: 252 seconds]
ghost43 has joined #bitcoin-core-dev
f321x has joined #bitcoin-core-dev
bomb-on has joined #bitcoin-core-dev
afiore has quit [Remote host closed the connection]
afiore has joined #bitcoin-core-dev
bomb-on has quit [Quit: aллилѹіа!]
f321x has quit [Remote host closed the connection]
f321x has joined #bitcoin-core-dev
bomb-on has joined #bitcoin-core-dev
bomb-on has quit [Client Quit]
bomb-on has joined #bitcoin-core-dev
f321x has quit [Remote host closed the connection]
f321x has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] maflcko opened pull request #34053: lint: Remove confusing, redundant, and brittle lint-spelling (master...2512-less-lint) https://github.com/bitcoin/bitcoin/pull/34053
l0rinc has joined #bitcoin-core-dev
Guyver2 has joined #bitcoin-core-dev
Guyver2 has left #bitcoin-core-dev [Closing Window]
<bitcoin-git> [bitcoin] sedited opened pull request #34054: net processing: Add ibd check before processing block for txdownloadman (master...txdownloadman_ibd_check) https://github.com/bitcoin/bitcoin/pull/34054
f321x has quit [Ping timeout: 252 seconds]
f321x has joined #bitcoin-core-dev
l0rinc has quit [Quit: l0rinc]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 260 seconds]
<bitcoin-git> [bitcoin] fanquake closed pull request #32730: p2p: avoid traversing blocks (twice) during IBD (master...2025_net_avoid_traversing_block_twice) https://github.com/bitcoin/bitcoin/pull/32730
<bitcoin-git> [bitcoin] fanquake pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/d155fc12a0c7...597b8be223d4
<bitcoin-git> bitcoin/master cea443e Anthony Towns: net: Pass time to InactivityChecks fuctions
<bitcoin-git> bitcoin/master 5f5c1ea Anthony Towns: net: Cache -capturemessages setting
<bitcoin-git> bitcoin/master 597b8be merge-script: Merge bitcoin/bitcoin#34025: net: Waste less time in socket handling
<bitcoin-git> [bitcoin] fanquake merged pull request #34025: net: Waste less time in socket handling (master...202512-netsplit-opt) https://github.com/bitcoin/bitcoin/pull/34025
f321x has quit [Ping timeout: 252 seconds]
<fanquake> l0rinc: why are the benchmarking runs being done with IPC disabled? i.e in https://github.com/bitcoin/bitcoin/pull/31132#issuecomment-3617721711
f321x has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
f321x has quit [Ping timeout: 252 seconds]
f321x_ has joined #bitcoin-core-dev
l0rinc has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] HowHsu opened pull request #34057: test: add some txgraph tests (master...get_worst_chunk) https://github.com/bitcoin/bitcoin/pull/34057
<bitcoin-git> [bitcoin] billymcbip opened pull request #34058: test: Improve code coverage for pubkey checks (master...cov) https://github.com/bitcoin/bitcoin/pull/34058
<bitcoin-git> [bitcoin] fanquake pushed 4 commits to master: https://github.com/bitcoin/bitcoin/compare/597b8be223d4...938d7aacabd0
<bitcoin-git> bitcoin/master f2fd1aa Roman Zeyde: blockstorage: return an error code from `ReadRawBlock()`
<bitcoin-git> bitcoin/master 4e2af1c Roman Zeyde: blockstorage: allow reading partial block data from storage
<bitcoin-git> bitcoin/master 0713529 Roman Zeyde: rest: allow reading partial block data from storage
<bitcoin-git> [bitcoin] fanquake merged pull request #33657: rest: allow reading partial block data from storage (master...romanz/rest-blockdata) https://github.com/bitcoin/bitcoin/pull/33657
<sipa> brunoerg: do you incorporate fuzz tests in your mutation testing?
<brunoerg> sipa: no, only unit and functional tests.
<sipa> brunoerg: would it be worth including the fuzz tests? it doesn't need a full fuzz build, just running the normal-build fuzz binary on the qa-assets corpus
<brunoerg> Yes, I will add it. I tested the txgraph mutants with fuzzing + qa-assets corpus and it killed over 95% of the mutants, much better then unit+func.
<sipa> that's because txgraph basically only has fuzz tests :p
<sipa> cool, but that explains your report in #34057
<corebot> https://github.com/bitcoin/bitcoin/issues/34057 | test: add some txgraph tests by HowHsu · Pull Request #34057 · bitcoin/bitcoin · GitHub
<brunoerg> yes, but fuzzing is known for not being able to kill mutants for obvious reasons. however, i did an empirical evaluation on some targets and showed that fuzzing can achieve a high mutation score depending on how the target is written - https://sol.sbc.org.br/index.php/sast/article/view/36885/36671.
<brunoerg> sipa: yes, it explains. i will add fuzzing and update the report.
<dergoegge> brunoerg: you could also add bitcoinfuzz (the script interpreter differential fuzzing should be able to kill a lot of interpreter mutants)
brunoerg has quit [Ping timeout: 250 seconds]
brunoerg has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] maflcko opened pull request #34059: refactor: Use NodeClock::time_point for m_addr_token_timestamp (master...2512-net-less-GetTime) https://github.com/bitcoin/bitcoin/pull/34059
brunoerg has quit [Ping timeout: 260 seconds]
bugs_ has joined #bitcoin-core-dev
jonatack has joined #bitcoin-core-dev
kevkevin has joined #bitcoin-core-dev
kevkevin has quit [Remote host closed the connection]
l0rinc has quit [Quit: l0rinc]
l0rinc has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
<brunoerg> dergoegge: yes, great idea!
kevkevin has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 265 seconds]
f321x_ has quit [Ping timeout: 252 seconds]
<bitcoin-git> [bitcoin] stratospher opened pull request #34060: test: fix race condition in p2p_v2_misbehaving.py peerid assertion (master...2025_12_fix_peerid_race) https://github.com/bitcoin/bitcoin/pull/34060
<bitcoin-git> [bitcoin] marcofleon opened pull request #34061: fuzz: Fix bugs in `clusterlin_postlinearize_tree` target (master...2025/12/clusterlin-fuzz-fixes) https://github.com/bitcoin/bitcoin/pull/34061
jonatack has quit [Ping timeout: 252 seconds]
bitdex has quit [Quit: = ""]
bomb-on has quit [Quit: aллилѹіа!]
brunoerg has joined #bitcoin-core-dev
bomb-on has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 244 seconds]
brunoerg has joined #bitcoin-core-dev
l0rinc has quit [Quit: l0rinc]
l0rinc has joined #bitcoin-core-dev
abubakarsadiq has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] rustaceanrob opened pull request #34063: Make `transaction_indentifier` hex string constructor evaluated at comptime (master...12-12-consteval-txid) https://github.com/bitcoin/bitcoin/pull/34063
bugs_ has quit [Quit: Leaving]
saturday7 has joined #bitcoin-core-dev
saturday- has quit [Ping timeout: 240 seconds]
f321x has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 256 seconds]
saturday7 has quit [Ping timeout: 240 seconds]
saturday7 has joined #bitcoin-core-dev
<andrewtoth> fanquake: I'm not sure about l0rinc, but for me when I run cmake -B build it gives me an error message that tells me to solve it by building with -DENABLE_IPC=OFF. So of course I take the easy route.
<sipa> may be worth investigating why IPC doesn't work for you!
<sipa> (independently of the value of benchmarking/testing)
<andrewtoth> It says I need to install capn proto
f321x has quit [Quit: f321x]
<sipa> yeah
<fanquake> If it’s possible for you to resolve, great. As it’d be good for benchmarking to remain as close as possible to default/release build config
<l0rinc> I haven't seen the mention: as andrewtoth also mentioned, when it was introduced it wasn't working reliably, was easier to ignore it. I will check again if it's still needed, thanks for the hint
<andrewtoth> Running the upgrade command in doc/build-unix.md does not work. I get an incompatible version.
<fanquake> what systems are you on?
<andrewtoth> pop_os 22.04
<fanquake> If your package mama get is installing 1.1 or 1.2 and you’re still seeing problems, please open an issue. If it’s pre 1 or 1.0, I’d suggest installing 1.2 from source
<andrewtoth> It's installing 0.8
<sipa> i assume "mama get" is a typo for "manager", but i like the notion of a "package mama"
<andrewtoth> yeah I thought that's what the cool kids call it so didn't mention it
kevkevin has quit [Remote host closed the connection]
kevkevin has joined #bitcoin-core-dev
l0rinc has quit [Quit: l0rinc]
kevkevin has quit [Remote host closed the connection]
l0rinc has joined #bitcoin-core-dev
WizJin_ has joined #bitcoin-core-dev
WizJin__ has quit [Ping timeout: 256 seconds]
jerryf has quit [Remote host closed the connection]
jerryf has joined #bitcoin-core-dev
abubakarsadiq has quit [Quit: Connection closed for inactivity]
WizJin__ has joined #bitcoin-core-dev
WizJin__ has quit [Max SendQ exceeded]
WizJin_ has quit [Ping timeout: 245 seconds]
WizJin__ has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
WizJin__ has quit [Max SendQ exceeded]
WizJin__ has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 252 seconds]
kevkevin has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
l0rinc has quit [Quit: l0rinc]
scgbckbone has joined #bitcoin-core-dev
jerryf has quit [Remote host closed the connection]
jerryf has joined #bitcoin-core-dev
memset has quit [Remote host closed the connection]
memset has joined #bitcoin-core-dev
<sipa> andrewtoth: i guess we're not cool
<andrewtoth> my package mama thinks I'm cool
<brunoerg> sipa: fyi running mutation testing for #32545 - including fuzzing :) as soon as I have the results will share them.
<corebot> https://github.com/bitcoin/bitcoin/issues/32545 | Replace cluster linearization algorithm with SFL by sipa · Pull Request #32545 · bitcoin/bitcoin · GitHub
<sipa> brunoerg: cool!
l0rinc has joined #bitcoin-core-dev
___nick___ has joined #bitcoin-core-dev
___nick___ has quit [Client Quit]
scgbckbone has quit [Changing host]
scgbckbone has joined #bitcoin-core-dev
<scgbckbone> hey, just tested current bitcoind master, it seems it still does not support musig2 in miniscript..below is the policy I tested:
<scgbckbone> tr(musig(@0,@1,@2)/**,{and_v(v:pk(musig(@0,@1)/**),older(12960)),{and_v(v:pk(musig(@0,@2)/**),older(12960)),and_v(v:pk(musig(@1,@2)/**),older(12960))}})
<scgbckbone> Q: is musig inside miniscript already supported or not? Is there any branch attempting to (I couldn't find any).
<scgbckbone> I'm getting following error:
<scgbckbone> error code: -5
<scgbckbone> error message: Invalid musig() expression
<scgbckbone> error is caused by the first leaf where "expr" after split is missing terminating parentheses of musig expression
___nick___ has joined #bitcoin-core-dev
brunoerg has quit [Read error: Connection reset by peer]
<sipa> @0 is not valid miniscript
<sipa> you need a key expression there
<sipa> also i don't know about /**
<sipa> scgbckbone: those look like they're BIP 388 wallet policies, not descriptors
___nick___ has quit [Ping timeout: 246 seconds]
___nick___ has joined #bitcoin-core-dev
bomb-on has quit [Quit: aллилѹіа!]
bomb-on has joined #bitcoin-core-dev
<scgbckbone> yeah, I didn't want to paste descriptor here, as it becomes mess - this is BIP388 policy - but the error I pasted is from actual descriptor
<scgbckbone> 'tr(musig([4f7b462a/88h/1h/0h]tpubDDEx9JidouucRTTS2ULRvkjVkt3ENkjUhJ8tTACNbzgmd9UTVZzXKbN8YGnxAekbBUPe93VgH46uPR484fgwYcmCy9fvcc6rhUXgLkezq5h,[7ca287c4/88h/1h/0h]tpubDCv3Ckn9UAMUDgZmbEkJNrLdAnNZai5CzdCUyEdCW7pTbPPfRKDiV1VSMkCfNzpdXLT3ZFLL4z88w5szrYwv4T2NECgodJzk32uupzTtcuF,[0f056943/88h/1h/0h]tpubDD8WTkCBJwj5WHR8S1gr2y64LcFSwEic38zGRKEuHXUUPRacZyTR
<scgbckbone> EJp9vZ5fayGVqEzzBacffrkK1g4CJRA5PgXE7MkrsLcUWbeixkLaNM6)/<0;1>/*,{and_v(v:pk(musig([4f7b462a/88h/1h/0h]tpubDDEx9JidouucRTTS2ULRvkjVkt3ENkjUhJ8tTACNbzgmd9UTVZzXKbN8YGnxAekbBUPe93VgH46uPR484fgwYcmCy9fvcc6rhUXgLkezq5h,[7ca287c4/88h/1h/0h]tpubDCv3Ckn9UAMUDgZmbEkJNrLdAnNZai5CzdCUyEdCW7pTbPPfRKDiV1VSMkCfNzpdXLT3ZFLL4z88w5szrYwv4T2NECgodJzk32uupzTtcuF)/<0
<scgbckbone> ;1>/*),older(12960)),{and_v(v:pk(musig([4f7b462a/88h/1h/0h]tpubDDEx9JidouucRTTS2ULRvkjVkt3ENkjUhJ8tTACNbzgmd9UTVZzXKbN8YGnxAekbBUPe93VgH46uPR484fgwYcmCy9fvcc6rhUXgLkezq5h,[0f056943/88h/1h/0h]tpubDD8WTkCBJwj5WHR8S1gr2y64LcFSwEic38zGRKEuHXUUPRacZyTREJp9vZ5fayGVqEzzBacffrkK1g4CJRA5PgXE7MkrsLcUWbeixkLaNM6)/<0;1>/*),older(12960)),and_v(v:pk(musig([7ca28
<scgbckbone> 7c4/88h/1h/0h]tpubDCv3Ckn9UAMUDgZmbEkJNrLdAnNZai5CzdCUyEdCW7pTbPPfRKDiV1VSMkCfNzpdXLT3ZFLL4z88w5szrYwv4T2NECgodJzk32uupzTtcuF,[0f056943/88h/1h/0h]tpubDD8WTkCBJwj5WHR8S1gr2y64LcFSwEic38zGRKEuHXUUPRacZyTREJp9vZ5fayGVqEzzBacffrkK1g4CJRA5PgXE7MkrsLcUWbeixkLaNM6)/<0;1>/*),older(12960))}})'
<sipa> hmm, unsure then
<sipa> scgbckbone: it's certainly supposed to work; there are unit and functional tests that exercise musig descriptors
<scgbckbone> I briefly went over the test suite but I could only see tapscrits with simple pk(musig(...)) leaves - so I thought that maybe it is not implemented yet
<sipa> oh, i see, your question is specifically about using it inside miniscript
<sipa> achow101: ?
l0rinc has quit [Quit: l0rinc]
l0rinc has joined #bitcoin-core-dev
_xamelion has quit [Remote host closed the connection]
scgbckbone has quit [Ping timeout: 272 seconds]
___nick___ has quit [Ping timeout: 264 seconds]
__nick__ has joined #bitcoin-core-dev
memset has quit [Remote host closed the connection]
memset has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] sajadvilsin-cmyk opened pull request #34072: Create sajad (master...patch-1) https://github.com/bitcoin/bitcoin/pull/34072
<bitcoin-git> [bitcoin] DrahtBot closed pull request #34072: Create sajad (master...patch-1) https://github.com/bitcoin/bitcoin/pull/34072
l0rinc has quit [Ping timeout: 244 seconds]
scgbckbone has joined #bitcoin-core-dev
scgbckbone has quit [Changing host]
scgbckbone has joined #bitcoin-core-dev
jonatack has joined #bitcoin-core-dev