bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/c8bae2be341c...69986de383ff
<bitcoin-git> bitcoin/master b65a25a Martin Zumsande: log: improve addrman logging
<bitcoin-git> bitcoin/master 69986de fanquake: Merge bitcoin/bitcoin#22839: log: improve addrman logging
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake merged pull request #22839: log: improve addrman logging (master...202108_log_addrman) https://github.com/bitcoin/bitcoin/pull/22839
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
earnestly has quit [Ping timeout: 260 seconds]
jespada has quit [Quit: My MacBook has gone to sleep. ZZZzzz…]
schmidty has quit [Ping timeout: 264 seconds]
jamesob has quit [Ping timeout: 252 seconds]
jamesob has joined #bitcoin-core-dev
stick has quit [Ping timeout: 252 seconds]
stick has joined #bitcoin-core-dev
schmidty has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/69986de383ff...4229f71bf8c2
<bitcoin-git> bitcoin/master 17ae260 Cory Fields: build: remove build stubs for external leveldb
<bitcoin-git> bitcoin/master 4229f71 fanquake: Merge bitcoin/bitcoin#23282: build: remove build stubs for external leveld...
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake merged pull request #23282: build: remove build stubs for external leveldb (master...no_external_leveldb) https://github.com/bitcoin/bitcoin/pull/23282
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
freesprung has joined #bitcoin-core-dev
<cfields> kanzure: thanks for hosting :)
<jamesob> does anyone understand this test failure? I don't see why my PR would be causing it, but I also don't see the same failure on other PRs: https://github.com/bitcoin/bitcoin/pull/23289/checks?check_run_id=3955413522
<fanquake> jamesob: see #22936. I'd assume the descriptors failure is intermittent, which results in the other error, neither of which are related to your change
<gribble> https://github.com/bitcoin/bitcoin/issues/22936 | test: dont try and use os.killpg() on Windows by fanquake · Pull Request #22936 · bitcoin/bitcoin · GitHub
<jamesob> awesome, will review now
<jamesob> thanks fanquake
emcy has quit [Quit: Leaving]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake pushed 4 commits to master: https://github.com/bitcoin/bitcoin/compare/4229f71bf8c2...c53e95f22cc2
<bitcoin-git> bitcoin/master fac303c MarcoFalke: refactor: Remove unused MakeUCharSpan
<bitcoin-git> bitcoin/master fa2d611 MarcoFalke: style: Sort
<bitcoin-git> bitcoin/master fa1e5de MarcoFalke: scripted-diff: Move bloom to src/common
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake merged pull request #23137: Move-only: bloom to src/common (master...2109-srcCommon) https://github.com/bitcoin/bitcoin/pull/23137
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
emcy has joined #bitcoin-core-dev
mikehu44 has joined #bitcoin-core-dev
gnaf has quit [Ping timeout: 260 seconds]
mikehu44_ has joined #bitcoin-core-dev
mikehu44_ has quit [Ping timeout: 258 seconds]
mikehu44_ has joined #bitcoin-core-dev
mikehu44_ has quit [Ping timeout: 260 seconds]
mikehu44_ has joined #bitcoin-core-dev
mikehu44_ has quit [Ping timeout: 260 seconds]
mikehu44_ has joined #bitcoin-core-dev
bitdex has joined #bitcoin-core-dev
mikehu44_ has quit [Ping timeout: 260 seconds]
mikehu44_ has joined #bitcoin-core-dev
gnaf has joined #bitcoin-core-dev
freesprung has quit [Quit: zzz]
gnaf has quit [Ping timeout: 258 seconds]
cmirror has quit [Remote host closed the connection]
cmirror has joined #bitcoin-core-dev
freesprung has joined #bitcoin-core-dev
BlueMatt has quit [Quit: Quit]
BlueMatt has joined #bitcoin-core-dev
belcher has quit [Ping timeout: 260 seconds]
ArctVaulMarsHMPJ has joined #bitcoin-core-dev
belcher has joined #bitcoin-core-dev
freesprung has quit [Ping timeout: 260 seconds]
ArctVaulMarsHMPJ has quit [Quit: EXIT]
gnaf has joined #bitcoin-core-dev
freesprung has joined #bitcoin-core-dev
freesprung has quit [Changing host]
freesprung has joined #bitcoin-core-dev
freesprung has quit [Quit: zzz]
freesprung has joined #bitcoin-core-dev
mikehu44 has quit [Quit: https://quassel-irc.org - Chat comfortably. Anywhere.]
mikehu44 has joined #bitcoin-core-dev
TheRec has quit [Read error: Connection reset by peer]
TheRec has joined #bitcoin-core-dev
TheRec has quit [Changing host]
TheRec has joined #bitcoin-core-dev
baakeydow has quit [Ping timeout: 245 seconds]
yanmaani1 has quit [Remote host closed the connection]
yanmaani1 has joined #bitcoin-core-dev
baakeydow has joined #bitcoin-core-dev
BlueMatt_ has joined #bitcoin-core-dev
BlueMatt has quit [Ping timeout: 264 seconds]
mikehu44 has quit [Quit: https://quassel-irc.org - Chat comfortably. Anywhere.]
vasild has quit [Remote host closed the connection]
vasild has joined #bitcoin-core-dev
vasild has quit [Remote host closed the connection]
vasild has joined #bitcoin-core-dev
smartin has joined #bitcoin-core-dev
goatpig has joined #bitcoin-core-dev
vasild has quit [Remote host closed the connection]
vasild has joined #bitcoin-core-dev
ravan has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to 22.x: https://github.com/bitcoin/bitcoin/compare/12b315484d1a...56156a1f08b0
<bitcoin-git> bitcoin/22.x e959b46 fanquake: build: explicitly disable libsecp256k1 openssl based tests
<bitcoin-git> bitcoin/22.x 56156a1 MarcoFalke: Merge bitcoin/bitcoin#23315: [22.x] build: explicitly disable libsecp256k1...
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #23315: [22.x] build: explicitly disable libsecp256k1 openssl based tests (22.x...22_x_disable_libsecp256k1_openssl_tests) https://github.com/bitcoin/bitcoin/pull/23315
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
gnaf has quit [Quit: Konversation terminated!]
earnestly has joined #bitcoin-core-dev
ravan has quit [Remote host closed the connection]
Guyver2 has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> bitcoin/master fa44406 MarcoFalke: ci: Disable syscall sandbox in valgrind functional tests
<bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/c53e95f22cc2...3e3f53c77e73
<bitcoin-git> bitcoin/master 3e3f53c MarcoFalke: Merge bitcoin/bitcoin#23328: ci: Disable syscall sandbox in valgrind funct...
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #23328: ci: Disable syscall sandbox in valgrind functional tests (master...2110-ciValSan) https://github.com/bitcoin/bitcoin/pull/23328
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/3e3f53c77e73...9aa4ddb410b2
<bitcoin-git> bitcoin/master 9aa4ddb MarcoFalke: Merge bitcoin/bitcoin#23287: test: get and decode tx with a single `gettra...
<bitcoin-git> bitcoin/master 130ee48 Sebastian Falbesoner: test: get and decode tx with a single `gettransaction` RPC call
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #23287: test: get and decode tx with a single `gettransaction` RPC call (master...202110-test-fetch_and_decode_tx_with_single_RPC_call) https://github.com/bitcoin/bitcoin/pull/23287
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
Guest48 has quit [Quit: Client closed]
dviola has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/9aa4ddb410b2...548ad5ef7f60
<bitcoin-git> bitcoin/master 4ac8c89 Sebastian Falbesoner: test: check that bumpfee RPC fails for txs with descendants in mempool
<bitcoin-git> bitcoin/master 548ad5e MarcoFalke: Merge bitcoin/bitcoin#23281: test: check that bumpfee RPC fails for txs wi...
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #23281: test: check that bumpfee RPC fails for txs with descendants in mempool (master...202110-test-bumpfee_test_descendant_in_mempool) https://github.com/bitcoin/bitcoin/pull/23281
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
dodo has quit [Remote host closed the connection]
haakon has quit [Ping timeout: 252 seconds]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/548ad5ef7f60...88fc7950f8db
<bitcoin-git> bitcoin/master b7884dd brunoerg: test: bip125-replaceable in listsinceblock
<bitcoin-git> bitcoin/master 88fc795 MarcoFalke: Merge bitcoin/bitcoin#23267: test: bip125-replaceable in listsinceblock
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #23267: test: bip125-replaceable in listsinceblock (master...2021-10-test-bip125-listsinceblock) https://github.com/bitcoin/bitcoin/pull/23267
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
emcy_ has joined #bitcoin-core-dev
haakon has joined #bitcoin-core-dev
emcy has quit [Ping timeout: 260 seconds]
haakon has quit [Ping timeout: 260 seconds]
sebx2a has quit [Ping timeout: 252 seconds]
sebx2a has joined #bitcoin-core-dev
vasild has quit [Remote host closed the connection]
vasild has joined #bitcoin-core-dev
haakon has joined #bitcoin-core-dev
dodo has joined #bitcoin-core-dev
AaronvanW has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke opened pull request #23331: test: Replace log with assert_equal in wallet_abandonconflict (master...2110-testAssert) https://github.com/bitcoin/bitcoin/pull/23331
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
mikehu44 has joined #bitcoin-core-dev
mikehu44 has quit [Ping timeout: 260 seconds]
Guyver2 has left #bitcoin-core-dev [Closing Window]
promag has joined #bitcoin-core-dev
promag_ has quit [Read error: Connection reset by peer]
freesprung has quit [Ping timeout: 260 seconds]
esraa has joined #bitcoin-core-dev
esraa has quit [Client Quit]
emcy has joined #bitcoin-core-dev
luke-jr has quit [Quit: ZNC - http://znc.sourceforge.net]
luke-jr has joined #bitcoin-core-dev
emcy_ has quit [Ping timeout: 260 seconds]
Victorsueca has joined #bitcoin-core-dev
luke-jr has quit [Quit: ZNC - http://znc.sourceforge.net]
luke-jr has joined #bitcoin-core-dev
luke-jr has quit [Quit: ZNC - http://znc.sourceforge.net]
luke-jr has joined #bitcoin-core-dev
mikehu44 has joined #bitcoin-core-dev
ravan has joined #bitcoin-core-dev
jespada has joined #bitcoin-core-dev
bitdex has quit [Quit: = ""]
ravan has quit [Ping timeout: 252 seconds]
goatpig has quit [Remote host closed the connection]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke opened pull request #23332: doc: Fix CWalletTx::Confirmation doc (master...2110-docWalletConfirm) https://github.com/bitcoin/bitcoin/pull/23332
jtrag has joined #bitcoin-core-dev
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
ravan has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] theStack opened pull request #23333: wallet: fix segfault by avoiding invalid default-ctored `external_spk_managers` entry (master...202110-wallet-fix_getwalletinfo_segfault_after_importing_descriptor) https://github.com/bitcoin/bitcoin/pull/23333
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
sipsorcery has joined #bitcoin-core-dev
gnaf has joined #bitcoin-core-dev
mikehu44_ has quit [Ping timeout: 260 seconds]
yanmaani1 has quit [Ping timeout: 276 seconds]
white5moke has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/88fc7950f8db...ee1294f155a4
<bitcoin-git> bitcoin/master 96f469f Jon Atack: netinfo: print peer counts for all reachable networks
<bitcoin-git> bitcoin/master ee1294f W. J. van der Laan: Merge bitcoin/bitcoin#23324: Print peer counts for all reachable networks ...
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] laanwj merged pull request #23324: Print peer counts for all reachable networks in -netinfo (master...netinfo-print-peer-counts-for-all-reachable-networks) https://github.com/bitcoin/bitcoin/pull/23324
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
yanmaani1 has joined #bitcoin-core-dev
sipsorcery has quit [Ping timeout: 260 seconds]
yanmaani1 has quit [Ping timeout: 276 seconds]
yanmaani1 has joined #bitcoin-core-dev
yanmaani1 has quit [Ping timeout: 276 seconds]
sipsorcery has joined #bitcoin-core-dev
yanmaani1 has joined #bitcoin-core-dev
Talkless has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/ee1294f155a4...f41aa81c996f
<bitcoin-git> bitcoin/master be7f413 =: Fix K1/K2 use in the comments in ChaCha20-Poly1305 AEAD
<bitcoin-git> bitcoin/master f41aa81 W. J. van der Laan: Merge bitcoin/bitcoin#23271: crypto: Fix K1/K2 use in the comments in ChaC...
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] laanwj merged pull request #23271: crypto: Fix K1/K2 use in the comments in ChaCha20-Poly1305 AEAD (master...fix-k1-k2) https://github.com/bitcoin/bitcoin/pull/23271
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
sipsorcery has quit [Ping timeout: 260 seconds]
sipsorcery has joined #bitcoin-core-dev
yanmaani1 has quit [Ping timeout: 276 seconds]
Guest48 has joined #bitcoin-core-dev
white5moke has quit [Quit: Leaving...]
white5moke has joined #bitcoin-core-dev
white5moke has quit [Remote host closed the connection]
yanmaani1 has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/f41aa81c996f...8a083bc5b57f
<bitcoin-git> bitcoin/master fadf118 MarcoFalke: p2p: Use mocktime for ping timeout
<bitcoin-git> bitcoin/master 8a083bc W. J. van der Laan: Merge bitcoin/bitcoin#23218: p2p: Use mocktime for ping timeout
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
mikehu44 has quit [Ping timeout: 260 seconds]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] laanwj merged pull request #23218: p2p: Use mocktime for ping timeout (master...2110-mockPingTime) https://github.com/bitcoin/bitcoin/pull/23218
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/8a083bc5b57f...179ce098333b
<bitcoin-git> bitcoin/master 1946af2 Kennan Mell: Add comment to COIN constant.
<bitcoin-git> bitcoin/master 179ce09 W. J. van der Laan: Merge bitcoin/bitcoin#23293: doc: Add comment to COIN constant.
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] laanwj merged pull request #23293: doc: Add comment to COIN constant (master...master) https://github.com/bitcoin/bitcoin/pull/23293
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
Guest9057 has joined #bitcoin-core-dev
yanmaani1 has quit [Ping timeout: 276 seconds]
yanmaani1 has joined #bitcoin-core-dev
jonatack has joined #bitcoin-core-dev
bomb-on has joined #bitcoin-core-dev
<hebasto> meeting?
<josibake> +1
<michaelfolkson> Don't think my proposed meeting topics have been picked up on https://gnusha.org/bitcoin-core-dev/proposedmeetingtopics.txt
<michaelfolkson> Ah I think gnusha was down, that may be it
<michaelfolkson> You here laanwj?
<meshcollider> <michaelfolkson> #proposedmeetingtopic Encouraging people to propose meeting topics for the weekly IRC Core dev meetings
<meshcollider> <michaelfolkson> #proposedmeetingtopic Attempting to transition to full RBF over multiple major versions
<laanwj> #startmetting
<jonasschnelli> hi
<laanwj> #startmeeting
<meshcollider> hi
<sipa> hi
<cfields> hi
<hebasto> hi
<michaelfolkson> hi
<kvaciral[m]> hi
<ariard> hi
<jonatack> hi
<lightlike> hi
<laanwj> #bitcoin-core-dev Meeting: achow101 _aj_ amiti ariard BlueMatt cfields Chris_Stewart_5 darosior digi_james dongcarl elichai2 emilengler fanquake fjahr gleb glozow gmaxwell gwillen hebasto instagibbs jamesob jarolrod jb55 jeremyrubin jl2012 jnewbery jonasschnelli jonatack jtimon kallewoof kanzure kvaciral laanwj lightlike luke-jr maaku marcofalke meshcollider michagogo moneyball morcos
<laanwj> nehan NicolasDorier paveljanik petertodd phantomcircuit promag provoostenator ryanofsky sdaftuar sipa vasild
<achow101> hi
<jamesob> hi
<jonasschnelli> #proposedmeetingtopic some personal information from myside
<josibake> hi
Guest59 has joined #bitcoin-core-dev
<sipa> #proposedmeetingtopic: 23306 proposes (and is a first step towards) getting rid of the prefer-port-8333 policy on the network
Guest5960 has joined #bitcoin-core-dev
Guest59 has quit [Client Quit]
<laanwj> other proposed meeting topics: tag 0.20.3-final (fanquake) Encouraging people to propose meeting topics for the weekly IRC Core dev meetings (michaelfolkson) Attempting to transition to full RBF over multiple major versions (michaelfolkson)
<laanwj> #topic High priority for review
<fjahr> hi
<michaelfolkson> Mine aren't urgent and can be pushed to another week
lsilva_ has joined #bitcoin-core-dev
Guest5960 has quit [Client Quit]
yanmaani1 has quit [Remote host closed the connection]
<laanwj> https://github.com/bitcoin/bitcoin/projects/8 has 10 blockers, 1 chasing concept ACK
Guest92 has joined #bitcoin-core-dev
yanmaani1 has joined #bitcoin-core-dev
<laanwj> anything to add/remove, that is ready for review, or that you'd like to give an update on?
<meshcollider> I'd quite like to have #16807 on there
<gribble> https://github.com/bitcoin/bitcoin/issues/16807 | Let validateaddress locate error in Bech32 address by meshcollider · Pull Request #16807 · bitcoin/bitcoin · GitHub
<ariard> maybe either #22674 or #23121, i think the former is more ready
<gribble> https://github.com/bitcoin/bitcoin/issues/22674 | validation: mempool validation and submission for packages of 1 child + parents by glozow · Pull Request #22674 · bitcoin/bitcoin · GitHub
<gribble> https://github.com/bitcoin/bitcoin/issues/23121 | [policy] check ancestor feerate in RBF, remove BIP125 Rule2 by glozow · Pull Request #23121 · bitcoin/bitcoin · GitHub
<laanwj> meshcollider: added
<meshcollider> Thanks!
<laanwj> ariard: i don't have an opinion which one, maybe someone else has
<ariard> yeah if glozow is around :)
Talkless has quit [Quit: Konversation terminated!]
<cfields> just a quick note: #23114 (minisketch integration, already listed as hi prio) is ~ready, but also requires review of minisketch itself. So... review beg for minisketch itself as well :)
<gribble> https://github.com/bitcoin/bitcoin/issues/23114 | Add minisketch subtree and integrate into build/test by fanquake · Pull Request #23114 · bitcoin/bitcoin · GitHub
<sipa> ACK minisketch!
* sipa hdies
<sipa> *hides
<josibake> reading that as sip dies was much more dramatic
jarthur has joined #bitcoin-core-dev
<michaelfolkson> Definite ACK for Minisketch. You mean on the Core PR cfields?
<sipa> i meant mine as: i'm obviously ack on the minisketch code itself, but i doubt it counts for much
<cfields> michaelfolkson: I mean the minisketch impl itself also needs review, not just the concept of merging it into core.
<laanwj> ariard: ok, added #22674 as it is more clearly a blocker for further work
<michaelfolkson> So that is reviewing the Minisketch repo rather than a particular PR?
<gribble> https://github.com/bitcoin/bitcoin/issues/22674 | validation: mempool validation and submission for packages of 1 child + parents by glozow · Pull Request #22674 · bitcoin/bitcoin · GitHub
<michaelfolkson> That's great that the Minisketch build stuff is ~ready
<laanwj> i'll have a look at the minisketch impl
Guest92 has quit [Quit: Client closed]
<michaelfolkson> Repo: https://github.com/sipa/minisketch
<sipa> FWIW, there have been two review clubs on the minisketch algorithm and code; https://bitcoincore.reviews/minisketch-26-2 and https://bitcoincore.reviews/minisketch
<laanwj> neat
<cfields> sipa: ah, thanks, good to know. I missed those.
<cfields> anyway, didn't mean to turn it into a topic, just a quick note.
<michaelfolkson> And post those reviews of the repo generally on #23114
<gribble> https://github.com/bitcoin/bitcoin/issues/23114 | Add minisketch subtree and integrate into build/test by fanquake · Pull Request #23114 · bitcoin/bitcoin · GitHub
<sipa> oh, there were three; https://bitcoincore.reviews/minisketch-26 too
<laanwj> haah
<jonatack> those were really interesting review clubs. i recall testing and reviewing minisketch, will return to it
<laanwj> so it's not exactly that it had a lack of review
smartin has quit [Quit: smartin]
<ariard> yeah i read the paper a while back https://eprint.iacr.org/2003/235.pdf, it was fun
<sipa> regarding testing, there are exhaustive tests for various small fields (smaller than the one we'd use, but most code is shared across)
<cfields> heh, ok, review beg retracted. I was just responding to a few "reviewed the integration but not the impl" on that PR, but apparently review has been happening too :)
<michaelfolkson> cfields: A few months ago :)
<laanwj> cfields: it wasn't entirely clear to me either so it's good to have brought it up imo
<michaelfolkson> +1
<laanwj> #topic Some personal information from myside (jonasschnelli)
<jonasschnelli> As most of you have probably recognized, I was pretty inactive during the last months
<jonasschnelli> Today, I'd like to inform you that I will step down as maintainer and contributor
<sipa> jonasschnelli: sad to see you go
<cfields> :(
<jonasschnelli> I will remove my committer rights immediately
<laanwj> hope you'll be back some day!
<jonasschnelli> The reasons are mostly a shift in interest as well as rising legal risks (it wasn't you :)
<laanwj> but i get it
<jonasschnelli> and who know,... i might come back later down the road.
<cfields> jonasschnelli: hope everything's ok. there will always room for you here :)
<jamesob> Sad to see you go jonasschnelli but enjoy what other things life has to offer
<jamesob> Come back anytime :)
<sipa> and of course, good luck on wherever your path may lead
<jonasschnelli> thanks all.
<jonasschnelli> As for the macOS signing keys, I think either achow or fanquake would be a good fit
<hebasto> come back soon :)
<luke-jr> aren't the signing keys held by a corporate entity?
<laanwj> yes,good luck in whatever else you're going to do
rex4539 has joined #bitcoin-core-dev
<jonasschnelli> luke-jr: legaly yes,.. but technically someone needs to sign it
<jonasschnelli> I’ll remain the legal „president“ of the „Bitcoin Core Code Signing Association“ in Switzerland (the construct for getting code sign certificates, currently active for macOS but only a backup for the Delaware LLC that holds the win certificate)
<jonasschnelli> Off course, I’ll stick around here and can help where requested.
<jonatack> sad to hear that but understandable...i hope you'll be catching some waves and enjoying life
<cfields> jonasschnelli: also, thanks for being responsible and being willing to revoke your own access. much appreciated.
<laanwj> jonatack: that's good to hear
<jonasschnelli> sure things!
<laanwj> cfields: +1
<sipa> indeed
<jonasschnelli> If someone likes to continue bitcoinbuilds.org, please DM me (server costs are covered)
<meshcollider> Thanks for all your work over the years jonasschnelli! Best of luck for whatever comes next for you :)
<jonasschnelli> thanks meshcollider!
<achow101> sad to see you go, and good luck with whatever else you decide to do
<michaelfolkson> +1
<luke-jr> jonasschnelli: does it need much babysitting?
<jonasschnelli> luke-jr: yes... it need some tweaking (and some love)
<ariard> Yeah thanks for the numerous years of hard work, hope you'll be back
<achow101> in terms of the macOS code signing key, I can take that over if everyone feels that isn't too centralizing
<laanwj> jonasschnelli: re if no one else does it i'm ok with doing it
<achow101> I can also get one issued to the Delaware LLC
<laanwj> +bitcoinbuilds
<luke-jr> achow101: we shouldn't be putting trust in these keys either way *shrug*
<josibake> echoing everyone else, thank your for your contributions and best of luck in whatever you do next
<luke-jr> laanwj: thanks
<josibake> thank you*
<jonasschnelli> laanwj: great to hear. It's quite a mess and someone really loving shell scipts and virtualization should tackle it
<fjahr> jonasschnelli: thank you also from me for all the things already mentioned!
<laanwj> #topic Getting rid of the prefer-port-8333 policy (sipa)
<sipa> just a very short topic
<laanwj> [#23306 proposes (and is a first step towards) getting rid of the prefer-port-8333 policy on the network]
<gribble> https://github.com/bitcoin/bitcoin/issues/23306 | Make AddrMan support multiple ports per IP by sipa · Pull Request #23306 · bitcoin/bitcoin · GitHub
<sipa> it's all in the PR, i just wanted to make sure people saw
rex4539 has quit []
<laanwj> will check it out!
<sipa> as it's a potentially far-reaching policy change (the PR doesn't actually change the policy, but it's a preparation for it)
<jonatack> +1
<sipa> unless someone has questions/comments, that's it for my topic
<laanwj> concept ack anyhow
<michaelfolkson> I would guess there are some lazy habits where 8333 is etched into people's minds.
<michaelfolkson> But not a strong rationale for not changing that
<laanwj> i'll skip fanquake's topic as i think he meant 0.20.2, and that already happened Oct 18
<fanquake> yes I’ve made that tag already
<fanquake> we’ve had a few signed builders for 0.20.2 as well
<laanwj> i also built it but not uploaded sigs yet, will do
<laanwj> #topic Encouraging people to propose meeting topics for the weekly IRC Core dev meetings (michaelfolkson)
<fanquake> jonasschnelli: ! Hope you are doing ok, and maybe back to Core dev some day.
<josibake> are there any existing guidelines for what a "good" topic is?
<michaelfolkson> Ok very short one. Just to encourage people to propose meeting topics for this meeting. The number of topics have been pretty sparse in recent weeks/months and it is much better to have too many than next to none
<michaelfolkson> We can always push non-urgent to future weeks
<michaelfolkson> josibake: No but that sounds like a good idea
<michaelfolkson> It can be a PR though or a mini Core project though (imo)
<michaelfolkson> There is the Core PR review club too of course
<laanwj> josibake: dunno, you know you can always just ask in the channel
<michaelfolkson> And we are doing an online livestreamed session on AssumeUTXO
<laanwj> if it's relevant to current development in the repository and there's people willing to talk about it it's a good topic, that's pretty wide
<sipa> i like having short meetings tbh
<michaelfolkson> You can leave early though sipa :)
<sipa> i mean: we don't need to fill meetings because meetings should be filled
Kaizen_Kintsugi has quit [Remote host closed the connection]
<michaelfolkson> The P2P ones have effectively been cancelled
<sipa> sure, if there is a topic, let's discuss it
<josibake> laanwj: thanks
<ariard> michaelfolkson: some people are attending multiple meetings, lightning, etc, short meetings are good :)
<michaelfolkson> I guess but people can leave if they aren't interested in the topic
<laanwj> i like short meetings but also, it's good if the time available is used in a productive way
<laanwj> if it ends with "high priority for review" every time it's also a bit demotivating
<jamesob> yeah, agree
<michaelfolkson> Just a thought that there are avenues for getting more review on your PR if you are frustrated (which imo includes this meeting)
jarthur_ has joined #bitcoin-core-dev
<jamesob> I'm going to go on a tirade about nits at some point, so I'll queue that up for a future meeting :)
<laanwj> hehh
<michaelfolkson> PR review nits?
<jamesob> yes
<michaelfolkson> The TL;DR?
<laanwj> i mean some things you acn also bring up outside meeting, not everything needs to be in the meeting
jarthur has quit [Ping timeout: 264 seconds]
Kaizen_Kintsugi has joined #bitcoin-core-dev
<laanwj> just depends
<jamesob> ugh I don't know. They can just be really demotivating and distracting, but I need to collect my thoughts and figure out if I have any actually-actionable things to say
<michaelfolkson> Ok
<laanwj> any cast this gets veeeery meta
<jamesob> it was 40% a joke
<michaelfolkson> Haha ok next topic (as we've got ariard here and he doesn't like meetings)
<michaelfolkson> Let's cover the full RBF topic
<laanwj> #topic Attempting to transition to full RBF over multiple major versions (michaelfolkson)
<laanwj> jamesob: 40% joke is the best approach to things
<jamesob> lol
<ariard> ahah
<michaelfolkson> Ok I'm happy to pass this over to ariard but he previously posted this to the mailing list https://lists.linuxfoundation.org/pipermail/bitcoin-dev/2021-June/019074.html
<luke-jr> how about just stop trying to dictate to users what policies they can or can't use? <.<
<jonatack> jamesob: (i see nits ideally as suggested lightly, and up to the author and that's, that really)
<michaelfolkson> luke-jr: Core has to pick a default
<laanwj> luke-jr: i understood that's the idea with full RBF, it drops a fair amount of policy restrictions
<ariard> luke-jr: if you mean simplifying our policy, this proposal is going in that sense ? but ultimately we *do* have to pickup defaults and they're free of implications
<luke-jr> michaelfolkson: not really; and right now, Core is forcing its "default" on users
<sipa> my view is generally: come up with an actual workable and worked-out policy, and propose it - beyond that it seems like a ML discussion primarily
<laanwj> you can't really get rid of policy entirely because we need it for anti-DoS etc
<sipa> at a very high level, i'm concept ack - i think it's inevitable long term that the network will tend to more rational policies, and full RBF is a step towards that
<luke-jr> Knots has configurable RBF, and I'm happy to re-open the Core PR if people are willing to accept/review it
<michaelfolkson> sipa: It has been proposed by ariard. And I think he's right to attempt a roadmap for changing it
<sipa> but it all depends on the details
<ariard> michaelfolkson: do you have a link towards achow101 BerkeleyDB deprecation plan you were mentioning offline last week ?
<sipa> michaelfolkson: i don't see a concrete policy, but i may have missed it
<sipa> "full RBF" isn't a policy, it's an aspiration :)
gene has joined #bitcoin-core-dev
<achow101> ariard: #20160
<gribble> https://github.com/bitcoin/bitcoin/issues/20160 | Proposed Timeline for Legacy Wallet and BDB removal · Issue #20160 · bitcoin/bitcoin · GitHub
<michaelfolkson> Right ^
<michaelfolkson> I think a similar roadmap could be attempted for full RBF by default (though I hadn't considered configurable option)
<ariard> achow101: thanks, i'll look at it and see how to adapt for full-rbf, i'll open a PR soon with config knob full-rbf=false
<laanwj> ariard: +1
<michaelfolkson> There still has to be a default even with configurable option
<luke-jr> ariard: ^
<luke-jr> ariard: the code already exists, just a matter of people accepting/reviewing it for Core
<sipa> i have some thoughts, but i'll respond on the ML
<ariard> luke-jr: for now, i'll only to propose a config knob with full-rbf=false, i don't think you can configure that for now ?
<michaelfolkson> There will be some business opposition but if we think long term this is the right thing to do we should attempt to change it over multiple versions giving people plenty of advance warning
<luke-jr> ariard: yes, that's one of the 3 configurations in that branch/Knots
<laanwj> having a config know for it in core at all is a start
<laanwj> changing the default is much further away
<laanwj> and probably much more controversial
<luke-jr> mempoolreplacement=none / fee,optin / fee,-optin
<josibake> laanwj:+1 just adding it as a config option shouldn't be controversial, right?
<ariard> yes i'm still worried about the Dos against multi-party funding transactions but we have years ahead before i think it starts to be a real issue
<luke-jr> josibake: it was last time I tried
<laanwj> josibake: not to me at least, given how many people want this
<michaelfolkson> Right, there will be some opposition. I have tried speaking to some Lightning businesses to understand their use of Lightning zero conf channels
<laanwj> luke-jr: when was that? 2015?
<luke-jr> laanwj: probably
<ariard> luke-jr: ah okay for the branch, gonna look
<michaelfolkson> Maybe last words but I think that was wrapped up in the block size drama. I am hoping a long(ish) term plan this time around will be less controversial
<michaelfolkson> But we'll see. I think we should attempt this long term (over multiple major versions) transition to full RBF
<laanwj> luke-jr: it does seem attitudes changed a bit since then
<michaelfolkson> It may fail
<luke-jr> ariard: just pushed the latest rebase (Oct 8th)
<luke-jr> laanwj: great
<ariard> michaelfolkson: though i've been reached out privately by a exchange operator disagreeing with full-rbf after posting on the ML
<cfields> ariard: suggest they respond publicly :)
<sipa> +1
<michaelfolkson> Right, I think the discussion should be out in the open
<michaelfolkson> If people really don't want it then the attempt may fail
<ariard> cfields: yeah will ping them back once PR is open!
<michaelfolkson> Or at least summarize their arguments ariard
<cfields> ariard: great, thanks!
<michaelfolkson> We don't need to know who it is if they want to stay anonymous
<luke-jr> michaelfolkson: eh? it's not a consensus change
<luke-jr> anyone can run full RBF today if they want to. or block RBF entirely if they prefer that.
<michaelfolkson> luke-jr: Right but we should still consult? If people really don't want it?
<michaelfolkson> At least collect those arguments
<luke-jr> michaelfolkson: if people don't want it, they can just not enable it..
<laanwj> right, but it's good to give people time to adapt to it at least before changing the default
<sipa> luke-jr: i don't think that's a useful response
<sipa> luke-jr: in their mind - for good or bad reasons - the problem is others enabling it
<sipa> (presumably)
<luke-jr> sipa: but they have no say over what other people do
<michaelfolkson> I think generally if we expect opposition/controversy it is good to reach out
<michaelfolkson> Even if it isn't consensus change
<laanwj> no, they don't, but a default change would not be something done lightly
<sipa> luke-jr: which is much easier to address publicly :)
<luke-jr> laanwj: perhaps, but that's down the road past just an option ;)
<laanwj> clearly for your own nodes you can run with whatever policy you want
<michaelfolkson> Also whether we like it or not we are going to have to consider policy more seriously because of Lightning and higher levels
<laanwj> and i definitely expect miners to start acting rationally in this regard
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke opened pull request #23334: fuzz: Descriptor wallet (master...2110-fuzzWall) https://github.com/bitcoin/bitcoin/pull/23334
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
<luke-jr> michaelfolkson: anything that depends on policy for security, is broken by design
<sipa> laanwj: yeah, it may take a few halvings, though
<laanwj> i mean, it's not something you can just post a rant against on the mailing list then ignore
<michaelfolkson> luke-jr: It isn't optimal sure. But I still think having Lightning ecosystem is better than ignoring it
<luke-jr> michaelfolkson: Lightning shouldn't depend on policy
<michaelfolkson> But anyway, let's move it to GitHub issue an mailing list
<michaelfolkson> luke-jr: But it does!
<luke-jr> last I heard, quite a few nodes and miners already use full RBF for years
<luke-jr> michaelfolkson: they should fix it then, regardless of this discussion
<michaelfolkson> It is suboptimal but the best we can do currently for Lightning design
<sipa> i feel like this is a bit like the Linux mantra "don't break userspace"; userspace may be depending on un-guaranteed behavior for dumb/bad reasons, but that doesn't mean you can just go and break it
<ariard> luke-jr: can you make the expectation that the most economically-rational policy is going to be run in majority over the p2p network ?
<luke-jr> ariard: no
<laanwj> i'd prefer to leave it at adding an option for now as well
<laanwj> 1 minute to go
<michaelfolkson> Ok we should wrap up. Want to respect people's time
<michaelfolkson> Thanks
<laanwj> michaelfolkson: please leave time management to me
<michaelfolkson> Sorry :)
<laanwj> but you're right anyhow
<laanwj> #endmeeting
Kaizen_Kintsugi has quit [Ping timeout: 260 seconds]
<jonasschnelli> (and yes, happy to shared the tweaked code of the meeting irc bot)
<luke-jr> ariard: anyone can start mining (in theory), so to expect *some* miners to have such a policy may be reasonable
<laanwj> jonasschnelli: you run the meetings irc bot?
<jonasschnelli> fanquake: Thanks. All good here. No drama or so. Just time to move on.
<jonasschnelli> laanwj: yes. But looks like its down
<ariard> (btw if you have coredev travel reimbursements applications to submit, please send then on my mail antoine.riard@gmail.com)
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/179ce098333b...12ff8993bcc7
<bitcoin-git> bitcoin/master fa6c62f MarcoFalke: test: Replace log with assert_equal in wallet_abandonconflict
<bitcoin-git> bitcoin/master 12ff899 MarcoFalke: Merge bitcoin/bitcoin#23331: test: Replace log with assert_equal in wallet...
core-meetingbot has joined #bitcoin-core-dev
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #23331: test: Replace log with assert_equal in wallet_abandonconflict (master...2110-testAssert) https://github.com/bitcoin/bitcoin/pull/23331
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
<cfields> sipa: sorry if I implied that minisketch was desperate for review btw. clearly just a case of me being out of the loop for a while.
<jonasschnelli> laanwj: it's a modified version of supybot.plugins.MeetBot from debian I think
<ariard> luke-jr: i got a point that users might be willingly to adopt a more privacy-preserving policy, and it might at odd with an economically-rational one (though hopefully not!)
<ariard> it has been discussed multiple times on the Lightning side to have LN nodes connecting directly to few miners, but i'm still meeeeh on the censorship-resistance of that
<laanwj> cfields: it can always use more review :)
<luke-jr> ariard: btw, should I open the fullrbf branch as a PR?
<laanwj> jonasschnelli: thanks; i know very little about existing IRC bot projects tbh, but it was kind of useful, and we should probably keep it going
<fanquake> Also, in regard to the macOS signing keys, fwiw I also currently have access to the Apple developer account that is associated with those keys
<luke-jr> (bleh, looks like it needs another rebase)
<jonasschnelli> fanquake: great. I think you should keep that. You can't download the private key from there (but create a new one AFAIK).
<jonasschnelli> happy to hand over the key to either you or achow101 (depending on how you all think how we should do it)
<achow101> it's a shared developer account?
<jonasschnelli> it can have multiple accounts/roles
<glozow> oh no i missed the meeting 😅 thanks ariard laanwj, #22674 is my top priority as well
<gribble> https://github.com/bitcoin/bitcoin/issues/22674 | validation: mempool validation and submission for packages of 1 child + parents by glozow · Pull Request #22674 · bitcoin/bitcoin · GitHub
<sipa> cfields: no worries
<achow101> it'd definitely be easier if we had just one code signer
<ariard> luke-jr: i have time for doing it in the coming weeks, wanna observed first the deprecation plan from #20160 and explain at best the impact of opt-in RBF on upper layers
<gribble> https://github.com/bitcoin/bitcoin/issues/20160 | Proposed Timeline for Legacy Wallet and BDB removal · Issue #20160 · bitcoin/bitcoin · GitHub
<laanwj> glozow: thanks, glad we picked the right one :)
<glozow> :)
<sipa> i don't really see why deprecation of BDB has to do with a network policy change
* luke-jr either ☺
<achow101> I think it's just for the roadmap format
<cfields> heh, I was also wondering what I was missing.
Kaizen_Kintsugi has joined #bitcoin-core-dev
<laanwj> just the idea of a very long term roadmap i guess
<ariard> sipa: afaik we don't have that much guidelines to propose potentially-impactful network policy change, though see that conv https://lists.linuxfoundation.org/pipermail/bitcoin-dev/2021-September/019400.html on sequence lock upgrades defect where it was raised what it could looks like
Guest54 has joined #bitcoin-core-dev
Guest54 has quit [Client Quit]
Guest9057 has quit [Quit: Client closed]
ghost43 has quit [Remote host closed the connection]
ghost43 has joined #bitcoin-core-dev
gene has quit [Ping timeout: 276 seconds]
gene has joined #bitcoin-core-dev
Kaizen_Kintsugi has quit [Remote host closed the connection]
Kaizen_Kintsugi has joined #bitcoin-core-dev
Kaizen_Kintsugi has quit [Ping timeout: 252 seconds]
sipsorcery has quit [Ping timeout: 260 seconds]
Kaizen_Kintsugi has joined #bitcoin-core-dev
Guest39 has joined #bitcoin-core-dev
<Guest39> hello friends
Kaizen_Kintsugi has quit [Ping timeout: 258 seconds]
sipsorcery has joined #bitcoin-core-dev
<Guest39> is satoshi nakamoto alive
<Guest39> i want to know this
Guest39 has quit [Quit: Client closed]
luke-jr has quit [Quit: ZNC - http://znc.sourceforge.net]
luke-jr has joined #bitcoin-core-dev
Kaizen_Kintsugi has joined #bitcoin-core-dev
Kaizen_Kintsugi has quit [Remote host closed the connection]
Kaizen_Kintsugi has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] joankaradimov opened pull request #23335: Include a missing header (master...include-missing-header) https://github.com/bitcoin/bitcoin/pull/23335
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
sipsorcery has quit [Ping timeout: 260 seconds]
jesseposner_ has joined #bitcoin-core-dev
jesseposner has quit [Ping timeout: 245 seconds]
AaronvanW has quit [Quit: Leaving...]
Kaizen_Kintsugi has quit [Remote host closed the connection]
jamesob has quit [Quit: Connection closed for inactivity]
Kaizen_Kintsugi has joined #bitcoin-core-dev