< phantomcircuit> so it seems that with a few modifications 0.7.2 will build on a modern system and sync (albeit slowly)
< phantomcircuit> i'd like to do some kind of 0.7.2.1 release, but in a way that it's obviously not maintained
< phantomcircuit> any ideas?
< luke-jr> phantomcircuit: why not just finish v0.7.3?
< luke-jr> v0.7.2 IIRC lacks the changes needed for the 2013 hardfork
< luke-jr> also, that sounds like a lot of work if you mean backporting all the softforks
< phantomcircuit> luke-jr, no i mean the minimal changes required to get it to build
< phantomcircuit> and to work with modern openssl
< luke-jr> ah
< luke-jr> well still, I'd do it based on 0.7.3
< luke-jr> [rc3]
< luke-jr> looks like the old stable branch repo is defunct, so I pushed the 0.7.x branch to my personal github repo
< bitcoin-git> [bitcoin] xujc opened pull request #14989: Delete JSON-RPC-interface.md (master...patch-1) https://github.com/bitcoin/bitcoin/pull/14989
< bitcoin-git> [bitcoin] fanquake closed pull request #14989: Delete JSON-RPC-interface.md (master...patch-1) https://github.com/bitcoin/bitcoin/pull/14989
< bitcoin-git> [bitcoin] MeshCollider pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b545a6e68965...27f5a295d740
< bitcoin-git> bitcoin/master 8b9171c Ben Woosley: wallet: Initialize stop_block to nullptr in CWallet::ScanForWalletTransactions...
< bitcoin-git> bitcoin/master 27f5a29 MeshCollider: Merge #14957: wallet: Initialize stop_block in CWallet::ScanForWalletTransactions...
< bitcoin-git> [bitcoin] MeshCollider closed pull request #14957: wallet: Initialize stop_block in CWallet::ScanForWalletTransactions (master...stop-block-null) https://github.com/bitcoin/bitcoin/pull/14957
< gkrizek> I requested an invite to the Bitcoin Core Slack and haven't heard anything. How can I get an invite? Is IRC or Slack used more?
< booyah> gkrizek: the regular meetings, are happening here on irc
< gkrizek> booyah: Thanks! I'll probably just stick to IRC then
< luke-jr> gkrizek: I don't think any Core devs actually use the Slack at this point
< gkrizek> luke-jr: Ha, thanks for the heads up.
< sipa> no, it's just a community effort
< gkrizek> Looking at the master branch in bitcoin/bitcoin, I see a lot of PRs are merged with failing tests. Why is this? Are the tests just flaky?
< sipa> the appveyor tests are experimental, and we ignore them for merging
< sipa> unfortunately we can't prevent it from marking the github PR as invalid
< sipa> *failed
< gkrizek> Understood. But it seems like Travis fails at a similar frequency...
< gkrizek> sipa: I'm new and want to start contributing. I'm just trying to understand the flow and requirements. Mainly curious how important/accurate those test suites are.
< gwillen> travis is generally pretty reliable
< gwillen> I think travis fails are typically treated as real, although I admit I saw one just today that I'm pretty sure was spurious
< sipa> there are occasionally failures due to timeouts or so
< gwillen> but in that case all but one platform showed as passing, which I think is a good clue
< gwillen> you can also inspect the output
< gwillen> so I think the answer is "the tests must actually pass, but spurious CI failures that aren't actually the result of failing tests happen sometimes"
< gkrizek> Ok, awesome. Thanks for the explanation!
< phantomcircuit> gwillen, travis randomly fails too
< phantomcircuit> it seems like the tests are much more expensive to run than they used to be (which is fine, but breaking)
< bitcoin-git> [bitcoin] jonasschnelli pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/27f5a295d740...e7b88ecbc920
< bitcoin-git> bitcoin/master 4d454dc Hennadii Stepanov: Refactoring with QString::toNSString...
< bitcoin-git> bitcoin/master e7b88ec Jonas Schnelli: Merge #14975: qt: Refactoring with QString::toNSString()...
< bitcoin-git> [bitcoin] jonasschnelli closed pull request #14975: qt: Refactoring with QString::toNSString() (master...20181216-macos-shownotification-refactor) https://github.com/bitcoin/bitcoin/pull/14975
< bitcoin-git> [bitcoin] hebasto opened pull request #14990: qt: Use bilinear filtering for icon (master...20181218-icon-bilinear-filtering) https://github.com/bitcoin/bitcoin/pull/14990
< bitcoin-git> [bitcoin] suprraz opened pull request #14991: Doc build macos (master...doc_build_macos) https://github.com/bitcoin/bitcoin/pull/14991
< bitcoin-git> [bitcoin] hebasto closed pull request #14990: qt: Use bilinear filtering for icon (master...20181218-icon-bilinear-filtering) https://github.com/bitcoin/bitcoin/pull/14990
< bitcoin-git> [bitcoin] benthecarman closed pull request #14977: rpc: GetKeyOrigin check in getaddressinfo RPC (master...getaddressinfo_rpc_refactor_GetKeyOrigin) https://github.com/bitcoin/bitcoin/pull/14977
< bitcoin-git> [bitcoin] practicalswift opened pull request #14993: rpc: Fix data race (UB) in InterruptRPC() (master...rpc-datarace) https://github.com/bitcoin/bitcoin/pull/14993
< wumpus> have any problems been reported with rc1 yet?
< wumpus> if not, maybe time to tag final
< fanquake> Haven't heard anything
< fanquake> Bunch of 0.17 backports to merge after tagging
< sat11> thx bitcoin
< promag> #14981 can be merged
< gribble> https://github.com/bitcoin/bitcoin/issues/14981 | rpc: Clarifying RPC getrawtransactions time help text by benthecarman · Pull Request #14981 · bitcoin/bitcoin · GitHub
< instagibbs> this seems like an odd magic number in the mempool caching code: https://github.com/bitcoin/bitcoin/blob/master/src/txmempool.cpp#L603
< instagibbs> can someone explain that block height 1,000,000
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/e7b88ecbc920...b545a7e33775
< bitcoin-git> bitcoin/master 82687b5 luciana: docs: add NSIS setup/install steps to windows docs
< bitcoin-git> bitcoin/master b545a7e MarcoFalke: Merge #14950: docs: add NSIS setup/install steps to windows docs...
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #14950: docs: add NSIS setup/install steps to windows docs (master...rebase-update-windows-nsis) https://github.com/bitcoin/bitcoin/pull/14950
< bitcoin-git> [bitcoin] MarcoFalke pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/b545a7e33775...f055389cb957
< bitcoin-git> bitcoin/master 069752b practicalswift: build: Enable functional tests in the ThreadSanitizer (TSan) build job
< bitcoin-git> bitcoin/master 5e5138a Chun Kuan Lee: travis: Use trap and set -e errtrace
< bitcoin-git> bitcoin/master eaf4070 practicalswift: Add suppression for InterruptRPC (fRPCRunning) data race
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #14829: travis: Enable functional tests in the ThreadSanitizer (TSan) build job (master...tsan) https://github.com/bitcoin/bitcoin/pull/14829
< bitcoin-git> [bitcoin] MarcoFalke pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/f055389cb957...d4197812d47d
< bitcoin-git> bitcoin/master fa0d3c4 MarcoFalke: test: Undo thread_local g_insecure_rand_ctx
< bitcoin-git> bitcoin/master fa61202 MarcoFalke: test: Add comment to g_insecure_rand_ctx
< bitcoin-git> bitcoin/master d419781 MarcoFalke: Merge #14985: test: Remove thread_local from test_bitcoin...
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #14985: test: Remove thread_local from test_bitcoin (master...Mf1812-testNoThreadLocal) https://github.com/bitcoin/bitcoin/pull/14985
< jamesob> #14336 ready for merge?
< gribble> https://github.com/bitcoin/bitcoin/issues/14336 | net: implement poll by pstratem · Pull Request #14336 · bitcoin/bitcoin · GitHub
< jamesob> I guess could use a few more tested ACKs
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/d4197812d47d...f080c65a09a8
< bitcoin-git> bitcoin/master fa9a5bc MarcoFalke: RPCHelpMan: Support required arguments after optional ones
< bitcoin-git> bitcoin/master f080c65 MarcoFalke: Merge #14875: RPCHelpMan: Support required arguments after optional ones...
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #14875: RPCHelpMan: Support required arguments after optional ones (master...Mf1812-rpcOptional) https://github.com/bitcoin/bitcoin/pull/14875
< promag> ryanofsky: ping
< bitcoin-git> [bitcoin] Empact opened pull request #14998: [WIP] Run CI against ubuntu 14.04 (master...qt52) https://github.com/bitcoin/bitcoin/pull/14998