14:00:07 <achow101> #startmeeting 14:00:14 <jamesob> hi 14:00:15 <TheCharlatan> hi 14:00:16 <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:19 <glozow> hi 14:00:24 <brunoerg> hi 14:00:27 <pinheadmz> waves 14:00:28 <hebasto> hi 14:00:33 <sipa> hi 14:00:35 <Murch> Howdy 14:00:38 <achow101> #topic package relay updates (glozow) 14:00:40 <furszy> hi 14:00:40 <dergoegge> hi 14:00:40 <LarryRuane> Hi 14:00:42 <stickies-v> hi 14:00:57 <darosior> hi 14:00:57 <lightlike> Hi 14:01:03 <abubakarsadiq> hi 14:01:15 <kouloumos> hi 14:01:33 <glozow> I am still working on the changes, got distracted by some other things this week. 14:02:31 <achow101> looks like #28031 has been getting some review too 14:02:34 <gribble> https://github.com/bitcoin/bitcoin/issues/28031 | Package Relay 1/3: Introduce TxPackageTracker as Orphan Resolution Module by glozow ÷ Pull Request #28031 ÷ bitcoin/bitcoin ÷ GitHub 14:03:09 <glozow> Yeah. dergoegge pointed out it might make sense to wrap more stuff in a TxDownloadManager given the fact that TxRequestTracker and TxPackageTracker need to be synchronized 14:04:20 <achow101> glozow: by "the changes" you mean the stuff that was discussed for #26711? 14:04:25 <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:04:36 <glozow> yes 14:05:02 <glozow> and also updating #28031, have a ci failure and some reviews 14:05:04 <gribble> https://github.com/bitcoin/bitcoin/issues/28031 | Package Relay 1/3: Introduce TxPackageTracker as Orphan Resolution Module by glozow ÷ Pull Request #28031 ÷ bitcoin/bitcoin ÷ GitHub 14:05:11 <achow101> cool 14:05:19 <achow101> #topic BIP 324 updates (sipa) 14:06:28 <achow101> #27985 was merged, so next should be #27993? 14:06:30 <gribble> https://github.com/bitcoin/bitcoin/issues/27985 | Add support for RFC8439 variant of ChaCha20 by sipa ÷ Pull Request #27985 ÷ bitcoin/bitcoin ÷ GitHub 14:06:31 <gribble> https://github.com/bitcoin/bitcoin/issues/27993 | Make poly1305 support incremental computation + modernize by sipa ÷ Pull Request #27993 ÷ bitcoin/bitcoin ÷ GitHub 14:06:37 <sipa> Yeah! 14:07:44 <achow101> #topic libbitcoinkernel updates (TheCharlatan) 14:07:54 <TheCharlatan> https://github.com/bitcoin/bitcoin/pull/28053 got merged, moving a shutdown call out of the kernel. 14:08:16 <TheCharlatan> https://github.com/bitcoin/bitcoin/pull/28048 is close to the finish line. Merging it would conclude the "prune the unnecessary .cpp files (and their respective headers)" (stage 1, step2) part of the project, which is nice :) 14:08:34 <TheCharlatan> So I think #28048 should be the priority for now. 14:08:37 <gribble> https://github.com/bitcoin/bitcoin/issues/28048 | kernel: Remove StartShutdown calls from validation code by ryanofsky ÷ Pull Request #28048 ÷ bitcoin/bitcoin ÷ GitHub 14:08:54 <TheCharlatan> Next to continuing the work on improving the shutdown handling, we'll then move on towards cleaving the last bits of boost and leveldb includes from our kernel headers. 14:09:04 <TheCharlatan> This is important because we don't want to ship those in our library headers. 14:09:36 <TheCharlatan> that's it :) 14:09:43 <achow101> neat 14:09:47 <achow101> i've updated the board 14:09:57 <achow101> #topic assumeutxo updates (jamesob) 14:10:19 <jamesob> Nothing new since last week; #27746 is still the thing we're looking to get merged. Seems pretty close, a few ACKs (maybe some stale?). 14:10:23 <gribble> https://github.com/bitcoin/bitcoin/issues/27746 | Rework validation logic for assumeutxo by sdaftuar ÷ Pull Request #27746 ÷ bitcoin/bitcoin ÷ GitHub 14:10:32 <achow101> sdaftuar needs to rebase it 14:10:41 <achow101> one of the kernel prs had a conflict 14:10:58 <sipa> He's on vacation, not sure how active he'll be during the summer. 14:11:03 <jamesob> I think he may or may not be indulging in Chaincode Summer, so not sure what the timing will look like on that... 14:11:44 <jamesob> Maybe I should open a replacement PR that can be rebased? 14:12:11 <achow101> seems like that would be a good idea 14:12:44 <furszy> or he could give you push access to his repo 14:12:45 <jamesob> Maybe I'll message him to see what his availability looks like for the next few weeks 14:13:34 <achow101> github has a "maintainers can push to this pr" option, but it's not enabled on 27746 14:14:25 <achow101> but hopefully we can get those changes in soon 14:14:54 <achow101> #topic Ad-hoc high priority for review 14:15:07 <sdaftuar> oh hi -- sorry i have been unexpectedly swamped. i might have some time this weekend to review the latest comments and rebase! 14:15:19 <achow101> anything to add or remove from https://github.com/orgs/bitcoin/projects/1/views/4 14:15:27 <achow101> sdaftuar: great! 14:15:35 <dergoegge> could i get #27499? #28043 depends on it 14:15:37 <gribble> https://github.com/bitcoin/bitcoin/issues/27499 | net processing, refactor: Decouple PeerManager from gArgs by dergoegge ÷ Pull Request #27499 ÷ bitcoin/bitcoin ÷ GitHub 14:15:38 <gribble> https://github.com/bitcoin/bitcoin/issues/28043 | fuzz: Test headers pre-sync through p2p interface by dergoegge ÷ Pull Request #28043 ÷ bitcoin/bitcoin ÷ GitHub 14:15:43 <jamesob> sdaftuar: thanks 14:16:03 <achow101> dergoegge: added 14:16:14 <dergoegge> #28043 could give us some more confidence in doing #25725 14:16:16 <gribble> https://github.com/bitcoin/bitcoin/issues/28043 | fuzz: Test headers pre-sync through p2p interface by dergoegge ÷ Pull Request #28043 ÷ bitcoin/bitcoin ÷ GitHub 14:16:17 <gribble> https://github.com/bitcoin/bitcoin/issues/25725 | consensus: Remove mainnet checkpoints by sdaftuar ÷ Pull Request #25725 ÷ bitcoin/bitcoin ÷ GitHub 14:16:59 <achow101> any other topics to discuss? 14:19:02 <achow101> #endmeeting