14:01:13 <achow101> #startmeeting 14:01:22 <hebasto> hi 14:01:26 <kanzure> hi 14:01:27 <glozow> hi 14:01:31 <achow101> #bitcoin -core-dev Meeting: achow101 _aj_ amiti ariard aureleoules b10c BlueMatt brunoerg cfields darosior dergoegge dongcarl fanquake fjahr furszy gleb glozow hebasto instagibbs jamesob jarolrod jonatack josibake kallewoof kanzure kouloumos kvaciral laanwj LarryRuane lightlike luke-jr MacroFake Murch phantomcircuit pinheadmz promag provoostenator ryanofsky sdaftuar S3RK stickies-v sipa theStack TheCharlatan vasild 14:01:34 <stickies-v> hi 14:01:41 <brunoerg> hi 14:01:43 <pinheadmz> hello 14:01:49 <instagibbs> hi 14:01:50 <dergoegge> hi 14:02:02 <achow101> There are no pre-proposed meeting topics this week. Are there any last minute topics to add to the list? 14:02:07 <furszy> hi 14:02:14 <TheCharlatan> hi 14:02:14 <_aj_> hi 14:02:21 <sipa> hi 14:02:38 <achow101> #topic assumeutxo updates (jamesob) 14:03:40 <achow101> #24008 was closed, and from conversations in ##bitcoin-assumeutxo, it sounds like the main pr (#27596) is the focus now 14:03:42 <cfields> hi 14:03:44 <gribble`> https://github.com/bitcoin/bitcoin/issues/24008 | assumeutxo: net_processing changes by jamesob ÷ Pull Request #24008 ÷ bitcoin/bitcoin ÷ GitHub 14:03:46 <gribble`> https://github.com/bitcoin/bitcoin/issues/27596 | assumeutxo (2) by jamesob ÷ Pull Request #27596 ÷ bitcoin/bitcoin ÷ GitHub 14:03:51 <kouloumos> hi 14:04:43 <instagibbs> "I'm in the car" he says 14:05:01 <instagibbs> so last I heard, he's taking in sdaftuar's suggestions for the net processing piece 14:05:12 <darosior> hi 14:05:48 <achow101> #topic package relay updates (glozow) 14:06:06 <josie> hi 14:06:19 <glozow> #27742 CI is green! No reviews yet... anything I can do to help encourage reviews? Or answer any questions? 14:06:21 <gribble`> https://github.com/bitcoin/bitcoin/issues/27742 | [NO MERGE] BIP331 Ancestor Package Relay by glozow ÷ Pull Request #27742 ÷ bitcoin/bitcoin ÷ GitHub 14:06:30 <instagibbs> ##bitcoin-assumeutxo 14:06:30 <instagibbs> ^ where to discuss more often, he says 14:06:34 <instagibbs> fin 14:06:38 <willcl-ark> hi 14:06:41 <glozow> If people are looking for something easier to review, the BIP PR is updated and open for review: https://github.com/bitcoin/bips/pull/1382 14:07:05 <instagibbs> I've worked through the orphan handling piece, but all that code is relatively new to me 14:07:11 <achow101> glozow: #26711 is still the next step? 14:07:14 <gribble`> https://github.com/bitcoin/bitcoin/issues/26711 | validate package transactions with their in-package ancestor sets by glozow ÷ Pull Request #26711 ÷ bitcoin/bitcoin ÷ GitHub 14:07:15 <glozow> instagibbs: awesome, thanks so much! 14:07:33 <glozow> yes, #26711 is the mempool PR open for review 14:07:34 <instagibbs> I'm waiting for real PR to bother you with my suggestions, will go through it a couple more times once open 14:07:36 <gribble`> https://github.com/bitcoin/bitcoin/issues/26711 | validate package transactions with their in-package ancestor sets by glozow ÷ Pull Request #26711 ÷ bitcoin/bitcoin ÷ GitHub 14:08:22 <glozow> ok, i will assume approach is "ok" and open the first PR 14:08:52 <achow101> cool 14:08:54 <instagibbs> first set is just encapsulation of orphan handling, I think that's safe to open 14:08:57 <instagibbs> and has benefits already 14:09:02 <glozow> correct, I will open that 14:09:10 <glozow> was just hoping for like 1 comment or so ð 14:09:31 <achow101> #topic libbitcoinkernel updates (TheCharlatan) 14:10:30 <achow101> seems like #27711 should be the next pr for libbitcoinkernel? 14:10:31 <gribble`> https://github.com/bitcoin/bitcoin/issues/27711 | kernel: Remove shutdown from kernel library by TheCharlatan ÷ Pull Request #27711 ÷ bitcoin/bitcoin ÷ GitHub 14:10:39 <TheCharlatan> kernel PRs are progressing, the next two are now undrafted. 14:10:45 <TheCharlatan> The shutdown discussion is continued in #27711. 14:10:47 <gribble`> https://github.com/bitcoin/bitcoin/issues/27711 | kernel: Remove shutdown from kernel library by TheCharlatan ÷ Pull Request #27711 ÷ bitcoin/bitcoin ÷ GitHub 14:10:58 <fanquake> ð 14:11:04 <TheCharlatan> PR #27576 that should finally get rid of the ArgsManager in kernel code. 14:11:05 <gribble`> https://github.com/bitcoin/bitcoin/issues/27576 | kernel: Remove args, settings, chainparams, chainparamsbase from kernel library by TheCharlatan ÷ Pull Request #27576 ÷ bitcoin/bitcoin ÷ GitHub 14:11:22 <cfields> woohoo :) 14:11:27 <josie> TheCharlatan: nice! 14:11:28 <TheCharlatan> thank you reviewers :) 14:12:19 <achow101> #topic BIP 324 updates (sipa) 14:12:34 <sipa> Not much to report, I'm afraid. 14:13:29 <Murch> Hi 14:13:32 <achow101> still waiting for review on https://github.com/bitcoin-core/secp256k1/pull/1129? 14:13:45 <sipa> Indeed. 14:14:21 <achow101> #topic Ad-hoc high priority for review 14:14:42 <achow101> Anything to add/remove from https://github.com/orgs/bitcoin/projects/1/views/4? 14:15:25 <achow101> or are there any other topics to discuss this week? 14:15:47 <pinheadmz> #27101 14:15:49 <gribble`> https://github.com/bitcoin/bitcoin/issues/27101 | Support JSON-RPC 2.0 when requested by client by pinheadmz ÷ Pull Request #27101 ÷ bitcoin/bitcoin ÷ GitHub 14:15:52 <pinheadmz> anyone else want JSON RPC 2.0 ? 14:16:10 <pinheadmz> not sand, not a big rock, more of a medium sized pebble. for the jar. 14:16:36 <achow101> added 14:18:49 <achow101> #endmeeting