2017-05-03

< bitcoin-git> [bitcoin] sipa opened pull request #10322: Use hardware timestamps in RNG seeding (master...rdtsc) https://github.com/bitcoin/bitcoin/pull/10322
< Chris_Stewart_5> It looks like travis-ci bans people that use bitcoin as a dependency in .travis.yml

2017-05-02

< bitcoin-git> [bitcoin] sipa opened pull request #10321: Use FastRandomContext for all tests (master...fast_rand_tests) https://github.com/bitcoin/bitcoin/pull/10321
< bitcoin-git> [bitcoin] sipa closed pull request #10299: Remove OpenSSL (master...simplerandom) https://github.com/bitcoin/bitcoin/pull/10299
< bitcoin-git> [bitcoin] practicalswift opened pull request #10319: Remove unused argument from MarkBlockAsInFlight(...) (master...remove-unused-argument) https://github.com/bitcoin/bitcoin/pull/10319
< bitcoin-git> [bitcoin] jnewbery opened pull request #10318: [tests] fix wait_for_inv() (master...fix_wait_for_inv) https://github.com/bitcoin/bitcoin/pull/10318
< gribble> https://github.com/bitcoin/bitcoin/issues/10285 | net: refactor the connection process. moving towards async connections. by theuni · Pull Request #10285 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10285 | net: refactor the connection process. moving towards async connections. by theuni · Pull Request #10285 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] practicalswift opened pull request #10317: Remove unused Python imports (master...remove-unused-python-imports-ii) https://github.com/bitcoin/bitcoin/pull/10317
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #10137: Remove unused import. Remove accidental trailing semicolons. (master...python-20170402) https://github.com/bitcoin/bitcoin/pull/10137
< bitcoin-git> bitcoin/master 75171f0 MarcoFalke: Merge #10137: Remove unused import. Remove accidental trailing semicolons....
< bitcoin-git> bitcoin/master b8d9a86 practicalswift: Remove unused import
< bitcoin-git> bitcoin/master 73b3721 practicalswift: Remove accidental trailing semicolons
< bitcoin-git> [bitcoin] MarcoFalke pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/3c5e6c94caf4...75171f099e82
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #10255: [test] Add test for listaddressgroupings (master...test_listaddressgroupings) https://github.com/bitcoin/bitcoin/pull/10255
< bitcoin-git> bitcoin/master 3c5e6c9 MarcoFalke: Merge #10255: [test] Add test for listaddressgroupings...
< bitcoin-git> bitcoin/master dadfee3 Jimmy Song: [test] Add test for listaddressgroupings...
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/dc8fc0c73beb...3c5e6c94caf4
< bitcoin-git> [bitcoin] parazyd opened pull request #10316: fix contrib/debian builds; prefer qt5 (master...debian-packaging) https://github.com/bitcoin/bitcoin/pull/10316
< bitcoin-git> [bitcoin] MarcoFalke 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/master dc8fc0c MarcoFalke: Merge #10198: [tests] Remove is_network_split from functional test framework...
< bitcoin-git> bitcoin/master c9cc76d John Newbery: [tests] Remove is_network_split from funtional test cases
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/8f3e38477e30...dc8fc0c73beb
< bitcoin-git> bitcoin/master 8f3e384 Wladimir J. van der Laan: Merge #10169: [tests] Remove func test code duplication...
< bitcoin-git> bitcoin/master 2a52ae6 John Newbery: Remove duplicate method definitions in NodeConnCB subclasses...
< bitcoin-git> bitcoin/master 52e15aa John Newbery: Adds helper functions to NodeConnCB...
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/2580ff81f442...8f3e38477e30
< bitcoin-git> [bitcoin] laanwj closed pull request #10314: Remove unused forward declaration for non-existent ScriptPubKeyToJSON(...) (master...remove-unused-function) https://github.com/bitcoin/bitcoin/pull/10314
< bitcoin-git> bitcoin/master 2580ff8 Wladimir J. van der Laan: Merge #10314: Remove unused forward declaration for non-existent ScriptPubKeyToJSON(...)...
< bitcoin-git> bitcoin/master 3fe8a83 practicalswift: Remove unused forward declaration for non-existent ScriptPubKeyToJSON(...)
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/1d75597ba6e8...2580ff81f442
< bitcoin-git> [bitcoin] laanwj closed pull request #10260: [doc] Minor corrections to osx dependencies (master...osx-doc-update) https://github.com/bitcoin/bitcoin/pull/10260
< bitcoin-git> bitcoin/master 1d75597 Wladimir J. van der Laan: Merge #10260: [doc] Minor corrections to osx dependencies...
< bitcoin-git> bitcoin/master 661caf8 fanquake: [doc] Minor corrections to osx dependencies
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/faf2dea5ea81...1d75597ba6e8
< bitcoin-git> [bitcoin] laanwj closed pull request #10234: [net] listbanned RPC and QT should show correct banned subnets (master...list_banned_correctly) https://github.com/bitcoin/bitcoin/pull/10234
< bitcoin-git> bitcoin/master faf2dea Wladimir J. van der Laan: Merge #10234: [net] listbanned RPC and QT should show correct banned subnets...
< bitcoin-git> bitcoin/master d6732d8 John Newbery: [tests] update disconnect_ban.py test case to work with listbanned
< bitcoin-git> bitcoin/master 77c54b2 John Newbery: [net] listbanned RPC and QT should show correct banned subnets
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/a3e756b7d610...faf2dea5ea81
< bitcoin-git> [bitcoin] practicalswift opened pull request #10314: Remove unused forward declaration for non-existent ScriptPubKeyToJSON(...) (master...remove-unused-function) https://github.com/bitcoin/bitcoin/pull/10314
< bitcoin-git> [bitcoin] instagibbs opened pull request #10313: [Consensus] Add constant for maximum stack size (master...stackconst) https://github.com/bitcoin/bitcoin/pull/10313
< bitcoin-git> [bitcoin] jonasschnelli closed pull request #10093: [Qt] Don't add arguments of sensitive command to console window (master...2017/03/qt_console) https://github.com/bitcoin/bitcoin/pull/10093
< bitcoin-git> bitcoin/master a3e756b Jonas Schnelli: Merge #10093: [Qt] Don't add arguments of sensitive command to console window...
< bitcoin-git> bitcoin/master 7278537 Jonas Schnelli: [Qt] Don't add arguments of sensitive command to console window
< bitcoin-git> [bitcoin] jonasschnelli pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/f4a6180607a8...a3e756b7d610
< bitcoin-git> [bitcoin] laanwj closed pull request #10175: Remove excess logic. (master...excess-logic) https://github.com/bitcoin/bitcoin/pull/10175
< bitcoin-git> bitcoin/master f4a6180 Wladimir J. van der Laan: Merge #10175: Remove excess logic....
< bitcoin-git> bitcoin/master 9a763d4 practicalswift: Remove excess logic.
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/e7b90f78f554...f4a6180607a8
< bitcoin-git> [bitcoin] laanwj closed pull request #10180: [trivial] Fix typos (tempoarily → temporarily, inadvertantly → inadvertently) (master...inadvertantly-and-tempoarily) https://github.com/bitcoin/bitcoin/pull/10180
< bitcoin-git> bitcoin/master e7b90f7 Wladimir J. van der Laan: Merge #10180: [trivial] Fix typos (tempoarily → temporarily, inadvertantly → inadvertently)...
< bitcoin-git> bitcoin/master 66082e0 practicalswift: [trivial] Fix typos (tempoarily → temporarily, inadvertantly → inadvertently)
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/0e8499c53f9e...e7b90f78f554
< bitcoin-git> [bitcoin] btcdrak opened pull request #10311: Remove unused args from GetFetchFlags() (master...getflags) https://github.com/bitcoin/bitcoin/pull/10311
< bitcoin-git> [bitcoin] laanwj closed pull request #10281: doc: Add RPC interface guidelines (master...2017_04_rpc_if_guidelines) https://github.com/bitcoin/bitcoin/pull/10281
< bitcoin-git> bitcoin/master 0e8499c Wladimir J. van der Laan: Merge #10281: doc: Add RPC interface guidelines...
< bitcoin-git> bitcoin/master c26655e Wladimir J. van der Laan: doc: Add RPC interface guidelines
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/e4bbd3d230f2...0e8499c53f9e
< bitcoin-git> [bitcoin] kallewoof opened pull request #10310: [doc] Add hint about getmempoolentry to getrawmempool help. (master...doc-getrawmempool-getmempoolentry) https://github.com/bitcoin/bitcoin/pull/10310
< bitcoin-git> [bitcoin] kallewoof closed pull request #10304: [rpc] Allow a txid param in getrawmempool (master...getrawmempool-include-txid) https://github.com/bitcoin/bitcoin/pull/10304

2017-05-01

< bitcoin-git> [bitcoin] sipa closed pull request #10292: Improved efficiency in COutPoint constructors (master...master) https://github.com/bitcoin/bitcoin/pull/10292
< bitcoin-git> bitcoin/master e4bbd3d Pieter Wuille: Merge #10292: Improved efficiency in COutPoint constructors...
< bitcoin-git> bitcoin/master 4fbae77 Marcos Mayorga: Improved efficiency in COutPoint constructors
< bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/c45da32047ca...e4bbd3d230f2
< bitcoin-git> [bitcoin] sipa closed pull request #10305: Fix potential NPD introduced in b297426c (master...2017-05-fix-10290-npd) https://github.com/bitcoin/bitcoin/pull/10305
< bitcoin-git> bitcoin/master c45da32 Pieter Wuille: Merge #10305: Fix potential NPD introduced in b297426c...
< bitcoin-git> bitcoin/master 70d3945 Matt Corallo: Fix potential NPD introduced in b297426c...
< bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/ab1f1d324691...c45da32047ca
< bitcoin-git> [bitcoin] sipa closed pull request #10309: Trivial: remove extra character from comment (master...typoquestion) https://github.com/bitcoin/bitcoin/pull/10309
< bitcoin-git> bitcoin/master ab1f1d3 Pieter Wuille: Merge #10309: Trivial: remove extra character from comment...
< bitcoin-git> bitcoin/master 3503716 CryptAxe: Trivial: remove extra character from comment
< bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/e2b99b13131b...ab1f1d324691
< bitcoin-git> [bitcoin] CryptAxe opened pull request #10309: Trivial: remove extra character from comment (master...typoquestion) https://github.com/bitcoin/bitcoin/pull/10309
< bitcoin-git> [bitcoin] tjps opened pull request #10308: [wallet] Securely erase potentially sensitive keys/values (master...tjps_secure_erase) https://github.com/bitcoin/bitcoin/pull/10308
< bitcoin-git> [bitcoin] jnewbery opened pull request #10307: [tests] allow zmq test to be run in out-of-tree builds (master...fix_zmq_test_out_of_tree) https://github.com/bitcoin/bitcoin/pull/10307
< SopaXorzTaker> sipa, what's this bounty? http://permalink.gmane.org/gmane.comp.bitcoin.devel/3089
< bitcoin-git> [bitcoin] tjps closed pull request #10182: [scheduler] Switched CScheduler to C++11 threading primitives (master...tjps_scheduler) https://github.com/bitcoin/bitcoin/pull/10182
< gribble> https://github.com/bitcoin/bitcoin/issues/10294 | [Wallet] unset change position when there is no change by instagibbs · Pull Request #10294 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] TheBlueMatt opened pull request #10305: Fix potential NPD introduced in b297426c (master...2017-05-fix-10290-npd) https://github.com/bitcoin/bitcoin/pull/10305
< bitcoin-git> [bitcoin] laanwj closed pull request #10294: [Wallet] unset change position when there is no change (master...fixchangepos) https://github.com/bitcoin/bitcoin/pull/10294
< bitcoin-git> bitcoin/master e2b99b1 Wladimir J. van der Laan: Merge #10294: [Wallet] unset change position when there is no change...
< bitcoin-git> bitcoin/master 7c58863 Gregory Sanders: [Wallet] unset change position when there is no change on exact match
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/492d22f92919...e2b99b13131b
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #10264: [test] Add tests for getconnectioncount, getnettotals and ping (master...test_netrpccalls) https://github.com/bitcoin/bitcoin/pull/10264
< bitcoin-git> bitcoin/master 85f950a Jimmy Song: [test] Add tests for getconnectioncount, getnettotals and ping...
< bitcoin-git> bitcoin/master 492d22f MarcoFalke: Merge #10264: [test] Add tests for getconnectioncount, getnettotals and ping...
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/9c33ffd38764...492d22f92919
< bitcoin-git> [bitcoin] kallewoof opened pull request #10304: [rpc] Allow a txid param in getrawmempool (master...getrawmempool-include-txid) https://github.com/bitcoin/bitcoin/pull/10304
< kallewoof> Maybe bitcoin could use libgcrypt (GnuPG uses it for keys, I think; it has a bunch of random things) instead of OpenSSL. Though it seems to be LGPL. Some parts of it are dual MPL-compat though, like random-drbg.c
< gmaxwell> okay we could do something that would perhaps mostly only be used by other bitcoin things.
< wumpus> but someone else cooporating on it outside the context of bitcoin core (which is pretty narrow) isn't necessarily bad
<@wumpus> gmaxwell: sipa: I'm just worried that taking up the concern of doing proper seeding in bitcoin core itself, without anyone else looking at it or using it, will result in a worse outcome for us. Allthe platform specific stuff is a hell to maintain, and it forces us to stay up to date in developments in that area
< bitcoin-git> bitcoin/master 9c33ffd Wladimir J. van der Laan: Merge #8824: Refactor TxToJSON() and ScriptPubKeyToJSON()...
< bitcoin-git> bitcoin/master 0ff9320 jonnynewbs: refactor TxToJSON() and ScriptPubKeyToJSON()
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/8d6d43e9338a...9c33ffd38764
< bitcoin-git> [bitcoin] laanwj closed pull request #10290: Add -stopatheight for benchmarking (master...shutdown_at_height) https://github.com/bitcoin/bitcoin/pull/10290
< bitcoin-git> bitcoin/master 8d6d43e Wladimir J. van der Laan: Merge #10290: Add -stopatheight for benchmarking...
< bitcoin-git> bitcoin/master b297426 Pieter Wuille: Add -stopatheight for benchmarking
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/80c3a734298e...8d6d43e9338a
< bitcoin-git> [bitcoin] kallewoof opened pull request #10303: [bench] Include ms/blk stats in Connect* benchmarks. (master...bench-add-per-block-stats) https://github.com/bitcoin/bitcoin/pull/10303

2017-04-30

< NicolasDorier> well yes I will cache the address on the server side, and just make a request to bitcoin core to know if it was used before returning it to the server
< SopaXorzTaker> banned me from #bitcoin for posting a malware link, explicitly marked like this and obsured: [https://]malware[.]coin.stealer
< sipa> bitcoin core doesn't support gaps or reconstruction from a seed at all
< NicolasDorier> is there a way in Bitcoin Core to get an unused address ? I am tempted to call getnewaddress everytimes, but doing so would create big gap in by BIP32 path, which would make rescanning fail
< bitcoin-git> [bitcoin] spencerlievens opened pull request #10302: [Makefile] Alphabetically Reorder addrdb.cpp (master...patch-2) https://github.com/bitcoin/bitcoin/pull/10302
< afk11> if this should be in #bitcoin-dev please someone let me know
< afk11> keeping in mind most bitcoin libraries only sign scripts that don't have any degrees of freedom with the pathway that's executed.. I'm starting out by iterating over script opcodes and making a tree out of possible execution pathways as they come
< bitcoin-git> [bitcoin] jameshilliard opened pull request #10301: Check if sys/random.h is required for getentropy. (master...getentropy-rand) https://github.com/bitcoin/bitcoin/pull/10301

2017-04-29

< gmaxwell> I posted about it before, rng is seeded based on a collection of different pointers and timestamps. And in Bitcoin on windows, based on a screenshot.
< gribble> https://github.com/bitcoin/bitcoin/issues/5885 | [WIP] Replace OpenSSL PRNG with built-in Fortuna implementation by sipa · Pull Request #5885 · bitcoin/bitcoin · GitHub
< jonasschnelli> Would unpack stuff from #5885 (https://github.com/bitcoin/bitcoin/pull/5885/files#diff-35f8a407f8c21cda300a45f50b6e9c74L5) make sense?
< gmaxwell> wumpus: your comment is needed at https://github.com/bitcoin-core/secp256k1/pull/440
< bitcoin-git> [bitcoin] sipa opened pull request #10299: Remove OpenSSL (master...simplerandom) https://github.com/bitcoin/bitcoin/pull/10299
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #10256: [test] Add test for gettxout to wallet.py (master...test_gettxout) https://github.com/bitcoin/bitcoin/pull/10256
< bitcoin-git> bitcoin/master 80c3a73 MarcoFalke: Merge #10256: [test] Add test for gettxout to wallet.py...
< bitcoin-git> bitcoin/master dd1ea59 Jimmy Song: [test] Add gettxout call...
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/4c924011f535...80c3a734298e

2017-04-28

< bitcoin-git> [bitcoin] fanquake closed pull request #10298: jslee1301 patch 1 (0.11...gh-jslee1301-patch-1) https://github.com/bitcoin/bitcoin/pull/10298
< bitcoin-git> [bitcoin] gh-jslee1301 opened pull request #10298: jslee1301 patch 1 (0.11...gh-jslee1301-patch-1) https://github.com/bitcoin/bitcoin/pull/10298
< bitcoin-git> [bitcoin] sipa opened pull request #10297: Simplify DisconnectBlock arguments/return value (master...disconnect_enum) https://github.com/bitcoin/bitcoin/pull/10297
< bitcoin-git> [bitcoin] ryanofsky opened pull request #10295: [qt] Move some WalletModel functions into CWallet (master...pr/ipc-move) https://github.com/bitcoin/bitcoin/pull/10295
< bitcoin-git> [bitcoin] instagibbs opened pull request #10294: [Wallet] unset change position when there is no change (master...fixchangepos) https://github.com/bitcoin/bitcoin/pull/10294
< bsm117532> bitcoin-cli -regtest fundrawtransaction 01000000000106220000000000001600149bf98693c5892c3e7b6d991cfeebf146285f59bd00000000 '{"changePosition": 1}'
< sipa> SopaXorzTaker: https://lists.linuxfoundation.org/pipermail/bitcoin-dev/2015-July/009697.html read this first before you say great
< sipa> until bitcoin core 0.11
< achow101> SopaXorzTaker: the original bitcoin client was supposed to include a marketplace thing
< SopaXorzTaker> wumpus, sipa: here's an actual question about the bitcoin core
< bitcoin-git> [bitcoin] mm-s opened pull request #10292: Improved efficiency in COutPoint constructors (master...master) https://github.com/bitcoin/bitcoin/pull/10292
< bitcoin-git> [bitcoin] mm-s closed pull request #10277: Improved efficiency in constructors of COutPoint (master...master) https://github.com/bitcoin/bitcoin/pull/10277

2017-04-27

< bitcoin-git> [bitcoin] practicalswift reopened pull request #9544: [trivial] Add end of namespace comments. Improve consistency. (master...consistent-use-of-end-of-namespace-comments) https://github.com/bitcoin/bitcoin/pull/9544
< bitcoin-git> [bitcoin] practicalswift closed pull request #9544: [trivial] Add end of namespace comments. Improve consistency. (master...consistent-use-of-end-of-namespace-comments) https://github.com/bitcoin/bitcoin/pull/9544
< jtimon> btw BlueMatt thanks for https://github.com/bitcoin/bitcoin/pull/771 I shouldn't look at it much and look at the new things instead but like these things if I find the time
< jtimon> wumpus: btw, it would be nice to put BlueMatt 's libconsensus-related PRs in https://github.com/bitcoin/bitcoin/projects/6 (and always tag anything there with "consensus")
<@wumpus> see "Ubuntu 16.04 Windows cross-build" on https://github.com/bitcoin/bitcoin/projects for a collection of issues
< gribble> https://github.com/bitcoin/bitcoin/issues/7729 | rpc: introduce label API for wallet by laanwj · Pull Request #7729 · bitcoin/bitcoin · GitHub
< BlueMatt> luke-jr: thats ridiculous, you're living in a world where people have the time go read a ton of bitcoin core docs/code before running it, and do...neither of which are true
< gribble> https://github.com/bitcoin/bitcoin/issues/7729 | rpc: introduce label API for wallet by laanwj · Pull Request #7729 · bitcoin/bitcoin · GitHub
< petertodd> gmaxwell: btw, re: your comment in the meeting, no-one's funding me to do any work on Bitcoin Core these days
< BlueMatt> jtimon: conf flag seems neater, no one checks their bitcoin datadir
< jtimon> it should still be relatively easy for users to get out of the stuck situation in case they can't upgrade in the same system or something, like maybe deleting a filed named ~/.bitcoin/DELETE_ME_ONLY_IF_YOU_CANT_UPGRADE_IN_THIS_SYSTEM or something
< luke-jr> BlueMatt: it's only slightly shorter than Bitcoin's current lifetime
< jtimon> or shouldn't I recommend bitcoin core for a wallet?
< jtimon> luke-jr: let's say my friend remembers what I told him about being up to date 6 years and 11 months after I helped him install bitcoin core
< BlueMatt> luke-jr: 110% sends the wrong message. if i expected any reasonable person to see that and think "I need to think for myself about what consensus of the network is" I'd be happy with it, but realistically the only people reading that will think "oh, I have to switch to the latest thing from Bitcoin Core, for whatever Bitcoin Core is according to my local google server"
< jtimon> I think if it's optional and disabled by default it kind of defeats the point, but I certainly don't want that for myself or the users I recommend to use bitcoin core
< gribble> https://github.com/bitcoin/bitcoin/issues/10282 | Expire bitcoind & bitcoin-qt 7-8 years after its last change by luke-jr · Pull Request #10282 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10195 | Switch chainstate db and cache to per-txout model by sipa · Pull Request #10195 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10148 | [WIP] Use non-atomic flushing with block replay by sipa · Pull Request #10148 · bitcoin/bitcoin · GitHub
< * jtimon> checks https://github.com/bitcoin/bitcoin/pull/8855 is on the priority list
< gribble> https://github.com/bitcoin/bitcoin/issues/10195 | Switch chainstate db and cache to per-txout model by sipa · Pull Request #10195 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10148 | [WIP] Use non-atomic flushing with block replay by sipa · Pull Request #10148 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10285 | net: refactor the connection process. moving towards async connections. by theuni · Pull Request #10285 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/7729 | rpc: introduce label API for wallet by laanwj · Pull Request #7729 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10195 | Switch chainstate db and cache to per-txout model by sipa · Pull Request #10195 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10199 | Better fee estimates by morcos · Pull Request #10199 · bitcoin/bitcoin · GitHub
<@wumpus> anything to add to project https://github.com/bitcoin/bitcoin/projects/8?
<@wumpus> jonasschnelli: I meant in bitcoin core; I don't know about other implementations
< sipa> (by bitcoin core)
< bitcoin-git> [bitcoin] sipa opened pull request #10290: Add -stopatheight for benchmarking (master...shutdown_at_height) https://github.com/bitcoin/bitcoin/pull/10290
< gribble> https://github.com/bitcoin/bitcoin/issues/10279 | Add a CChainState class to validation.cpp to take another step towards clarifying internal interfaces by TheBlueMatt · Pull Request #10279 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/771 | CBlockStore by TheBlueMatt · Pull Request #771 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10240 | [WIP] Add basic HD wallet restore functionality by jonasschnelli · Pull Request #10240 · bitcoin/bitcoin · GitHub
<@wumpus> #bitcoin-core-dev Meeting: wumpus sipa gmaxwell jonasschnelli morcos luke-jr btcdrak sdaftuar jtimon cfields petertodd kanzure bluematt
< bitcoin-git> [bitcoin] laanwj closed pull request #10075: Remove unused C++ code not covered by unit tests (master...unused) https://github.com/bitcoin/bitcoin/pull/10075
< bitcoin-git> bitcoin/master 4c92401 Wladimir J. van der Laan: Merge #10075: Remove unused C++ code not covered by unit tests...
< bitcoin-git> bitcoin/master b51aaf1 practicalswift: Remove unused C++ code not covered by unit tests
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/a550f6e415fd...4c924011f535
< bitcoin-git> [bitcoin] sipa closed pull request #10283: Cleanup: reduce to one GetMinimumFee call signature (master...oneGetMinimumFee) https://github.com/bitcoin/bitcoin/pull/10283
< bitcoin-git> bitcoin/master a550f6e Pieter Wuille: Merge #10283: Cleanup: reduce to one GetMinimumFee call signature...
< bitcoin-git> bitcoin/master 3edbd79 Alex Morcos: cleanup: reduce to one GetMinimumFee call signature
< bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/47535d7c3ec7...a550f6e415fd
< bitcoin-git> [bitcoin] jimmysong opened pull request #10287: [tests] Update Unit Test for addrman.h/addrman.cpp (master...test_addrman) https://github.com/bitcoin/bitcoin/pull/10287
< bitcoin-git> [bitcoin] TheBlueMatt opened pull request #10286: Call wallet notify callbacks in scheduler thread (without cs_main) (master...2017-01-wallet-cache-inmempool-4) https://github.com/bitcoin/bitcoin/pull/10286
< gribble> https://github.com/bitcoin/bitcoin/issues/9254 | [depends] ZeroMQ 4.2.2 by fanquake · Pull Request #9254 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] theuni opened pull request #10285: net: refactor the connection process. moving towards async connections. (master...connman-events6) https://github.com/bitcoin/bitcoin/pull/10285

2017-04-26

< bitcoin-git> [bitcoin] MarcoFalke closed pull request #10280: [test] Unit test amount.h/amount.cpp (master...test_amount) https://github.com/bitcoin/bitcoin/pull/10280
< bitcoin-git> bitcoin/master 47535d7 MarcoFalke: Merge #10280: [test] Unit test amount.h/amount.cpp...
< bitcoin-git> bitcoin/master dcb69fd Jimmy Song: [test] Unit test amount.h/amount.cpp...
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/cf5782508a3f...47535d7c3ec7
< bitcoin-git> [bitcoin] morcos opened pull request #10284: Always log debug information for fee calculation in CreateTransaction (master...debugEstimates) https://github.com/bitcoin/bitcoin/pull/10284
< bitcoin-git> [bitcoin] morcos opened pull request #10283: Cleanup: reduce to one GetMinimumFee call signature (master...oneGetMinimumFee) https://github.com/bitcoin/bitcoin/pull/10283
< bitcoin-git> [bitcoin] luke-jr opened pull request #10282: Expire bitcoind & bitcoin-qt 7-8 years after its last change (master...timebomb) https://github.com/bitcoin/bitcoin/pull/10282
< bitcoin-git> [bitcoin] laanwj closed pull request #9614: [wallet] [refactor] Simplify getbalance implementation (master...pr/getbalance-cleanup) https://github.com/bitcoin/bitcoin/pull/9614
< bitcoin-git> bitcoin/master cf57825 Wladimir J. van der Laan: Merge #9614: [wallet] [refactor] Simplify getbalance implementation...
< bitcoin-git> bitcoin/master 02d9f50 Russell Yanofsky: [wallet] Remove unneeded legacy getbalance code
< bitcoin-git> bitcoin/master 82b7dc3 Russell Yanofsky: [wallet] Add GetLegacyBalance method to simplify getbalance RPC...
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/6fdb31916589...cf5782508a3f
< bitcoin-git> [bitcoin] practicalswift closed pull request #9545: Add final where appropriate (master...add-overrides-where-appropriate) https://github.com/bitcoin/bitcoin/pull/9545
< bitcoin-git> [bitcoin] fanquake closed pull request #9363: Reduce fingerprinting attacks (master...MinChainWorkBlocktxns) https://github.com/bitcoin/bitcoin/pull/9363
< bitcoin-git> [bitcoin] laanwj closed pull request #9743: Fix several potential issues found by sanitizers (master...fsanitize) https://github.com/bitcoin/bitcoin/pull/9743
< bitcoin-git> bitcoin/master 6fdb319 Wladimir J. van der Laan: Merge #9743: Fix several potential issues found by sanitizers...
< bitcoin-git> bitcoin/master 1d31093 Pieter Wuille: fix tsan: utiltime race on nMockTime
< bitcoin-git> bitcoin/master 321bbc2 Pieter Wuille: fix ubsan: bitcoin-tx: not initialize context before IsFullyValid
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/bd9ec0ef1ea7...6fdb31916589
< bitcoin-git> [bitcoin] laanwj closed pull request #9670: contrib: github-merge improvements (master...2017_01_ghmerge_update) https://github.com/bitcoin/bitcoin/pull/9670
< bitcoin-git> bitcoin/master bd9ec0e Wladimir J. van der Laan: Merge #9670: contrib: github-merge improvements...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/8979f4569e0e...bd9ec0ef1ea7
< bitcoin-git> bitcoin/master b508424 Wladimir J. van der Laan: contrib: github-merge improvements...
< bitcoin-git> [bitcoin] laanwj closed pull request #10273: [scripts] Minor improvements to `macdeployqtplus` script. (master...macdeployqtplus) https://github.com/bitcoin/bitcoin/pull/10273
< bitcoin-git> bitcoin/master 8979f45 Wladimir J. van der Laan: Merge #10273: [scripts] Minor improvements to `macdeployqtplus` script....
< bitcoin-git> bitcoin/master e8babc4 Chris Gavin: Use `with` in `macdeployqtplus` script.
< bitcoin-git> bitcoin/master 4f3ac7d Chris Gavin: Remove unused variable from `macdeployqtplus` script.
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/8254a8ae213b...8979f4569e0e
< bitcoin-git> [bitcoin] laanwj closed pull request #10278: [test] Add Unit Test for GetListenPort (master...test_listenport) https://github.com/bitcoin/bitcoin/pull/10278
< bitcoin-git> bitcoin/master 1b14449 Jimmy Song: [test] Add Unit Test for GetListenPort...
< bitcoin-git> bitcoin/master 8254a8a Wladimir J. van der Laan: Merge #10278: [test] Add Unit Test for GetListenPort...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/9c94fb6c3257...8254a8ae213b
< bitcoin-git> [bitcoin] laanwj closed pull request #9930: Trivial: Correct indentation and bracing (master...fix-indent) https://github.com/bitcoin/bitcoin/pull/9930
< bitcoin-git> [bitcoin] laanwj opened pull request #10281: doc: Add RPC interface guidelines (master...2017_04_rpc_if_guidelines) https://github.com/bitcoin/bitcoin/pull/10281
< bitcoin-git> [bitcoin] jimmysong opened pull request #10280: [test] Unit test amount.h/amount.cpp (master...test_amount) https://github.com/bitcoin/bitcoin/pull/10280
< bitcoin-git> [bitcoin] TheBlueMatt opened pull request #10279: Add a CChainState class to validation.cpp to take another step towards clarifying internal interfaces (master...2016-12-cconsensus) https://github.com/bitcoin/bitcoin/pull/10279

2017-04-25

< jonasschnelli> testnet-seed.bitcoin.jonasschnelli.ch should be up and running... do you use tor?
< ryan-c> host testnet-seed.bitcoin.jonasschnelli.ch | grep ' has address ' | awk '{print$4}' | xargs -n1 -I IP bitcoin-cli -testnet addnode IP add
< gribble> https://github.com/bitcoin/bitcoin/issues/10210 | Frozen (for 15 minutes) on shutdown · Issue #10210 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10215 | Check interruptNet during dnsseed lookups by TheBlueMatt · Pull Request #10215 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10209 | Stalled shutdown · Issue #10209 · bitcoin/bitcoin · GitHub
< BlueMatt> can we close #10209 now that we fixed the "regenerate bitcoin-config.h" issue and #10210 now that #10215 is merged?
< ryan-c> BlueMatt: It's the binary release from bitcoin.org, sha256sum 860a046514961aeead8b640f1a321da8eb575d727fd8c37de5a4cee5908bbd9d
< ryan-c> I told bitcoind 0.14.0 to shutdown via bitcoin-cli over four hours ago, and it's still running. strace shows it waiting on a futex call. Is there anything I should do before sending it a kill -9? The system is not low on memory.
< sipa> that doesn't mean there is no advantage for HT cores in bitcoin block validation, as ECDSA verification isn't the only thing involved
<@wumpus> cfields: jnewbery had already agreed with that, just he still had some concern: https://github.com/bitcoin/bitcoin/pull/10234#issuecomment-295354958
< bitcoin-git> [bitcoin] jimmysong opened pull request #10278: [test] Add Unit Test for GetListenPort (master...test_listenport) https://github.com/bitcoin/bitcoin/pull/10278
< bitcoin-git> [bitcoin] mm-s opened pull request #10277: Improved efficiency in constructors of COutPoint (master...master) https://github.com/bitcoin/bitcoin/pull/10277
< bitcoin-git> [bitcoin] laanwj closed pull request #10225: [test] Add aborttrescan tests (master...abort-rescan-tests) https://github.com/bitcoin/bitcoin/pull/10225
< bitcoin-git> bitcoin/master e0a7e19 Wladimir J. van der Laan: Merge #10225: [test] Add aborttrescan tests...
< bitcoin-git> bitcoin/master ed60970 Karl-Johan Alm: [test] Test abortrescan command.
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/c29a0d48129d...e0a7e1994e6f
< bitcoin-git> [bitcoin] laanwj closed pull request #10265: [wallet] [moveonly] Check non-null pindex before potentially referencing (master...fix-check-pindex-scanforwallettx) https://github.com/bitcoin/bitcoin/pull/10265
< bitcoin-git> bitcoin/master c29a0d4 Wladimir J. van der Laan: Merge #10265: [wallet] [moveonly] Check non-null pindex before potentially referencing...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/cb007e4346cf...c29a0d48129d
< bitcoin-git> bitcoin/master c36ea69 Karl-Johan Alm: [wallet] Make sure pindex is non-null before possibly referencing in LogPrintf call.
< gribble> https://github.com/bitcoin/bitcoin/issues/10234 | [net] listbanned RPC and QT should show correct banned subnets by jnewbery · Pull Request #10234 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10234 | [net] listbanned RPC and QT should show correct banned subnets by jnewbery · Pull Request #10234 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj closed pull request #10263: Trivial: fix fee estimate write error log message (master...coinbase) https://github.com/bitcoin/bitcoin/pull/10263
< bitcoin-git> bitcoin/master cb007e4 Wladimir J. van der Laan: Merge #10263: Trivial: fix fee estimate write error log message...
< bitcoin-git> bitcoin/master 94807be CryptAxe: Trivial: fix fee estimate write error log message
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/95f5e4407502...cb007e4346cf
< bitcoin-git> [bitcoin] knocte opened pull request #10276: contrib/verifybinaries: allow filtering by platform (master...filterByPlatformInVerifySh) https://github.com/bitcoin/bitcoin/pull/10276
< bitcoin-git> [bitcoin] laanwj closed pull request #10270: Remove Clang workaround for Boost 1.46 (master...remove-boost-clang-workaround) https://github.com/bitcoin/bitcoin/pull/10270
< bitcoin-git> bitcoin/master 95f5e44 Wladimir J. van der Laan: Merge #10270: Remove Clang workaround for Boost 1.46...
< bitcoin-git> bitcoin/master 93dbb15 fanquake: Remove Clang workaround for Boost 1.46
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/54e2d87e792f...95f5e4407502
< bitcoin-git> [bitcoin] laanwj closed pull request #10274: Replace by fee v0.11.2 (master...replace-by-fee-v0.11.2) https://github.com/bitcoin/bitcoin/pull/10274
< bitcoin-git> [bitcoin] kallewoof opened pull request #10275: [rpc] Allow fetching tx directly from specified block in getrawtransaction (master...gettx-with-blockhash) https://github.com/bitcoin/bitcoin/pull/10275
< bitcoin-git> [bitcoin] ericchong opened pull request #10274: Replace by fee v0.11.2 (master...replace-by-fee-v0.11.2) https://github.com/bitcoin/bitcoin/pull/10274
< bitcoin-git> [bitcoin] chrisgavin opened pull request #10273: [scripts] Minor improvements to `macdeployqtplus` script. (master...macdeployqtplus) https://github.com/bitcoin/bitcoin/pull/10273
<@wumpus> unless someone implements https://github.com/bitcoin/bitcoin/issues/8973 to make onion connections come in on either an alternative port or a unix socket, it's unsafe to whitelist 127.0.0.1 [by default]
< bitcoin-git> [bitcoin] practicalswift closed 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] laanwj closed pull request #10272: [Tests] Prevent warning: variable 'x' is uninitialized (master...20170425_FastRandomContext_test_warnings) https://github.com/bitcoin/bitcoin/pull/10272
< bitcoin-git> bitcoin/master 54e2d87 Wladimir J. van der Laan: Merge #10272: [Tests] Prevent warning: variable 'x' is uninitialized...
< bitcoin-git> bitcoin/master 5ec8836 Pavel Janík: Prevent warning: variable 'x' is uninitialized
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/c73af5416b66...54e2d87e792f
< bitcoin-git> [bitcoin] paveljanik opened pull request #10272: [Tests] Prevent warning: variable 'x' is uninitialized (master...20170425_FastRandomContext_test_warnings) https://github.com/bitcoin/bitcoin/pull/10272
< bitcoin-git> [bitcoin] fanquake opened pull request #10271: Use std::thread::hardware_concurrency, instead of Boost, to determine available cores (master...replace-boost-getnumcores) https://github.com/bitcoin/bitcoin/pull/10271
< bitcoin-git> [bitcoin] fanquake opened pull request #10270: Remove Clang workaround for Boost 1.46 (master...remove-boost-clang-workaround) https://github.com/bitcoin/bitcoin/pull/10270

2017-04-24

< bitcoin-git> [bitcoin] sipa closed pull request #10249: Switch CCoinsMap from boost to std unordered_map (master...stdcoinmap) https://github.com/bitcoin/bitcoin/pull/10249
< bitcoin-git> bitcoin/master c73af54 Pieter Wuille: Merge #10249: Switch CCoinsMap from boost to std unordered_map...
< bitcoin-git> bitcoin/master e6756ad Pieter Wuille: Switch CCoinsMap from boost to std unordered_map
< bitcoin-git> bitcoin/master 344a2c4 Pieter Wuille: Add support for std::unordered_{map,set} to memusage.h
< bitcoin-git> [bitcoin] sipa pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/fa1ac2881f2a...c73af5416b66
< gribble> https://github.com/bitcoin/bitcoin/issues/10260 | [doc] Minor corrections to osx dependencies by fanquake · Pull Request #10260 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj closed pull request #9951: Wallet database handling abstractions/simplifications (master...2017_03_wallet_dbwrapper) https://github.com/bitcoin/bitcoin/pull/9951
< bitcoin-git> bitcoin/master 071c955 Wladimir J. van der Laan: wallet: Get rid of fFileBacked...
< bitcoin-git> bitcoin/master be9e1a9 Wladimir J. van der Laan: wallet: Reduce references to global bitdb environment
< bitcoin-git> bitcoin/master 71afe3c Wladimir J. van der Laan: wallet: Introduce database handle wrapper...
< bitcoin-git> [bitcoin] laanwj pushed 7 new commits to master: https://github.com/bitcoin/bitcoin/compare/342b9bc3907e...fa1ac2881f2a
< gribble> https://github.com/bitcoin/bitcoin/issues/10192 | Cache full script execution results in addition to signatures by TheBlueMatt · Pull Request #10192 · bitcoin/bitcoin · GitHub
< fanquake> Since we seem to be giving Boost removal a good shot for 0.15, does anyone have suggestions for replacing GetNumCores? https://github.com/bitcoin/bitcoin/blob/master/src/util.cpp#L881
<@wumpus> "bitcoin mining on mars"
<@wumpus> BlueMatt: ah yes the "apply all the buzzwords to bitcoin mining, see what sticks" approach
< BlueMatt> wumpus: the same bottom-of-the-barrel buzzword bingo that came up with "using normal computers to simulate quantum computers to do bitcoin mining"
< bitcoin-git> [bitcoin] laanwj closed pull request #9792: FastRandomContext improvements and switch to ChaCha20 (master...chacha) https://github.com/bitcoin/bitcoin/pull/9792
< bitcoin-git> bitcoin/master e04326f Pieter Wuille: Add ChaCha20
< bitcoin-git> bitcoin/master 663fbae Pieter Wuille: FastRandom benchmark
< bitcoin-git> bitcoin/master c21cbe6 Pieter Wuille: Introduce FastRandomContext::randbool()
< bitcoin-git> [bitcoin] laanwj pushed 6 new commits to master: https://github.com/bitcoin/bitcoin/compare/1b25b6df0f08...342b9bc3907e
<@wumpus> "using deep learning frameworks in bitcoin mining", that's hilarious, what bottom-of-the-barrel buzzword factory did that come out of
< bitcoin-git> [bitcoin] kallewoof opened pull request #10267: New -readconfig argument for including external configuration files (master...feature-config-readconfig) https://github.com/bitcoin/bitcoin/pull/10267
< bitcoin-git> [bitcoin] fanquake closed pull request #10266: Update policy.h (master...patch-1) https://github.com/bitcoin/bitcoin/pull/10266
< bitcoin-git> [bitcoin] 1mbtxn opened pull request #10266: Update policy.h (master...patch-1) https://github.com/bitcoin/bitcoin/pull/10266

2017-04-23

< bitcoin-git> [bitcoin] kallewoof opened pull request #10265: [wallet] Check non-null pindex before potentially referencing (master...fix-check-pindex-scanforwallettx) https://github.com/bitcoin/bitcoin/pull/10265
< gmaxwell> Can someone please close: https://github.com/bitcoin/bitcoin/issues/6876
< bitcoin-git> [bitcoin] jimmysong opened pull request #10264: [test] Add tests for getconnectioncount, getnettotals and ping (master...test_netrpccalls) https://github.com/bitcoin/bitcoin/pull/10264
< bitcoin-git> [bitcoin] CryptAxe reopened pull request #10263: Trivial: fix fee estimate write error log message (master...coinbase) https://github.com/bitcoin/bitcoin/pull/10263
< bitcoin-git> [bitcoin] CryptAxe closed pull request #10263: Trivial: fix fee estimate write error log message (master...coinbase) https://github.com/bitcoin/bitcoin/pull/10263