2016-11-17

< shangzhou> bitcoin.org's download link.
< shangzhou> gmaxwell: i did some search this morning at baidu. Searched "c1726ccc50635795c942c7d7e51d979c4f83a3d17f8982e9d02a114a15fef419" only one result (a thread http://8btc.com/thread-41510-1-5.html) someone use Avast complain that bitcoin 0.13.1 win exe downloaded from bitcoin.org was blocked by the avast. Search something like "bitcoin 0.13.1 win exe" top one is
< BlueMatt> -rw------- 1 matt matt 1.7T Nov 16 16:44 /home/matt/.bitcoin/debug.log

2016-11-16

< Chris_Stewart_5> Why does this test fail for EVAL_FALSE and not WITNESS_PROGRAM_MISMATCH? https://github.com/bitcoin/bitcoin/blob/master/src/test/data/script_tests.json#L1257
< theymos> fyi, some apparently-popular antivirus is flagging Bitcoin Core 0.13.1: https://www.virustotal.com/en/file/c1726ccc50635795c942c7d7e51d979c4f83a3d17f8982e9d02a114a15fef419/analysis/
< bitcoin-git> [bitcoin] jonasschnelli closed pull request #9173: demo (master...master) https://github.com/bitcoin/bitcoin/pull/9173
< bitcoin-git> [bitcoin] demodun opened pull request #9173: demo (master...master) https://github.com/bitcoin/bitcoin/pull/9173
< bitcoin-git> [bitcoin] laanwj reopened pull request #8747: [rpc] Fix transaction size comments and RPC help text. (master...rpc_comments) https://github.com/bitcoin/bitcoin/pull/8747
<@wumpus> or original VT-100 terminals or whatnot :) bitcoin isn't that old
< gribble> https://github.com/bitcoin/bitcoin/issues/9166 | listtransactions returns invalid JSON when comment contain non-UTF8 special chars · Issue #9166 · bitcoin/bitcoin · GitHub
<@wumpus> and that is how everyone uses it. We're not going to support non-UTF-8 JSON in bitcoin core.
< gribble> https://github.com/bitcoin/bitcoin/issues/7940 | [WIP] Fuzzing framework by pstratem · Pull Request #7940 · bitcoin/bitcoin · GitHub
< jonasschnelli> BLTS: maybe #bitcoin or #bitcoin-dev but not #bitcoin-core-dev I guess
< bitcoin-git> [bitcoin] laanwj opened pull request #9172: Resurrect pstratem's "Simple fuzzing framework" (master...2016_11_fuzzing_framework) https://github.com/bitcoin/bitcoin/pull/9172
< bitcoin-git> bitcoin/master cb2ed30 MarcoFalke: Merge #9155: [trivial] update comments for tx weight...
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #9155: [trivial] update comments for tx weight (master...master) https://github.com/bitcoin/bitcoin/pull/9155
< bitcoin-git> bitcoin/master 07ede5d Brian Deery: update comments for tx weight
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/0a6d48d9ed60...cb2ed300a89e
< bitcoin-git> [bitcoin] laanwj closed pull request #8747: [rpc] Fix transaction size comments and RPC help text. (master...rpc_comments) https://github.com/bitcoin/bitcoin/pull/8747
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #9168: [qa] add assert_raises_message to check specific error message (master...ba-assert-raises) https://github.com/bitcoin/bitcoin/pull/9168
< bitcoin-git> bitcoin/master 307acdd mrbandrews: [qa] add assert_raises_message to check specific error message
< bitcoin-git> bitcoin/master 0a6d48d MarcoFalke: Merge #9168: [qa] add assert_raises_message to check specific error message...
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/434e683f7be6...0a6d48d9ed60
< bitcoin-git> [bitcoin] jonasschnelli opened pull request #9171: Introduce out-of-band block requests (master...2016/11/spv_step_1) https://github.com/bitcoin/bitcoin/pull/9171
<@wumpus> CubicEarth: this is not a channel for general bitcoin related questions, please use #bitcoin
< bitcoin-git> [bitcoin] laanwj closed pull request #9133: Unset fImporting for loading mempool (master...noImportLoadMempool) https://github.com/bitcoin/bitcoin/pull/9133
< bitcoin-git> bitcoin/master 79f755d Alex Morcos: Unset fImporting for loading mempool
< bitcoin-git> bitcoin/master 434e683 Wladimir J. van der Laan: Merge #9133: Unset fImporting for loading mempool...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/62af16463833...434e683f7be6
< bitcoin-git> [bitcoin] laanwj closed pull request #9131: fNetworkActive is not protected by a lock, use an atomic (master...2016/11/net_toggle) https://github.com/bitcoin/bitcoin/pull/9131
< bitcoin-git> bitcoin/master 079142b Jonas Schnelli: fNetworkActive is not protected by a lock, use an atomic
< bitcoin-git> bitcoin/master 62af164 Wladimir J. van der Laan: Merge #9131: fNetworkActive is not protected by a lock, use an atomic...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/4333b1c4ea74...62af16463833
< bitcoin-git> [bitcoin] laanwj closed pull request #9151: [qa] proxy_test: Calculate hardcoded port numbers (master...Mf1611-qaPort) https://github.com/bitcoin/bitcoin/pull/9151
< bitcoin-git> bitcoin/master 4333b1c Wladimir J. van der Laan: Merge #9151: [qa] proxy_test: Calculate hardcoded port numbers...
< bitcoin-git> bitcoin/master fa80ef8 MarcoFalke: [qa] proxy_test: Calculate hardcoded port numbers instead
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/918ea16dc08d...4333b1c4ea74
< bitcoin-git> [bitcoin] laanwj closed pull request #9169: build: fix qt5.7 build under macOS (master...fix-objcxx-std) https://github.com/bitcoin/bitcoin/pull/9169
< bitcoin-git> bitcoin/master 70266e9 Cory Fields: build: fix qt5.7 build under macOS...
< bitcoin-git> bitcoin/master 918ea16 Wladimir J. van der Laan: Merge #9169: build: fix qt5.7 build under macOS...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/6eeac6e30d65...918ea16dc08d
< jonasschnelli> sipa: come one! s/hash rate</a></li>/hash rate</a></li><li class="hilight"><a accesskey="h" href="http://bitcoin.sipa.be/ver9-10k.png">ver9-10k</a></li>/
< sipa> gmaxwell: http://bitcoin.sipa.be
< gmaxwell> for bitcoin I do think we need DRD run on it more often, it's just a bit frustrating because its so slow.
<@wumpus> bitcoin: where even the configure scripts yell at you
< * luke-jr> cowers in fear of the new Bitcoin Core configure that yells at you :P

2016-11-15

< cfields> morcos: top two commits here: https://github.com/theuni/bitcoin/commits/prevector-move
< gribble> https://github.com/bitcoin/bitcoin/issues/9125 | Make CBlock a vector of shared_ptr of CTransactions by sipa · Pull Request #9125 · bitcoin/bitcoin · GitHub
< cfields> morcos: assuming bitcoin-qt build succeeds after that, then yes, that's enough
< bitcoin-git> [bitcoin] theuni opened pull request #9169: build: fix qt5.7 build under macOS (master...fix-objcxx-std) https://github.com/bitcoin/bitcoin/pull/9169
< bitcoin-git> [bitcoin] mrbandrews opened pull request #9168: [qa] add assert_raises_message to check specific error message (master...ba-assert-raises) https://github.com/bitcoin/bitcoin/pull/9168
< bitcoin-git> [bitcoin] morcos opened pull request #9167: IsAllFromMe (master...IsAllFromMe) https://github.com/bitcoin/bitcoin/pull/9167
< gribble> https://github.com/bitcoin/bitcoin/issues/8589 | Inline CTxInWitness inside CTxIn (on top of #8580) by sipa · Pull Request #8589 · bitcoin/bitcoin · GitHub
< timothy> drizzt@liara ~ % bitcoin-cli listtransactions '*' 2
< thermoman> jonasschnelli, luke-jr: https://github.com/bitcoin/bitcoin/issues/9166
< bitcoin-git> [bitcoin] instagibbs opened pull request #9165: [trivial] SendMoney: use already-calculated balance (master...triv-curbal) https://github.com/bitcoin/bitcoin/pull/9165
< bitcoin-git> [bitcoin] sipa closed pull request #9164: [trivial] credit values are CAmount (master...intcredit) https://github.com/bitcoin/bitcoin/pull/9164
< bitcoin-git> bitcoin/master 6eeac6e Pieter Wuille: Merge #9164: [trivial] credit values are CAmount...
< bitcoin-git> bitcoin/master 20c3215 Gregory Sanders: credit values are CAmount
< bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/018a4eb120dc...6eeac6e30d65
< jonasschnelli> Now testing bitcoin-cli
< thermoman> bitcoin-cli says can't parse reply
< jonasschnelli> This one is easy to review if someone has 2-3 minutes of boringness: https://github.com/bitcoin/bitcoin/pull/9142
< sdaftuar> thrasher`: i don't know where that test case came from, but i do recall this github comment that may be of help: https://github.com/bitcoin/bitcoin/pull/5620#issuecomment-69351549
< bitcoin-git> [bitcoin] instagibbs opened pull request #9164: [trivial] credit values are CAmount (master...intcredit) https://github.com/bitcoin/bitcoin/pull/9164
< bitcoin-git> bitcoin/master f54e460 Wladimir J. van der Laan: Merge #9132: Make strWalletFile const...
< bitcoin-git> [bitcoin] laanwj closed pull request #9132: Make strWalletFile const (master...2016/11/strWalletFile_const) https://github.com/bitcoin/bitcoin/pull/9132
< bitcoin-git> bitcoin/master f734505 Jonas Schnelli: Make strWalletFile const
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/770364b8eaf7...f54e4605fc31
< bitcoin-git> [bitcoin] laanwj closed pull request #9160: [trivial] Fix hungarian variable name (master...hungarian) https://github.com/bitcoin/bitcoin/pull/9160
< bitcoin-git> bitcoin/master 770364b Wladimir J. van der Laan: Merge #9160: [trivial] Fix hungarian variable name...
< bitcoin-git> bitcoin/master ec34648 Russell Yanofsky: [trivial] Fix hungarian variable name...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b42291334651...770364b8eaf7
< gribble> https://github.com/bitcoin/bitcoin/issues/8831 | Replace CWalletDB::ReadKeyValue with CWallet::LoadKeyValue by pstratem · Pull Request #8831 · bitcoin/bitcoin · GitHub
< achow101> Squidicuz: yes, bitcoincore.org is Bitcoin Core's site
< shinyg> Maybe bitcoin is a software stack but Core is just software
< shinyg> makes sense, how about is Bitcoin Core a Solution stack or software as some describe it/
< achow101> they're trying to separate bitcoin core from bitcoin.org as bitcoin.org is for bitcoin in general whilst bitcoin core is a specific project
< sipa> but bitcoin core has its own project website now
< shinyg> so was there a specific reason why new releases stopped coming from bitcoin.org?
< achow101> well they're listed here: https://bitcoin.org/en/about-us#sponsorship

2016-11-14

< achow101> also, IIRC bitcoin.org and the bitcoin foundation are no longer related
< achow101> shinyg: blockstream does not fund Bitcoin Core development at all. There are developers who work on Bitcoin Core who also have jobs at Blockstream. The people who actually partially fund development are the MIT DCI who actually pay some people to work on Core (wladimir, cory)
< shinyg> I have recently expanded the Wikipedia article for Bitcoin Core by a factor of 10. Could someone spend just a few minutes to see if there are any major ommission or errors?
< shinyg> Hi people, I was after a little assistance from those who know more about Bitcoin Core and those who support Wikipedia.
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #9136: sync_blocks cleanup (master...sync-clean) https://github.com/bitcoin/bitcoin/pull/9136
< bitcoin-git> bitcoin/master 7943b13 Russell Yanofsky: [qa] Avoid 2 list comprehensions in sync_blocks
< bitcoin-git> bitcoin/master 05e57cc Russell Yanofsky: [qa] Fix sync_blocks timeout argument...
< bitcoin-git> bitcoin/master fd6bb70 Russell Yanofsky: [qa] Improve sync_blocks error messages.
< bitcoin-git> [bitcoin] MarcoFalke pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/924745dd6f5f...b42291334651
< bitcoin-git> [bitcoin] ryanofsky opened pull request #9160: [trivial] Fix hungarian variable name (master...hungarian) https://github.com/bitcoin/bitcoin/pull/9160
< bitcoin-git> [bitcoin] ryanofsky opened pull request #9159: [qa] Wait for specific block announcement in p2p-compactblocks (master...cmpct-announce-wait) https://github.com/bitcoin/bitcoin/pull/9159
< gribble> https://github.com/bitcoin/bitcoin/issues/8580 | Make CTransaction actually immutable by sipa · Pull Request #8580 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/9125 | Make CBlock a vector of shared_ptr of CTransactions by sipa · Pull Request #9125 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] Leviathn closed pull request #9158: Removal of "free transaction" logic from codebase (master...patch-1) https://github.com/bitcoin/bitcoin/pull/9158
< bitcoin-git> [bitcoin] Leviathn opened pull request #9158: Removal of "free transaction" logic from codebase (master...patch-1) https://github.com/bitcoin/bitcoin/pull/9158
< bitcoin-git> [bitcoin] jonasschnelli opened pull request #9156: Add compile and link options echo to configure (master...2016/11/configure) https://github.com/bitcoin/bitcoin/pull/9156
< gmaxwell> okay, bitcoin's sha256 is 237.1 million bytes per second, openssl is 320.7 million bytes per second.. and the chacha is 358.6 million bytes per second. (openssl at slight disadvantage due to 8k vs 1m size, but it doesn't matter much)
< btcdrak> gmaxwell: no no, we're not allowed to sign that, because it's Roger Ver's attempt to snub Bitcoin Core. Looks like a more watered down version of the first leak http://pastebin.com/cgkqcWBS.
< bitcoin-git> bitcoin/master fa42e4a MarcoFalke: [qt] Make network disabled icon 50% opaque
< bitcoin-git> bitcoin/master 924745d Jonas Schnelli: Merge #9145: [qt] Make network disabled icon 50% opaque...
< bitcoin-git> [bitcoin] jonasschnelli closed pull request #9145: [qt] Make network disabled icon 50% opaque (master...Mf1611-qtNetworkIcon) https://github.com/bitcoin/bitcoin/pull/9145
< bitcoin-git> [bitcoin] jonasschnelli pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/87ab49e4fe38...924745dd6f5f
< bitcoin-git> [bitcoin] carryforward opened pull request #9155: [trivial] update comments for tx weight (master...master) https://github.com/bitcoin/bitcoin/pull/9155
< bitcoin-git> [bitcoin] fanquake closed pull request #9154: Add recent checkpoints to remove CPU load during sync (master...f/new-checkpoints) https://github.com/bitcoin/bitcoin/pull/9154
< bitcoin-git> [bitcoin] XertroV opened pull request #9154: Add recent checkpoints to remove CPU load during sync (master...f/new-checkpoints) https://github.com/bitcoin/bitcoin/pull/9154

2016-11-13

< bitcoin-git> [bitcoin] luke-jr opened pull request #9152: Wallet/RPC: sweepprivkeys method to scan UTXO set and send to local wallet (master...sweepprivkeys) https://github.com/bitcoin/bitcoin/pull/9152
< bitcoin-git> [bitcoin] luke-jr closed pull request #9150: Qt: Make network disabled icon larger and 50% opaque (master...gui_netdisabled_icon_enlarge) https://github.com/bitcoin/bitcoin/pull/9150
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #9151: [qa] proxy_test: Calculate hardcoded port numbers (master...Mf1611-qaPort) https://github.com/bitcoin/bitcoin/pull/9151
< bitcoin-git> [bitcoin] luke-jr opened pull request #9150: Qt: Make network disabled icon larger and 50% opaque (master...gui_netdisabled_icon_enlarge) https://github.com/bitcoin/bitcoin/pull/9150
< fanquake> Sure, this list will show you the current "big" changes for the 0.14.0 release: https://github.com/bitcoin/bitcoin/milestone/21
< fanquake> Victorsueca If you'd just like to test stuff in general, start here : https://github.com/bitcoin/bitcoin/issues?q=is%3Aopen+is%3Aissue+label%3AWindows . See if you can recreate/investigate issues. If so post some info about the system your using, how you tested it etc.

2016-11-12

< gribble> https://github.com/bitcoin/bitcoin/issues/9148 | Wallet RPCs can return stale info due to ProcessNewBlock Race · Issue #9148 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #9145: [qt] Make network disabled icon 50% opaque (master...Mf1611-qtNetworkIcon) https://github.com/bitcoin/bitcoin/pull/9145
< bitcoin-git> [bitcoin] fanquake opened pull request #9144: [Trivial] Correct waitforblockheight example help text (master...rpc-commands) https://github.com/bitcoin/bitcoin/pull/9144
< bitcoin-git> [bitcoin] jonasschnelli opened pull request #9143: Refactor ZapWalletTxes to avoid layer vialotions (master...2016/11/wallet_db_refactoring_1) https://github.com/bitcoin/bitcoin/pull/9143
< bitcoin-git> [bitcoin] jonasschnelli opened pull request #9142: Move -salvagewallet, -zap(wtx) to where they belong (master...2016/11/wallet_init) https://github.com/bitcoin/bitcoin/pull/9142
< bitcoin-git> [bitcoin] jonasschnelli opened pull request #9141: Remove unnecessary calls to CheckFinalTx (master...2016/11/istrusted) https://github.com/bitcoin/bitcoin/pull/9141
< bitcoin-git> [bitcoin] luke-jr closed pull request #8388: [0.13] mining: Optimise for typical mining use with blockmaxsize (0.13...blockmaxsize_opti-0.13) https://github.com/bitcoin/bitcoin/pull/8388
< bitcoin-git> [bitcoin] luke-jr closed pull request #7534: Minimal RPC & wallet support for CLTV-enabled multisig addresses (master...cltv_address) https://github.com/bitcoin/bitcoin/pull/7534
< bitcoin-git> [bitcoin] luke-jr opened pull request #9140: Bugfix: Correctly replace generated headers and fail cleanly (master...bugfix_genheaders) https://github.com/bitcoin/bitcoin/pull/9140

2016-11-11

< bitcoin-git> [bitcoin] ryanofsky opened pull request #9139: Change sync_blocks to pick smarter maxheight (master...sync-height) https://github.com/bitcoin/bitcoin/pull/9139
< bitcoin-git> [bitcoin] morcos closed pull request #9123: Remove extraneous LogPrint from fee estimation (master...eliminateFeeWarning) https://github.com/bitcoin/bitcoin/pull/9123
< bitcoin-git> [bitcoin] morcos opened pull request #9138: Improve fee estimation (master...refactorFeeEstimation) https://github.com/bitcoin/bitcoin/pull/9138
< bitcoin-git> [bitcoin] ryanofsky opened pull request #9137: WIP: Allow wallet key import RPCs to track TxOut amounts on -prune nodes (master...import-pruned) https://github.com/bitcoin/bitcoin/pull/9137
< bitcoin-git> [bitcoin] ryanofsky opened pull request #9136: sync_blocks cleanup (master...sync-clean) https://github.com/bitcoin/bitcoin/pull/9136
< GoogleHater> My Bitcoin Core does not finding *.onion peers.
< GoogleHater> Why speed limit isn't implemented in Bitcoin Core?
< bitcoin-git> [bitcoin] morcos opened pull request #9133: Unset fImporting for loading mempool (master...noImportLoadMempool) https://github.com/bitcoin/bitcoin/pull/9133
< bitcoin-git> [bitcoin] jonasschnelli opened pull request #9131: fNetworkActive is not protected by a lock, use an atomic (master...2016/11/net_toggle) https://github.com/bitcoin/bitcoin/pull/9131
< gribble> https://github.com/bitcoin/bitcoin/issues/8456 | [RPC] Simplified bumpfee command. by mrbandrews · Pull Request #8456 · bitcoin/bitcoin · GitHub
< fanquake> One thing I wanted you to take a look at this this diff generated by your diff-tool -> https://github.com/bitcoin/bitcoin/pull/8808#issuecomment-259657066 , the bottom change seems to significant to be part of the PR.
< gribble> https://github.com/bitcoin/bitcoin/issues/9100 | tx_valid: re-order inputs to how they are encoded by dcousens · Pull Request #9100 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] paveljanik opened pull request #9130: Mention the new network toggle functionality in the tooltip. (master...20161111_disable_network_tooltip) https://github.com/bitcoin/bitcoin/pull/9130
< gribble> https://github.com/bitcoin/bitcoin/issues/9100 | tx_valid: re-order inputs to how they are encoded by dcousens · Pull Request #9100 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj closed pull request #9124: Use better name for local variable to prevent -Wshadow compiler warning (master...20161110_Wshadow_benchcheckblock) https://github.com/bitcoin/bitcoin/pull/9124
< bitcoin-git> bitcoin/master 87ab49e Wladimir J. van der Laan: Merge #9124: Use better name for local variable to prevent -Wshadow compiler warning...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/bfc7aad00884...87ab49e4fe38
< bitcoin-git> bitcoin/master bf49f10 Pavel Janík: Use better name for local variable to prevent -Wshadow compiler warning
< gribble> https://github.com/bitcoin/bitcoin/issues/9101 | [Wallet] Do not parse ssValue in CWalletDB::Recover by pstratem · Pull Request #9101 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/9124 | Use better name for local variable to prevent -Wshadow compiler warning by paveljanik · Pull Request #9124 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/8983 | WIP: Log block height and size when received by rebroad · Pull Request #8983 · bitcoin/bitcoin · GitHub
<@wumpus> ooh capitalizing Bitcoin just now that internet is no longer going to be capitalized :)
< gribble> https://github.com/bitcoin/bitcoin/issues/9064 | unify capitalization of "bitcoin address" by s-matthew-english · Pull Request #9064 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj closed pull request #9115: Mention reporting security issues responsibly (master...20161109_secissues) https://github.com/bitcoin/bitcoin/pull/9115
< bitcoin-git> bitcoin/master bfc7aad Wladimir J. van der Laan: Merge #9115: Mention reporting security issues responsibly...
< bitcoin-git> bitcoin/master 7d1de30 Pavel Janík: Mention reporting security issues responsibly
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/ab914a65301b...bfc7aad00884
< gribble> https://github.com/bitcoin/bitcoin/issues/9115 | Mention reporting security issues responsibly by paveljanik · Pull Request #9115 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] luke-jr closed pull request #8889: Qt/ModalOverlay: Use theme tooltip colours (master...overlay_theme) https://github.com/bitcoin/bitcoin/pull/8889
< jonasschnelli> Luke-Jr: how do we continue with https://github.com/bitcoin/bitcoin/pull/8889? Close?
< bitcoin-git> [bitcoin] jonasschnelli closed pull request #8996: Network activity toggle (master...networkactive) https://github.com/bitcoin/bitcoin/pull/8996
< bitcoin-git> bitcoin/master 32efa79 Jon Lund Steffensen: Qt: Add GUI feedback and control of network activity state....
< bitcoin-git> bitcoin/master e38993b Jon Lund Steffensen: RPC: Add "togglenetwork" method to toggle network activity temporarily...
< bitcoin-git> bitcoin/master 7c9a98a Jon Lund Steffensen: Allow network activity to be temporarily suspended....
< bitcoin-git> [bitcoin] jonasschnelli pushed 7 new commits to master: https://github.com/bitcoin/bitcoin/compare/7977a1157a3a...ab914a65301b
<@wumpus> (where "upstream" is the name of your remote for bitcoin/bitcoin, big chance it is "origin" for you)
< bitcoin-git> [bitcoin] laanwj closed pull request #9058: Fixes for p2p-compactblocks.py test timeouts on travis (#8842) (master...fix-8842-sync-timeouts) https://github.com/bitcoin/bitcoin/pull/9058
< bitcoin-git> bitcoin/master dac53b5 Russell Yanofsky: Modify getblocktxn handler not to drop requests for old blocks...
< bitcoin-git> bitcoin/master 55bfddc Russell Yanofsky: [qa] Fix stale data bug in test_compactblocks_not_at_tip...
< bitcoin-git> bitcoin/master 47e9659 Russell Yanofsky: [qa] Fix bug in compactblocks v2 merge...
< bitcoin-git> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/46027e8668ec...7977a1157a3a
< fanquake> Have a look at the github-merge script in this dir https://github.com/bitcoin/bitcoin/tree/master/contrib/devtools
< bitcoin-git> [bitcoin] laanwj closed pull request #9113: Return the type when it's unknown instead of throwing an exception (master...ReturnNotThrow) https://github.com/bitcoin/bitcoin/pull/9113
< bitcoin-git> [bitcoin] laanwj closed pull request #9112: Avoid ugly exception in log on unknown inv type (master...2016_11_invtype_debugging) https://github.com/bitcoin/bitcoin/pull/9112
< bitcoin-git> bitcoin/master 46027e8 Wladimir J. van der Laan: Merge #9112: Avoid ugly exception in log on unknown inv type...
< bitcoin-git> bitcoin/master e9f25dd Wladimir J. van der Laan: Avoid ugly exception in log on unknown inv type...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/21e6c6b569c5...46027e8668ec
<@wumpus> phantomcircuit: jonasschnelli made such a tool https://github.com/bitcoin/bitcoin/pull/8745 - but yeah that's out of scope
< gribble> https://github.com/bitcoin/bitcoin/issues/9112 | Avoid ugly exception in log on unknown inv type by laanwj · Pull Request #9112 · bitcoin/bitcoin · GitHub
< jonasschnelli> For multiwallet, this PR is large but simple to review: https://github.com/bitcoin/bitcoin/pull/8764
<@wumpus> bumpfee needs to be on RPC first e.g. see https://github.com/bitcoin/bitcoin/pull/8456
< Victorsueca> wumpus: you mean that bitcoin core should have it's own random service like, for example, OpenPGP that uses other computationally intensive processes running on your system?
< bitcoin-git> [bitcoin] laanwj closed pull request #9114: [depends] Set OSX_MIN_VERSION to 10.8 (master...depends-min-osx-10-8) https://github.com/bitcoin/bitcoin/pull/9114
< bitcoin-git> bitcoin/master 21e6c6b Wladimir J. van der Laan: Merge #9114: [depends] Set OSX_MIN_VERSION to 10.8...
< bitcoin-git> bitcoin/master 617c96d fanquake: [depends] Set OSX_MIN_VERSION to 10.8
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/71bc39eb7483...21e6c6b569c5
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #9129: One fDisconnect to rule them all (master...OnefDisconnect) https://github.com/bitcoin/bitcoin/pull/9129
< bitcoin-git> [bitcoin] rebroad opened pull request #9129: One fDisconnect to rule them all (master...OnefDisconnect) https://github.com/bitcoin/bitcoin/pull/9129
< bitcoin-git> [bitcoin] theuni opened pull request #9128: net: Decouple CConnman and message serialization (master...connman-send) https://github.com/bitcoin/bitcoin/pull/9128
< bitcoin-git> [bitcoin] sipa opened pull request #9125: Make CBlock a vector of shared_ptr of CTransactions (master...sharedblock) https://github.com/bitcoin/bitcoin/pull/9125

2016-11-10

< sipa> you're free to ask questions about Bitcoin and Bitcoin Core's development model here
< Lauda> Oops wrong chat, thought this was #bitcoin. Sorry.
< MarcoFalke> FreeUser: The "Bitcoin Core Developers" is just the short version of all the credits (which you can find in the release notes)
< FreeUser> I can see "Bitcoin Core Developers" while loading Bitcoin Core.
< Chris_Stewart_5> Damn! I paid that guy all my doge coin for a bitcoin core dev sticker..
< gmaxwell> Perhaps you're a Bitcoin Core dev, have you submitted a patch? :)
< FreeUser> Official means from Bitcoin Core devs
< gmaxwell> whats a Bitcoin Core devs? there is no such thing as membership.
< sipa> Knots is a fork of Bitcoin Core, maintained by different people
< FreeUser> Is Bitcoin Knots an official wallet?
< FreeUser> How to disable alerts in old Bitcoin Core versions?
< bitcoin-git> [bitcoin] paveljanik opened pull request #9124: Use better name for local variable to prevent -Wshadow compiler warning (master...20161110_Wshadow_benchcheckblock) https://github.com/bitcoin/bitcoin/pull/9124
< gribble> https://github.com/bitcoin/bitcoin/issues/9058 | Fixes for p2p-compactblocks.py test timeouts on travis (#8842) by ryanofsky · Pull Request #9058 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/9058 | Fixes for p2p-compactblocks.py test timeouts on travis (#8842) by ryanofsky · Pull Request #9058 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/8550 | [Qt] Add interactive mempool graph by jonasschnelli · Pull Request #8550 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/8501 | Add mempool statistics collector by jonasschnelli · Pull Request #8501 · bitcoin/bitcoin · GitHub
< jonasschnelli> petertodd: yes. 8501 can work without 8550 and has the most bitcoin-core wide impact.
< gribble> https://github.com/bitcoin/bitcoin/issues/8501 | Add mempool statistics collector by jonasschnelli · Pull Request #8501 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/8501 | Add mempool statistics collector by jonasschnelli · Pull Request #8501 · bitcoin/bitcoin · GitHub
< jonasschnelli> BlueMatt: yeah! Bumpfee should get some review. Guys! https://github.com/bitcoin/bitcoin/pull/8456
< gribble> https://github.com/bitcoin/bitcoin/issues/8776 | Wallet refactoring leading up to multiwallet by luke-jr · Pull Request #8776 · bitcoin/bitcoin · GitHub
< jonasschnelli> So.. theres a project: https://github.com/bitcoin/bitcoin/projects/2
< bitcoin-git> [bitcoin] morcos opened pull request #9123: Remove extraneous LogPrint from fee estimation (master...eliminateFeeWarning) https://github.com/bitcoin/bitcoin/pull/9123
< NicolasDorier> well bitcoin core has a wallet part
< sipa> bitcoin core has a perfectly good wallet
< btcdrak> Yeah I also think purespv seems wrong for Bitcoin Core.
< BlueMatt> I'm not sure if its worth the effort to make bitcoin core a competitive spv wallet - they mostly all already support connecting to a trusted node (though need auth)
<@wumpus> being able to use bitcoin core as a full SPV node would be useful, especially with a local node that does validate
< BlueMatt> wumpus: because if you're not gonna be upgrading to full trust model why are you running bitcoin core as a wallet?
< jonasschnelli> For those who haven seen it, there a full working PR for the hybrid SPV: https://github.com/bitcoin/bitcoin/pull/9076
< sipa> ok, "client mode" as bitcoin's creator called it then
<@wumpus> I think the idea has always been to have some kind of SPV mode in bitcoin core, yes
< jonasschnelli> I wanted to ask if we'd like to see some form of "SPV" (wrong term, i agree) mode in Bitcoin-Core, if it's worth to continue the work
<@wumpus> #bitcoin-core-dev Meeting: wumpus sipa gmaxwell jonasschnelli morcos luke-jr btcdrak sdaftuar jtimon cfields petertodd kanzure bluematt instagibbs phantomcircuit codeshark michagogo marcofalke paveljanik NicolasDorier jl2012
< gribble> https://github.com/bitcoin/bitcoin/issues/9075 | Decouple peer-processing-logic from block-connection-logic (#3) by TheBlueMatt · Pull Request #9075 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj closed pull request #9049: Remove duplicatable duplicate-input check from CheckTransaction (master...2016-10-bench-checkblock) https://github.com/bitcoin/bitcoin/pull/9049
< bitcoin-git> bitcoin/master e2b3fb3 Matt Corallo: Optimize vInOutPoints insertion a bit
< bitcoin-git> bitcoin/master eecffe5 Matt Corallo: Remove redundant duplicate-input check from CheckTransaction
< bitcoin-git> bitcoin/master b2e178a Matt Corallo: Add deserialize + CheckBlock benchmarks, and a full block hex
< bitcoin-git> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/e5364991daec...71bc39eb7483
< jonasschnelli> Bip151 encrypted message packaging (https://github.com/bitcoin/bips/blob/master/bip-0151.mediawiki#encrypted-messages-structure) allows that. Because from the protocol perspective, combining them seems to have no downside
< bitcoin-git> bitcoin/master e536499 MarcoFalke: Merge #9097: [qa] Rework sync_* and preciousblock.py...
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #9097: [qa] Rework sync_* and preciousblock.py (master...Mf1611-qaSyncAndPrecious) https://github.com/bitcoin/bitcoin/pull/9097
< bitcoin-git> bitcoin/master fac1141 MarcoFalke: [qa] preciousblock: Use assert_equal and BitcoinTestFramework.__init__
< bitcoin-git> bitcoin/master fa97ccb MarcoFalke: [qa] util: Rework sync_*()...
< bitcoin-git> [bitcoin] MarcoFalke pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/4a2b170c075c...e5364991daec
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #9122: fix getnettotals RPC description about timemillis. (master...fix-getnettotals-rpc-desc) https://github.com/bitcoin/bitcoin/pull/9122
< bitcoin-git> bitcoin/master 4a2b170 MarcoFalke: Merge #9122: fix getnettotals RPC description about timemillis....
< bitcoin-git> bitcoin/master a79f864 Masahiko Hyuga: fix getnettotals RPC description about timemillis.
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/aab102cbae6b...4a2b170c075c
< bitcoin-git> [bitcoin] laanwj closed pull request #9121: Initialize variable to prevent compiler warning (master...20161110_rpcdump_uninitialized) https://github.com/bitcoin/bitcoin/pull/9121
< bitcoin-git> bitcoin/master aab102c Wladimir J. van der Laan: Merge #9121: Initialize variable to prevent compiler warning...
< bitcoin-git> bitcoin/master bdcba6d Pavel Janík: Initialize variable to prevent compiler warning
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/537e0cb252a3...aab102cbae6b
< bitcoin-git> [bitcoin] laanwj closed pull request #9120: bug: Missed one "return false" in recent refactoring in #9067 (master...fixExitCodesBitcoinTx) https://github.com/bitcoin/bitcoin/pull/9120
< bitcoin-git> bitcoin/master 537e0cb Wladimir J. van der Laan: Merge #9120: bug: Missed one "return false" in recent refactoring in #9067...
< bitcoin-git> bitcoin/master 45d372f UdjinM6: Missed one "return false" in recent refactoring in #9067
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/faec09bc7f03...537e0cb252a3
< bitcoin-git> [bitcoin] visvirial opened pull request #9122: fix getnettotals RPC description about timemillis. (master...fix-getnettotals-rpc-desc) https://github.com/bitcoin/bitcoin/pull/9122
< bitcoin-git> [bitcoin] paveljanik opened pull request #9121: Initialize variable to prevent compiler warning (master...20161110_rpcdump_uninitialized) https://github.com/bitcoin/bitcoin/pull/9121
< gribble> https://github.com/bitcoin/bitcoin/issues/8895 | Better SigCache Implementation by JeremyRubin · Pull Request #8895 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/9075 | Decouple peer-processing-logic from block-connection-logic (#3) by TheBlueMatt · Pull Request #9075 · bitcoin/bitcoin · GitHub

2016-11-09

< gribble> https://github.com/bitcoin/bitcoin/issues/9058 | Fixes for p2p-compactblocks.py test timeouts on travis (#8842) by ryanofsky · Pull Request #9058 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] UdjinM6 opened pull request #9120: bug: Missed one "return false" in recent refactoring in #9067 (master...fixExitCodesBitcoinTx) https://github.com/bitcoin/bitcoin/pull/9120
< bitcoin-git> [bitcoin] laanwj closed pull request #9118: Remove requireGreater argment from TxConfirmStats::EstimateMedianVal() (master...remove-dead-code) https://github.com/bitcoin/bitcoin/pull/9118
< bitcoin-git> [bitcoin] laanwj closed pull request #9048: [0.13 backport] Fix handling of invalid compact blocks (0.13...fix-invalid-cb-ban-0.13) https://github.com/bitcoin/bitcoin/pull/9048
< bitcoin-git> [bitcoin] dooglus opened pull request #9118: Remove requireGreater argment from TxConfirmStats::EstimateMedianVal() (master...remove-dead-code) https://github.com/bitcoin/bitcoin/pull/9118
< bitcoin-git> bitcoin/master faec09b Wladimir J. van der Laan: Merge #8894: [Testing] Include fRelay in mininode version messages...
< bitcoin-git> bitcoin/master e5d682f jnewbery: Fix mininode version message format
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/fb156100f9b4...faec09bc7f03
< bitcoin-git> [bitcoin] laanwj closed pull request #9111: Remove unused variable UNLIKELY_PCT from fees.h (master...remove-unused-unlikely-pct) https://github.com/bitcoin/bitcoin/pull/9111
< bitcoin-git> bitcoin/master d8edf03 fanquake: Remove unused var UNLIKELY_PCT from fees.h
< bitcoin-git> bitcoin/master fb15610 Wladimir J. van der Laan: Merge #9111: Remove unused variable UNLIKELY_PCT from fees.h...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/f445d886126c...fb156100f9b4
< bitcoin-git> bitcoin/master f445d88 Wladimir J. van der Laan: Revert "Check and enable -Wshadow by default."...
< bitcoin-git> [bitcoin] laanwj pushed 1 new commit to master: https://github.com/bitcoin/bitcoin/commit/f445d886126c00814c71b855fc2f36fdd7e11098
<@wumpus> not just me *multiple people* tested it here: https://github.com/bitcoin/bitcoin/pull/8794
< gribble> https://github.com/bitcoin/bitcoin/issues/8808 | Do not shadow variables (gcc set) by paveljanik · Pull Request #8808 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/8808 | Do not shadow variables (gcc set) by paveljanik · Pull Request #8808 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/8808 | Do not shadow variables (gcc set) by paveljanik · Pull Request #8808 · bitcoin/bitcoin · GitHub
< Chris_Stewart_5> Is the CHECKSIG operation missing in BIP141 for P2WPKH & P2WPKH nested inside of P2SH? https://github.com/bitcoin/bips/blob/master/bip-0141.mediawiki#p2wpkh
< bitcoin-git> [bitcoin] theuni opened pull request #9117: net: don't send feefilter messages before the version handshake is complete (master...feefilter-assert) https://github.com/bitcoin/bitcoin/pull/9117
< MarcoFalke> I think I was hitting the assert in net.h https://github.com/bitcoin/bitcoin/blob/e9847303e708ab71b4d2c22ceb7d65c89615e18a/src/net.h#L772 ... I couldn't reproduce and debug.log only says ProcessMessages(version, 102 bytes) FAILED peer=10 http://pastebin.ubuntu.com/23451144/
< jonasschnelli> cfields: here I pass in Lambda https://github.com/bitcoin/bitcoin/pull/9076/files#diff-b2bb174788c7409b671c46ccc86034bdR3824 ... the function structure defines the parameter as std::function, does this always result in a copy for the lambda?
< bitcoin-git> [bitcoin] laanwj closed pull request #8794: Enable -Wshadow by default (master...20160922_Wshadow_enable) https://github.com/bitcoin/bitcoin/pull/8794
< bitcoin-git> bitcoin/master 359bac7 Pavel Janík: Add notes about variable names and shadowing
< bitcoin-git> bitcoin/master e0477f6 Wladimir J. van der Laan: Merge #8794: Enable -Wshadow by default...
< bitcoin-git> bitcoin/master fd5654c Pavel Janík: Check and enable -Wshadow by default.
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/e81df49644c2...e0477f6d2072
< bitcoin-git> [bitcoin] laanwj closed pull request #9039: Various serialization simplifcations and optimizations (master...simpleserial) https://github.com/bitcoin/bitcoin/pull/9039
< bitcoin-git> bitcoin/master fad9b66 Pieter Wuille: Make nType and nVersion private and sometimes const...
< bitcoin-git> bitcoin/master c2c5d42 Pieter Wuille: Make streams' read and write return void...
< bitcoin-git> bitcoin/master 50e8a9c Pieter Wuille: Remove unused ReadVersion and WriteVersion...
< bitcoin-git> [bitcoin] laanwj pushed 10 new commits to master: https://github.com/bitcoin/bitcoin/compare/e9847303e708...e81df49644c2
< bitcoin-git> [bitcoin] ondrejsika opened pull request #9116: Allow getblocktemlate for not connected regtest node (master...master) https://github.com/bitcoin/bitcoin/pull/9116
< bitcoin-git> [bitcoin] paveljanik opened pull request #9115: Mention reporting security issues responsibly (master...20161109_secissues) https://github.com/bitcoin/bitcoin/pull/9115
< bitcoin-git> [bitcoin] fanquake opened pull request #9114: [depends] Set OSX_MIN_VERSION to 10.8 (master...depends-min-osx-10-8) https://github.com/bitcoin/bitcoin/pull/9114
< bitcoin-git> [bitcoin] rebroad opened pull request #9113: Return the type when it's unknown instead of throwing an exception (master...ReturnNotThrow) https://github.com/bitcoin/bitcoin/pull/9113
< bitcoin-git> [bitcoin] laanwj opened pull request #9112: Avoid ugly exception in log on unknown inv type (master...2016_11_invtype_debugging) https://github.com/bitcoin/bitcoin/pull/9112
< bitcoin-git> bitcoin/master 1077577 Andrew Chow: Fix auto-deselection of peers
< bitcoin-git> bitcoin/master e984730 Wladimir J. van der Laan: Merge #8874: Multiple Selection for peer and ban tables...
< bitcoin-git> bitcoin/master addfdeb Andrew Chow: Multiple Selection for peer and ban tables...
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/924de0bd75a7...e9847303e708
< bitcoin-git> [bitcoin] fanquake opened pull request #9111: Remove unused variable UNLIKELY_PCT from fees.h (master...remove-unused-unlikely-pct) https://github.com/bitcoin/bitcoin/pull/9111
< gribble> https://github.com/bitcoin/bitcoin/issues/1625 | Bitcoin-Qt: exception in CInv::GetCommand() when using -onlynet="IPv6" · Issue #1625 · bitcoin/bitcoin · GitHub