1 2017-03-07 00:33:20 0|bitcoin-git|13bitcoin/06master 148455e36 15practicalswift: [test] Avoid reading a potentially uninitialized variable in tx_invalid-test...
2 2017-03-07 00:33:20 0|bitcoin-git|[13bitcoin] 15MarcoFalke pushed 2 new commits to 06master: 02 https://github.com/bitcoin/bitcoin/compare/72fb5158b1c8...19be26afe3d0
3 2017-03-07 00:33:21 0|bitcoin-git|13bitcoin/06master 1419be26a 15MarcoFalke: Merge #9555: [test] Avoid reading a potentially uninitialized variable in tx_invalid-test (transaction_tests.cpp)...
4 2017-03-07 00:33:31 0|bitcoin-git|[13bitcoin] 15MarcoFalke closed pull request #9555: [test] Avoid reading a potentially uninitialized variable in tx_invalid-test (transaction_tests.cpp) (06master...06avoid-ub-in-tx_invalid-test) 02 https://github.com/bitcoin/bitcoin/pull/9555
5 2017-03-07 03:04:40 0|gmaxwell|dear god, UIs are hard.
6 2017-03-07 03:04:41 0|gmaxwell|19:02 < pawn> gmaxwell: I have the new version. I opened the console. I ran dumpwallet, and it's not giving anything back to me. I passed wallet.dat and /Users/pawn/Library/Application\ Support/Bitcoin/wallet.dat as an argument.
7 2017-03-07 03:05:33 0|sipa|:'(
8 2017-03-07 03:05:49 0|gmaxwell|I had him back up first.
9 2017-03-07 03:11:16 0|gwillen|that is ... not a failure mode I ever would have thought of. Users find very creative ways to hurt themselves. I'm glad you had him back up.
10 2017-03-07 03:18:29 0|gmaxwell|saw things similar prior. (button for a most common symbol in my words isn't working)
11 2017-03-07 04:38:45 0|achow101|Does anyone else have a compiler warning for wallet/db.cpp:620?
12 2017-03-07 04:46:59 0|phantomcircuit|achow101, compiler warnings are hard to reproduce because they all have different ideas of what to warn about
13 2017-03-07 04:58:19 0|gmaxwell|achow101: whats the warning?
14 2017-03-07 04:58:56 0|gmaxwell|if it's a shadow warning, mention it on #9911
15 2017-03-07 04:58:58 0|gribble|https://github.com/bitcoin/bitcoin/issues/9911 | Wshadow: various gcc fixes by paveljanik ÷ Pull Request #9911 ÷ bitcoin/bitcoin ÷ GitHub
16 2017-03-07 05:01:50 0|achow101|it is a shadow warning
17 2017-03-07 05:02:37 0|achow101|looks like paveljanik already got it and fixed it
18 2017-03-07 05:50:31 0|paveljanik|yup :-)
19 2017-03-07 08:22:16 0|bitcoin-git|[13bitcoin] 15practicalswift opened pull request #9936: [trivial] Fix three typos introduced into walletdb.h in yesterday's merge of commit 7184e25 (06master...06fix-typos-introduced-in-commit-7184e25c) 02 https://github.com/bitcoin/bitcoin/pull/9936
20 2017-03-07 08:53:17 0|bitcoin-git|[13bitcoin] 15laanwj opened pull request #9937: rpc: Prevent `dumpwallet` from overwriting files (06master...062017_03_walletdump_nooverwrite) 02 https://github.com/bitcoin/bitcoin/pull/9937
21 2017-03-07 09:33:44 0|bitcoin-git|[13bitcoin] 15laanwj closed pull request #9928: Allow verify-commit.sh to just verify the HEAD commit (Use non-recursive verification by default) (06master...062017/03/vc_simple) 02 https://github.com/bitcoin/bitcoin/pull/9928
22 2017-03-07 09:48:59 0|wumpus|going to tag 0.14.0 final in a bit
23 2017-03-07 09:50:26 0|wumpus|gah, the github-merge script is not working anymore
24 2017-03-07 09:52:21 0|bitcoin-git|[13bitcoin] 15laanwj pushed 3 new commits to 06master: 02 https://github.com/bitcoin/bitcoin/compare/19be26afe3d0...00c13ea637d6
25 2017-03-07 09:52:22 0|bitcoin-git|13bitcoin/06master 14b3ec305 15Matt Corallo: Fix bashisms in verify-commits and always check top commit's tree
26 2017-03-07 09:52:22 0|bitcoin-git|13bitcoin/06master 14f20e664 15Matt Corallo: Check gpg version before setting --weak-digest
27 2017-03-07 09:52:23 0|bitcoin-git|13bitcoin/06master 1400c13ea 15Wladimir J. van der Laan: Merge #9932: Fix verify-commits on travis and always check top commit's tree...
28 2017-03-07 09:52:45 0|bitcoin-git|[13bitcoin] 15laanwj closed pull request #9932: Fix verify-commits on travis and always check top commit's tree (06master...062017-03-fix-verify-commits) 02 https://github.com/bitcoin/bitcoin/pull/9932
29 2017-03-07 09:54:26 0|bitcoin-git|[13bitcoin] 15laanwj pushed 1 new commit to 06master: 02 https://github.com/bitcoin/bitcoin/commit/309bf16257b2395ce502017be627186b749ee749
30 2017-03-07 09:54:27 0|bitcoin-git|13bitcoin/06master 14309bf16 15Wladimir J. van der Laan: devtools: Fix a syntax error typo...
31 2017-03-07 10:01:21 0|bitcoin-git|[13bitcoin] 15laanwj pushed 4 new commits to 06master: 02 https://github.com/bitcoin/bitcoin/compare/309bf16257b2...779f2f97478c
32 2017-03-07 10:01:22 0|bitcoin-git|13bitcoin/06master 1473296f5 15Matt Corallo: CScheduler boost->std::function, use millisecs for times, not secs
33 2017-03-07 10:01:22 0|bitcoin-git|13bitcoin/06master 14735d9b5 15Matt Corallo: Use CScheduler for wallet flushing, remove ThreadFlushWalletDB
34 2017-03-07 10:01:23 0|bitcoin-git|13bitcoin/06master 140235be1 15Matt Corallo: Rename FlushWalletDB -> CompactWalletDB, add function description
35 2017-03-07 10:01:36 0|bitcoin-git|[13bitcoin] 15laanwj closed pull request #9605: Use CScheduler for wallet flushing, remove ThreadFlushWalletDB (06master...062017-01-cscheduler-cleanups) 02 https://github.com/bitcoin/bitcoin/pull/9605
36 2017-03-07 10:03:54 0|bitcoin-git|13bitcoin/06master 1402c57b5 15Suhas Daftuar: Optimize GetWitnessHash() for non-segwit transactions
37 2017-03-07 10:03:54 0|bitcoin-git|[13bitcoin] 15laanwj pushed 2 new commits to 06master: 02 https://github.com/bitcoin/bitcoin/compare/779f2f97478c...6015df5b855f
38 2017-03-07 10:03:55 0|bitcoin-git|13bitcoin/06master 146015df5 15Wladimir J. van der Laan: Merge #9912: Optimize GetWitnessHash() for non-segwit transactions...
39 2017-03-07 10:04:16 0|bitcoin-git|[13bitcoin] 15laanwj closed pull request #9912: Optimize GetWitnessHash() for non-segwit transactions (06master...062017-03-optimize-witness-hash) 02 https://github.com/bitcoin/bitcoin/pull/9912
40 2017-03-07 10:15:06 0|bitcoin-git|[13bitcoin] 15laanwj closed pull request #9937: rpc: Prevent `dumpwallet` from overwriting files (06master...062017_03_walletdump_nooverwrite) 02 https://github.com/bitcoin/bitcoin/pull/9937
41 2017-03-07 10:15:58 0|wumpus|crap, travis is *still* failing with #9932
42 2017-03-07 10:16:00 0|gribble|https://github.com/bitcoin/bitcoin/issues/9932 | An error has occurred and has been logged. Please contact this bot's administrator for more information.
43 2017-03-07 10:16:21 0|luke-jr|gribble also XD
44 2017-03-07 10:16:21 0|Victorsueca|lol
45 2017-03-07 10:16:32 0|Victorsueca|everything seems to fail today
46 2017-03-07 10:17:19 0|wumpus|yes it's still morning and I've already had it with today
47 2017-03-07 10:28:34 0|jeremyrubin|wumpus: Aw hang in there
48 2017-03-07 10:29:06 0|jeremyrubin|wumpus: crack some bubbly whn you're done ;)
49 2017-03-07 10:51:05 0|wumpus|hah
50 2017-03-07 10:53:00 0|wumpus|* [new tag] v0.14.0 -> v0.14.0
51 2017-03-07 10:54:42 0|jonasschnelli|\o/
52 2017-03-07 10:54:47 0|Ylbam|+1
53 2017-03-07 11:03:39 0|btcdrak|woo
54 2017-03-07 11:06:24 0|shangzhou|\o/
55 2017-03-07 11:10:02 0|jeremyrubin|yay!
56 2017-03-07 12:06:27 0|bitcoin-git|[13bitcoin] 15JeremyRubin closed pull request #8464: [WIP] "Lockfree" Checkqueue Implementation (06master...06lockfree-checkqueue-squashed) 02 https://github.com/bitcoin/bitcoin/pull/8464
57 2017-03-07 12:20:45 0|bitcoin-git|[13bitcoin] 15JeremyRubin opened pull request #9938: Lock-Free CheckQueue (06master...06PR-lockfree-checkqueue) 02 https://github.com/bitcoin/bitcoin/pull/9938
58 2017-03-07 12:25:54 0|jeremyrubin|Alright going to sleep after writing that one up
59 2017-03-07 12:48:55 0|jonasschnelli|659878f197b29e05fcd89f3bd18a17d0d0952d479bbc6c95f4aecc7f0cd104e5 bitcoin-0.14.0-osx-unsigned.dmg
60 2017-03-07 13:09:17 0|wumpus|05edd35d4b119ec92ecedde3b2da6a69078161741421da8ed039908a8e223405 :/
61 2017-03-07 13:11:53 0|wumpus|oh wait nm
62 2017-03-07 13:12:29 0|wumpus|built the rc3 again instead of final :)
63 2017-03-07 13:14:23 0|jonasschnelli|14898a9b54eb8e88eeec2f29024f2fb47a6b54dbc0e8c587eea4bec0a13fb951 bitcoin-0.14.0-win-unsigned.tar.gz
64 2017-03-07 13:14:34 0|jonasschnelli|06e6ceeb687e784e9aaad45e9407c7eed5f7e9c9bbe44083179287f54f0f9f2b bitcoin-0.14.0-x86_64-linux-gnu.tar.gz
65 2017-03-07 13:59:35 0|bitcoin-git|[13bitcoin] 15joaopaulofonseca opened pull request #9939: Cache vout IsMine() value on `CWallet::AvailableCoins()` (06master...06enhancement/cache-wallet-available-coins-output-ismine) 02 https://github.com/bitcoin/bitcoin/pull/9939
66 2017-03-07 14:42:38 0|jonasschnelli|Hmm... just did an importmulti from the Qt console... the rescan window does not show up.
67 2017-03-07 14:50:59 0|achow101|my gitian builds match jonasschnelli's
68 2017-03-07 14:51:37 0|jonasschnelli|achow101: great
69 2017-03-07 15:06:21 0|wumpus|mine match jonasschnelli's too
70 2017-03-07 15:52:25 0|cfields|woohoo, final tag :)
71 2017-03-07 15:52:58 0|cfields|I'm about to head to the airport, will be able to sign in ~6hrs
72 2017-03-07 16:12:30 0|bitcoin-git|[13bitcoin] 15laanwj pushed 5 new commits to 06master: 02 https://github.com/bitcoin/bitcoin/compare/6015df5b855f...3178b2c740f1
73 2017-03-07 16:12:31 0|bitcoin-git|13bitcoin/06master 141f98abe 15Russell Yanofsky: Factor out CWallet::nTimeSmart computation into a method....
74 2017-03-07 16:12:31 0|bitcoin-git|13bitcoin/06master 14c6b82d1 15Russell Yanofsky: Add tests for CWalletTx::nTimeSmart
75 2017-03-07 16:12:32 0|bitcoin-git|13bitcoin/06master 146c996c2 15Russell Yanofsky: Add documentation describing CWallet::nTimeSmart....
76 2017-03-07 16:35:27 0|morcos|wumpus: #9602 has 5+ ACK's can we go ahead and merge while its clean? The only quibbling seems to be about more comment cleanups and some outstanding lack of consensus on breaking priortisetransaction API. That change can always be modified later.
77 2017-03-07 16:35:29 0|gribble|https://github.com/bitcoin/bitcoin/issues/9602 | Remove coin age priority and free transactions - implementation by morcos ÷ Pull Request #9602 ÷ bitcoin/bitcoin ÷ GitHub
78 2017-03-07 16:56:15 0|BlueMatt|wumpus: hmm, the travis error on 309bf16 i can reproduce - I dont match your sha512 tree
79 2017-03-07 17:10:09 0|bitcoin-git|[13bitcoin] 15TheBlueMatt opened pull request #9940: Fix verify-commits on OSX, update for new bad Tree-SHA512, point travis to different keyservers (06master...062017-03-verify-commits-no-recursion) 02 https://github.com/bitcoin/bitcoin/pull/9940
80 2017-03-07 18:04:43 0|wumpus|morcos: ok
81 2017-03-07 18:05:45 0|wumpus|BlueMatt: I've done nothing special, just used github-merge.py, no clue why that SHA512 wouldn't match :/
82 2017-03-07 18:07:45 0|wumpus|BlueMatt: maybe we should revert the SHA512 change until we have a clue what is happening here
83 2017-03-07 18:09:06 0|wumpus|let's see if I can reproduce it myself, anyhow
84 2017-03-07 18:09:58 0|wumpus|309bf16 doesn't even have a SHA512
85 2017-03-07 18:12:01 0|wumpus|it was a quick fixup (without github-merge) to get github-merge to work again
86 2017-03-07 18:31:49 0|bitcoin-git|13bitcoin/06master 14ddf58c7 15MarcoFalke: wallet: Remove sendfree...
87 2017-03-07 18:31:49 0|bitcoin-git|[13bitcoin] 15laanwj pushed 14 new commits to 06master: 02 https://github.com/bitcoin/bitcoin/compare/3178b2c740f1...30ff3a2fc95c
88 2017-03-07 18:31:50 0|bitcoin-git|13bitcoin/06master 1412839cd 15Alex Morcos: [rpc] Remove estimatepriority and estimatesmartpriority....
89 2017-03-07 18:31:50 0|bitcoin-git|13bitcoin/06master 14272b25a 15Alex Morcos: [mining] Remove -blockprioritysize....
90 2017-03-07 18:32:04 0|bitcoin-git|[13bitcoin] 15laanwj closed pull request #9602: Remove coin age priority and free transactions - implementation (06master...06nopriority) 02 https://github.com/bitcoin/bitcoin/pull/9602
91 2017-03-07 18:34:18 0|morcos|yay!! thanks and sorry for the pestering! (#9548 can go too now, i wasn't sure if it conflicted, but it doesnt)
92 2017-03-07 18:34:19 0|gribble|https://github.com/bitcoin/bitcoin/issues/9548 | Remove min reasonable fee by morcos ÷ Pull Request #9548 ÷ bitcoin/bitcoin ÷ GitHub
93 2017-03-07 18:41:50 0|btcdrak|nice work morcos!
94 2017-03-07 18:42:38 0|wumpus|morcos: no problem, if a pull is ready it should be merged
95 2017-03-07 18:42:44 0|wumpus|yes great work :)
96 2017-03-07 18:49:22 0|bitcoin-git|[13bitcoin] 15laanwj pushed 4 new commits to 06master: 02 https://github.com/bitcoin/bitcoin/compare/30ff3a2fc95c...47510ad3dd51
97 2017-03-07 18:49:24 0|bitcoin-git|13bitcoin/06master 142a7b56c 15Alex Morcos: CBlockPolicyEstimator now uses hard coded minimum bucket feerate
98 2017-03-07 18:49:24 0|bitcoin-git|13bitcoin/06master 14ac9d3d2 15Alex Morcos: Change fee estimation bucket limit variable names
99 2017-03-07 18:49:25 0|bitcoin-git|13bitcoin/06master 14ad82cb0 15Alex Morcos: Remove unnecessary min fee argument in CTxMemPool constructor
100 2017-03-07 18:49:39 0|bitcoin-git|[13bitcoin] 15laanwj closed pull request #9548: Remove min reasonable fee (06master...06removeMinReasonableFee) 02 https://github.com/bitcoin/bitcoin/pull/9548
101 2017-03-07 18:56:10 0|BlueMatt|wumpus: well that explains why it didnt match :P
102 2017-03-07 19:37:29 0|bitcoin-git|[13bitcoin] 15morcos opened pull request #9942: Refactor CBlockPolicyEstimator (06master...06moveTxConfirmStats) 02 https://github.com/bitcoin/bitcoin/pull/9942
103 2017-03-07 19:45:31 0|bitcoin-git|[13bitcoin] 15jnewbery opened pull request #9943: Make script.py wildcard importable. (06master...06rpctestsprimitives) 02 https://github.com/bitcoin/bitcoin/pull/9943
104 2017-03-07 19:46:03 0|bitcoin-git|[13bitcoin] 15jnewbery closed pull request #9876: Remove wildcard imports from qa (06master...06remove_wildcard_imports) 02 https://github.com/bitcoin/bitcoin/pull/9876
105 2017-03-07 20:20:44 0|achow101|github as this review feature where you can submit all of your comments as one review and that goes out to recipients as one email. I ask that people please use that to avoid spamming people's inboxes with emails for every single comment you make
106 2017-03-07 20:20:55 0|achow101|s/as this/has this/
107 2017-03-07 20:21:03 0|BlueMatt|achow101: sadly you cant respond in batch
108 2017-03-07 20:21:12 0|BlueMatt|so you get a batch review, and then a million response emails :(
109 2017-03-07 20:21:37 0|BlueMatt|also, I'd generally recommend a threaded email client to fix this issue :P
110 2017-03-07 20:22:33 0|achow101|well that's annoying then.
111 2017-03-07 20:22:38 0|sipa|you can't respond in batch?
112 2017-03-07 20:22:44 0|sipa|really?
113 2017-03-07 20:24:49 0|BlueMatt|sipa: i dont think so? Did I miss something?
114 2017-03-07 20:24:54 0|BlueMatt|maybe i did...hmmm
115 2017-03-07 20:26:42 0|achow101|BlueMatt: if you respond in the "view changes" tab, you can respond in a batch with the "start a review" button when you click to reply
116 2017-03-07 20:29:05 0|BlueMatt|ahh, ok
117 2017-03-07 20:29:12 0|BlueMatt|but, really, also, threaded email client and folders :)
118 2017-03-07 20:30:14 0|achow101|I have a threaded email client. I just have everything expanded by default and saw a ton of emails for all of your comments :)
119 2017-03-07 20:44:28 0|bitcoin-git|[13bitcoin] 15sipa closed pull request #9909: tests: Add FindEarliestAtLeast test for edge cases (06master...06pr/faltest) 02 https://github.com/bitcoin/bitcoin/pull/9909
120 2017-03-07 20:44:37 0|sipa|oops, misclick
121 2017-03-07 21:28:25 0|kanzure|should decoderawtransaction calculate total input amounts?
122 2017-03-07 21:28:39 0|sipa|no
123 2017-03-07 21:29:19 0|kanzure|is there another way for an rpc user to do basic sanity checks on transactions like that?
124 2017-03-07 21:29:30 0|sipa|if they have the inputs, sure
125 2017-03-07 21:29:38 0|sipa|which should be trivial if it's their own
126 2017-03-07 21:29:44 0|sipa|and nearly impossible otherwise
127 2017-03-07 21:30:01 0|sipa|(would require a fully indexed blockchain)
128 2017-03-07 21:30:06 0|kanzure|which rpc command though?
129 2017-03-07 21:30:09 0|achow101|kanzure: there was a proposal for a verifyrawtransaction but it was controversial so it got dropped
130 2017-03-07 21:30:22 0|achow101|the only way to check if a transaction is valid is to send it
131 2017-03-07 21:30:25 0|sipa|kanzure: if you have -txindex, use getrawtransaction
132 2017-03-07 21:30:29 0|sipa|kanzure: otherwise, don't
133 2017-03-07 21:30:46 0|kanzure|getrawtransaction seems to only take txid
134 2017-03-07 21:30:54 0|kanzure|oh you mean check inputs via getrawtransaction
135 2017-03-07 21:30:54 0|sipa|yes, you fetch all the inputs
136 2017-03-07 21:30:59 0|kanzure|ok ok fine, thanks
137 2017-03-07 21:31:06 0|gmaxwell|we need to specify a new raw transaction format that can carry inputs... but without segwit it's insufficient to verify things.
138 2017-03-07 21:31:22 0|sipa|kanzure: or use gettransaction
139 2017-03-07 21:31:26 0|luke-jr|"(note: this is a temporary file, to be added-to by anybody, and moved to release-notes at release time)" made it into the final tag XD
140 2017-03-07 21:31:29 0|sipa|(which only works if it's your own transaction)
141 2017-03-07 21:31:36 0|gmaxwell|but it should be segwit only because of that.
142 2017-03-07 21:32:02 0|kanzure|context: i am doing a simple user interface for a 8-of-12 signing ritual, checking some other nodes to make sure the total inputs add up to something reasonable and expected
143 2017-03-07 21:32:10 0|gmaxwell|top
144 2017-03-07 21:32:53 0|kanzure|sipa: no, iterating over inputs via getrawtransaction is good enough for now. also giving inputs in the transaction format sounds good to me.
145 2017-03-07 21:33:26 0|sipa|kanzure: still, it requires a full txindex node
146 2017-03-07 21:34:20 0|sipa|kanzure: unless all the outputs are available
147 2017-03-07 21:34:25 0|sipa|which i guess will be the case for you
148 2017-03-07 21:34:45 0|sipa|s/outputs are available/inputs are unspent/
149 2017-03-07 22:07:26 0|luke-jr|hm, we lost bitcoin.conf.5 (manpage)
150 2017-03-07 22:20:35 0|BlueMatt|any idea in what pr?
151 2017-03-07 22:20:44 0|BlueMatt|its there there in contrib/debian
152 2017-03-07 23:11:36 0|bitcoin-git|[13bitcoin] 15jnewbery opened pull request #9945: Improve logging in bctest.py if there is a formatting mismatch (06master...06improve_bctest_logging) 02 https://github.com/bitcoin/bitcoin/pull/9945
153 2017-03-07 23:13:06 0|luke-jr|BlueMatt: not in 0.14/master?
154 2017-03-07 23:15:32 0|BlueMatt|oh, nvm, I'm thinking of contrib/debain/examples/
155 2017-03-07 23:21:04 0|bitcoin-git|[13bitcoin] 15pinheadmz opened pull request #9946: Fix build errors if spaces in path or parent directory (06master...06dirspaces2) 02 https://github.com/bitcoin/bitcoin/pull/9946