brunoerg has quit [Remote host closed the connection]
Evel-Knievel has quit [Ping timeout: 246 seconds]
Evel-Knievel has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 250 seconds]
brunoerg has joined #bitcoin-core-dev
Willtech has quit [Quit: Willtech]
Willtech has joined #bitcoin-core-dev
bomb-on has quit [Quit: aллилѹіа!]
test_ has joined #bitcoin-core-dev
_flood has quit [Ping timeout: 240 seconds]
Willtech has quit [Quit: Willtech]
Willtech has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 260 seconds]
<jamesob> hmmmmm have we totally-definitely-certainly outlawed `goto` use?
bitdex has quit [Ping timeout: 240 seconds]
bitdex has joined #bitcoin-core-dev
<_aj_> jamesob: "how can i do [pastebin] better than using goto?"
NorrinRadd has joined #bitcoin-core-dev
evanlinjin has joined #bitcoin-core-dev
<jamesob> _aj_: it's just so tempting to pretend I'm a kernel hacker rather than introduce yet another refactoring commit
<jamesob> and debatably what this would be replacing is less clear than goto IMO (using `continue` in a do-while to do poor man's recursion during init)
<jamesob> but I'll relent, despite always being on the lookout for innovative ways to make assumeutxo PRs harder to merge
<sipa> Unless you're using it in a super tight performance-critical loop, and there is no way to achieve the same performance with a more structured approach... I'd say no.
<jamesob> sipa: you mean "yes[, we have outlawed goto]"
brunoerg has joined #bitcoin-core-dev
a1ph4byte has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Remote host closed the connection]
evanlinjin has quit [Ping timeout: 240 seconds]
<sipa> we have a goto somewhere in the codebase!
evanlinjin has joined #bitcoin-core-dev
<sipa> there's at least one in src/crypto/poly1305 i believe
<sipa> (but it's code copied from elsewhere)
<jamesob> IIRC dongcarl tried to introduce one recently to the libbitcoinkernel client code
<dongcarl> Yup
<dongcarl> It's example code and will be moved to src/kernel/example in future PRs tho
<luke-jr> aren't examples usually under contrib/ ?
evanlinjin has quit [Ping timeout: 240 seconds]
brunoerg has quit [Ping timeout: 260 seconds]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
<jeremyrubin> jamesob: show me a code snippet and i'll show you goto free code
evanlinjin has joined #bitcoin-core-dev
evanlinjin has quit [Ping timeout: 240 seconds]
bitdex has quit [Ping timeout: 240 seconds]
bitdex has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Remote host closed the connection]
cmirror has quit [Remote host closed the connection]
cmirror has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 250 seconds]
evanlinjin has joined #bitcoin-core-dev
mikehu44 has joined #bitcoin-core-dev
mikehu44 has quit [Ping timeout: 276 seconds]
mikehu44 has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 250 seconds]
evanlinjin has quit [Ping timeout: 240 seconds]
mikehu44 has quit [Ping timeout: 256 seconds]
mikehu44 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/f0a834e2f10a...4381681e554d
<bitcoin-git> bitcoin/master 786b3a7 Andrew Chow: tests: Do not always create a descriptor wallet in wallet_createwallet
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
<bitcoin-git> bitcoin/master 4381681 MacroFake: Merge bitcoin/bitcoin#25011: tests: Do not always create a descriptor wall...
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #25011: tests: Do not always create a descriptor wallet in wallet_createwallet (master...fix-legacy-createwallet-test) https://github.com/bitcoin/bitcoin/pull/25011
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
Alina-malina has quit [Ping timeout: 246 seconds]
Alina-malina has joined #bitcoin-core-dev
mikehu44_ has joined #bitcoin-core-dev
mikehu44 has quit [Ping timeout: 256 seconds]
evanlinjin has joined #bitcoin-core-dev
evanlinjin has quit [Remote host closed the connection]
evanlinjin has joined #bitcoin-core-dev
belcher has quit [Ping timeout: 250 seconds]
salvatoshi has joined #bitcoin-core-dev
belcher has joined #bitcoin-core-dev
sipsorcery has joined #bitcoin-core-dev
hashfunc1ebd has joined #bitcoin-core-dev
belcher has quit [Ping timeout: 250 seconds]
mikehu44_ has quit [Ping timeout: 272 seconds]
mikehu44 has joined #bitcoin-core-dev
bitdex has quit [Ping timeout: 240 seconds]
Willtech has quit [Remote host closed the connection]
belcher has joined #bitcoin-core-dev
bitdex has joined #bitcoin-core-dev
<laanwj> if you're running a patched version you should be able to ask git for the necessary information yourself, though
<laanwj> so i'm still not really convinced
<laanwj> jamesob: yes, goto has no place in C++ code, in C code it's sometimes ok
<laanwj> in C code it's useful (often, the least insane way) to do cleanup in case of errors, in C++ code you should use RAII for cleanup instead
<laanwj> please avoid "kernel hacking" in this project if possible, if dumb, easy to understand code (in far as that's even possible with C++) suffices, go for that, the less tricks and surprises the better
Guyver2 has joined #bitcoin-core-dev
<laanwj> anyhow wrt the merge list: the only thing git log --merges v.... v.... *doesn't* do is sort the PRs per category, but this is a painstaking process anyway, there's some automation using the github API and labels but it's often only a heuristic and still needs careful checking
<laanwj> for 23.0 i just didn't get around to that, if someone else does i'm not going to stop them
<laanwj> well, for next release i man, for 23.0 it's too late
<laanwj> mean*
brunoerg has quit [Ping timeout: 240 seconds]
hashfunc1ebd has quit [Ping timeout: 272 seconds]
sipsorcery has quit [Ping timeout: 240 seconds]
sipsorcery has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke pushed 4 commits to master: https://github.com/bitcoin/bitcoin/compare/4381681e554d...9446de160f37
<bitcoin-git> bitcoin/master 74cd038 fanquake: refactor: fix includes in src/init
<bitcoin-git> bitcoin/master c79ad93 fanquake: refactor: fix includes in src/compat
<bitcoin-git> bitcoin/master 9b0a13a fanquake: tidy: Add include-what-you-use
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #24831: tidy: add include-what-you-use (master...include_what_you_use) https://github.com/bitcoin/bitcoin/pull/24831
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
AaronvanW has joined #bitcoin-core-dev
robertspigler has quit [Quit: You have been kicked for being idle]
brunoerg has quit [Ping timeout: 260 seconds]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] laanwj opened pull request #25012: doc: Update blurb in README.md (master...2022-04-readme-blurb) https://github.com/bitcoin/bitcoin/pull/25012
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
NorrinRadd has quit [Ping timeout: 256 seconds]
jonatack has joined #bitcoin-core-dev
NorrinRadd has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] laanwj closed pull request #25012: doc: Update blurb in README.md (master...2022-04-readme-blurb) https://github.com/bitcoin/bitcoin/pull/25012
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke opened pull request #25013: Remove cs_main from verifymessage, move msg utils to new file (master...2204-rpc-misc-🔌) https://github.com/bitcoin/bitcoin/pull/25013
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
brunoerg has joined #bitcoin-core-dev
evanlinjin has quit [Ping timeout: 240 seconds]
<laanwj> re: #25012 I really hoped to remove the 'whether bitcoin is experimental' concern from our repo once and for all, but seems it just comes back in a different form then, really, never mind
<gribble> https://github.com/bitcoin/bitcoin/issues/25012 | doc: Update blurb in README.md by laanwj · Pull Request #25012 · bitcoin/bitcoin · GitHub
<laanwj> 'File "..." contains a shebang line, but has the file permission 775 instead of the expected executable permission 755. Do "chmod 755 test/lint/lint-python-mutable-default-parameters.py" (or remove the shebang line)' huh
<fanquake> laanwj: I think removing that is fine
<laanwj> fanquake: thanks
<_aj_> laanwj: <3
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke pushed 9 commits to master: https://github.com/bitcoin/bitcoin/compare/9446de160f37...b51e60f91472
<bitcoin-git> bitcoin/master 5921b86 Carl Dong: init: Reset mempool and chainman via reconstruction
<bitcoin-git> bitcoin/master 98f4bda Carl Dong: refactor: Convert warningcache to std::array
<bitcoin-git> bitcoin/master 6e747e8 Anthony Towns: validation: default initialize and guard chainman members
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #22564: refactor: Move mutable globals cleared in `::UnloadBlockIndex` to `BlockManager` (master...2021-07-kirby-inhale-global-muts) https://github.com/bitcoin/bitcoin/pull/22564
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
<laanwj> _aj_: 💜
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/b51e60f91472...85aea18ae660
<bitcoin-git> bitcoin/master 85aea18 laanwj: Merge bitcoin/bitcoin#24982: tests: Port `lint-all.sh` to `lint-all.py`
<bitcoin-git> bitcoin/master 29f44fe hiago: Converting `lint-all.sh` to `lint-all.py`.
evanlinjin has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] laanwj merged pull request #24982: tests: Port `lint-all.sh` to `lint-all.py` (master...port_lint-all.sh_to_python) https://github.com/bitcoin/bitcoin/pull/24982
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
evanlinjin has quit [Quit: Konversation terminated!]
evanlinjin has joined #bitcoin-core-dev
evanlinjin has quit [Client Quit]
evanlinjin has joined #bitcoin-core-dev
evanlinjin_ has joined #bitcoin-core-dev
evanlinjin has quit [Ping timeout: 240 seconds]
Guest50 has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] laanwj opened pull request #25015: test: Use permissions from git in lint-files.py (master...2022-04-lint-permissions-git) https://github.com/bitcoin/bitcoin/pull/25015
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
mudsip has joined #bitcoin-core-dev
mikehu44_ has joined #bitcoin-core-dev
mikehu44 has quit [Ping timeout: 272 seconds]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> bitcoin/master fa82a1e MacroFake: lint: Mention NONFATAL_UNREACHABLE in lint-assertions.py
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/85aea18ae660...e36c612e5a9b
<bitcoin-git> bitcoin/master e36c612 fanquake: Merge bitcoin/bitcoin#24988: lint: Mention NONFATAL_UNREACHABLE in lint-as...
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
<bitcoin-git> [bitcoin] fanquake merged pull request #24988: lint: Mention NONFATAL_UNREACHABLE in lint-assertions.py (master...2204-lint-abort-🐛) https://github.com/bitcoin/bitcoin/pull/24988
mikehu44 has joined #bitcoin-core-dev
mikehu44_ has quit [Ping timeout: 276 seconds]
mudsip has quit []
dougefish has quit [Ping timeout: 250 seconds]
dougefish has joined #bitcoin-core-dev
test_ is now known as _flood
bitdex has quit [Ping timeout: 240 seconds]
bitdex has joined #bitcoin-core-dev
bitdex has quit [Quit: = ""]
<jamesob> jeremyrubin: no trouble constructing the goto-free code, just entails a lot of refactoring etc.
<jamesob> laanwj: don't worry, I was kidding about the "kernel hacking" comment
<jamesob> attempted humor not conveying well I guess
<_aj_> laanwj: or 🍦
evanlinjin_ has quit [Quit: Konversation terminated!]
<laanwj> jamesob: sorry for taking it so serious, it's hard to gauge sometimes!
<laanwj> _aj_: yess
<jamesob> laanwj: no worries, I was just delirious and in a rambling mood during a rebase last night :)
<laanwj> jamesob: i meann remmeber that we have an actual system call sandbox in bitcoin core, it's scary close to kernel hacking
<jamesob> haha, good point!
jespada has quit [Ping timeout: 240 seconds]
jespada has joined #bitcoin-core-dev
<jonatack> 10 PRINT "HELLO WORLD" 20 GOTO 10
<sipa> what a basic joke
<jonatack> I think that was the first program I ever wrote :D
<sipa> mine's probably close
<jonatack> indeed basic
<jamesob> you guys are old-timers! my first program was in *visual* basic, truly high-tech stuff
<sipa> which version? i used visual basic 3, 5, and 6
gnaf has joined #bitcoin-core-dev
<jamesob> all I remember was that it was on Windows ME...
<sipa> My condolences.
<jamesob> lol
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake pushed 12 commits to master: https://github.com/bitcoin/bitcoin/compare/e36c612e5a9b...dd17c42a1642
<bitcoin-git> bitcoin/master 1392e8e Carl Dong: build: Don't add unrelated libs to LIBTEST_*
<bitcoin-git> bitcoin/master 64caf94 Carl Dong: build: Remove vestigial LIBLEVELDB_SSE42
<bitcoin-git> bitcoin/master 05d1525 Carl Dong: build: Create .la library for crc32c
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake merged pull request #24322: [kernel 1/n] Introduce initial `libbitcoinkernel` (master...2022-02-libbitcoinkernel-p1-minimal) https://github.com/bitcoin/bitcoin/pull/24322
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
<laanwj> in MSX basic you could do this cursed thing where you stripped the line numbers, and replaced gotos by assignments to the interpreter pointer in the token stream—
<laanwj> there are things much worse than gotos :)
<sipa> Duff's device?
Guest50 has quit [Quit: Client closed]
<laanwj> yes, i'd say that qualifies
<laanwj> or what about the gcc computed goto extension
<laanwj> maybe it could be combined with a Duff's device for extra curse points
<cfields> libbitcoinkernel \o/
<sipa> laanwj: Throw in another longjmp() too
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke closed pull request #20030: validation: Remove useless call to mempool->clear() (master...2009-valMemClear) https://github.com/bitcoin/bitcoin/pull/20030
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
<laanwj> cfields: oh no
<lightlike> maybe disguise the goto as "do{...} while(false)" with breaks and hope no one will notice (I think we had this kind of thing until recently)
<laanwj> sipa: ah ye our old friend longjmp, was surpised to see it actually used in freetype in #582
<gribble> https://github.com/bitcoin/bitcoin/issues/582 | Bitcoin-Qt signmessage GUI by luke-jr · Pull Request #582 · bitcoin/bitcoin · GitHub
<sipa> That doesn't simulate a goto; it's a trick to construct a macro that works without branches in an if then else
<roconnor> oh, "with breaks"
<roconnor> ... wait, I don't get it.
<sipa> (not that we permit branchless if/then/elses in the first place, but it doesn't hurt...)
<roconnor> https://stackoverflow.com/a/257421 -- okay I'm impressed.
<sipa> lightlike roconnor: that approach has the advance that it still works in C++ code with new variables defined in the body (in C++ a goto cannot cross the definition of a variable)
<sipa> I don't know... I think that do ... break ... break ... while(0) pattern is a slight improvement over a goto, but I can see it'd be controversial
<roconnor> For me it has the advantage that it is supported by VST while goto is not.
<laanwj> i'd agree, it's "an improvement over a goto" but then also everything is said, if awkward constructions can completely be avoided that'd be even better
<roconnor> C has fewer ways of avoiding akward constructions than C++ does.
<laanwj> yes, very true
<sipa> at some point you just resort to an asm { ... } block ;)
<laanwj> sometimes a high level language just gets in the way :)
a1ph4byte has quit [Quit: Client closed]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke reopened pull request #19909: txmempool: Remove unused clear() member function (master...2009-noTxpoolClear) https://github.com/bitcoin/bitcoin/pull/19909
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
mikehu44 has quit [Ping timeout: 256 seconds]
pk-b2 has joined #bitcoin-core-dev
a1ph4byte has joined #bitcoin-core-dev
salvatoshi has quit [Ping timeout: 256 seconds]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
pk-b2 has quit [Quit: My MacBook has gone to sleep. ZZZzzz…]
Kaizen_Kintsugi_ has quit [Remote host closed the connection]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
bfsfhkacjzgcytf9 has joined #bitcoin-core-dev
<cfields> laanwj: "oh no" ?
<laanwj> cfields: i mean in the sense that there's an actual kernel in bitcoin now so we can't have anything against "kernel hacking"
salvatoshi has joined #bitcoin-core-dev
<laanwj> jamesob won
<sipa> "kernel" and "core" are practically synonyms anyway
jonatack has quit [Ping timeout: 248 seconds]
<cfields> oh :)
<laanwj> but yeah i can see how that could be interpreted wrong, to be clear i really like libbitcoinkernel
jonatack has joined #bitcoin-core-dev
pk-b2 has joined #bitcoin-core-dev
pk-b2 has quit [Client Quit]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] jonatack opened pull request #25016: blockstorage, refactor: improve GetFirstStoredBlock() (master...GetFirstStoredBlock-improvements) https://github.com/bitcoin/bitcoin/pull/25016
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
salvatoshi has quit [Ping timeout: 260 seconds]
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/dd17c42a1642...47b8256da872
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
<bitcoin-git> bitcoin/master 47b8256 laanwj: Merge bitcoin/bitcoin#24937: test: Remove previous release check in featur...
<bitcoin-git> bitcoin/master fafd674 MarcoFalke: test: Remove previous release check
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] laanwj merged pull request #24937: test: Remove previous release check in feature_taproot.py (master...2204-noPrevRelTap-🏥) https://github.com/bitcoin/bitcoin/pull/24937
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] laanwj pushed 4 commits to master: https://github.com/bitcoin/bitcoin/compare/47b8256da872...8730bd3fc87c
<bitcoin-git> bitcoin/master bbbcb96 Hennadii Stepanov: build, refactor: Fix indentation
<bitcoin-git> bitcoin/master 65cddf6 Hennadii Stepanov: build: Fix `AC_CHECK_HEADERS` and `AC_CHECK_LIB` for `miniupnpc` package
<bitcoin-git> bitcoin/master 1659034 Hennadii Stepanov: build: Fix `AC_CHECK_HEADERS` and `AC_CHECK_LIB` for `libnatpmp` package
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] laanwj merged pull request #24958: build: Fix macOS Apple M1 build with miniupnpc and libnatpmp. Again :) (master...220424-flags) https://github.com/bitcoin/bitcoin/pull/24958
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
sipsorcery has quit [Remote host closed the connection]
sipsorcery has joined #bitcoin-core-dev
Guest60 has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] jonatack opened pull request #25017: validation: make the CScriptCheck::swap() member function `noexcept` (master...swap-noexcept) https://github.com/bitcoin/bitcoin/pull/25017
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
Guest60 has quit [Client Quit]
bomb-on has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Remote host closed the connection]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] scgbckbone opened pull request #25019: normalize (lowercase) master fp in ExternalSigner::SignTransaction (master...lower_master_fp_of_ext_signer_in_SignTransaction) https://github.com/bitcoin/bitcoin/pull/25019
bitcoin-git has left #bitcoin-core-dev [#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
Kaizen_Kintsugi_ has quit [Remote host closed the connection]
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] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/8730bd3fc87c...dabec990135c
<bitcoin-git> bitcoin/master dabec99 MacroFake: Merge bitcoin/bitcoin#24956: Call CHECK_NONFATAL only once where needed
<bitcoin-git> bitcoin/master fab34d3 MarcoFalke: Call CHECK_NONFATAL only once where needed
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #24956: Call CHECK_NONFATAL only once where needed (master...2204-once-check-tip-🦅) https://github.com/bitcoin/bitcoin/pull/24956
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
Kaizen_Kintsugi_ has quit [Remote host closed the connection]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] hebasto opened pull request #25020: contrib: Check symbols in the `bitcoinconsensus` shared library (master...220428-shared) https://github.com/bitcoin/bitcoin/pull/25020
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] achow101 pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/dabec990135c...4cf9fa0b662a
<bitcoin-git> bitcoin/master 2052e3a Martin Zumsande: wallet: ignore chainStateFlushed notifications while attaching chain
<bitcoin-git> bitcoin/master 4cf9fa0 Andrew Chow: Merge bitcoin/bitcoin#24984: wallet: ignore chainStateFlushed notification...
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] achow101 merged pull request #24984: wallet: ignore chainStateFlushed notifications while attaching chain (master...202204_wallet_rescan) https://github.com/bitcoin/bitcoin/pull/24984
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
<hebasto> cfields: in the context of your https://github.com/bitcoin/bitcoin/pull/24322#issuecomment-1112453779 mind looking into #24387 ?
<gribble> https://github.com/bitcoin/bitcoin/issues/24387 | build: Fail to build `libmultiprocess` for the `arm-linux-gnueabihf` target · Issue #24387 · bitcoin/bitcoin · GitHub
<laanwj> #startmeeting
<core-meetingbot> Meeting started Thu Apr 28 19:00:52 2022 UTC. The chair is laanwj. Information about MeetBot at https://bitcoin.jonasschnelli.ch/ircmeetings.
<core-meetingbot> Available commands: action commands idea info link nick
<hebasto> 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 larryruane lightlike luke-jr maaku marcofalke meshcollider michagogo moneyball
<laanwj> morcos nehan NicolasDorier paveljanik petertodd phantomcircuit promag provoostenator ryanofsky sdaftuar sipa vasild
<glozow> hi
<sipsorcery> hi
<sipa> hi
<laanwj> welcome to the weekly general bitcoin-core-dev meeting!
<laanwj> there have been no topics proposed in advance using #proposedmeetingtopic, any last minute ones people would like to discuss today?
<cfields> hi
<jonatack> 10 print "hi" 20 goto 10
<dongcarl> I can give a very quick update/announcement on libbitcoinkernel if no other topics are there
<lightlike> hi
<laanwj> dongcarl: sounds good to me, let's have that as topic after high prio for review
<dongcarl> cool
<jonatack> dongcarl: post-merge ack modulo the clang build chatter ;)
<kanzure> hi
<jeremyrubin> Hi
<laanwj> #topic High priority for review
<core-meetingbot> topic: High priority for review
<laanwj> currently 11 blockers 1 chasing concept ACK https://github.com/bitcoin/bitcoin/projects/8
<laanwj> anything to add/remove?
<achow101> hi
<laanwj> or that's on the list and (almost) ready for merge
<lightlike> #24322 was merged and can be removed
<gribble> https://github.com/bitcoin/bitcoin/issues/24322 | [kernel 1/n] Introduce initial `libbitcoinkernel` by dongcarl · Pull Request #24322 · bitcoin/bitcoin · GitHub
<glozow> #23443 seems close, though needs rebase
<gribble> https://github.com/bitcoin/bitcoin/issues/23443 | p2p: Erlay support signaling by naumenkogs · Pull Request #23443 · bitcoin/bitcoin · GitHub
<jonatack> #24757 for me, please
<gribble> https://github.com/bitcoin/bitcoin/issues/24757 | build, ci: add `DEBUG_LOCKCONTENTION` to --enable-debug and CI by jonatack · Pull Request #24757 · bitcoin/bitcoin · GitHub
<laanwj> lightlike: even better! should we add the next one dongcarl?
<laanwj> glozow: thanks!
<dongcarl> Yes! #24410 it is
<gribble> https://github.com/bitcoin/bitcoin/issues/24410 | [kernel 2a/n] Split hashing/index `GetUTXOStats` codepaths, decouple from `coinstatsindex` by dongcarl · Pull Request #24410 · bitcoin/bitcoin · GitHub
<laanwj> jonatack: added
<sipa> kernel 2a is written in c++2a?
<dongcarl> Hahaha I wish!
<laanwj> dongcarl: added
<dongcarl> laanwj: Thanks!
<laanwj> #topic Quick update on libbitcoinkernel (dongcarl)
<core-meetingbot> topic: Quick update on libbitcoinkernel (dongcarl)
<dongcarl> Hi all, after the merge of #24322, the initial build systems changes of libbitcoinkernel are done.
<gribble> https://github.com/bitcoin/bitcoin/issues/24322 | [kernel 1/n] Introduce initial `libbitcoinkernel` by dongcarl · Pull Request #24322 · bitcoin/bitcoin · GitHub
<dongcarl> We will now be doing the more meaningful part of decoupling consensus from other modules. I hope that these PRs will have a wider review base, since it doesn’t require any knowledge of build system minutiae.
<dongcarl> The first PR ready for review is #24410, which is fairly straightforward.
<gribble> https://github.com/bitcoin/bitcoin/issues/24410 | [kernel 2a/n] Split hashing/index `GetUTXOStats` codepaths, decouple from `coinstatsindex` by dongcarl · Pull Request #24410 · bitcoin/bitcoin · GitHub
<dongcarl> A lot of these PRs are not dependent on each other, so at times we will have multiple PRs that are ready for review and not based on one another. There’s no need to wait for a PR to be merged to review the other one!
<jonatack> laanwj: ty!
<dongcarl> One last note: I’m happy to chat about any of these PRs to get people up to speed. It also helps me know what parts are unclear so I can improve my docs/commit messages.
<dongcarl> Hit me up on any communication channel, and I'm also more than happy to do a video/audio call for better bandwidth
* dongcarl is done, but can answer questions if people have any
<sipa> Cool.
<laanwj> dongcarl: thanks for the update
<dongcarl> :)
<laanwj> sounds clear to me, there's a lot to review but hopefully we can make quite some progress before 24.0
<dongcarl> Yup that is my hope too. If anyone spots anything that I could be doing to make things easier to review, let me know!
<hebasto> dongcarl: thanks!
<jonatack> dongcarl: do you recommend we developers build with --enable-experimental-util-chainstate or --with-experimental-kernel-lib? (for the moment i have both in my bash alias)
<dongcarl> jonatack: setting --enable-experimental-util-chainstate will automatically turn on --with-experimental-kernel-lib, unless you set --without-experimental-kernel-lib, in which case configure will fail
<laanwj> heh
<jonatack> ok, thanks, so --enable-experimental-util-chainstate sgtm then (and [kernel 2a/n] looks straightforward to review)
<dongcarl> jonatack: :-)
<cfields> atm the results are intended to be not very usable, but the kernel lib and bitcoin-chainstate (excluding some windows weirdness) should at least build successfully. Please report if that's not the case anywhere.
<dongcarl> Many thanks to cfields who got me through various tough situations in kernel 1/n and in upcoming PRs
<cfields> thanks ryanofsky for great review as well.
<dongcarl> ryan truly is of the sky
<laanwj> i guess that concludes the meeting, or does anyone have any last last minute topic
<jonatack> following the review feedback by both of you on [kernel 1a] was enlightening, as were the excellent commit messages
<laanwj> thanks everyone for attending, see you next week
<laanwj> #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 Apr 28 19:23:11 2022 UTC.
<dongcarl> jonatack: Haha I'm glad you think so! :-)
<dongcarl> laanwj: Thanks!
salvatoshi has joined #bitcoin-core-dev
<jamesob> dongcarl: looking forward to reviewing 2/n!
Talkless has quit [Quit: Konversation terminated!]
<dongcarl> jamesob: 🎉
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> bitcoin/master 606ce05 Andrew Chow: Merge bitcoin/bitcoin#18554: wallet: ensure wallet files are not reused ac...
<bitcoin-git> [bitcoin] achow101 pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/4cf9fa0b662a...606ce05ec225
<bitcoin-git> bitcoin/master 9687659 Seibart Nedor: wallet: ensure wallet files are not reused across chains
<bitcoin-git> bitcoin/master 5f21321 Seibart Nedor: tests: add tests for cross-chain wallet use prevention
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] achow101 merged pull request #18554: wallet: ensure wallet files are not reused across chains (master...wallet-file-reuse-prevention) https://github.com/bitcoin/bitcoin/pull/18554
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bfsfhkacjzgcytf9 has quit [Ping timeout: 276 seconds]
<fanquake> jonatack: what clang build chatter ?
salvatoshi has quit [Ping timeout: 246 seconds]
<jonatack> can be ironed gradually along with any other more significant issues that come up
jarthur has joined #bitcoin-core-dev
dlb76 has joined #bitcoin-core-dev
<dongcarl> achow101: Known failure? https://cirrus-ci.com/task/4571677035069440
<cfields> hebasto: i'm not ignoring you btw, just investigating more fully and waiting to put all my thoughts in one place.
a1ph4byte has quit [Quit: Client closed]
<achow101> dongcarl: not to me
sipsorcery has quit [Ping timeout: 248 seconds]
Guyver2 has quit [Remote host closed the connection]
sipsorcery has joined #bitcoin-core-dev
<jonatack> fixing
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] jonatack opened pull request #25021: test: fix intermittent wallet_encryption functional test (master...fix-intermittent-wallet_encryption-test) https://github.com/bitcoin/bitcoin/pull/25021
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
sipsorcery has quit [Ping timeout: 250 seconds]
tecnecio has joined #bitcoin-core-dev
<hebasto> cfields: thank you
bomb-on has quit [Quit: aллилѹіа!]