Kaizen_Kintsugi_ has quit [Ping timeout: 250 seconds]
jarthur has quit [Quit: jarthur]
vysn has joined #bitcoin-core-dev
AaronvanW has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [gui] promag opened pull request #594: scripted-diff: replace deprecated Q_OS_MAC with Q_OS_MACOS (master...q_os_macos) https://github.com/bitcoin-core/gui/pull/594
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 240 seconds]
AaronvanW has quit [Ping timeout: 272 seconds]
brunoerg has quit [Remote host closed the connection]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
Willtech has quit [Quit: Willtech]
Willtech has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] rnapoles closed pull request #24983: Build Windows version with msys2 build (master...windows-msys2-build) https://github.com/bitcoin/bitcoin/pull/24983
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
Kaizen_Kintsugi_ has quit [Remote host closed the connection]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 250 seconds]
brunoerg has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] rnapoles opened pull request #24985: doc,test: Compilation for 64-bit Windows with msys2 (master...msys2-windows-build) https://github.com/bitcoin/bitcoin/pull/24985
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
Willtech has quit [Quit: Willtech]
Willtech has joined #bitcoin-core-dev
mikehu44 has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 260 seconds]
evanlinjin has quit [Remote host closed the connection]
evanlinjin has joined #bitcoin-core-dev
AaronvanW has joined #bitcoin-core-dev
evanlinjin has quit [Remote host closed the connection]
evanlinjin has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 240 seconds]
evanlinjin has quit [Remote host closed the connection]
evanlinjin has joined #bitcoin-core-dev
AaronvanW has quit [Ping timeout: 240 seconds]
brunoerg has joined #bitcoin-core-dev
bitdex has quit [Quit: = ""]
Guest6680 has joined #bitcoin-core-dev
Guest6680 has quit [Client Quit]
evanlinjin has quit [Remote host closed the connection]
evanlinjin has joined #bitcoin-core-dev
a1ph4byte18 has joined #bitcoin-core-dev
evanlinjin has quit [Remote host closed the connection]
evanlinjin has joined #bitcoin-core-dev
cmirror has quit [Remote host closed the connection]
cmirror has joined #bitcoin-core-dev
hashfunc1ebd has joined #bitcoin-core-dev
evanlinjin has quit [Remote host closed the connection]
evanlinjin has joined #bitcoin-core-dev
a1ph4byte18 has quit [Ping timeout: 252 seconds]
a1ph4byte has quit [Ping timeout: 252 seconds]
brunoerg has quit [Ping timeout: 250 seconds]
Kaizen_Kintsugi_ has quit [Remote host closed the connection]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
evanlinjin has quit [Remote host closed the connection]
evanlinjin has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 250 seconds]
hashfunc1ebd has quit [Ping timeout: 250 seconds]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
mikehu44 has quit [Ping timeout: 256 seconds]
mikehu44_ has joined #bitcoin-core-dev
mikehu44 has joined #bitcoin-core-dev
x88x88x has quit [Remote host closed the connection]
mikehu44_ has quit [Ping timeout: 256 seconds]
x88x88x has joined #bitcoin-core-dev
sipsorcery has joined #bitcoin-core-dev
AaronvanW has joined #bitcoin-core-dev
sudoforge has quit [Ping timeout: 256 seconds]
a1ph4byte has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 260 seconds]
AaronvanW has quit [Ping timeout: 246 seconds]
Kaizen_Kintsugi_ has quit [Ping timeout: 240 seconds]
mikehu44_ has joined #bitcoin-core-dev
mikehu44 has quit [Ping timeout: 256 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 240 seconds]
brunoerg has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 250 seconds]
<vasild> midnight: Hello :)
sipsorcery has quit [Ping timeout: 240 seconds]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
<PaperSword> Hello
<laanwj> provoostenator: simply said because the "ghi" script it uses (https://github.com/gkrizek/ghi) is based on github's original bot which didn't have a persistent IRC mode either, PRs welcome though :)
<laanwj> speaking of element, it would be straightforward on matrix to keep the notificator in the room, IRC's lack of persistent sessions between connections makes it harder to do
<laanwj> that said, ghi doesn't currently have a matrix backend either
<laanwj> hebasto: yeah, and time to create a 0.24 issue
<laanwj> 24.0*
Guyver2 has joined #bitcoin-core-dev
AaronvanW has joined #bitcoin-core-dev
<laanwj> (created a feature request for ghi: https://github.com/gkrizek/ghi/issues/27)
brunoerg has quit [Ping timeout: 260 seconds]
vasild has quit [Remote host closed the connection]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] hebasto opened pull request #24986: build: Move `consensus/merkle` module out from `libbitcoinconsensus` (master...220426-merkle) https://github.com/bitcoin/bitcoin/pull/24986
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
sudoforge 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/1e7db37e76c5...1ae65b4c5fbf
<bitcoin-git> bitcoin/master 1ae65b4 MacroFake: Merge bitcoin/bitcoin#24968: Move only: Move TxOrphange tests to orphange_...
<bitcoin-git> bitcoin/master b8f17fb John Newbery: [tests] Move TxOrphange tests to orphange_tests.cpp
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #24968: Move only: Move TxOrphange tests to orphange_tests.cpp (master...202204_move_orphanage_tests) https://github.com/bitcoin/bitcoin/pull/24968
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
vasild has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 248 seconds]
brunoerg has joined #bitcoin-core-dev
<laanwj> proposed release schedule for 24.0 https://github.com/bitcoin/bitcoin/issues/24987
evanlinjin has quit [Ping timeout: 240 seconds]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke opened pull request #24988: lint: Mention NONFATAL_UNREACHABLE in lint-assertions.py (master...2204-lint-abort-🐛) https://github.com/bitcoin/bitcoin/pull/24988
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/master 709af67 w0xlt: p2p: replace RecursiveMutex `m_total_bytes_sent_mutex` with Mutex
<bitcoin-git> bitcoin/master 8be75fd w0xlt: p2p: add assertions and negative TS annotations for `m_total_bytes_sent_mu...
<bitcoin-git> bitcoin/master a237a06 w0xlt: scripted-diff: rename cs_totalBytesSent -> m_total_bytes_sent_mutex
<bitcoin-git> [bitcoin] laanwj pushed 4 commits to master: https://github.com/bitcoin/bitcoin/compare/1ae65b4c5fbf...a19f641a80cb
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] laanwj merged pull request #24157: p2p: Replace RecursiveMutex `cs_totalBytesSent` with Mutex and rename it (master...cs_totalBytesSent_mutex) https://github.com/bitcoin/bitcoin/pull/24157
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
<hebasto> a half is done :) -- 22 of 43 tasks in #19303
<gribble> https://github.com/bitcoin/bitcoin/issues/19303 | Replace all of the RecursiveMutex instances with the Mutex ones · Issue #19303 · bitcoin/bitcoin · GitHub
Kaizen_Kintsugi_ has quit [Ping timeout: 240 seconds]
<vasild> hebasto: wow! :)
<laanwj> hebasto: neat!
<vasild> the first 90% of the work takes the 90% of the time and the rest takes the other 90% of the time
<hebasto> in thread safety direction, I think we should focus on #24931 now
<gribble> https://github.com/bitcoin/bitcoin/issues/24931 | Strengthen thread safety assertions by ajtowns · Pull Request #24931 · bitcoin/bitcoin · GitHub
An0rak has joined #bitcoin-core-dev
AaronvanW has quit [Ping timeout: 272 seconds]
sudoforge has quit [Quit: 404]
<PaperSword> hebasto: nice
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
mikehu44 has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] promag opened pull request #24989: scripted-diff: rename BytePtr to AsBytePtr (master...rename-byteptr) https://github.com/bitcoin/bitcoin/pull/24989
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
mikehu44_ has quit [Ping timeout: 256 seconds]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake pushed 4 commits to master: https://github.com/bitcoin/bitcoin/compare/a19f641a80cb...f436bfd126ad
<bitcoin-git> bitcoin/master 4fad7e4 Kiminuo: test: add unit tests for `SplitString` helper
<bitcoin-git> bitcoin/master 9cc8e87 Sebastian Falbesoner: refactor: introduce single-separator split helper `SplitString`
<bitcoin-git> bitcoin/master a62e844 MarcoFalke: fuzz: add `SplitString` fuzz target
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake merged pull request #22953: refactor: introduce single-separator split helper (boost::split replacement) (master...202109-refactor-boost_split_replacement_single_sep) https://github.com/bitcoin/bitcoin/pull/22953
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
<_aj_> hebasto: btw, happy to split up 24931, don't really understand which parts are easier/harder though
brunoerg has quit [Ping timeout: 256 seconds]
brunoerg has joined #bitcoin-core-dev
jonatack has joined #bitcoin-core-dev
pigeons has quit [Ping timeout: 256 seconds]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] vasild opened pull request #24991: init: allow startup with -onlynet=onion -listenonion=1 (master...onlynet_onion_with_listenonion_is_ok) https://github.com/bitcoin/bitcoin/pull/24991
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
Kaizen_Kintsugi_ has quit [Ping timeout: 256 seconds]
evanlinjin has joined #bitcoin-core-dev
AaronvanW has joined #bitcoin-core-dev
<vasild> _aj_: hebasto: I didn't like that there is now "Mutex" and "GlobalMutex" types. IMO nothing can justify that.
<vasild> if there is a PR without that, I would look deeper into it (and possibly ACK it)
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake closed pull request #15112: build: Optionally enable -Wzero-as-null-pointer-constant (master...zero-as-null-pointer-constant) https://github.com/bitcoin/bitcoin/pull/15112
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
<_aj_> vasild: "nothing can justify that" -- so even if it avoided 1000 remote exploits and made everything 100 times fast it would still be unacceptable? i'm not sure if there's some language thing here, but i find it really hard to understand your take on locking
brunoerg has quit [Ping timeout: 248 seconds]
Guyver2_ has joined #bitcoin-core-dev
Guyver2 has quit [Ping timeout: 246 seconds]
brunoerg has joined #bitcoin-core-dev
mikehu44 has quit [Quit: https://quassel-irc.org - Chat comfortably. Anywhere.]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
<laanwj> _aj_: it depends, i don't know the particular context, but say, sometimes understanding of things advances and some things turn out to be really never suitable for use and always bad design, recursive mutexes probably fit in that category
jonatack has quit [Ping timeout: 272 seconds]
<_aj_> laanwj: GuardMutex is literally just an empty subclass of Mutex so that AssertLockNotHeld(Mutex) can have stronger thread safety annotations than AssertLockNotedHeld(GuardMutex) while otherwise behaving exactly the same
<_aj_> err, GlobalMutex
bitcoin-git has joined #bitcoin-core-dev
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/f436bfd126ad...30c1c6ed80c5
<bitcoin-git> bitcoin/master 30c1c6e fanquake: Merge bitcoin/bitcoin#24979: Precomputed hashes are note #16 in BIP341
<bitcoin-git> bitcoin/master df08c23 Gregory Sanders: Precomputed hashes are note #16 in BIP341
<laanwj> _aj_: that doesn't sound bad
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake merged pull request #24979: Precomputed hashes are note #16 in BIP341 (master...bip341_note_16) https://github.com/bitcoin/bitcoin/pull/24979
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
evanlinjin has quit [Remote host closed the connection]
<_aj_> laanwj: (i mean replacing "GlobalMutex g_x_mutex; X x GUARDED_BY(g_x_mutex);" with "class ProtectedX { Mutex m; X x GUARDED_BY(m); .. }; ProtectedX g_x;" is an improvement that does away with GlobalMutex, but also involves moving the code that accesses "x" into the new class, which is a bunch more work than a scripted diff)
evanlinjin has joined #bitcoin-core-dev
evanlinjin has quit [Client Quit]
evanlinjin_ has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/30c1c6ed80c5...269dcad16e3d
<bitcoin-git> bitcoin/master dac44fc Martin Zumsande: init: disallow reindex-chainstate with optional indexes
<bitcoin-git> bitcoin/master 269dcad fanquake: Merge bitcoin/bitcoin#24789: init, index: disallow indexes when running re...
<bitcoin-git> bitcoin/master 62e1428 Martin Zumsande: doc: Add note that -reindex will rebuild optional indexes
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake merged pull request #24789: init, index: disallow indexes when running reindex-chainstate (master...202204_reindex_corrupt) https://github.com/bitcoin/bitcoin/pull/24789
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
jonatack has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 260 seconds]
Kaizen_Kintsugi_ has quit [Ping timeout: 250 seconds]
evanlinjin_ has quit [Quit: Konversation terminated!]
<vasild> _aj_: my point is this: "if there is a PR without that, I would look deeper into it (and possibly ACK it)"
<vasild> or is it not possible to split GlobalMutex away from #24931
<gribble> https://github.com/bitcoin/bitcoin/issues/24931 | Strengthen thread safety assertions by ajtowns · Pull Request #24931 · bitcoin/bitcoin · GitHub
<vasild> ?
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
<_aj_> vasild: negative assertions don't work with global scope mutexes; as it stands we don't have any AssertLockNotHeld(g_cs_x) except when g_cs_x is already a recursivemutex so the first two commits work without specially annotating the global non-recursive mutexes. but adding negative annotations for LOCK can't be done on global mutexes
* vasild looking...
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake closed pull request #16037: rpc: Enable wallet import on pruned nodes (master...2019-05-importwallet-pruned) https://github.com/bitcoin/bitcoin/pull/16037
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake closed pull request #18061: util: Pass size to ParseHex to assist preallocation (master...2020-02-remove-strlen) https://github.com/bitcoin/bitcoin/pull/18061
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
evanlinjin has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake closed pull request #15729: rpc: Raise error in getbalance if minconf is not zero (master...2019-04-drop-getbalance-minconf) https://github.com/bitcoin/bitcoin/pull/15729
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake closed pull request #20277: test: Extend p2p_ibd_tx_relay.py to verify no-transaction are requested during IBD (master...2020-10-ibd-txrelay) https://github.com/bitcoin/bitcoin/pull/20277
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake closed pull request #18964: rpc, wallet: Scan mempool after import* (master...2020-05-fix-18954) https://github.com/bitcoin/bitcoin/pull/18964
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake closed pull request #14641: rpc: Add min_conf option to fund transaction calls (master...2018-11-fundrawtransaction) https://github.com/bitcoin/bitcoin/pull/14641
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake closed pull request #18471: qa: Test shared validation interface (master...2020-03-test-shared-validation-interface) https://github.com/bitcoin/bitcoin/pull/18471
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
jespada_ has quit [Ping timeout: 260 seconds]
jespada has joined #bitcoin-core-dev
evanlinjin has quit [Quit: Konversation terminated!]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake closed pull request #20752: [fuzzer] generate new seeds in an infinite loop with random parameters (master...2020-12-fuzz-test-runner-infinite-generate) https://github.com/bitcoin/bitcoin/pull/20752
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
aleggg has quit [Ping timeout: 240 seconds]
aleggg has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake closed pull request #21260: wallet: indicate whether a transaction is in the mempool (master...show-conflicted-transactions-not-in-mempool) https://github.com/bitcoin/bitcoin/pull/21260
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake closed pull request #21284: rpc: add the add_inputs option to bumpfee/psbtbumpfee (master...add-add-inputs-option-to-bumpfee) https://github.com/bitcoin/bitcoin/pull/21284
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
evanlinjin has joined #bitcoin-core-dev
evanlinjin has quit [Client Quit]
Guyver2_ has quit [Quit: Going offline, see ya! (www.adiirc.com)]
evanlinjin has joined #bitcoin-core-dev
jespada has quit [Ping timeout: 250 seconds]
evanlinjin has quit [Client Quit]
jespada has joined #bitcoin-core-dev
evanlinjin 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/269dcad16e3d...23ebd7a8027f
<bitcoin-git> bitcoin/master fa870e3 MarcoFalke: Remove not needed clang-format off comments
<bitcoin-git> bitcoin/master 23ebd7a laanwj: Merge bitcoin/bitcoin#24959: Remove not needed clang-format off comments
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] laanwj merged pull request #24959: Remove not needed clang-format off comments (master...2204-no-format-off-🔝) https://github.com/bitcoin/bitcoin/pull/24959
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
Kaizen_Kintsugi_ has quit [Ping timeout: 240 seconds]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] tulio150 opened pull request #24992: PT-BR tray menu translation fix (master...patch-1) https://github.com/bitcoin/bitcoin/pull/24992
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake closed pull request #24992: PT-BR tray menu translation fix (master...patch-1) https://github.com/bitcoin/bitcoin/pull/24992
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake closed pull request #18606: test: checks that bitcoin-cli autocomplete is in sync (master...feature-auto-cli) https://github.com/bitcoin/bitcoin/pull/18606
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake closed pull request #21085: test: enable self.chain = 'main' to work in python bitcoin test framework (master...mainettests) https://github.com/bitcoin/bitcoin/pull/21085
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
a1ph4byte29 has joined #bitcoin-core-dev
a1ph4byte29 has quit [Client Quit]
a1ph4byte has quit [Quit: Client closed]
Kaizen_Kintsugi_ has quit [Ping timeout: 250 seconds]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> bitcoin/master e532748 practicalswift: Don't use zero as null pointer constant (-Wzero-as-null-pointer-constant)
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
<bitcoin-git> bitcoin/master 9c96f10 fanquake: tidy: enable modernize-use-nullptr
<bitcoin-git> [bitcoin] fanquake pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/23ebd7a8027f...cc3877f8317f
<bitcoin-git> bitcoin/master cc3877f fanquake: Merge bitcoin/bitcoin#24971: tidy: modernize-use-nullptr
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake merged pull request #24971: tidy: modernize-use-nullptr (master...clang_tidy_nullptr) https://github.com/bitcoin/bitcoin/pull/24971
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
sipsorcery has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake closed pull request #23225: test: reject float at satoshi_round (master...20211006) https://github.com/bitcoin/bitcoin/pull/23225
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
evanlinjin has quit [Remote host closed the connection]
evanlinjin has joined #bitcoin-core-dev
bfsfhkacjzgcytf9 has quit [Ping timeout: 256 seconds]
<achow101> #24644 and #24977 rfm?
<gribble> https://github.com/bitcoin/bitcoin/issues/24644 | wallet: add tracepoints and algorithm information to coin selection by achow101 · Pull Request #24644 · bitcoin/bitcoin · GitHub
<gribble> https://github.com/bitcoin/bitcoin/issues/24977 | rpc: Explain active and internal in listdescriptors by achow101 · Pull Request #24977 · bitcoin/bitcoin · GitHub
<jonatack> #24150 too?
<gribble> https://github.com/bitcoin/bitcoin/issues/24150 | refactor: move index class members from protected to private by jonatack · Pull Request #24150 · bitcoin/bitcoin · GitHub
bitcoin-git has joined #bitcoin-core-dev
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/cc3877f8317f...f4005af3ec45
<bitcoin-git> bitcoin/master 4637bbe Andrew Chow: rpc: Explain active and internal in listdescriptors
<bitcoin-git> bitcoin/master f4005af fanquake: Merge bitcoin/bitcoin#24977: rpc: Explain active and internal in listdescr...
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake merged pull request #24977: rpc: Explain active and internal in listdescriptors (master...better-listdescriptors-docs) https://github.com/bitcoin/bitcoin/pull/24977
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
<fanquake> Will look at the tracepoints
<fanquake> 24150 should be easy to rebase, and conflicts with 21726, which I think is close to mergeable
<jonatack> ok
jarthur has joined #bitcoin-core-dev
Guyver2 has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/f4005af3ec45...f654cdb89cc4
<bitcoin-git> bitcoin/master ff4a38a Hennadii Stepanov: build: Fix configuring depends with cmake
<bitcoin-git> bitcoin/master f654cdb fanquake: Merge bitcoin/bitcoin#24392: build: Fix configuring depends with cmake
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake merged pull request #24392: build: Fix configuring depends with cmake (master...220219-cmake) https://github.com/bitcoin/bitcoin/pull/24392
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake closed pull request #23100: Abort functional tests early upon low drive space (master...20210926) https://github.com/bitcoin/bitcoin/pull/23100
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
Kaizen_Kintsugi_ has quit [Ping timeout: 240 seconds]
Guyver2 has left #bitcoin-core-dev [Closing Window]
a1ph4byte has joined #bitcoin-core-dev
evanlinjin has quit [Ping timeout: 240 seconds]
evanlinjin has joined #bitcoin-core-dev
ifrantar has joined #bitcoin-core-dev
ifrantar77 has joined #bitcoin-core-dev
ifrantar77 has quit [Client Quit]
ifrantar has quit [Ping timeout: 252 seconds]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/f654cdb89cc4...833add0f48b0
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
<bitcoin-git> bitcoin/master bae4561 João Barbosa: scripted-diff: rename BytePtr to AsBytePtr
<bitcoin-git> bitcoin/master 833add0 fanquake: Merge bitcoin/bitcoin#24989: scripted-diff: rename BytePtr to AsBytePtr
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake merged pull request #24989: scripted-diff: rename BytePtr to AsBytePtr (master...rename-byteptr) https://github.com/bitcoin/bitcoin/pull/24989
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
Kaizen_Kintsugi_ has quit [Ping timeout: 276 seconds]
salvatoshi has quit [Ping timeout: 272 seconds]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
salvatoshi has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Remote host closed the connection]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
salvatoshi has quit [Ping timeout: 250 seconds]
real_or_random has quit [Quit: ZNC 1.8.2 - https://znc.in]
jonatack has quit [Ping timeout: 256 seconds]
vasild has quit [Ping timeout: 240 seconds]
vasild has joined #bitcoin-core-dev
<josibake> does anyone have a guide/docs for getting setup with tracepoints? i just realized that despite compiling with --enable-usdt, the functional tests are skipping due to tracepoints not being enabled
a1ph4byte has quit [Quit: Client closed]
roconnor_ has joined #bitcoin-core-dev
roconnor has quit [Ping timeout: 250 seconds]
roconnor_ is now known as roconnor
<Murch> b10c recently made me aware that you needed to also install systemtap
Talkless has joined #bitcoin-core-dev
<Murch> I think it was only mentioned in the install documentation, but not in the tracepoint readme
Kaizen_Kintsugi_ has quit [Remote host closed the connection]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
<b10c> Murch: yeah, we should have some getting started documentation in there
Kaizen_Kintsugi_ has quit [Ping timeout: 240 seconds]
<josibake> murch: that was indeed the missing piece. i got it running but it is spitting out warnings about macros being redfined. looks like the tests are running fine, tho.
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
<Murch> Yeah, that threw me off at first as well, but the three warnings in the beginning are just that, the script runs fine after
<achow101> does it still require sudo?
<josibake> achow101: yes, i had to run the test with sudo to get around the BPF permissions check
a1ph4byte has joined #bitcoin-core-dev
<josibake> murch: looks like the warnings are a known issue and you can get rid of them by compiling bcc from source
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] brunoerg opened pull request #24993: contrib: close file after manipulating it in `linearize-data` (master...2022-04-contrib-close-file) https://github.com/bitcoin/bitcoin/pull/24993
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
<josibake> i just compiled bcc from source and ran the functional tests without any warnings from bcc. might be good to mention this if we add docs for "getting started" with bcc
Guest46 has joined #bitcoin-core-dev
Guest46 has quit [Client Quit]
sipsorcery has quit [Ping timeout: 240 seconds]
<b10c> achow101: laanwj found a way to do it without sudo - I haven't tried this yet https://github.com/bitcoin/bitcoin/pull/24358#issuecomment-1083149220
<b10c> josibake: oh, he mentioned the same issue you had
gst94 has joined #bitcoin-core-dev
<josibake> b10c, laanwj: yep, same error. i fixed it by compiling bcc from source
arturomf94[m] has joined #bitcoin-core-dev
gst94 has quit [Client Quit]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] hebasto opened pull request #24994: build: Fix `libbitcoinconsensus` Windows builds with `DEBUG=1` (master...220426-consensus) https://github.com/bitcoin/bitcoin/pull/24994
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
sipsorcery has joined #bitcoin-core-dev
mekster669490 has joined #bitcoin-core-dev
mekster66949 has quit [Read error: Connection reset by peer]
mekster669490 is now known as mekster66949
<laanwj> josibake: good to know, bcc is still under heavy development it seems
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> bitcoin/master 912f1ed Andrew Chow: wallet: track which coin selection algorithm produced a SelectionResult
<bitcoin-git> bitcoin/master 8e3f39e Andrew Chow: wallet: Add some tracepoints for coin selection
<bitcoin-git> bitcoin/master 15b5838 Andrew Chow: wallet: compute waste for SelectionResults of preset inputs
<bitcoin-git> [bitcoin] fanquake pushed 6 commits to master: https://github.com/bitcoin/bitcoin/compare/833add0f48b0...260ede1d998b
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake merged pull request #24644: wallet: add tracepoints and algorithm information to coin selection (master...tracepoints-coin-selection) https://github.com/bitcoin/bitcoin/pull/24644
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
Kaizen_Kintsugi_ has quit [Remote host closed the connection]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
evanlinjin has quit [Remote host closed the connection]
evanlinjin has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> bitcoin/master 2561823 Fabian Jahr: blockstorage: Add prune locks to BlockManager
<bitcoin-git> bitcoin/master f08c9fb Fabian Jahr: Index: Use prune locks for blockfilterindex
<bitcoin-git> [bitcoin] fanquake pushed 7 commits to master: https://github.com/bitcoin/bitcoin/compare/260ede1d998b...34ae04d77509
<bitcoin-git> bitcoin/master 231fc7b Fabian Jahr: refactor: Introduce GetFirstStoredBlock helper function
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake merged pull request #21726: Improve Indices on pruned nodes via prune blockers (master...prune_improve) https://github.com/bitcoin/bitcoin/pull/21726
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
<fanquake> Anyone seeing p2p_ibd_txrelay.py fail?
<fanquake> sporadic failure has disappeared
<sipa> so sporadic failure seems sporadic
bomb-on has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/34ae04d77509...bd616bc16a3a
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
<bitcoin-git> bitcoin/master bd616bc fanquake: Merge bitcoin/bitcoin#24917: Make BlockManager::LoadBlockIndex private
<bitcoin-git> bitcoin/master fa1970f MarcoFalke: Make BlockManager::LoadBlockIndex private
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake merged pull request #24917: Make BlockManager::LoadBlockIndex private (master...2204-priv-🔚) https://github.com/bitcoin/bitcoin/pull/24917
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
<fanquake> embarrassed I even reported it
bomb-on has quit [Read error: Connection reset by peer]
bomb-on has joined #bitcoin-core-dev
Talkless has quit [Quit: Konversation terminated!]
AaronvanW has quit [Quit: Leaving...]
real_or_random has joined #bitcoin-core-dev
evanlinjin has quit [Ping timeout: 240 seconds]
a1ph4byte has quit [Ping timeout: 252 seconds]
rodarmor has joined #bitcoin-core-dev
otoburb has quit [Quit: leaving]
lightningbot has quit [Killed (NickServ (GHOST command used by lightningbot`!lightningb@azure.erisian.com.au))]
lightningbot has joined #bitcoin-core-dev
otoburb has joined #bitcoin-core-dev
a1ph4byte has joined #bitcoin-core-dev
vysn has quit [Ping timeout: 250 seconds]
jonatack has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Remote host closed the connection]
jonatack has quit [Ping timeout: 248 seconds]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
jonatack has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 240 seconds]
brunoerg has quit [Remote host closed the connection]
sipsorcery has quit [Ping timeout: 250 seconds]
sipsorcery has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 260 seconds]
bitdex has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
bomb-on has quit [Quit: aллилѹіа!]
geyaeb has quit [Remote host closed the connection]
sipsorcery has quit [Ping timeout: 240 seconds]
geyaeb has joined #bitcoin-core-dev
belcher has quit [Ping timeout: 276 seconds]
belcher has joined #bitcoin-core-dev
vysn has joined #bitcoin-core-dev
BUSY has quit [Read error: Connection reset by peer]
BUSY has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Remote host closed the connection]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev