03:04 < bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/3657aee2d25c...657b82cef0e8

03:04 < bitcoin-git> bitcoin/master 5cb7ee6 Anthony Towns: net: improve code documentation for dns seed behaviour

03:04 < bitcoin-git> bitcoin/master 657b82c fanquake: Merge #19084: net: improve code documentation for dns seed behaviour

03:04 < 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

03:14 < luke-jr> the mid-stage between osx-unsigned and osx-signed isn't deterministic anymore?

03:16 < luke-jr> hmm, apparently just not for me

03:23 < phantomcircuit> sipa, any idea what the bit length of the remainder should be for a 1:10^9 gcs encoding?

03:27 < fanquake> luke-jr if you upload your bitcoin-0.20.0-osx-unsigned.tar.gz somewhere I can take a look

03:29 < sipa> phantomcircuit: range is 0..MN-1, where M=1/fprate

03:29 < sipa> and N is number of elements in the set

03:30 < sipa> so if you want 1 in a billion fprate, log2(N)+30

03:34 < luke-jr> fanquake: http://luke.dashjr.org/tmp/code/bitcoin-0.20.0-osx-unsigned.tar.gz

03:35 < fanquake> Thanks. Will diff shortly

03:58 < fanquake> Weird

04:00 < luke-jr> ?

04:00 < fanquake> The difference is only in the tar.gz: https://gist.github.com/fanquake/0189129764edd9aa9d89fd274f790176

04:00 < fanquake> When I unzip and compare genisoimage, they are identical

04:00 < luke-jr> file permissions?

04:01 < luke-jr> hmm

04:02 < luke-jr> are you sure? :/

04:03 < fanquake> Added stat output to the gist

04:06 < luke-jr> what line is that stat format?

04:08 < luke-jr> is it at all possible you extracted my tar.gz under fanquake/?

04:08 < luke-jr> my geniso definitely has the 6044 6066

04:08 < fanquake> I will nuke both and recheck

04:12 < fanquake> Yea yours has the 6044 6066. Mine has the 6041 6063.

04:13 < fanquake> I must have mis-hashed previously. Have updated the shasums in the gist

04:13 < fanquake> Any idea what would cause the difference?

04:15 < luke-jr> no :/

04:15 < luke-jr> not sure what those numbers are

04:16 < luke-jr> looks like part of the symbol table

04:16 < luke-jr> 00000000002b5300 l O .data 0000000000000004 warn_sysfs.6066

04:17 < luke-jr> could I have a newer library?

04:18 < luke-jr> weird, not finding warn_sysfs in /lib or /usr/lib

04:19 < fanquake> The only instance of warn_sysfs in cdrkit is here: https://github.com/ystk/debian-cdrkit/blob/a5022f1207160d30554d87d15af7ee164a35a71f/libusal/scsi-linux-sg.c#L402

04:20 < luke-jr> weird

04:20 < luke-jr> post-preprocessing line number?

04:21 < luke-jr> in which case it could be any header

04:22 < * fanquake> is unhappy he started looking at this before making lunch

04:22 < luke-jr> I guess at this point it's probably unimportant to resolve

04:24 < midnight> brutal. :(

04:26 < phantomcircuit> sipa, doesn't seem to have really helped, lower false positive rate but the overall time remained about the same

08:22 < bitcoin-git> [bitcoin] skmcontrib opened pull request #19152: build: improve build OS configure output (master...fixbuildos) https://github.com/bitcoin/bitcoin/pull/19152

09:17 < stevenroose> Review appreciated on https://github.com/rust-bitcoin/rust-bitcoin/pull/428

09:17 < stevenroose> It's a small change to disallow uncompressed pubkeys in Address::p2[w]pkh.

09:18 < stevenroose> Someone suggests a follow-up change to also parse p2wsh scripts for pushes that are valid uncompressed pubkeys and disallow that.

09:19 < stevenroose> I'm personally a bit reluctant to do that, so I won't be including it in #428.

09:19 < gribble> https://github.com/bitcoin/bitcoin/issues/428 | Fix for small change outputs by sipa · Pull Request #428 · bitcoin/bitcoin · GitHub

09:23 < bitcoin-git> [bitcoin] S3RK opened pull request #19153: Feature/mempool compatibility test (master...feature/mempool_compatibility_test) https://github.com/bitcoin/bitcoin/pull/19153

10:53 < fanquake> wumpus / sipa: May need to block development-lab

10:56 < 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

11:01 < 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

11:05 < bitcoin-git> [bitcoin] laanwj pushed 4 commits to master: https://github.com/bitcoin/bitcoin/compare/657b82cef0e8...42065518d937

11:05 < bitcoin-git> bitcoin/master faabc6e MarcoFalke: doc: Remove release notes of 0.20.1 release

11:05 < bitcoin-git> bitcoin/master fa1a916 MarcoFalke: doc: Add release-notes-0.20.0.md

11:05 < bitcoin-git> bitcoin/master fa13c18 MarcoFalke: doc: Move 0.21 fragments into the main release notes

11:06 < 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

11:25 < bitcoin-git> [bitcoin] MarcoFalke pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/42065518d937...0f55294cc165

11:25 < bitcoin-git> bitcoin/master 6666c82 MarcoFalke: fuzz: Give CNode ownership to ConnmanTestMsg in process_message fuzz harne...

11:25 < bitcoin-git> bitcoin/master fab860a MarcoFalke: fuzz: Stop nodes in process_message* fuzzers

11:25 < bitcoin-git> bitcoin/master 0f55294 MarcoFalke: Merge #18875: fuzz: Stop nodes in process_message* fuzzers

11:25 < 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

11:32 < bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/0f55294cc165...bdedfcf076a6

11:32 < bitcoin-git> bitcoin/master facede1 MarcoFalke: test: Check that invalid witness destinations can not be imported

11:32 < bitcoin-git> bitcoin/master bdedfcf MarcoFalke: Merge #18974: test: Check that invalid witness destinations can not be imp...

11:32 < 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

11:52 < wumpus> congratulations on the 0.20.0 release everyone 🚀

11:53 < wumpus> fanquake: done

11:53 < fanquake> 🚀

11:53 < fanquake> wumpus: thanks

11:54 < fanquake> & congrats

11:56 < jonatack> yay! \o/

13:28 < 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.

13:28 < vasild> 16:18 < hebasto> vasild: does TSan detect such deadlock possibilities?

13:29 < vasild> hebasto: just to answer your question -- tsan does not always detect such deadlocks

13:31 < vasild> at least the testcase in #19157 is not detected by tsan

13:31 < gribble> https://github.com/bitcoin/bitcoin/issues/19157 | Improve deadlock detection · Issue #19157 · bitcoin/bitcoin · GitHub

13:56 < bitcoin-git> [bitcoin] fanquake pushed 10 commits to master: https://github.com/bitcoin/bitcoin/compare/bdedfcf076a6...575589a62e39

13:56 < bitcoin-git> bitcoin/master fac2eee MarcoFalke: cirrus: Remove no longer needed install step

13:56 < bitcoin-git> bitcoin/master fa2ffe8 MarcoFalke: ci: Deduplicate DOCKER_EXEC

13:56 < bitcoin-git> bitcoin/master fa0d5ee MarcoFalke: ci: Set halt_on_error=1 for tsan

13:56 < bitcoin-git> [bitcoin] fanquake merged pull request #19041: ci: tsan with -stdlib=libc++-10 (master...2005-ciTsanFix) https://github.com/bitcoin/bitcoin/pull/19041

14:20 < bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/575589a62e39...234fabab9062

14:20 < bitcoin-git> bitcoin/master bd7e530 Kiminuo: This PR adds initial support for type hints checking in python scripts.

14:20 < bitcoin-git> bitcoin/master 234faba fanquake: Merge #18210: test: type hints in Python tests

14:21 < 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

14:33 < bitcoin-git> [bitcoin] vasild opened pull request #19158: sync: remove unneeded LockData::invlockorders (master...remove_invlockorders) https://github.com/bitcoin/bitcoin/pull/19158

14:38 < troygiorshev> #proposedmeetingtopic peer logging

14:38 < 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

14:38 < troygiorshev> https://gist.github.com/troygiorshev/f2fedca9b2ab8a2c6449c92d92e17678

15:00 < 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

15:35 < 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

16:03 < stevenroose> Is there a way to get a wallet xpub from RPC?

16:10 < sipa> for a legacy wallet, no, as it uses hardened derivation, so an xpub is meaningless

16:10 < 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

16:11 < stevenroose> sipa: from what version did Core move away from hardened derivation?

16:12 < sipa> stevenroose: none

16:12 < stevenroose> sipa: non-legacy wallet you mean a descriptor wallet? but those are only opt-in?

16:12 < sipa> descriptor wallets use descriptors, which may use an xpub

16:12 < stevenroose> from what version are descriptor wallets supported? (well let me read the link first, thanks jonatack sipa )

16:12 < sipa> stevenroose: none so far

16:13 < sipa> (they're in master, so likely 0.21)

16:13 < stevenroose> oh, hmm

16:15 < 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)

16:49 < bitcoin-git> [bitcoin] MarcoFalke opened pull request #19162: ci: tsan gui (master...2006-ciTsanGui) https://github.com/bitcoin/bitcoin/pull/19162

17:15 < bitcoin-git> [bitcoin] MarcoFalke pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/234fabab9062...a1c0e5fce139

17:15 < bitcoin-git> bitcoin/master 47be28c fanquake: validation: use std::chrono in CChainState::FlushStateToDisk()

17:15 < bitcoin-git> bitcoin/master 789e9dd fanquake: validation: use std::chrono in IsCurrentForFeeEstimation()

17:15 < bitcoin-git> bitcoin/master a1c0e5f MarcoFalke: Merge #19088: validation: use std::chrono throughout some validation funct...

17:15 < 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

17:22 < achow101> stevenroose: dumpwallet puts the xprv at the top of the file

17:39 < stevenroose> achow101: epic thanks

18:47 < bitcoin-git> [bitcoin] vasild closed pull request #19158: sync: remove unneeded LockData::invlockorders (master...remove_invlockorders) https://github.com/bitcoin/bitcoin/pull/19158

19:12 < ja> on transifex, how do you see the details of a warning? nothing happens when i click the exclamation mark

22:55 < bitcoin-git> [bitcoin] MarcoFalke opened pull request #19164: ci: tsan with wallet (master...2006-ciTsanWallet) https://github.com/bitcoin/bitcoin/pull/19164