< * luke-jr> wonders if we'll ever get to the poitn where tests passing means there are no RC bug reports
< fanquake> Have pushed up some signed sigs for 0.17.2rc2
< bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/7d4bc60f1fee...630ec7bf41fe
< bitcoin-git> bitcoin/master fa8d65f MarcoFalke: doc: Fix doxygen comment for SignTransaction in rpc/rawtransaction_util
< bitcoin-git> bitcoin/master 630ec7b fanquake: Merge #16900: doc: Fix doxygen comment for SignTransaction in rpc/rawtrans...
< bitcoin-git> [bitcoin] fanquake merged pull request #16900: doc: Fix doxygen comment for SignTransaction in rpc/rawtransaction_util (master...1909-docRpcUtil) https://github.com/bitcoin/bitcoin/pull/16900
< bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/630ec7bf41fe...587003d380a0
< bitcoin-git> bitcoin/master 14c6a2d Sjors Provoost: [doc] update brew instruction for doxygen
< bitcoin-git> bitcoin/master 587003d fanquake: Merge #16914: doc: Update homebrew instruction for doxygen
< bitcoin-git> [bitcoin] fanquake merged pull request #16914: doc: Update homebrew instruction for doxygen (master...2019/09/doxygen-homebrew) https://github.com/bitcoin/bitcoin/pull/16914
< bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/587003d380a0...a73775e4d590
< bitcoin-git> bitcoin/master 96299a9 fridokus: Test: Move common function assert_approx() into util.py
< bitcoin-git> bitcoin/master a73775e fanquake: Merge #16917: tests: Move common function assert_approx() into util.py
< bitcoin-git> [bitcoin] fanquake merged pull request #16917: tests: Move common function assert_approx() into util.py (master...master) https://github.com/bitcoin/bitcoin/pull/16917
< bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/a73775e4d590...04321494aea6
< bitcoin-git> bitcoin/master 72a18a7 practicalswift: tests: Add information on how to add Vulture suppressions
< bitcoin-git> bitcoin/master 0432149 fanquake: Merge #16921: tests: Add information on how to add Vulture suppressions
< bitcoin-git> [bitcoin] fanquake merged pull request #16921: tests: Add information on how to add Vulture suppressions (master...vulture-suppressions) https://github.com/bitcoin/bitcoin/pull/16921
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/04321494aea6...f8b0b190aa9d
< bitcoin-git> bitcoin/master fa2e038 MarcoFalke: test: Fix extra_args in wallet_import_rescan.py
< bitcoin-git> bitcoin/master f8b0b19 MarcoFalke: Merge #16920: test: Fix extra_args in wallet_import_rescan.py
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #16920: test: Fix extra_args in wallet_import_rescan.py (master...1909-testUseCorrectPythonSyntax) https://github.com/bitcoin/bitcoin/pull/16920
< bitcoin-git> [bitcoin] promag opened pull request #16923: wallet: Fix duplicates fileid exception on start (master...2019-09-fix-duplicate-exception) https://github.com/bitcoin/bitcoin/pull/16923
< bitcoin-git> [bitcoin] nmarley opened pull request #16926: depends: Add OpenSSL termios fix for musl libc (master...btc-openssl-musl-libc-patch) https://github.com/bitcoin/bitcoin/pull/16926
< bitcoin-git> [bitcoin] nmarley opened pull request #16927: depends: Refresh ZeroMQ 4.3.1 patch (master...btc-zmq431-patches) https://github.com/bitcoin/bitcoin/pull/16927
< instagibbs> can someone point me to where the "Config setting for %s only applied on %s network when in [%s] section." stuff is actually enforced?
< instagibbs> Having trouble tracing it
< instagibbs> that line is just where an init warning is thrown then it continue
< ryanofsky> instagibbs, you can check the two UseDefaultSection calls. if it returns false the settings get thrown away
< instagibbs> ah-hah thanks
< instagibbs> ryanofsky, oh I was getting thrown off that network-specific args are fine in default area *when read by -cli*
< bitcoin-git> [bitcoin] luke-jr closed pull request #16228: Bugfix: make check: Only run tests that were compiled (master...bugfix_raii_check_fail) https://github.com/bitcoin/bitcoin/pull/16228
< achow101> any particular reason that we use CAutoFile in a lot of places instead of CBufferedFile?
< sipa> CAutoFile is older
< sipa> we needed CBufferedFile for rescanning block files as it supports rewinding
< lightlike> sipa: maybe you are interested to have a look at #16577, which fixes a bug after which it should actually support rewinding
< gribble> https://github.com/bitcoin/bitcoin/issues/16577 | util: CBufferedFile fixes and unit test by LarryRuane · Pull Request #16577 · bitcoin/bitcoin · GitHub