2017-04-17

< bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/c5e9e428a919...a077a90da88f
< bitcoin-git> [bitcoin] sipa closed pull request #9693: Prevent integer overflow in ReadVarInt. (master...varint_maxsize) https://github.com/bitcoin/bitcoin/pull/9693
< bitcoin-git> bitcoin/master c5e9e42 Pieter Wuille: Merge #9693: Prevent integer overflow in ReadVarInt....
< bitcoin-git> bitcoin/master 45f0961 Gregory Maxwell: Prevent integer overflow in ReadVarInt....
< bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/f4db00f9a548...c5e9e428a919
< SopaXorzTaker> wumpus, sipa, the large bitcoin collider client script is untrustworthy
< bitcoin-git> [bitcoin] laanwj opened pull request #10220: Experiment: test: Disable fsync in travis tests (master...2017_04_tests_eatmydata) https://github.com/bitcoin/bitcoin/pull/10220
< bitcoin-git> [bitcoin] jimmysong opened pull request #10219: Tests: Order Python Tests By Duration (master...order_tests_by_duration) https://github.com/bitcoin/bitcoin/pull/10219

2017-04-16

< sipa> this is not a support channel; ask in #bitcoin or on stack exchange
< __0> got this issue https://github.com/bitcoin/bitcoin/issues/7970 what to do? apt-get install libdb-dev already did
< MarcoFalke> github removed all pulls from our leveldb repo after the transition. Maybe send them a follow up sipa? E.g. https://github.com/bitcoin-core/leveldb/issues/17
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #8319: [qa] wallet*.py: Check for salvagewallet regressions (master...Mf1607-qaSalv) https://github.com/bitcoin/bitcoin/pull/8319
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #10078: [qa] Fundraw: Use named args to limit scope of names (master...Mf1703-qaNamedShadow) https://github.com/bitcoin/bitcoin/pull/10078

2017-04-15

< jonasschnelli> <*highlight>[22:09:48] <BlueMatt:#bitcoin-core-dev> jonasschnelli: hey
<@wumpus> trying to compile bitcoin core with c++17 clang runs into some curious things: https://github.com/laanwj/bitcoin/commit/7a05ed0835fa962064a9c5198ab568109ab02d85 explicit bool coercion required at some places, as well as ?: needing to be exactly the right type *and* constness
< bitcoin-git> [bitcoin] NicolasDorier opened pull request #10216: [Wallet] FundRawTransaction can accept pre-set inputs whose parent is not yet broadcasted (master...fundrawtransaction2) https://github.com/bitcoin/bitcoin/pull/10216
< bitdroid> does bitcoin core have a android app?

2017-04-14

< bitcoin-git> [bitcoin] TheBlueMatt opened pull request #10215: Check interruptNet during dnsseed lookups (master...2017-04-dnsseed-break) https://github.com/bitcoin/bitcoin/pull/10215
< bitcoin-git> [bitcoin] KibbledJiveElkZoo closed pull request #10183: [RPC] Don't default rescan on private/public key imports. (master...rpc_rescan_fixes) https://github.com/bitcoin/bitcoin/pull/10183
< bitcoin-git> [bitcoin] rawodb closed pull request #10214: Abortable rescans & rpc abortrescan command (master...pr/rpc_abort_rescan) https://github.com/bitcoin/bitcoin/pull/10214
< bitcoin-git> [bitcoin] rawodb opened pull request #10214: Abortable rescans & rpc abortrescan command (master...pr/rpc_abort_rescan) https://github.com/bitcoin/bitcoin/pull/10214
< bitcoin-git> [bitcoin] practicalswift opened pull request #10212: Make sure parameter names in .cpp and .h files are in sync (master...make-doxygen-happy-by-using-consistent-parameter-names) https://github.com/bitcoin/bitcoin/pull/10212
< bitcoin-git> [bitcoin] kallewoof opened pull request #10211: [doc] Contributor fixes & new "finding reviewers" section (master...contributor-finding-reviewers) https://github.com/bitcoin/bitcoin/pull/10211
<@wumpus> the current form has quite some risks while it activates, which wouldn't really be our prefered way of working, see the mailing list for example gmaxwell's writeup here: https://lists.linuxfoundation.org/pipermail/bitcoin-dev/2017-April/014152.html
< bitcoin-git> bitcoin/0.14 33fadc2 Wladimir J. van der Laan: doc: Update release notes pre-rc2
< bitcoin-git> [bitcoin] laanwj pushed 1 new commit to 0.14: https://github.com/bitcoin/bitcoin/commit/33fadc20bae4828788d6d82c582c457adc6941e1
< bitcoin-git> bitcoin/0.14 348a717 Wladimir J. van der Laan: qt: translations update pre-rc2
< bitcoin-git> [bitcoin] laanwj pushed 1 new commit to 0.14: https://github.com/bitcoin/bitcoin/commit/348a71701db4cb29a762c7d60aba34380ee1d403
< bitcoin-git> [bitcoin] laanwj closed pull request #10205: [0.14] net: define NodeId as an int64_t (0.14...10176-backport) https://github.com/bitcoin/bitcoin/pull/10205
< bitcoin-git> bitcoin/0.14 30fa231 Cory Fields: net: define NodeId as an int64_t...
< bitcoin-git> bitcoin/0.14 3c79602 John Newbery: [rpc] rename disconnectnode argument...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to 0.14: https://github.com/bitcoin/bitcoin/compare/06909df179e7...30fa231011a6
< bitcoin-git> [bitcoin] laanwj closed pull request #10204: [rpc] rename disconnectnode argument (master...rename_disconnect_node_argument) https://github.com/bitcoin/bitcoin/pull/10204
< bitcoin-git> bitcoin/master f4db00f Wladimir J. van der Laan: Merge #10204: [rpc] rename disconnectnode argument...
< bitcoin-git> bitcoin/master 883154c John Newbery: [rpc] rename disconnectnode argument
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b7365f0545b1...f4db00f9a548
< jonasschnelli> Hmm.... I don't really understand why getaddrinfo on OSX does cache forever... https://github.com/bitcoin/bitcoin/blob/1a5aaabb8a3d67a039ad120bb5d8d418467cac4e/src/netbase.cpp#L104
< bitcoin-git> [bitcoin] jonasschnelli reopened pull request #10208: [wallet] Rescan abortability (master...rescan-abortability) https://github.com/bitcoin/bitcoin/pull/10208
< bitcoin-git> [bitcoin] jonasschnelli closed pull request #10208: [wallet] Rescan abortability (master...rescan-abortability) https://github.com/bitcoin/bitcoin/pull/10208
< bitcoin-git> [bitcoin] kallewoof opened pull request #10208: [wallet] Rescan abortability (master...rescan-abortability) https://github.com/bitcoin/bitcoin/pull/10208
< bitcoin-git> [bitcoin] kallewoof closed pull request #9872: [qa] Multi-chain support in test framework (master...qa-multi-chain-support) https://github.com/bitcoin/bitcoin/pull/9872
< kallewoof> I'm not sure. I talked to wumpus about it. See e.g. https://github.com/bitcoin/bitcoin/pull/9387#issuecomment-268213151
< kallewoof> cfields: This is the part, I was a bit off on the struct names, but: https://github.com/bitcoin/bitcoin/pull/9387/files#diff-321303fddcf725df060981d626a05df9R234
< gribble> https://github.com/bitcoin/bitcoin/issues/2 | Long-term, safe, store-of-value · Issue #2 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/9517 | [refactor] Switched httpserver.cpp to use RAII wrapped libevents. by kallewoof · Pull Request #9517 · bitcoin/bitcoin · GitHub
< gmaxwell> /home/gmaxwell/src/bitcoin/src/support/events.h:37: multiple definition of `obtain_event(event_base*, int, short, void (*)(int, short, void*), void*)'
< gmaxwell> test/test_test_bitcoin-raii_event_tests.o:/home/gmaxwell/src/bitcoin/src/./support/events.h:30: first defined here
< gribble> https://github.com/bitcoin/bitcoin/issues/9517 | [refactor] Switched httpserver.cpp to use RAII wrapped libevents. by kallewoof · Pull Request #9517 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] wtogami opened pull request #10207: Clarify importprivkey help text ... example of blank label without rescan (master...importprivkey) https://github.com/bitcoin/bitcoin/pull/10207

2017-04-13

< bitcoin-git> [bitcoin] dramaticbulgarian closed pull request #10206: Chainparams: In ReceivedBlockTransactions() (master...chainparams) https://github.com/bitcoin/bitcoin/pull/10206
< bitcoin-git> [bitcoin] dramaticbulgarian opened pull request #10206: Chainparams: In ReceivedBlockTransactions() (master...chainparams) https://github.com/bitcoin/bitcoin/pull/10206
< bitcoin-git> [bitcoin] theuni opened pull request #10205: net: define NodeId as an int64_t (0.14...10176-backport) https://github.com/bitcoin/bitcoin/pull/10205
< gribble> https://github.com/bitcoin/bitcoin/issues/8694 | Basic multiwallet support by luke-jr · Pull Request #8694 · bitcoin/bitcoin · GitHub
< BlueMatt> wumpus: i think you can remove the merged PRs from https://github.com/bitcoin/bitcoin/projects/8
< petertodd> spudowiar: I dropped out of a physics degree for bitcoin (though I was doing it part time while working at a startup...)
< gribble> https://github.com/bitcoin/bitcoin/issues/10044 | Run functional tests in `make check` by jnewbery · Pull Request #10044 · bitcoin/bitcoin · GitHub
<@wumpus> morcos: BlueMatt: ok added them to the project https://github.com/bitcoin/bitcoin/projects/8
< gribble> https://github.com/bitcoin/bitcoin/issues/9942 | Refactor CBlockPolicyEstimator by morcos · Pull Request #9942 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/9792 | FastRandomContext improvements and switch to ChaCha20 by sipa · Pull Request #9792 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10179 | Give CValidationInterface Support for calling notifications on the CScheduler Thread by TheBlueMatt · Pull Request #10179 · bitcoin/bitcoin · GitHub
< ryanofsky> and qt seemed like it might be a less controversial place to start since we already have separate bitcoind and bitcoin-qt executables
< gribble> https://github.com/bitcoin/bitcoin/issues/10102 | bitcoin-qt: spawn bitcoind and communicate over pipe (Experimental, WIP) by ryanofsky · Pull Request #10102 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10102 | bitcoin-qt: spawn bitcoind and communicate over pipe (Experimental, WIP) by ryanofsky · Pull Request #10102 · bitcoin/bitcoin · GitHub
< gmaxwell> /mode #bitcoin-dev +b *!*@178.62.68.75
< gribble> https://github.com/bitcoin/bitcoin/issues/10143 | [net] Allow disconnectnode RPC to be called with node id by jnewbery · Pull Request #10143 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] jnewbery opened pull request #10204: [rpc] rename disconnectnode argument (master...rename_disconnect_node_argument) https://github.com/bitcoin/bitcoin/pull/10204
< gribble> https://github.com/bitcoin/bitcoin/issues/10179 | Give CValidationInterface Support for calling notifications on the CScheduler Thread by TheBlueMatt · Pull Request #10179 · bitcoin/bitcoin · GitHub
< BlueMatt> and bitcoin core is a big part of that
<@wumpus> #link Release schedule for 0.15.0 https://github.com/bitcoin/bitcoin/issues/9961
< gribble> https://github.com/bitcoin/bitcoin/issues/10189 | devtools/net: add a verifier for scriptable changes. Use it to make CNode::id private. by theuni · Pull Request #10189 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10193 | scripted-diff: Remove #include foreach.hpp> by jtimon · Pull Request #10193 · bitcoin bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10193 | scripted-diff: Remove #include foreach.hpp> by jtimon · Pull Request #10193 · bitcoin bitcoin · GitHub
< petertodd> wumpus: granted, if you ever compile bitcoin you're running stuff within the repo anyway
< gribble> https://github.com/bitcoin/bitcoin/issues/10189 | devtools/net: add a verifier for scriptable changes. Use it to make CNode::id private. by theuni · Pull Request #10189 · bitcoin/bitcoin · GitHub
< spudowiar> 2017-04-11 21:37:35 -- Mode #bitcoin [+b *!*@178.62.68.75] by gmaxwell
< wumpus> #bitcoin-core-dev Meeting: wumpus sipa gmaxwell jonasschnelli morcos luke-jr btcdrak sdaftuar jtimon cfields petertodd kanzure blue
< cfields_> sipa: for leveldb merge, whenever it's needed: https://github.com/theuni/bitcoin/commit/2cb7dda13884e44105f33c16e7e2c1a9aed46295
< bitcoin-git> [bitcoin] sipa closed pull request #9480: De-duplicate SignatureCacheHasher (master...refactor-signaturecachehasher-visibility) https://github.com/bitcoin/bitcoin/pull/9480
< bitcoin-git> bitcoin/master b7365f0 Pieter Wuille: Merge #9480: De-duplicate SignatureCacheHasher...
< bitcoin-git> bitcoin/master f9c8807 Jeremy Rubin: Deduplicate SignatureCacheHasher...
< bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/eab00d96dfe9...b7365f0545b1
< gribble> https://github.com/bitcoin/bitcoin/issues/9743 | Fix several potential issues found by sanitizers by sipa · Pull Request #9743 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/9480 | De-duplicate SignatureCacheHasher by JeremyRubin · Pull Request #9480 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/9942 | Refactor CBlockPolicyEstimator by morcos · Pull Request #9942 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj closed pull request #9665: Use cached [compact] blocks to respond to getdata messages (master...2017-02-processgetdata-cache) https://github.com/bitcoin/bitcoin/pull/9665
< bitcoin-git> bitcoin/master b49ad44 Matt Corallo: Add comment about cs_most_recent_block coverage
< bitcoin-git> bitcoin/master c47f5b7 Matt Corallo: Cache witness-enabled state with recent-compact-block-cache
< bitcoin-git> bitcoin/master efc135f Matt Corallo: Use cached [compact] blocks to respond to getdata messages
< bitcoin-git> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/cf8a8b1028d6...eab00d96dfe9
< gribble> https://github.com/bitcoin/bitcoin/issues/9665 | Use cached [compact] blocks to respond to getdata messages by TheBlueMatt · Pull Request #9665 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj closed pull request #10176: net: gracefully handle NodeId wrapping (master...nodeid-no-wrap) https://github.com/bitcoin/bitcoin/pull/10176
< bitcoin-git> bitcoin/master cf8a8b1 Wladimir J. van der Laan: Merge #10176: net: gracefully handle NodeId wrapping...
< bitcoin-git> bitcoin/master c851be4 Cory Fields: net: define NodeId as an int64_t...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/70f6f56e9dde...cf8a8b1028d6
< sipa> wumpus: it's annoying to create PRs against the bitcoin core leveldb repository
< bitcoin-git> [bitcoin] laanwj closed pull request #7148: [NO MERGE] Travis: Run nightly test suite (master...travis/nightly) https://github.com/bitcoin/bitcoin/pull/7148
< gribble> https://github.com/bitcoin/bitcoin/issues/7289 | [WIP] Make arguments reconfigurable at runtime via RPC by luke-jr · Pull Request #7289 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/7522 | Bugfix: Only use git for build info if the repository is actually the right one by luke-jr · Pull Request #7522 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] NicolasDorier opened pull request #10202: [Wallet] FundRawTransaction can fund a transaction with preset inputs found in the CoinView (master...fundrawtransaction) https://github.com/bitcoin/bitcoin/pull/10202
< bitcoin-git> [bitcoin] NicolasDorier closed pull request #10068: [Wallet] FundRawTransaction accepts preset non-wallet inputs (master...nonwalletinputs) https://github.com/bitcoin/bitcoin/pull/10068
< bitcoin-git> [bitcoin] mariodian opened pull request #10201: pass Consensus::Params& to ReceivedBlockTransactions() (master...consensusparams-receivedblocktransactions) https://github.com/bitcoin/bitcoin/pull/10201
< bitcoin-git> [bitcoin] laanwj closed pull request #10165: [Wallet] Refactoring by using CInputCoin instead of std::pair (master...inputcoin) https://github.com/bitcoin/bitcoin/pull/10165
< bitcoin-git> bitcoin/master f597dcb NicolasDorier: [Wallet] Simplify code using CInputCoin
< bitcoin-git> bitcoin/master e78bc45 NicolasDorier: [Wallet] Decouple CInputCoin from CWalletTx
< bitcoin-git> bitcoin/master fd44ac1 NicolasDorier: [Wallet] Rename std::pair<const CWalletTx*, unsigned int> to CInputCoin
< bitcoin-git> [bitcoin] laanwj pushed 5 new commits to master: https://github.com/bitcoin/bitcoin/compare/c9ff4f8ee601...70f6f56e9dde
< bitcoin-git> bitcoin/0.14 06909df Wladimir J. van der Laan: Merge #10185: [0.14] Mention dbcache memory changes in release notes...
< bitcoin-git> bitcoin/0.14 b7caa30 Suhas Daftuar: Mention dbcache memory changes in 0.14.1 release notes
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to 0.14: https://github.com/bitcoin/bitcoin/compare/55f641ca194b...06909df179e7
< bitcoin-git> [bitcoin] laanwj closed pull request #10186: Remove SYNC_TRANSACTION_NOT_IN_BLOCK magic number (master...remove_SYNC_TRANSACTION_NOT_IN_BLOCK_magic_number) https://github.com/bitcoin/bitcoin/pull/10186
< bitcoin-git> bitcoin/master c9ff4f8 Wladimir J. van der Laan: Merge #10186: Remove SYNC_TRANSACTION_NOT_IN_BLOCK magic number...
< bitcoin-git> bitcoin/master d0cd0bd John Newbery: Make CWallet::SyncTransactions() interface friendlier
< bitcoin-git> bitcoin/master 714e4ad John Newbery: AddToWalletIfInvolvingMe should test pIndex, not posInBlock
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/de01da7cad32...c9ff4f8ee601
< gribble> https://github.com/bitcoin/bitcoin/issues/10185 | [0.14] Mention dbcache memory changes in release notes by sdaftuar · Pull Request #10185 · bitcoin/bitcoin · GitHub
< sdaftuar> gmaxwell: is this what you have in mind? https://github.com/sdaftuar/bitcoin/tree/2017-04-braindead-simple-mining
< jtimon> sipa: I'm getting some errors related to prevector templating, perhaps you can help with https://github.com/bitcoin/bitcoin/pull/10193#issuecomment-293742166

2017-04-12

< berndj> gmaxwell, i think i started this line of thinking in #bitcoin, my version was only slightly different, where the coins become redeemable by anyone (i.e. probably a miner) if SW should fail to be active
< jtimon> yep, look at https://github.com/bitcoin/bitcoin/pull/10193 for an example (needs some squashing after completing it)
< cfields> BlueMatt: I don't see what's so terrible about automating this checking: https://github.com/bitcoin/bitcoin/pull/9902/commits/bac5c9cf643e9333479ac667426d0b70f8f3aa7f
< gribble> https://github.com/bitcoin/bitcoin/issues/10189 | devtools/net: add a verifier for scriptable changes. Use it to make CNode::id private. by theuni · Pull Request #10189 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] sdaftuar opened pull request #10200: Mining: Skip recent transactions if fee difference is small (master...2017-04-dont-mine-recent-tx) https://github.com/bitcoin/bitcoin/pull/10200
< gribble> https://github.com/bitcoin/bitcoin/issues/9480 | De-duplicate SignatureCacheHasher by JeremyRubin · Pull Request #9480 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/9942 | Refactor CBlockPolicyEstimator by morcos · Pull Request #9942 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] bulldozer00 closed pull request #10194: Remove unecessary friend keyword from the class definition (master...patch-1) https://github.com/bitcoin/bitcoin/pull/10194
< bitcoin-git> [bitcoin] laanwj closed pull request #10177: Changed "Send" button default status from true to false (master...ui_fixes) https://github.com/bitcoin/bitcoin/pull/10177
< bitcoin-git> bitcoin/master de01da7 Wladimir J. van der Laan: Merge #10177: Changed "Send" button default status from true to false...
< bitcoin-git> bitcoin/master 8c3e6c6 KibbledJiveElkZoo: Changed "Send" button default status from true to false...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/350b22497c7c...de01da7cad32
< bitcoin-git> [bitcoin] laanwj closed pull request #10150: [rpc] Add logging rpc (master...logging_rpc) https://github.com/bitcoin/bitcoin/pull/10150
< bitcoin-git> bitcoin/master 7fd50c3 John Newbery: allow libevent logging to be updated during runtime
< bitcoin-git> bitcoin/master 5255aca John Newbery: [rpc] Add logging RPC...
< bitcoin-git> bitcoin/master 4d9950d John Newbery: Set BCLog::LIBEVENT correctly for old libevent versions.
< bitcoin-git> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/b44adf92342a...350b22497c7c
< Chris_Stewart_5> arubi: Look at this function if you haven't seen it yet: https://github.com/bitcoin/bitcoin/blob/b83264d9c7a8ddb79f64bd9540caddc8632ef31f/src/script/interpreter.cpp#L186
< gribble> https://github.com/bitcoin/bitcoin/issues/10199 | Better fee estimates by morcos · Pull Request #10199 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] morcos opened pull request #10199: Better fee estimates (master...smarterfee) https://github.com/bitcoin/bitcoin/pull/10199
< Chris_Stewart_5> arubi: Like you mentioned you can tinker with the verify flags: https://github.com/bitcoin/bitcoin/blob/master/src/policy/policy.h#L52
< bitcoin-git> [bitcoin] jnewbery closed pull request #10198: [tests] Remove is_network_split from functional test framework (master...remove_is_network_split) https://github.com/bitcoin/bitcoin/pull/10198
< bitcoin-git> [bitcoin] jnewbery opened pull request #10198: [tests] Remove is_network_split from functional test framework (master...remove_is_network_split) https://github.com/bitcoin/bitcoin/pull/10198
< bitcoin-git> [bitcoin] jnewbery opened pull request #10197: Functional test warnings (master...functional_test_warnings) https://github.com/bitcoin/bitcoin/pull/10197
< bitcoin-git> [bitcoin] sdaftuar opened pull request #10196: Bugfix: PrioritiseTransaction updates the mempool tx counter (master...2017-04-prioritise-transaction) https://github.com/bitcoin/bitcoin/pull/10196
< bitcoin-git> [bitcoin] jnewbery reopened pull request #10191: [trivial] Remove unused submit block parameters argument (master...remove_submit_block_params) https://github.com/bitcoin/bitcoin/pull/10191
< bitcoin-git> [bitcoin] sipa opened pull request #10195: Switch chainstate db and cache to per-txout model (master...pertxoutcache) https://github.com/bitcoin/bitcoin/pull/10195
< bitcoin-git> [bitcoin] bulldozer00 opened pull request #10194: Remove unecessary friend keyword from the class definition (master...patch-1) https://github.com/bitcoin/bitcoin/pull/10194
< gribble> https://github.com/bitcoin/bitcoin/issues/8694 | Basic multiwallet support by luke-jr · Pull Request #8694 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/9694 | Importmulti cannot import bare private keys · Issue #9694 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] jtimon opened pull request #10193: scripted-diff: sed -i 's/BOOST_FOREACH(\(.*\),/for (\1 :/' ./src/qt/*.cpp (master...b14-10189-scripted-qt-foreach) https://github.com/bitcoin/bitcoin/pull/10193
< gribble> https://github.com/bitcoin/bitcoin/issues/3 | Encrypt wallet · Issue #3 · bitcoin/bitcoin · GitHub

2017-04-11

< gribble> https://github.com/bitcoin/bitcoin/issues/10189 | devtools/net: add a verifier for scriptable changes. Use it to make CNode::id private. by theuni · Pull Request #10189 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #10187: tests: Fix test_runner return value in case of skipped test (master...2017_04_fix_tracis) https://github.com/bitcoin/bitcoin/pull/10187
< bitcoin-git> bitcoin/master b44adf9 MarcoFalke: Merge #10187: tests: Fix test_runner return value in case of skipped test...
< bitcoin-git> bitcoin/master e96462f Wladimir J. van der Laan: tests: Fix test_runner return value in case of skipped test...
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/67023e9004ba...b44adf92342a
< bitcoin-git> [bitcoin] TheBlueMatt closed pull request #10125: Exit bitcoind immediately on shutdown instead of 200ms later (master...2017-03-faster-shutdown) https://github.com/bitcoin/bitcoin/pull/10125
< bitcoin-git> [bitcoin] TheBlueMatt opened pull request #10192: Cache full script execution results in addition to signatures (master...2017-04-cache-scriptchecks) https://github.com/bitcoin/bitcoin/pull/10192
< bitcoin-git> [bitcoin] jnewbery closed pull request #10191: [trivial] Remove unused submit block parameters argument (master...remove_submit_block_params) https://github.com/bitcoin/bitcoin/pull/10191
< bitcoin-git> [bitcoin] jnewbery opened pull request #10191: [trivial] Remove unused submit block parameters argument (master...remove_submit_block_params) https://github.com/bitcoin/bitcoin/pull/10191
< gribble> https://github.com/bitcoin/bitcoin/issues/10189 | devtools/net: add a verifier for scriptable changes. Use it to make CNode::id private. by theuni · Pull Request #10189 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] jnewbery opened pull request #10190: Mining functional tests (including regression test for submitblock) (master...mining_functional_test) https://github.com/bitcoin/bitcoin/pull/10190
< bitcoin-git> [bitcoin] theuni opened pull request #10189: devtools/net: add a verifier for scriptable changes. Use it to make CNode::id private. (master...private-nodeid) https://github.com/bitcoin/bitcoin/pull/10189
< gribble> https://github.com/bitcoin/bitcoin/issues/10148 | Use non-atomic flushing with block replay by sipa · Pull Request #10148 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj opened pull request #10187: tests: Fix test_runner return value in case of skipped test (master...2017_04_fix_tracis) https://github.com/bitcoin/bitcoin/pull/10187
< wumpus> https://travis-ci.org/bitcoin/bitcoin/jobs/220971234 shows as a failed build, but the build, the unit tests and the functional tests seem to pass
< bitcoin-git> [bitcoin] jnewbery opened pull request #10186: Remove SYNC_TRANSACTION_NOT_IN_BLOCK magic number (master...remove_SYNC_TRANSACTION_NOT_IN_BLOCK_magic_number) https://github.com/bitcoin/bitcoin/pull/10186
< gribble> https://github.com/bitcoin/bitcoin/issues/10182 | [scheduler] Switched CScheduler to C++11 threading primitives by tjps · Pull Request #10182 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] sdaftuar opened pull request #10185: [0.14] Mention dbcache memory changes in release notes (0.14...2017-04-relnotes-0.14.1) https://github.com/bitcoin/bitcoin/pull/10185
< bitcoin-git> [bitcoin] NicolasDorier opened pull request #10184: [Wallet] Worst case performance improvement on KeyPool filtering (master...hdinternalperf) https://github.com/bitcoin/bitcoin/pull/10184
< bitcoin-git> [bitcoin] KibbledJiveElkZoo opened pull request #10183: Don't default rescan on private/public key imports. (master...rpc_rescan_fixes) https://github.com/bitcoin/bitcoin/pull/10183
< jcorgan> as well as encryption using a key one can control, but with bitcoin traffic that seems less than essential
< bitcoin-git> [bitcoin] tjps opened pull request #10182: [scheduler] Switched CScheduler to C++11 threading primitives (master...tjps_scheduler) https://github.com/bitcoin/bitcoin/pull/10182
< bitcoin-git> [bitcoin] jtimon opened pull request #10181: Include cleanup (master...2017-04-10-includes) https://github.com/bitcoin/bitcoin/pull/10181

2017-04-10

< gribble> https://github.com/bitcoin/bitcoin/issues/10178 | Remove CValidationInterface::UpdatedTransaction by TheBlueMatt · Pull Request #10178 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] practicalswift opened pull request #10180: [trivial] Fix typos (tempoarily → temporarily, inadvertantly → inadvertently) (master...inadvertantly-and-tempoarily) https://github.com/bitcoin/bitcoin/pull/10180
< bitcoin-git> [bitcoin] TheBlueMatt opened pull request #10179: Give CValidationInterface Support for calling notifications on the CScheduler Thread (master...2017-01-wallet-cache-inmempool-3) https://github.com/bitcoin/bitcoin/pull/10179
< bitcoin-git> [bitcoin] laanwj closed pull request #9725: CValidationInterface Cleanups (master...2017-01-wallet-cache-inmempool-1) https://github.com/bitcoin/bitcoin/pull/9725
< bitcoin-git> bitcoin/master 29e6e23 Matt Corallo: Make ConnectTrace::blocksConnected private, hide behind accessors
< bitcoin-git> bitcoin/master 822000c Matt Corallo: Add pblock to connectTrace at the end of ConnectTip, not start...
< bitcoin-git> bitcoin/master f5e9a01 Matt Corallo: Include missing #include in zmqnotificationinterface.h
< bitcoin-git> [bitcoin] laanwj pushed 13 new commits to master: https://github.com/bitcoin/bitcoin/compare/e183ea2047b3...67023e9004ba
< bitcoin-git> [bitcoin] TheBlueMatt opened pull request #10178: Remove CValidationInterface::UpdatedTransaction (master...2017-01-wallet-cache-inmempool-2) https://github.com/bitcoin/bitcoin/pull/10178
< gribble> https://github.com/bitcoin/bitcoin/issues/10176 | net: gracefully handle NodeId wrapping by theuni · Pull Request #10176 · bitcoin/bitcoin · GitHub
< jtimon> weird, for some reason travis didn't checked https://github.com/bitcoin/bitcoin/pull/9494 after last rebase
< bitcoin-git> [bitcoin] KibbledJiveElkZoo opened pull request #10177: Changed "Send" button default status from true to false (master...ui_fixes) https://github.com/bitcoin/bitcoin/pull/10177
< bitcoin-git> [bitcoin] theuni opened pull request #10176: net: gracefully handle NodeId wrapping (master...nodeid-no-wrap) https://github.com/bitcoin/bitcoin/pull/10176
< bitcoin-git> [bitcoin] mrwhythat closed pull request #10161: Support for Tor's Single Onion Service (master...tor-single-onion-service) https://github.com/bitcoin/bitcoin/pull/10161
< bitcoin-git> [bitcoin] laanwj closed pull request #10164: Wallet: reduce excess logic InMempool() (master...patch-1) https://github.com/bitcoin/bitcoin/pull/10164
< bitcoin-git> bitcoin/master 3491476 Kewde: Wallet: reduce excess logic InMemPool()...
< bitcoin-git> bitcoin/master e183ea2 Wladimir J. van der Laan: Merge #10164: Wallet: reduce excess logic InMempool()...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/e19586a8a955...e183ea2047b3
< bitcoin-git> [bitcoin] laanwj closed pull request #10135: [p2p] Send the correct error code in reject messages (master...correctrejectmessages) https://github.com/bitcoin/bitcoin/pull/10135
< bitcoin-git> bitcoin/master e19586a Wladimir J. van der Laan: Merge #10135: [p2p] Send the correct error code in reject messages...
< bitcoin-git> bitcoin/master 5d08c9c John Newbery: Send the correct error code in reject messages
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/a27dbc55b1aa...e19586a8a955
< bitcoin-git> [bitcoin] laanwj closed pull request #9949: [bench] Avoid function call arguments which are pointers to uninitialized values (master...avoid-pointers-to-unitialized-values-in-function-calls) https://github.com/bitcoin/bitcoin/pull/9949
< bitcoin-git> bitcoin/master a27dbc5 Wladimir J. van der Laan: Merge #9949: [bench] Avoid function call arguments which are pointers to uninitialized values...
< bitcoin-git> bitcoin/master 218d915 practicalswift: [bench] Avoid function call arguments which are pointers to uninitialized values
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/51833a17347f...a27dbc55b1aa
< bitcoin-git> [bitcoin] laanwj closed pull request #10156: Fix for issues with startup and multiple monitors on windows. (master...masterMultiMonFix) https://github.com/bitcoin/bitcoin/pull/10156
< bitcoin-git> bitcoin/master 51833a1 Wladimir J. van der Laan: Merge #10156: Fix for issues with startup and multiple monitors on windows....
< bitcoin-git> bitcoin/master e9ff818 Allan Doensen: Fix for issues with startup and multiple monitors on windows.
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/1fa4ae67a330...51833a17347f
< bitcoin-git> [bitcoin] jonasschnelli closed pull request #9890: Add a button to open the config file in a text editor (master...master) https://github.com/bitcoin/bitcoin/pull/9890
< bitcoin-git> bitcoin/master 1fa4ae6 Jonas Schnelli: Merge #9890: Add a button to open the config file in a text editor...
< bitcoin-git> bitcoin/master 9ab9e7d Eric Shaw Jr: Add a button to open the config file in a text editor
< bitcoin-git> [bitcoin] jonasschnelli pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/64962aecb6be...1fa4ae67a330
< wumpus> midnightmagic: yes, the github info is being mirrored https://github.com/zw/bitcoin-gh-meta
< bitcoin-git> [bitcoin] practicalswift opened pull request #10175: Remove excess logic. (master...excess-logic) https://github.com/bitcoin/bitcoin/pull/10175
< bitcoin-git> [bitcoin] laanwj closed pull request #10166: Ignore Doxyfile generated from Doxyfile.in template. (master...20170407_gitignoreDoxyfile) https://github.com/bitcoin/bitcoin/pull/10166
< bitcoin-git> bitcoin/master 64962ae Wladimir J. van der Laan: Merge #10166: Ignore Doxyfile generated from Doxyfile.in template....
< bitcoin-git> bitcoin/master 9eaf718 Pavel Janík: Ignore Doxyfile generated from Doxyfile.in template.
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/813eacf81b73...64962aecb6be
< bitcoin-git> [bitcoin] laanwj closed pull request #10167: [trivial] Fix typo ("improssible" → "impossible") (master...improssible) https://github.com/bitcoin/bitcoin/pull/10167
< bitcoin-git> bitcoin/master 813eacf Wladimir J. van der Laan: Merge #10167: [trivial] Fix typo ("improssible" → "impossible")...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/ed09dd3f5a87...813eacf81b73
< bitcoin-git> bitcoin/master bcca57e practicalswift: [trivial] Fix typo ("improssible" → "impossible")
< wumpus> though it's possible to at least view the logs inthe console: https://github.com/bitcoin/bitcoin/pull/10159#issuecomment-292102755 however to get the identifiers you still need to visit the website
< bitcoin-git> [bitcoin] laanwj closed pull request #10142: Run bitcoin_test-qt under minimal QPA platform (master...pr/qtmin) https://github.com/bitcoin/bitcoin/pull/10142
< bitcoin-git> bitcoin/master ed09dd3 Wladimir J. van der Laan: Merge #10142: Run bitcoin_test-qt under minimal QPA platform...
< bitcoin-git> bitcoin/master bf10264 Russell Yanofsky: Run bitcoin_test-qt under minimal QPA platform...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/703202180bde...ed09dd3f5a87
< bitcoin-git> [bitcoin] laanwj closed pull request #10168: Fix build warning from #error text (master...fixbuildwarning) https://github.com/bitcoin/bitcoin/pull/10168
< bitcoin-git> bitcoin/master 7032021 Wladimir J. van der Laan: Merge #10168: Fix build warning from #error text...
< bitcoin-git> bitcoin/master dd7e43e John Newbery: Fix build warning from #error text
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/88799ea1b1c0...703202180bde

2017-04-09

< bitcoin-git> [bitcoin] sipa closed pull request #9599: [WIP] Perform validation in a separate thread to message handling. (master...ValidationThread) https://github.com/bitcoin/bitcoin/pull/9599
< bitcoin-git> [bitcoin] sipa closed pull request #8087: Introduce CBlockchain and move CheckBlockHeader (master...2016-05-22-cblockchain) https://github.com/bitcoin/bitcoin/pull/8087

2017-04-08

< gribble> https://github.com/bitcoin/bitcoin/issues/8263 | Qt 4.x EOL · Issue #8263 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] awemany opened pull request #10172: Fix opt-in RBF reliance on compiler integer size (master...rbf-numlimits-fix) https://github.com/bitcoin/bitcoin/pull/10172
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #10124: [test] Suppress test logging spam (master...suppress_test_logging_spam) https://github.com/bitcoin/bitcoin/pull/10124
< bitcoin-git> bitcoin/master 88799ea MarcoFalke: Merge #10124: [test] Suppress test logging spam...
< bitcoin-git> bitcoin/master 45ce471 John Newbery: Reduce spammy test logging...
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/df1ca9e93a61...88799ea1b1c0
< bitcoin-git> [bitcoin] jnewbery opened pull request #10171: [tests] Add node methods to test framework (master...add_node_methods_to_test_framework) https://github.com/bitcoin/bitcoin/pull/10171
< bitcoin-git> [bitcoin] jnewbery closed pull request #10111: [WIP] Remove NetworkThread from individual test cases (master...remove_network_thread_from_test_scripts) https://github.com/bitcoin/bitcoin/pull/10111

2017-04-07

< warren> https://github.com/bitcoin/bitcoin/issues/10170 "Normalized, Deterministic OSX SDK tarball" This is an opportunity for non-C++ developers to contribute to Bitcoin Core development.
< bitcoin-git> [bitcoin] jnewbery opened pull request #10169: [tests] Remove func test code duplication (master...remove_func_test_code_duplication) https://github.com/bitcoin/bitcoin/pull/10169
< bitcoin-git> [bitcoin] jnewbery closed pull request #10160: [WIP] updatepeer RPC (master...updatepeer) https://github.com/bitcoin/bitcoin/pull/10160
< gribble> https://github.com/bitcoin/bitcoin/issues/9942 | Refactor CBlockPolicyEstimator by morcos · Pull Request #9942 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] jnewbery opened pull request #10168: Fix build warning from #error text (master...fixbuildwarning) https://github.com/bitcoin/bitcoin/pull/10168
< jnewbery> looks like it's happening on travis too: https://travis-ci.org/bitcoin/bitcoin/jobs/219669340
< jnewbery> #error Client version information missing: wasn't defined by bitcoin-config.h nor defined any other way
< gribble> https://github.com/bitcoin/bitcoin/issues/10155 | build: Deduplicate version numbers by laanwj · Pull Request #10155 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] practicalswift opened pull request #10167: [trivial] Fix typo ("improssible" → "impossible") (master...improssible) https://github.com/bitcoin/bitcoin/pull/10167
< bitcoin-git> [bitcoin] paveljanik opened pull request #10166: Ignore Doxyfile generated from Doxyfile.in template. (master...20170407_gitignoreDoxyfile) https://github.com/bitcoin/bitcoin/pull/10166
< bitcoin-git> [bitcoin] laanwj closed pull request #10159: [tests] color test results and sort alphabetically (master...sort_test_results) https://github.com/bitcoin/bitcoin/pull/10159
< bitcoin-git> bitcoin/master d80baaa John Newbery: fixup - align summary row correctly and make colors/glyphs globals
< bitcoin-git> bitcoin/master bb92d83 John Newbery: [tests] Add unicode symbols for tests passing/failing/skipping
< bitcoin-git> bitcoin/master 63062bd John Newbery: [tests] color test results and sort alphabetically
< bitcoin-git> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/a5fd74667459...df1ca9e93a61
< karelb> ....oh, I should have asked this in #bitcoin , right
< bitcoin-git> [bitcoin] laanwj closed pull request #9681: Refactor Bumpfee, move core functionality to CWallet (master...2017/02/qt_bump) https://github.com/bitcoin/bitcoin/pull/9681
< bitcoin-git> bitcoin/master 2718db0 Jonas Schnelli: Restore invalid fee check (must be > 0)
< bitcoin-git> bitcoin/master 0337a39 Jonas Schnelli: Refactor Bumpfee core functionality
< bitcoin-git> bitcoin/master d1a95e8 Jonas Schnelli: Bumpfee move request parameter interaction to the top
< bitcoin-git> [bitcoin] laanwj pushed 10 new commits to master: https://github.com/bitcoin/bitcoin/compare/928695bee67e...a5fd74667459
< wumpus> btw: as bitcoin.org migrated to a new server, I'm waiting for things to be in order to upload 0.14.1rc1 binaries, this is why it's taking longer than normal
< bitcoin-git> [bitcoin] laanwj closed pull request #10155: build: Deduplicate version numbers (master...2017_03_version_bump) https://github.com/bitcoin/bitcoin/pull/10155
< bitcoin-git> bitcoin/master 08d9aee Wladimir J. van der Laan: build: Remove duplicate version information from src/clientversion.h...
< bitcoin-git> bitcoin/master 168a703 Wladimir J. van der Laan: doc: Make build system insert version in Doxyfile
< bitcoin-git> bitcoin/master b67eb8d Wladimir J. van der Laan: doc: Remove version numbers from READMEs...
< bitcoin-git> [bitcoin] laanwj pushed 5 new commits to master: https://github.com/bitcoin/bitcoin/compare/a3a2160b6650...928695bee67e
< gribble> https://github.com/bitcoin/bitcoin/issues/9681 | Refactor Bumpfee, move core functionality to CWallet by jonasschnelli · Pull Request #9681 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] jonasschnelli closed pull request #10043: Fix bug in CKey DER encoding (master...2017/03/fix_der_comp) https://github.com/bitcoin/bitcoin/pull/10043
< bitcoin-git> [bitcoin] laanwj closed pull request #10162: [trivial] Log calls to getblocktemplate (master...loggetblocktemplatecalls) https://github.com/bitcoin/bitcoin/pull/10162
< bitcoin-git> bitcoin/master a3a2160 Wladimir J. van der Laan: Merge #10162: [trivial] Log calls to getblocktemplate...
< bitcoin-git> bitcoin/master 1352092 John Newbery: Log calls to getblocktemplate
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/8c28670e92b6...a3a2160b6650
< bitcoin-git> [bitcoin] NicolasDorier opened pull request #10165: [Wallet] Refactoring by using CInputCoin instead of std::pair (master...inputcoin) https://github.com/bitcoin/bitcoin/pull/10165
< NicolasDorier> I rebased https://github.com/bitcoin/bitcoin/pull/9728 I would be happy if Hardware wallets people can review. I ate my own dog food, and it seems this PR would really help helping bitcoin core wore seemlessly with hardware wallet and normal wallet without code modification.
< bitcoin-git> [bitcoin] kallewoof closed pull request #9686: [WIP] RPC help tabulation (master...rpc-help-tabulation) https://github.com/bitcoin/bitcoin/pull/9686
< bitcoin-git> [bitcoin] kewde opened pull request #10164: Wallet: reduce excess logic InMemPool() (master...patch-1) https://github.com/bitcoin/bitcoin/pull/10164

2017-04-06

< jcorgan> patience is a virtue well-honed by participation in the bitcoin world :-)
< sturles> An UASF would need support from a supermajority of exchanges and payment processors, and preferably as many merchants as possible dealing with bitcoin directly.
< abpa> It's really for #bitcoin discussion not #bitcoin-core-dev
< jcorgan> but perhaps this particular thing should stay in #bitcoin, sorry guys
< rgrant> jcorgan: "FairHeader" has gained a following already in #bitcoin. "forward compatible" might stir up big blockers.
< jeremyrubin> In the future, you don't want some regulator charging in requesting feature changes to disable a "covert" bitcoin feature
< jeremyrubin> I think we need to be very careful with how we brand any changes happening. The reason we would like to ban it is *NOT* because it is covert. It is because it is incomaptible with Bitcoin's incentive systems (e.g., transaction selection to maximiuze fees) and it interferes with protocol development.
< jtimon> let's move the discussion to #bitcoin ?