< bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/3657aee2d25c...657b82cef0e8
< bitcoin-git> bitcoin/master 5cb7ee6 Anthony Towns: net: improve code documentation for dns seed behaviour
< bitcoin-git> bitcoin/master 657b82c fanquake: Merge #19084: net: improve code documentation for dns seed behaviour
< bitcoin-git> [bitcoin] fanquake merged pull request #19084: net: improve code documentation for dns seed behaviour (master...202005-dns-seed-doc) https://github.com/bitcoin/bitcoin/pull/19084
< luke-jr> the mid-stage between osx-unsigned and osx-signed isn't deterministic anymore?
< luke-jr> hmm, apparently just not for me
< phantomcircuit> sipa, any idea what the bit length of the remainder should be for a 1:10^9 gcs encoding?
< fanquake> luke-jr if you upload your bitcoin-0.20.0-osx-unsigned.tar.gz somewhere I can take a look
< sipa> phantomcircuit: range is 0..MN-1, where M=1/fprate
< sipa> and N is number of elements in the set
< sipa> so if you want 1 in a billion fprate, log2(N)+30
< fanquake> Thanks. Will diff shortly
< fanquake> Weird
< luke-jr> ?
< fanquake> The difference is only in the tar.gz: https://gist.github.com/fanquake/0189129764edd9aa9d89fd274f790176
< fanquake> When I unzip and compare genisoimage, they are identical
< luke-jr> file permissions?
< luke-jr> hmm
< luke-jr> are you sure? :/
< fanquake> Added stat output to the gist
< luke-jr> what line is that stat format?
< luke-jr> is it at all possible you extracted my tar.gz under fanquake/?
< luke-jr> my geniso definitely has the 6044 6066
< fanquake> I will nuke both and recheck
< fanquake> Yea yours has the 6044 6066. Mine has the 6041 6063.
< fanquake> I must have mis-hashed previously. Have updated the shasums in the gist
< fanquake> Any idea what would cause the difference?
< luke-jr> no :/
< luke-jr> not sure what those numbers are
< luke-jr> looks like part of the symbol table
< luke-jr> 00000000002b5300 l O .data 0000000000000004 warn_sysfs.6066
< luke-jr> could I have a newer library?
< luke-jr> weird, not finding warn_sysfs in /lib or /usr/lib
< luke-jr> weird
< luke-jr> post-preprocessing line number?
< luke-jr> in which case it could be any header
< * fanquake> is unhappy he started looking at this before making lunch
< luke-jr> I guess at this point it's probably unimportant to resolve
< midnight> brutal. :(
< phantomcircuit> sipa, doesn't seem to have really helped, lower false positive rate but the overall time remained about the same
< bitcoin-git> [bitcoin] skmcontrib opened pull request #19152: build: improve build OS configure output (master...fixbuildos) https://github.com/bitcoin/bitcoin/pull/19152
< stevenroose> Review appreciated on https://github.com/rust-bitcoin/rust-bitcoin/pull/428
< stevenroose> It's a small change to disallow uncompressed pubkeys in Address::p2[w]pkh.
< stevenroose> Someone suggests a follow-up change to also parse p2wsh scripts for pushes that are valid uncompressed pubkeys and disallow that.
< stevenroose> I'm personally a bit reluctant to do that, so I won't be including it in #428.
< gribble> https://github.com/bitcoin/bitcoin/issues/428 | Fix for small change outputs by sipa · Pull Request #428 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] S3RK opened pull request #19153: Feature/mempool compatibility test (master...feature/mempool_compatibility_test) https://github.com/bitcoin/bitcoin/pull/19153
< fanquake> wumpus / sipa: May need to block development-lab
< bitcoin-git> [bitcoin] laanwj opened pull request #19155: doc: Add historical release notes for 0.20.0 (master...2020_06_relnot_0.20.0) https://github.com/bitcoin/bitcoin/pull/19155
< bitcoin-git> [bitcoin] laanwj closed pull request #19155: doc: Add historical release notes for 0.20.0 (master...2020_06_relnot_0.20.0) https://github.com/bitcoin/bitcoin/pull/19155
< bitcoin-git> [bitcoin] laanwj pushed 4 commits to master: https://github.com/bitcoin/bitcoin/compare/657b82cef0e8...42065518d937
< bitcoin-git> bitcoin/master faabc6e MarcoFalke: doc: Remove release notes of 0.20.1 release
< bitcoin-git> bitcoin/master fa1a916 MarcoFalke: doc: Add release-notes-0.20.0.md
< bitcoin-git> bitcoin/master fa13c18 MarcoFalke: doc: Move 0.21 fragments into the main release notes
< bitcoin-git> [bitcoin] laanwj merged pull request #19146: doc: Remove release note fragments of 0.20.1 release, Add release-notes-0.20.0 (master...2006-docRel) https://github.com/bitcoin/bitcoin/pull/19146
< bitcoin-git> [bitcoin] MarcoFalke pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/42065518d937...0f55294cc165
< bitcoin-git> bitcoin/master 6666c82 MarcoFalke: fuzz: Give CNode ownership to ConnmanTestMsg in process_message fuzz harne...
< bitcoin-git> bitcoin/master fab860a MarcoFalke: fuzz: Stop nodes in process_message* fuzzers
< bitcoin-git> bitcoin/master 0f55294 MarcoFalke: Merge #18875: fuzz: Stop nodes in process_message* fuzzers
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #18875: fuzz: Stop nodes in process_message* fuzzers (master...2005-fuzzStopNodes) https://github.com/bitcoin/bitcoin/pull/18875
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/0f55294cc165...bdedfcf076a6
< bitcoin-git> bitcoin/master facede1 MarcoFalke: test: Check that invalid witness destinations can not be imported
< bitcoin-git> bitcoin/master bdedfcf MarcoFalke: Merge #18974: test: Check that invalid witness destinations can not be imp...
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #18974: test: Check that invalid witness destinations can not be imported (master...2005-testInvalidWitnessDestination) https://github.com/bitcoin/bitcoin/pull/18974
< wumpus> congratulations on the 0.20.0 release everyone 🚀
< wumpus> fanquake: done
< fanquake> 🚀
< fanquake> wumpus: thanks
< fanquake> & congrats
< jonatack> yay! \o/
< vasild> 16:09 < vasild> hebasto: https://paste.gg/p/anonymous/4f3dd080b5ef4a09a0cba6a6eaffc6a6 -- if somebody can confirm that this is indeed a problem and I am not delusional, then I will open a bug report about it.
< vasild> 16:18 < hebasto> vasild: does TSan detect such deadlock possibilities?
< vasild> hebasto: just to answer your question -- tsan does not always detect such deadlocks
< vasild> at least the testcase in #19157 is not detected by tsan
< gribble> https://github.com/bitcoin/bitcoin/issues/19157 | Improve deadlock detection · Issue #19157 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] fanquake pushed 10 commits to master: https://github.com/bitcoin/bitcoin/compare/bdedfcf076a6...575589a62e39
< bitcoin-git> bitcoin/master fac2eee MarcoFalke: cirrus: Remove no longer needed install step
< bitcoin-git> bitcoin/master fa2ffe8 MarcoFalke: ci: Deduplicate DOCKER_EXEC
< bitcoin-git> bitcoin/master fa0d5ee MarcoFalke: ci: Set halt_on_error=1 for tsan
< bitcoin-git> [bitcoin] fanquake merged pull request #19041: ci: tsan with -stdlib=libc++-10 (master...2005-ciTsanFix) https://github.com/bitcoin/bitcoin/pull/19041
< bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/575589a62e39...234fabab9062
< bitcoin-git> bitcoin/master bd7e530 Kiminuo: This PR adds initial support for type hints checking in python scripts.
< bitcoin-git> bitcoin/master 234faba fanquake: Merge #18210: test: type hints in Python tests
< bitcoin-git> [bitcoin] fanquake merged pull request #18210: test: type hints in Python tests (master...feature/type-hint-minimum) https://github.com/bitcoin/bitcoin/pull/18210
< bitcoin-git> [bitcoin] vasild opened pull request #19158: sync: remove unneeded LockData::invlockorders (master...remove_invlockorders) https://github.com/bitcoin/bitcoin/pull/19158
< troygiorshev> #proposedmeetingtopic peer logging
< troygiorshev> i have a proposal for extending the per-peer message capture and resource logging capabilities. if anyone's interested or think they would find this useful, feel free to take a look
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #19159: test: Make valgrind.supp work on aarch64 (master...2006-valgrindAarch64) https://github.com/bitcoin/bitcoin/pull/19159
< bitcoin-git> [bitcoin] ryanofsky opened pull request #19160: multiprocess: Add basic spawn and IPC support (master...pr/ipc-echo) https://github.com/bitcoin/bitcoin/pull/19160
< stevenroose> Is there a way to get a wallet xpub from RPC?
< sipa> for a legacy wallet, no, as it uses hardened derivation, so an xpub is meaningless
< jonatack> stevenroose: exporting an xpub from bitcoin core not yet but soon with descriptor wallets https://bitcoin.stackexchange.com/questions/90135/how-do-i-export-an-xpub-from-bitcoin-core-for-use-in-btcpayserver
< stevenroose> sipa: from what version did Core move away from hardened derivation?
< sipa> stevenroose: none
< stevenroose> sipa: non-legacy wallet you mean a descriptor wallet? but those are only opt-in?
< sipa> descriptor wallets use descriptors, which may use an xpub
< stevenroose> from what version are descriptor wallets supported? (well let me read the link first, thanks jonatack sipa )
< sipa> stevenroose: none so far
< sipa> (they're in master, so likely 0.21)
< stevenroose> oh, hmm
< stevenroose> is there a way to get the xpriv of the hardened derivation? (let's say hypothetically you're a total grotmongool and you don't care about theft; or, say, you are doing a poc and want to use Core instead of a custom wallet :D)
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #19162: ci: tsan gui (master...2006-ciTsanGui) https://github.com/bitcoin/bitcoin/pull/19162
< bitcoin-git> [bitcoin] MarcoFalke pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/234fabab9062...a1c0e5fce139
< bitcoin-git> bitcoin/master 47be28c fanquake: validation: use std::chrono in CChainState::FlushStateToDisk()
< bitcoin-git> bitcoin/master 789e9dd fanquake: validation: use std::chrono in IsCurrentForFeeEstimation()
< bitcoin-git> bitcoin/master a1c0e5f MarcoFalke: Merge #19088: validation: use std::chrono throughout some validation funct...
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #19088: validation: use std::chrono throughout some validation functions (master...validation_chrono) https://github.com/bitcoin/bitcoin/pull/19088
< achow101> stevenroose: dumpwallet puts the xprv at the top of the file
< stevenroose> achow101: epic thanks
< bitcoin-git> [bitcoin] vasild closed pull request #19158: sync: remove unneeded LockData::invlockorders (master...remove_invlockorders) https://github.com/bitcoin/bitcoin/pull/19158
< ja> on transifex, how do you see the details of a warning? nothing happens when i click the exclamation mark
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #19164: ci: tsan with wallet (master...2006-ciTsanWallet) https://github.com/bitcoin/bitcoin/pull/19164