14:00:43 <achow101> #startmeeting 14:00:48 <Murch> achow101: Meeting? 14:00:53 <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:00:55 <stickies-v> hi 14:00:57 <TheCharlatan> hi 14:00:57 <hebasto> hi 14:00:59 <jamesob> hi 14:01:03 <kouloumos> hi 14:01:09 <MacroFake> oi 14:01:13 <pinheadmz> hi 14:01:17 <dergoegge> hi 14:01:18 <sipa> hi 14:01:21 <LarryRuane> Hi 14:01:24 <furszy> hi 14:01:32 <achow101> Welcome to the weekly general IRC meeting 14:01:41 <Murch> GM 14:01:50 <achow101> there are no pre-propsoed meeting topics this week. are there any last minute topics to add to the list? 14:01:59 <theStack> hi 14:02:11 <darosior> hi 14:02:24 <achow101> #topic assumeutxo updates (jamesob) 14:02:43 <josie> hi 14:02:53 <glozow> hi 14:03:39 <jamesob> Not too much to report here - Suhas seems to have found a bug in some of the blockindex management code that I'm trying to resolve. I'm adding functional testcases and will be evaluating his proposed changes today. 14:04:10 <achow101> the blocker is still #24008? 14:04:10 <jamesob> I'm actively looking for an apprentice to take on some of the brunt work of refactoring, mostly as a convenient means to recruit someone newish into the project. 14:04:14 <gribble> https://github.com/bitcoin/bitcoin/issues/24008 | assumeutxo: net_processing changes by jamesob ÷ Pull Request #24008 ÷ bitcoin/bitcoin ÷ GitHub 14:04:33 <jamesob> Yeah, that's still the one. I think that change is okay, but maybe sdaftuar has another take on it 14:05:06 <achow101> ok 14:05:14 <TheCharlatan> is this bug addressed in #27746? 14:05:15 <jamesob> I'm not up for a huge refactoring adventure, so if we radically change the approach I might step back for a few months 14:05:16 <gribble> https://github.com/bitcoin/bitcoin/issues/27746 | Draft: rework validation logic for assumeutxo by sdaftuar ÷ Pull Request #27746 ÷ bitcoin/bitcoin ÷ GitHub 14:05:23 <jamesob> and shift my focus to other stuff 14:05:43 <sipa> sdaftuar: ping ? 14:05:58 <jamesob> TheCharlatan: yeah, allegedly, although I haven't looked at it myself yet. I came up with my own fix last night and will push it to the big branch shortly. 14:06:13 <jamesob> (#27596) 14:06:15 <gribble> https://github.com/bitcoin/bitcoin/issues/27596 | assumeutxo (2) by jamesob ÷ Pull Request #27596 ÷ bitcoin/bitcoin ÷ GitHub 14:06:35 <sipa> jamesob: I'm not sure that an apprentice is what assumeutxo needs 14:06:49 <jamesob> sipa: you may be right 14:07:46 <b10c> hi 14:08:57 <achow101> #topic package relay updates (glozow) 14:09:10 <glozow> Big Branch is up! #27742 14:09:12 <gribble> https://github.com/bitcoin/bitcoin/issues/27742 | [NO MERGE] BIP331 Ancestor Package Relay by glozow ÷ Pull Request #27742 ÷ bitcoin/bitcoin ÷ GitHub 14:09:26 <glozow> Seeking some approach feedback on that. Are p2p folks available to take a look? 14:10:29 <dergoegge> Yes I'll look at it 14:11:15 <lightlike> +1 14:11:16 <glozow> ðÂÂÂð 14:11:27 <glozow> thanks thanks thanks 14:11:29 <achow101> cool! 14:11:43 <achow101> #topic libbitcoinkernel updates (TheCharlatan) 14:12:04 <TheCharlatan> #27636 is waiting for re-review, currently has 1 ACK and three stale ACKs. Really happy that this is getting a lot of attention. 14:12:08 <gribble> https://github.com/bitcoin/bitcoin/issues/27636 | kernel: Remove util/system from kernel library, interface_ui from validation. by TheCharlatan ÷ Pull Request #27636 ÷ bitcoin/bitcoin ÷ GitHub 14:12:17 <TheCharlatan> I decided to move two of its commits that touched the shutdown handling to #27711 instead, since they were a bit controversial. 14:12:19 <gribble> https://github.com/bitcoin/bitcoin/issues/27711 | kernel: Remove shutdown from kernel library by TheCharlatan ÷ Pull Request #27711 ÷ bitcoin/bitcoin ÷ GitHub 14:12:37 <MacroFake> TheCharlatan: Do you want review or are you addressing the nits? 14:12:49 <TheCharlatan> I think molding the current shutdown behavior into something kernel friendly should be done with some prudence, so I hope this unblocks progress on both the other work remaining work for splitting out the few left over files from the kernel and being able to focus solely on shutdown routines. 14:13:58 <TheCharlatan> MacroFake, I'll address the nits in #27636 if I have to push again. 14:14:01 <gribble> https://github.com/bitcoin/bitcoin/issues/27636 | kernel: Remove util/system from kernel library, interface_ui from validation. by TheCharlatan ÷ Pull Request #27636 ÷ bitcoin/bitcoin ÷ GitHub 14:14:44 <MacroFake> Might be good to reply to them, so that reviewers know what is going on 14:15:12 <bitcoin-git> [bitcoin] fanquake opened pull request #27755: [24.x] rpc: Fix invalid bech32 handling (24.x...backport_27727_24_2) https://github.com/bitcoin/bitcoin/pull/27755 14:15:25 <TheCharlatan> will do 14:15:41 <achow101> #topic BIP 324 updates (sipa) 14:17:44 <cfields> hi 14:17:54 <achow101> I see that https://github.com/bitcoin-core/secp256k1/pull/1129 has gotten some more review and changes, so I suppose we're waiting on re-review of that? 14:18:35 <sipa> Not too much progress there, hope to see more soon. 14:18:59 <achow101> ok 14:18:59 <sipa> For 1129, I'm mostly waiting on a review by real_or_random still. 14:19:29 <achow101> #topic Ad-hoc high priority for review 14:20:00 <achow101> https://github.com/orgs/bitcoin/projects/1/views/4 anything to add or remove? 14:21:36 <achow101> or any other topics to discuss? 14:22:05 <fanquake> v25.0 has been tagged 14:22:10 <fanquake> Time to guix build it 14:22:55 <glozow> +1 14:23:04 <achow101> #endmeeting