< gmaxwell> sipa: 9295 calls to you, -- /merge me/ /merge me/
< bitcoin-git> [bitcoin] sipa pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/72bf1b3d0962...815640ec6af9
< bitcoin-git> bitcoin/master c24a4f5 Jonas Schnelli: [Wallet] Bugfix: FRT: don't terminate when keypool is empty
< bitcoin-git> bitcoin/master 1a6eacb Jonas Schnelli: [QA] add fundrawtransaction test on a locked wallet with empty keypool
< bitcoin-git> bitcoin/master 815640e Pieter Wuille: Merge #9295: [Wallet] Bugfix: Fundrawtransaction: don't terminate when keypool is empty...
< bitcoin-git> [bitcoin] sipa closed pull request #9295: [Wallet] Bugfix: Fundrawtransaction: don't terminate when keypool is empty (master...2016/12/fix_frt) https://github.com/bitcoin/bitcoin/pull/9295
< bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/815640ec6af9...d38b0d7a6b60
< bitcoin-git> bitcoin/master fe41f58 Russell Yanofsky: Remove undefined FetchCoins method declaration
< bitcoin-git> bitcoin/master d38b0d7 Pieter Wuille: Merge #9307: Remove undefined FetchCoins method declaration...
< bitcoin-git> [bitcoin] sipa closed pull request #9307: Remove undefined FetchCoins method declaration (master...pr/coins-delfetch) https://github.com/bitcoin/bitcoin/pull/9307
< bitcoin-git> [bitcoin] sipa pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/d38b0d7a6b60...a1dcf2e1087b
< bitcoin-git> bitcoin/master bf663f8 Alex Morcos: remove external usage of mempool conflict tracking
< bitcoin-git> bitcoin/master a874ab5 Alex Morcos: remove internal tracking of mempool conflicts for reporting to wallet
< bitcoin-git> bitcoin/master a1dcf2e Pieter Wuille: Merge #9240: Remove txConflicted...
< bitcoin-git> [bitcoin] sipa closed pull request #9240: Remove txConflicted (master...removeTxConflicted) https://github.com/bitcoin/bitcoin/pull/9240
< gmaxwell> There is now several percent of hashpower producing blocks with version=0x30000000 ... what is that?
< harrymm> gmaxwell: version=0x30000000 is being reported by coin.dance as a classic block
< btcdrak> harrymm: BIP109 was marked rejected, so it's currently meaningless. You should contact them.
< gmaxwell> I would have said it probably shouldn't have been marked rejected until it hit its programmatic timeout, but I don't believe any implementation had one.
< luke-jr> gmaxwell: it was when BU and Classic declared it dead
< gmaxwell> so what would happen if a 'marked dead' proposal was activated by the network in-fact (ignoring the non-penetration of classic in any part of the network, for a moment)?
< luke-jr> I presume we'd have to update the status of it
< luke-jr> although meeting its own criteria is not sufficient for this hardfork of course
< gmaxwell> yes, indeed.
< gmaxwell> in any case, point I was making was that the author abandoning it or even its biggest proponents doing do doesn't actually guarentee that its dead.
< luke-jr> well, their claim was that *everyone* had abandoned it, so I did some basic checking that none of the active proponents I knew of disagreed
< bitcoin-git> [bitcoin] rebroad opened pull request #9315: Request announcement by cmpctblock AFTER requesting cmpctblock/blocktxn (master...RequestAnnounceAfter) https://github.com/bitcoin/bitcoin/pull/9315
< btcdrak> gmaxwell: the code was also removed from the only implementation that shipped it.
< morcos> gmaxwell: dcousens: my suggestion would be to mark wallet txs as stale if they are expired from your mempool and then not auto-reaccept stale txs...
< morcos> then it would be nice if there were gui/rpc ways to identify these so they could be marked abandoned if manually they are determined to be so, or be double spent
< morcos> of course if there are OTHER people rebroadcasting your old txs that may not be the worlds best solution..
< morcos> this makes a lot more sense in my mind after 2 weeks, than 3 days... something that hasn't been confirmed in 2 weeks is in real danger never being confirmed.. 3 days, not so much
< morcos> Call for review. I know #9138 is a bit annoying to review, but I think its important to get in for 0.14, and I'd rather have it merged earlier rather than later.
< gribble> https://github.com/bitcoin/bitcoin/issues/9138 | Improve fee estimation by morcos · Pull Request #9138 · bitcoin/bitcoin · GitHub
< morcos> Fee estimation issues often only show up after a lot of usage, so better to get it some real testing... Also the multiple rebases make it hard to keep the review ACK's updated
< MurhS0xFF> anyone
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #9316: [wallet] Disable free transactions when relay is disabled (master...Mf1612-sendFreeNo) https://github.com/bitcoin/bitcoin/pull/9316
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/a1dcf2e1087b...76fcd9d50341
< bitcoin-git> bitcoin/master 9359f8a Alex Morcos: Wallet needs to stay unlocked for whole test
< bitcoin-git> bitcoin/master 76fcd9d MarcoFalke: Merge #9309: [qa] Wallet needs to stay unlocked for whole test...
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #9309: [qa] Wallet needs to stay unlocked for whole test (master...rarerpcfail) https://github.com/bitcoin/bitcoin/pull/9309