FrancisMr has quit [Remote host closed the connection]
brunoerg has quit [Ping timeout: 265 seconds]
brunoerg has joined #bitcoin-core-dev
szarka has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 260 seconds]
AaronvanW has quit [Quit: Leaving...]
brunoerg has joined #bitcoin-core-dev
Norrin has joined #bitcoin-core-dev
AmunRa has quit [Ping timeout: 255 seconds]
brunoerg has quit [Ping timeout: 246 seconds]
Norrin has quit [Ping timeout: 255 seconds]
Norrin has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 260 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 260 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 265 seconds]
Norrin has quit [Remote host closed the connection]
Norrin has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 248 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 264 seconds]
Guest38 has joined #bitcoin-core-dev
Guest38 has quit [Client Quit]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 260 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 265 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 268 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 256 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 252 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 265 seconds]
brunoerg has joined #bitcoin-core-dev
earnestly has quit [Ping timeout: 260 seconds]
brunoerg has quit [Ping timeout: 264 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 256 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 272 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 264 seconds]
as2333 has quit [Ping timeout: 264 seconds]
as2333 has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 268 seconds]
brunoerg has joined #bitcoin-core-dev
Norrin has quit [Ping timeout: 255 seconds]
brunoerg has quit [Ping timeout: 265 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 265 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 256 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 265 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 256 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 265 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 246 seconds]
cmirror has quit [Remote host closed the connection]
cmirror has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 260 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 260 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 265 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 246 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 260 seconds]
brunoerg has joined #bitcoin-core-dev
hashfunc has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 264 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 252 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 248 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 248 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 252 seconds]
brunoerg has joined #bitcoin-core-dev
hardtotell has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 246 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 265 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 248 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 256 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 252 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 260 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 264 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 265 seconds]
brunoerg has joined #bitcoin-core-dev
hashfunc has quit [Remote host closed the connection]
hashfunc has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 265 seconds]
brunoerg has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 252 seconds]
brunoerg has quit [Ping timeout: 252 seconds]
hashfunc has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 246 seconds]
hashfunc has quit [Ping timeout: 246 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 252 seconds]
sudoforge has quit [Quit: 404]
hashfunc has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
as2333 has quit [Remote host closed the connection]
brunoerg has quit [Ping timeout: 256 seconds]
brunoerg has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 260 seconds]
brunoerg has quit [Ping timeout: 252 seconds]
brunoerg has joined #bitcoin-core-dev
Guest79 has joined #bitcoin-core-dev
Guest31 has joined #bitcoin-core-dev
hashfunc has joined #bitcoin-core-dev
Guyver2 has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 268 seconds]
Guest31 has quit [Client Quit]
Guest79 has quit [Client Quit]
<michaelfolkson> #proposedmeetingtopic How to resolve a situation where a minority of maintainers refuse to comment on a new maintainer PR
salvatoshi has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 256 seconds]
hashfunc has joined #bitcoin-core-dev
dermoth has quit [Ping timeout: 260 seconds]
brunoerg has joined #bitcoin-core-dev
dermoth has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 265 seconds]
hashfunc has quit [Remote host closed the connection]
Guyver2_ has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
Guyver2 has quit [Ping timeout: 260 seconds]
brunoerg has quit [Ping timeout: 264 seconds]
earnestly has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake opened pull request #26877: doc: move errant release note to doc/ (master...move_rel_notes) https://github.com/bitcoin/bitcoin/pull/26877
hashfunc has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/fbe5e1220a35...535adce2721f
<bitcoin-git> bitcoin/master 94d0c41 fanquake: build: remove already tested headers from AC_CHECK_HEADERS
<bitcoin-git> bitcoin/master 535adce fanquake: Merge bitcoin/bitcoin#26825: build: remove already tested headers from AC_...
<bitcoin-git> [bitcoin] fanquake merged pull request #26825: build: remove already tested headers from AC_CHECK_HEADERS (master...remove_already_tested_headers) https://github.com/bitcoin/bitcoin/pull/26825
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 260 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 260 seconds]
hashfunc has quit [Ping timeout: 255 seconds]
<bitcoin-git> [bitcoin] fanquake opened pull request #26878: [24.x] Backports (24.x...24_x_backports) https://github.com/bitcoin/bitcoin/pull/26878
<bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/535adce2721f...bd7400453267
<bitcoin-git> bitcoin/master c28d461 fanquake: doc: move errant release note to doc/
<bitcoin-git> bitcoin/master bd74004 MarcoFalke: Merge bitcoin/bitcoin#26877: doc: move errant release note to doc/
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #26877: doc: move errant release note to doc/ (master...move_rel_notes) https://github.com/bitcoin/bitcoin/pull/26877
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 252 seconds]
hashfunc has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 256 seconds]
brunoerg has quit [Ping timeout: 252 seconds]
AaronvanW has joined #bitcoin-core-dev
hashfunc has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 260 seconds]
hashfunc has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 260 seconds]
hashfunc has joined #bitcoin-core-dev
Guest10 has joined #bitcoin-core-dev
Guest10 has quit [Client Quit]
Guest10 has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake opened pull request #26879: [23.x] ci: reaplce intel macos ci job (23.x...23_remove_intel_macos_ci) https://github.com/bitcoin/bitcoin/pull/26879
Guest83 has joined #bitcoin-core-dev
Guest83 has quit [Client Quit]
hashfunc has quit [Ping timeout: 260 seconds]
<bitcoin-git> [bitcoin] MarcoFalke opened pull request #26880: [24.x] ci: replace Intel macOS CI job (24.x...2301-ci-24.x-🎉) https://github.com/bitcoin/bitcoin/pull/26880
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 265 seconds]
Guest10 has quit [Quit: Client closed]
hashfunc has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 260 seconds]
brunoerg has quit [Ping timeout: 265 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 272 seconds]
hashfunc has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 260 seconds]
hashfunc has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 264 seconds]
hashfunc has quit [Ping timeout: 255 seconds]
brunoerg has joined #bitcoin-core-dev
hashfunc has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 256 seconds]
AaronvanW has quit [Remote host closed the connection]
brunoerg has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 260 seconds]
<bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/bd7400453267...edc3d1b296e3
<bitcoin-git> bitcoin/master fa1bf4e MarcoFalke: test: Fix intermittent timeout in p2p_permissions.py
<bitcoin-git> bitcoin/master edc3d1b MarcoFalke: Merge bitcoin/bitcoin#26854: test: Fix intermittent timeout in p2p_permiss...
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #26854: test: Fix intermittent timeout in p2p_permissions.py (master...2301-test-timeout-p2p-perm-💙) https://github.com/bitcoin/bitcoin/pull/26854
hashfunc has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 260 seconds]
<bitcoin-git> [bitcoin] MarcoFalke pushed 5 commits to 24.x: https://github.com/bitcoin/bitcoin/compare/0da38b6b0e74...5148145b3597
<bitcoin-git> bitcoin/24.x 644c030 MacroFake: ci: Remove unused package
<bitcoin-git> bitcoin/24.x ce2a072 MacroFake: ci: Allow PIP_PACKAGES on centos
<bitcoin-git> bitcoin/24.x 50ad39d Hennadii Stepanov: ci: Make `getopt` path architecture agnostic
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #26880: [24.x] ci: replace Intel macOS CI job (24.x...2301-ci-24.x-🎉) https://github.com/bitcoin/bitcoin/pull/26880
hashfunc has joined #bitcoin-core-dev
AaronvanW has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 260 seconds]
hashfunc has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 252 seconds]
hashfunc has joined #bitcoin-core-dev
<_aj_> hmm, the pyzmq 25.0 release from 40 minutes ago seems to be breaking CI for bitcoin-inquisition; maybe breaking master/24.0 too? https://cirrus-ci.com/task/6396380575432704
AaronvanW has quit [Ping timeout: 272 seconds]
hashfunc has quit [Ping timeout: 255 seconds]
Guyver2__ has joined #bitcoin-core-dev
Guyver2_ has quit [Ping timeout: 248 seconds]
Guyver2__ has quit [Read error: Connection reset by peer]
Guyver2__ has joined #bitcoin-core-dev
hashfunc has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 256 seconds]
dviola has joined #bitcoin-core-dev
<MacroFake> I wondered yesterday if we should pin the zmq dependency, but then figured that it is used so widely that it would be unlikely they introduced a bug
Norrin has joined #bitcoin-core-dev
hashfunc has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake opened pull request #26882: test: skip sqlite3 tests if it isn't available (master...func_tests_check_import_sqlite) https://github.com/bitcoin/bitcoin/pull/26882
hashfunc has quit [Ping timeout: 260 seconds]
<bitcoin-git> [bitcoin] stickies-v opened pull request #26883: refactor: follow-ups for #26695 (master...w/n26695-followups) https://github.com/bitcoin/bitcoin/pull/26883
hashfunc has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 260 seconds]
hashfunc has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 246 seconds]
kexkey has quit [Ping timeout: 256 seconds]
hashfunc has joined #bitcoin-core-dev
AaronvanW has joined #bitcoin-core-dev
kexkey has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 255 seconds]
hashfunc has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 246 seconds]
sudoforge has joined #bitcoin-core-dev
Norrin has quit [Ping timeout: 255 seconds]
hg has joined #bitcoin-core-dev
hashfunc has joined #bitcoin-core-dev
bitdex has quit [Quit: = ""]
Norrin has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 260 seconds]
hashfunc has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 246 seconds]
vasild has quit [Ping timeout: 255 seconds]
vasild has joined #bitcoin-core-dev
hashfunc has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 260 seconds]
vasild has quit [Ping timeout: 255 seconds]
vasild has joined #bitcoin-core-dev
hashfunc has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 260 seconds]
hashfunc has joined #bitcoin-core-dev
AmunRa has joined #bitcoin-core-dev
Norrin has quit [Ping timeout: 255 seconds]
<bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/edc3d1b296e3...fcd1a57be060
<bitcoin-git> bitcoin/master 672f7ad fanquake: doc: remove usages of C++11
<bitcoin-git> bitcoin/master fcd1a57 MarcoFalke: Merge bitcoin/bitcoin#26827: doc: use "std lib clock" over "C++11 clock"
hashfunc has quit [Ping timeout: 255 seconds]
hashfunc has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 260 seconds]
<bitcoin-git> [bitcoin] MarcoFalke closed pull request #26827: doc: use "std lib clock" over "C++11 clock" (master...doc_std_lib_clocks) https://github.com/bitcoin/bitcoin/pull/26827
hashfunc has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 252 seconds]
kexkey has quit [Ping timeout: 255 seconds]
kexkey has joined #bitcoin-core-dev
hashfunc has joined #bitcoin-core-dev
MrFrancis has joined #bitcoin-core-dev
Guyver2__ has left #bitcoin-core-dev [#bitcoin-core-dev]
hashfunc has quit [Ping timeout: 252 seconds]
hashfunc has joined #bitcoin-core-dev
<michaelfolkson> fanquake glozow: If you're around can you attend today's IRC meeting. I'd like to discuss the vasild maintainer situation, seems like a farce at this point
hashfunc has quit [Ping timeout: 260 seconds]
salvatoshi has quit [Ping timeout: 272 seconds]
hashfunc has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 252 seconds]
as2333 has joined #bitcoin-core-dev
hashfunc has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 248 seconds]
hashfunc has joined #bitcoin-core-dev
dviola has quit [Quit: WeeChat 3.7.1]
hashfunc has quit [Ping timeout: 255 seconds]
hashfunc has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 260 seconds]
hashfunc has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 246 seconds]
Talkless has joined #bitcoin-core-dev
halo has joined #bitcoin-core-dev
hashfunc has joined #bitcoin-core-dev
hg has quit [Ping timeout: 260 seconds]
<jamesob> sipa: is the tapleaf version 0xC0 declared as a constant anywhere in the code? can't find it
<sipa> TAPROOT_LEAF_TAPSCRIPT
hashfunc has quit [Ping timeout: 252 seconds]
<jamesob> thanks
dillontemp has joined #bitcoin-core-dev
hashfunc has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 255 seconds]
<achow101> #startmeeting
<core-meetingbot> Meeting started Thu Jan 12 19:00:28 2023 UTC. The chair is achow101. Information about MeetBot at https://bitcoin.jonasschnelli.ch/ircmeetings.
<core-meetingbot> Available commands: action commands idea info link nick
<michaelfolkson> hi
<achow101> #bitcoin-core-dev Meeting: achow101 aj amiti ariard b10c 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 larryruane lightlike luke-jr maaku marcofalke meshcollider michagogo moneyball morcos nehan NicolasDorier paveljanik petertodd
<achow101> phantomcircuit promag provoostenator ryanofsky sdaftuar sipa vasild
<jamesob> hi
<hebasto> hi
<vasild> hi
<achow101> There are 3 preproposed meeting topics
<brunoerg> hi
<achow101> but let's start with the usual
<achow101> #topic High priority for review
<core-meetingbot> topic: High priority for review
<furszy> hi
<kanzure> hi
<lightlike> hi
<achow101> https://github.com/orgs/bitcoin/projects/1 anything to add/remove/merge?
<MacroFake> hi
<MacroFake> I'd like to have #26039
<gribble> https://github.com/bitcoin/bitcoin/issues/26039 | refactor: Run type check against RPCArgs (1/2) by MarcoFalke · Pull Request #26039 · bitcoin/bitcoin · GitHub
<achow101> MacroFake: done
<MacroFake> thanks
<sipa> I'd like #26691
<gribble> https://github.com/bitcoin/bitcoin/issues/26691 | Update secp256k1 subtree to libsecp256k1 version 0.2.0 by sipa · Pull Request #26691 · bitcoin/bitcoin · GitHub
<achow101> sipa: added
kmartin85 has joined #bitcoin-core-dev
<achow101> #topic #25871 (bytes1440000)
<core-meetingbot> topic: #25871 (bytes1440000)
<gribble> https://github.com/bitcoin/bitcoin/issues/25871 | contrib: add vasild to trusted keys by vasild · Pull Request #25871 · bitcoin/bitcoin · GitHub
<gribble> https://github.com/bitcoin/bitcoin/issues/25871 | contrib: add vasild to trusted keys by vasild · Pull Request #25871 · bitcoin/bitcoin · GitHub
hashfunc has joined #bitcoin-core-dev
<michaelfolkson> I'm happy to pick this one up if bytes1440000 isn't here
<michaelfolkson> Basically the same one as mine
<michaelfolkson> So the PR to add vasild as a maintainer has now been open 5 months https://github.com/bitcoin/bitcoin/pull/25871
<achow101> I think it's clear that there is disagreement that vasild should be added as a maintainer, and so by that basis alone, should not be due to lack of consensus
<michaelfolkson> What is the reasoning?
<michaelfolkson> And why can't fanquake and/or glozow comment on the PR?
<achow101> however, this is basically new territory for us, as far as I can remember, all previous maintainers were added without any disagreement at all
<michaelfolkson> I'm assuming given the secrecy vasild supposedly has some dark secrets that we can't discuss
<michaelfolkson> Or that fanquake and glozow don't value discussing anything in public
<michaelfolkson> The silence is absurd
<michaelfolkson> What are next steps? Just let fanquake and glozow block whatever they want without disclosing why?
hashfunc has quit [Ping timeout: 260 seconds]
<sipa> Previous maintainers were also all selected because existing maintainers saw a need, and nominated somewhere. It's unclear to me where this is coming from here.
<sipa> *someone
<achow101> from what I can tell, I believe the concern is the lack of addressing the specific concerns brought up in https://github.com/bitcoin/bitcoin/pull/25871#issuecomment-1257032315
hashfunc has joined #bitcoin-core-dev
<michaelfolkson> I get you're neutral sipa but the other maintainers have ACKed it, many long term contributors have ACKed it. fanquake and glozow have to have a reason
<achow101> I think the removal of the net_processing scope was part of that, it doesn't address the concerns regarding review participation (or lack thereof)
<michaelfolkson> This can't be how the project is managed going forward
<michaelfolkson> It isn't fair on vasild
<michaelfolkson> Concerns were raised with glozow when she became maintainer and that was merged within days
<michaelfolkson> Ok so some suggestions for next steps?
<achow101> iirc they were addressed, sufficiently that those who raised them ack'd
<michaelfolkson> Close the PR? Wait for fanquake and glozow to decide the next maintainer?
<michaelfolkson> And provide no rationale for their choice?
<_aj_> next steps? stop trying to make vasild a maintainer when there's no particular need, work on making good PRs and reviewing them
instagibbs has left #bitcoin-core-dev [#bitcoin-core-dev]
instagibbs has joined #bitcoin-core-dev
<sipa> I'm mostly neutral on vasild being added. I'm not neutral on a maintainer being added in the first place - without having a discussion about why another one is needed, I don't understand why we're having this discussion in the first place.
<michaelfolkson> Ok so comment on the PR sipa and _aj_ that another maintainer isn't needed and NACK it
<vasild> some opinions on the PR state that the p2p/networking area is so big that it warrants two maintainers...
<MacroFake> michaelfolkson: sipa already said that on the pull
<michaelfolkson> MacroFake: He didn't NACK it
<instagibbs> mic check (sorry)
<achow101> instagibbs: working
<MacroFake> michaelfolkson: I think it is rude to request others to NACK a pull
hashfunc has quit [Ping timeout: 246 seconds]
<sipa> No, that is not what I'm saying. I'm not NACKing vasild, nor am I NACKing adding another maintainer. I'm saying I'd want to see a discussion about whether another one is needed. I haven
<sipa> 't seen that discussion.
<michaelfolkson> I think it is rude for a potential maintainer to be sitting there for 5 months not knowing what is going on
<lightlike> wasn't the original reason that laanwj (who was doing much of the merging in net) stopped being a maintainer?
<vasild> lightlike: yes, that was
<sipa> And as far as I'm concerned, it's existing maintainers who have to answer that question.
<michaelfolkson> I think it is rude for 2 maintainers to just ignore a PR for a new maintainer with so many ACKs
<michaelfolkson> And ignore requests for them to comment and appear at meetings
<_aj_> i've only found it hard to get review of changes in net; not merges once review's passed.
<_aj_> this entire topic seems much more like it's following https://www.openculture.com/2022/01/read-the-cias-simple-sabotage-field-manual.html rather than trying to solve actual problems to me
<michaelfolkson> Well the sabotage of Vasil being maintainer, yeah maybe
<michaelfolkson> You saying fanquake and glozow are CIA _aj_?
<sipa> wtf
<MacroFake> (slightly off topic: let's not mix up the low level net scope and the higher level p2p net processing scope, while there is overlap, they are largely different modules)
<vasild> If existent maintainers think there is no need for a new one, maybe it would be good to state so. And maybe some maintainer to claim that area? Btw that would put the discussion whether my "skills" are tuned to p2p or just networking in a strange light.
<achow101> michaelfolkson: why is it so important tht vasild become maintainer to you? Is it about him specifically or about the process?
<achow101> I could just nack and close the pr if it makes you feel better ...
<michaelfolkson> achow101: Vasil is being treated awfully. I hated when Luke was treated badly during Taproot activation and hate this
<michaelfolkson> Some people think they can do whatever they want and don't need to tell anyone anything. On a supposedly open source project
<michaelfolkson> If everyone else is happy with it then I'm in the minority
<michaelfolkson> But I wouldn't blame Vasil for moving on to other things. The communication is a joke
<achow101> frankly, I think opinions aren't being shared because of potential backlash from aggressive users such as yourself and bytes1440000
<michaelfolkson> Ok move on to the next topic then
<_aj_> the only lowish level net PR on high pri or PR status projects is #25515 which is in draft. adding #26837 might be a more productive step?
<gribble> https://github.com/bitcoin/bitcoin/issues/25515 | net, test: Virtualise CConnman and add initial PeerManager unit tests by dergoegge · Pull Request #25515 · bitcoin/bitcoin · GitHub
<gribble> https://github.com/bitcoin/bitcoin/issues/26837 | I2P network optimizations by vasild · Pull Request #26837 · bitcoin/bitcoin · GitHub
<achow101> _aj_: done
<michaelfolkson> I will consider Bitcoin Core a project where fanquake and glozow can make whatever decisions they want without communicating why from now on. I didn't think it was that way but apparently it is
<vasild> would be nice if 26837 makes it to the next release
<dergoegge> #25515 can be removed from high-prio, gotta get around to updating it (it got quite a few concept acks, which is what i wanted)
<gribble> https://github.com/bitcoin/bitcoin/issues/25515 | net, test: Virtualise CConnman and add initial PeerManager unit tests by dergoegge · Pull Request #25515 · bitcoin/bitcoin · GitHub
<lightlike> or even have 26837 as a backport candidate? (if it actually helps the stabililty of the i2p network)
<achow101> dergoegge: done
<achow101> #topic #26868 (bytes1440000)
<core-meetingbot> topic: #26868 (bytes1440000)
<gribble> https://github.com/bitcoin/bitcoin/issues/26868 | [WIP] doc: add MAINTAINERS.md by 1440000bytes · Pull Request #26868 · bitcoin/bitcoin · GitHub
<gribble> https://github.com/bitcoin/bitcoin/issues/26868 | [WIP] doc: add MAINTAINERS.md by 1440000bytes · Pull Request #26868 · bitcoin/bitcoin · GitHub
hashfunc has joined #bitcoin-core-dev
<achow101> The pr this is pulled from was closed due to lack of intereset. I suspect the same will happen again
<MacroFake> Looks like people can leave a comment on the pull, unless there is a need to discuss something on IRC?
<achow101> I think he just wants to bring attention to it
<achow101> michaelfolkson: I presume your proposed topic was sufficiently discussed already?
<michaelfolkson> achow101: Indeed
<achow101> does anyone have anything else they would like to discuss?
<_aj_> unrelatedly, is there any way to do pushes so that github better detects that PR's were merged, so they get the purple colour instead of the red colour? eg #26856 or #26827? maybe push to the repo the PR was from first, and delay for a few minutes before also pushing to the other (gui/main) one?
<gribble> https://github.com/bitcoin/bitcoin/issues/26856 | ci: Run one task with all tests on credits by MarcoFalke · Pull Request #26856 · bitcoin/bitcoin · GitHub
<gribble> https://github.com/bitcoin/bitcoin/issues/26827 | doc: use "std lib clock" over "C++11 clock" by fanquake · Pull Request #26827 · bitcoin/bitcoin · GitHub
<MacroFake> _aj_: Maybe a "Closes #pull_number" would help?
<MacroFake> Or we could ditch GitHub (hides)
<achow101> _aj_: I haven't noticed any patterns in how it detects it, seems rather spurious to me
<MacroFake> I contacted support a few years ago and they told me "they are working on it"
<MacroFake> it's an intermittent issue
hashfunc has quit [Ping timeout: 255 seconds]
brunoerg has quit [Remote host closed the connection]
<instagibbs> request a force-purple button
brunoerg has joined #bitcoin-core-dev
<achow101> #endmeeting
<core-meetingbot> topic: 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/ | Meeting topics http://gnusha.org/bitcoin-core-dev/proposedmeetingtopics.txt / http://gnusha.org/bitcoin-core-dev/proposedwalletmeetingtopics.txt
<core-meetingbot> Meeting ended Thu Jan 12 19:35:38 2023 UTC.
<_aj_> MacroFake: presumably there'll be a nostr based replacement for PR review any day now
<michaelfolkson> If anyone agrees with me on this please reach out privately. I am very surprised by the reaction on a supposedly open source project
<_aj_> https://github.com/isaacs/github/issues/363 -- seems like "closes #" also just marks it as red, not merged/purple
brunoerg has quit [Ping timeout: 265 seconds]
<bitcoin-git> [bitcoin] MarcoFalke pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/fcd1a57be060...f4ef856375c5
<bitcoin-git> bitcoin/master fa5dccb MarcoFalke: scripted-diff: ci: Rework docker naming
<bitcoin-git> bitcoin/master fa0584e MarcoFalke: ci: Stop and remove CI container
<bitcoin-git> bitcoin/master f4ef856 MarcoFalke: Merge bitcoin/bitcoin#26850: ci: Stop and remove CI container
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #26850: ci: Stop and remove CI container (master...2301-ci-container-stop-💺) https://github.com/bitcoin/bitcoin/pull/26850
dillontemp has quit [Quit: Client closed]
hashfunc has joined #bitcoin-core-dev
<jamesob> any good reason we don't have granular, optional logging in the script interpreter? Is it performance thing?
<jamesob> *is it a
<_aj_> for script debugging?
<sipa> have you tried btcdeb?
<jamesob> _aj_: yeah. sipa: yeah, sort of, but I found it inconvenient I couldn't just feed in hex (afaict?)
<achow101> jamesob: because the only people who need that are those write scripts by hand and they're insane :p
<jamesob> achow101: haha
<jamesob> sipa: if there's some good workflow for going from python CScripts -> btcdeb, I'd be curious to know
<sipa> I don't know, just wanted to make sure you were aware of the tool.
hashfunc has quit [Ping timeout: 260 seconds]
kmartin85 has quit [Ping timeout: 260 seconds]
brunoerg has joined #bitcoin-core-dev
hashfunc has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 256 seconds]
brunoerg has joined #bitcoin-core-dev
hashfunc has quit [Ping timeout: 260 seconds]
Talkless has quit [Quit: Konversation terminated!]
brunoerg has quit [Ping timeout: 265 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 246 seconds]
dviola has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 246 seconds]
dviola has quit [Quit: WeeChat 3.7.1]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 260 seconds]
dviola has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 256 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 268 seconds]
dviola has quit [Quit: WeeChat 3.8]
coreyphillips has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 246 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 265 seconds]
MrFrancis has quit [Ping timeout: 246 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 260 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 268 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 246 seconds]
brunoerg has joined #bitcoin-core-dev
halo has quit [Quit: WeeChat 3.8]
brunoerg has quit [Ping timeout: 264 seconds]
b_101 has joined #bitcoin-core-dev
hakinser has joined #bitcoin-core-dev
b_101_ has quit [Ping timeout: 272 seconds]
brunoerg has joined #bitcoin-core-dev
hakinser has quit [Quit: Lost terminal]
brunoerg has quit [Ping timeout: 260 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 264 seconds]
brunoerg has joined #bitcoin-core-dev
bytes1440000 has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 260 seconds]
<bytes1440000> Sorry I was not available in the meeting for some personal reasons. Many things that I would like to respond to however I had some points written for meeting earlier which I want to share.
brunoerg has joined #bitcoin-core-dev
<bytes1440000> #25871
<gribble> https://github.com/bitcoin/bitcoin/issues/25871 | contrib: add vasild to trusted keys by vasild · Pull Request #25871 · bitcoin/bitcoin · GitHub
<bytes1440000> Only 2 questions related to this PR: 1. Why is the pull request not merged? 2. Who has the admin access on Github?
<bytes1440000> I don't see any downsides of adding vasild as new maintainer and here are the positives after evaluating PR which has sufficient ACKs from maintainers and contributors:
brunoerg has quit [Ping timeout: 248 seconds]
<bytes1440000> 1. A project like bitcoin core that is considered one of the top 10 critical open source C++ project would get a maintainer for module that matters the most for a bitcoin node after QA
<bytes1440000> 2. Vasild and other contributors would be motivated even if took a few months
<bytes1440000> 3. Hopefully we can get more reviews in some p2p/net pull requests helping in close some of the open requests and encouraging contributors to improve things further with more PRs
<bytes1440000> 4. It will be a fair decision considering number of ACKs and disagreements being addressed in the pull request by author
MrFrancis has joined #bitcoin-core-dev
AaronvanW has quit [Quit: Leaving...]
<bytes1440000> #26868
<gribble> https://github.com/bitcoin/bitcoin/issues/26868 | [WIP] doc: add MAINTAINERS.md by 1440000bytes · Pull Request #26868 · bitcoin/bitcoin · GitHub
brunoerg has joined #bitcoin-core-dev
<bytes1440000> - Looking for overall feedback on adding this doc with minimal info
<bytes1440000> - 'call for maintainers' section could be added if enough contributors agree
<bytes1440000> - Can change 'selection' to a better word if there is any suggestion. A maintainers.md file with list of maintainers and basic stuff about role, adding new maintainer etc. does not affect this project in any negative way.
MrFrancis has quit [Ping timeout: 252 seconds]
<bytes1440000> Positives:
<bytes1440000> 1. Transparency
<bytes1440000> 2. New contributors would be aware of the process
<bytes1440000> 3. Less misinformation about bitcoin core and maintainers
<bytes1440000> 4. Could help in improving things related to maintainers for this project if we agree on some updates to this doc in future or current pull request
brunoerg has quit [Ping timeout: 260 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 255 seconds]
vasild has quit [Remote host closed the connection]
brunoerg has joined #bitcoin-core-dev
vasild has joined #bitcoin-core-dev
AmunRa has quit [Remote host closed the connection]
brunoerg has quit [Ping timeout: 260 seconds]