2016-08-26

< GitHub8> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/9a0ed08b40b1...c072b8fd95cd
< GitHub43> [bitcoin] sipa closed pull request #8109: Do not shadow member variables (master...20160527_shadow_httpserver) https://github.com/bitcoin/bitcoin/pull/8109
< GitHub107> bitcoin/master 9a0ed08 Pieter Wuille: Merge #8109: Do not shadow member variables...
< GitHub107> bitcoin/master ff8d279 Pavel Janík: Do not shadow member variables
< GitHub107> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/bb566761fbe1...9a0ed08b40b1
< helo> rinzes_: #bitcoin
< rinzes_> I am trying to find out how to remove bitcoin core completely
< GitHub154> [bitcoin] laanwj opened pull request #8604: build,doc: Update for 0.13.0+ and OpenBSD 5.9 (master...2016_08_openbsd_update) https://github.com/bitcoin/bitcoin/pull/8604
< sipa> SyntaxError: Non-ASCII character '\xf0' in file /home/pw/git/bitcoin/qa/rpc-tests/test_framework/util.py on line 180, but no encoding declared; see http://python.org/dev/peps/pep-0263/ for details
< GitHub183> [bitcoin] laanwj closed pull request #8547: Update btcdrak key with new expiry dates (master...btcdrak-key) https://github.com/bitcoin/bitcoin/pull/8547
< GitHub114> [bitcoin] laanwj closed pull request #8602: [trivial][doc] Mention ++i as preferred over i++ in dev notes (master...trvial-dev-notes) https://github.com/bitcoin/bitcoin/pull/8602
< GitHub182> bitcoin/master bb56676 Wladimir J. van der Laan: Merge #8602: [trivial][doc] Mention ++i as preferred over i++ in dev notes...
< GitHub182> bitcoin/master ab53207 fanquake: [trivial][doc] Mention ++i as preferred to i++ in dev notes
< GitHub182> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/fd37acaedab1...bb566761fbe1
< GitHub169> [bitcoin] laanwj closed pull request #8603: [trivial][doc] Mention gpg --refresh-keys in release-process.md (master...release-process-refresh-keys) https://github.com/bitcoin/bitcoin/pull/8603
< GitHub56> bitcoin/master c25083b fanquake: [trivial][doc] Mention gpg --refresh-keys in release-process.md
< GitHub56> bitcoin/master fd37aca Wladimir J. van der Laan: Merge #8603: [trivial][doc] Mention gpg --refresh-keys in release-process.md...
< GitHub56> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/12892dbb9fb6...fd37acaedab1
< wumpus> heh, bitcoin can no longer be compiled with gcc 4.9.3 with OpenBSD's default max data segment limit of 1536MB
< GitHub9> [bitcoin] fanquake opened pull request #8603: [trivial][doc] Mention gpg --refresh-keys in release-process.md (master...release-process-refresh-keys) https://github.com/bitcoin/bitcoin/pull/8603
< GitHub56> [bitcoin] fanquake opened pull request #8602: [trivial][doc] Mention ++i as preferred over i++ in dev notes (master...trvial-dev-notes) https://github.com/bitcoin/bitcoin/pull/8602
< GitHub44> [bitcoin] laanwj closed pull request #8227: tests: Re-enable RPC tests for Windows (master...2016_05_win_reenable_rpc_tests) https://github.com/bitcoin/bitcoin/pull/8227
< wumpus> btcdrak: do you have a plan for https://github.com/bitcoin/bitcoin/pull/7562?
< GitHub87> bitcoin/0.13 122fdfd Wladimir J. van der Laan: Merge #8600: [0.13.1]: Backport [wallet] rpc: Drop misleading option...
< GitHub7> [bitcoin] laanwj closed pull request #8600: [0.13.1]: Backport [wallet] rpc: Drop misleading option (0.13...Mf1608-walletRpcDropBackport) https://github.com/bitcoin/bitcoin/pull/8600
< GitHub87> bitcoin/0.13 526d2b0 MarcoFalke: [wallet] rpc: Drop misleading option...
< GitHub87> [bitcoin] laanwj pushed 2 new commits to 0.13: https://github.com/bitcoin/bitcoin/compare/f2306fbe0142...122fdfdae915
< GitHub7> [bitcoin] laanwj closed pull request #8590: Remove unused variables (master...Mf1608-unusedCode) https://github.com/bitcoin/bitcoin/pull/8590
< GitHub41> bitcoin/master fa6dc9f MarcoFalke: Remove unused variables
< GitHub41> bitcoin/master 12892db Wladimir J. van der Laan: Merge #8590: Remove unused variables...
< GitHub41> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/65837375d98b...12892dbb9fb6
< GitHub138> [bitcoin] laanwj closed pull request #7132: Add option to opt into full-RBF when sending funds (master...2015-11-opt-into-full-rbf-option) https://github.com/bitcoin/bitcoin/pull/7132
< GitHub161> [bitcoin] laanwj opened pull request #8601: Add option to opt into full-RBF when sending funds (rebase) (master...2016_08_full_rbf_option) https://github.com/bitcoin/bitcoin/pull/8601
< GitHub65> [bitcoin] MarcoFalke opened pull request #8600: [0.13.1]: Backport [wallet] rpc: Drop misleading option (0.13...Mf1608-walletRpcDropBackport) https://github.com/bitcoin/bitcoin/pull/8600
< GitHub156> [bitcoin] laanwj closed pull request #8581: [wallet] rpc: Drop misleading option (master...Mf1608-walletDropRpc) https://github.com/bitcoin/bitcoin/pull/8581
< GitHub197> bitcoin/master 6583737 Wladimir J. van der Laan: Merge #8581: [wallet] rpc: Drop misleading option...
< GitHub197> bitcoin/master fab5ecb MarcoFalke: [wallet] rpc: Drop misleading option
< GitHub197> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/c19f8a4a7795...65837375d98b
< jonasschnelli> sipa: maybe my fault.. I'm running https://github.com/bitcoin/bitcoin/pull/7685/files on top
< GitHub63> [bitcoin] laanwj closed pull request #8583: Show XTHIN in GUI (master...ShowXTHINinGUI) https://github.com/bitcoin/bitcoin/pull/8583
< GitHub123> bitcoin/master c19f8a4 Wladimir J. van der Laan: Merge #8583: Show XTHIN in GUI...
< GitHub123> bitcoin/master 4c3e2cb R E Broadley: Show XTHIN in GUI
< GitHub123> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/6c9f1b8c2405...c19f8a4a7795
< jonasschnelli> Yes. But IMO is a fact that people have stopped running bitcoin-qt because of its extensive resource consumption. But maybe I'm wrong here.
< gmaxwell> if you've figured out that bitcoin is the cause you are already more advanced than we should hope for, usually once they've figured that out bitcoin is doomed. :P
< gmaxwell> then we should fix it so that it doesn't screw up voip, _generally_, even when they don't know bitcoin is the cause, not by finding some obscure option.
< jonasschnelli> My goal is that people will not shut down bitcoin-qt in order to do a VoIP call (because restarting in pain in the ass).
< gmaxwell> So for example, bandwidth bursts are distrubing their VoIP, so they ask for a pause. We can provide, but 1% of the users having the problem will find it and use it, the rest will suffer or just remove bitcoin.
< jonasschnelli> sipa: regarding your profiles idea: https://github.com/bitcoin/bitcoin/issues/8437, how could setting profile values make sense? I guess in bitcoin.conf would be ideal (not another file), this would mean, it must be something like -profile0-par or -profile1-maxconnections, etc.
< GitHub110> [bitcoin] laanwj closed pull request #8561: Show "end" instead of many zeros when getheaders request received with a hashStop of Null (master...LessGetheadersZeros) https://github.com/bitcoin/bitcoin/pull/8561
< GitHub129> bitcoin/master 259ee09 R E Broadley: Show "end" instead of many zeros when getheaders request received with a hashStop of Null.
< GitHub129> bitcoin/master 6c9f1b8 Wladimir J. van der Laan: Merge #8561: Show "end" instead of many zeros when getheaders request received with a hashStop of Null...
< GitHub129> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/53f8f226bd1d...6c9f1b8c2405
< jonasschnelli> I'd like to merge https://github.com/bitcoin/bitcoin/pull/8371 soon... thanks for reviews
< GitHub164> [bitcoin] laanwj closed pull request #8595: [Wallet] Ensure <0.13 clients can't open HD wallets (master...2016/07/hd_minversion) https://github.com/bitcoin/bitcoin/pull/8595
< GitHub48> [bitcoin] fivepiece opened pull request #8598: Fix displaying of invalid and non-minimal small pushes as numbers (master...fixasm) https://github.com/bitcoin/bitcoin/pull/8598
< GitHub134> [bitcoin] jonasschnelli closed pull request #8135: [OSX] fix make deploy when compiling on OSX (master...2016/06/makedeploy) https://github.com/bitcoin/bitcoin/pull/8135
< GitHub61> [bitcoin] rebroad opened pull request #8597: [WIP] Move code from VERACK to VERSION (since VERACK is not requied) (master...NotDependentOnVerack) https://github.com/bitcoin/bitcoin/pull/8597
< dcousens> phantomcircuit: nevermind, its just bitcoin-cli being smart :)
< GitHub198> [bitcoin] rebroad opened pull request #8596: Feeler code and debugging. (master...FeelerFixes) https://github.com/bitcoin/bitcoin/pull/8596
< kanzure> 18:53 < kanzure> saurik: bitcoin core just a bunch of people hanging out on irc doing stuff. so i pinged you about it.
< kanzure> 16:54 < saurik> so, I just spent a bunch of time making certain I didn't miss anything via IRC, and I've also checked e-mail. were they intending to tell me anything? I am kind of in shock that of all projects, bitcoin, another open source project, is now continuing the pattern of "we found something mysterious that we fixed that we don't bother mentioning directly to upstream"
< GitHub143> [bitcoin] rebroad opened pull request #8595: [Wallet] Ensure <0.13 clients can't open HD wallets (master...2016/07/hd_minversion) https://github.com/bitcoin/bitcoin/pull/8595

2016-08-25

< GitHub34> [bitcoin] gmaxwell opened pull request #8594: Do not add random inbound peers to addrman. (master...no_inbound_addr) https://github.com/bitcoin/bitcoin/pull/8594
< gmaxwell> jonasschnelli: that is going a bit far, ultimately if your computere is totally compromised you probably can't use bitcoin correctly no matter having a hardware wallet or offline signing or what not.. at most you can slow the bleeding.
< gmaxwell> jonasschnelli: in the case of the detatched signer which I think we should use in the GUI by default, it would be the encrypted master key. But bitcoin-core wouldn't really know or care what it is.
< jonasschnelli> If we scale and all uses that jump on the nicely scaled bitcoin bus and use webbased or heavy centarlized insecure walltes... we can stop focus on consensus and p2p
< CodeShark> wallet stuff really needs to be a separate project - bitcoin-core-dev should focus on consensus and p2p stuff :p
< btcdrak> yes. detached signing is more for the bitcoin-dev ML
< wumpus> it makes no sense to try to support detached signing in bitcoin core until there is some kind of standard, if every hw vendor is going to hook in their own code in their own way it isgoing to be a mess
< jonasschnelli> To shortly come back to afk11 topic: there has been a proposal for detached signing: https://lists.linuxfoundation.org/pipermail/bitcoin-dev/2016-August/013008.html
< jonasschnelli> and sandboxing is impossible for bitcoin core right now
< jonasschnelli> OSX code signatures are from "Bitcoin Foundation"... anyone could hold the key
< petertodd> jonasschnelli: I suspect the code-signature-verification is what increases the security of bitcoin-core :)
< jonasschnelli> The question is, does code-signing really increases the security of bitcoin-core...
< sipa> oh, let's just switch bitcoin to use prime factoring as PoW
< petertodd> afk11: that's off-topic to Bitcoin Core development right now
< gmaxwell> #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
< sipa> we need #bitcoin-core-org-dev :)
< wumpus> please keep this channel restricted to bitcoin core development
< btcdrak> btw, this is the PR https://github.com/bitcoin/bitcoin/pull/7351
< sipa> who is bitcoin.org's maintainers, and who is bitcoin.org itself?
< luke-jr> what happened to bitcoin.org's maintainers (not bitcoin.org itself) offering to host bitcoincore.org also?
< afk11> bitcoincore.org is behind CloudFlare - following the gitian instructions lead to a 403 if CloudFlare decides it doesn't like you. https://github.com/bitcoin/bitcoin/blob/master/doc/release-process.md#fetch-and-create-inputs-first-time-or-when-dependency-versions-change
< luke-jr> MarcoFalke: https://github.com/bitcoin/bitcoin/pull/6996 my rebase already has the tests taken care of too..
< GitHub39> [bitcoin] jl2012 opened pull request #8593: Verify all incoming txs unless the witness stripped size is >100kB (master...verifyalltx) https://github.com/bitcoin/bitcoin/pull/8593
< GitHub7> [bitcoin] hejunbok closed pull request #8592: 0.13 (master...0.13) https://github.com/bitcoin/bitcoin/pull/8592
< GitHub80> [bitcoin] hejunbok opened pull request #8592: 0.13 (master...0.13) https://github.com/bitcoin/bitcoin/pull/8592
< GitHub76> [bitcoin] sipa closed pull request #8591: Do not relay or mine excessive sighash transactions (master...NoExcessiveSighashTxs) https://github.com/bitcoin/bitcoin/pull/8591
< GitHub149> [bitcoin] MarcoFalke closed pull request #8579: Performance: Prefer prefix operator for non-primitive types (master...Mf1608-perfIter) https://github.com/bitcoin/bitcoin/pull/8579
< GitHub181> [bitcoin] rebroad opened pull request #8591: Do not relay or mine excessive sighash transactions (master...NoExcessiveSighashTxs) https://github.com/bitcoin/bitcoin/pull/8591
< GitHub172> [bitcoin] sipa closed pull request #8462: Move AdvertiseLocal debug output to net category (master...advertiselocal) https://github.com/bitcoin/bitcoin/pull/8462
< GitHub69> bitcoin/master 53f8f22 Pieter Wuille: Merge #8462: Move AdvertiseLocal debug output to net category...
< GitHub69> bitcoin/master f13c1ba Michael Rotarius: Move AdvertiseLocal debug output to net category
< GitHub69> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/0606f95b1ea8...53f8f226bd1d
< jonasschnelli> Travis failed on master: https://travis-ci.org/bitcoin/bitcoin/jobs/155043866#L2347
< GitHub83> [bitcoin] MarcoFalke opened pull request #8590: Remove unused variables (master...Mf1608-unusedCode) https://github.com/bitcoin/bitcoin/pull/8590
< GitHub179> bitcoin/master 0606f95 Jonas Schnelli: Merge #7579: [Qt] show network/chain errors in the GUI...
< GitHub28> [bitcoin] jonasschnelli closed pull request #7579: [Qt] show network/chain errors in the GUI (master...2016/02/gui_alert) https://github.com/bitcoin/bitcoin/pull/7579
< GitHub179> bitcoin/master 2f32c82 Jonas Schnelli: [Qt] show network/chain errors in the GUI
< GitHub179> [bitcoin] jonasschnelli pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/9d0f43b7ca72...0606f95b1ea8
< GitHub120> [bitcoin] sipa closed pull request #8226: contributing.md: Fix formatting (line lengths and smart quotes) (master...binns-contributing-formatting) https://github.com/bitcoin/bitcoin/pull/8226
< GitHub54> bitcoin/master 9d0f43b Pieter Wuille: Merge #8226: contributing.md: Fix formatting (line lengths and smart quotes)...
< GitHub54> bitcoin/master be1d451 Will Binns: contributing.md: Fix formatting...
< GitHub54> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/d26234a9e2fa...9d0f43b7ca72
< GitHub60> [bitcoin] sipa closed pull request #7984: Optional parameter for rescans to start at a specified height (master...rescan-fromheight) https://github.com/bitcoin/bitcoin/pull/7984
< GitHub70> [bitcoin] sipa closed pull request #8452: Code simplification: inline CTxInWitness inside CTxIn (master...segwitinline) https://github.com/bitcoin/bitcoin/pull/8452
< GitHub152> [bitcoin] sipa opened pull request #8589: Inline CTxInWitness inside CTxIn (on top of #8580) (master...segwitinlinepain) https://github.com/bitcoin/bitcoin/pull/8589
< GitHub68> [bitcoin] jonasschnelli closed pull request #8487: Persist the datadir after option reset (master...persist-datadir) https://github.com/bitcoin/bitcoin/pull/8487
< GitHub187> bitcoin/master d26234a Jonas Schnelli: Merge #8487: Persist the datadir after option reset...
< GitHub187> bitcoin/master 57acb82 Andrew Chow: Load choose datadir dialog after options reset
< GitHub187> bitcoin/master 15df3c1 Andrew Chow: Persist the datadir after option reset...
< GitHub187> [bitcoin] jonasschnelli pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/95a983d56dbd...d26234a9e2fa
< luke-jr> wumpus: I wonder if a non-clean build dir (old bitcoin-config.h?) might also cause this
< wumpus> luke-jr: I had this problem with qt5 too, with the out-of-tree changes, btw https://github.com/bitcoin/bitcoin/pull/7911#issuecomment-212413442 But that was fixed. Bu tmay be a similar issue
< luke-jr> https://github.com/bitcoin/bitcoin/issues/8586 should probably be reopened.
< GitHub76> [bitcoin] MarcoFalke closed pull request #8578: [test] Remove unused code (master...Mf1608-qaUnused) https://github.com/bitcoin/bitcoin/pull/8578
< GitHub185> bitcoin/master 95a983d MarcoFalke: Merge #8578: [test] Remove unused code...
< GitHub185> bitcoin/master fa1cf9e MarcoFalke: [test] Remove unused code
< GitHub185> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/026c6edac947...95a983d56dbd
< wumpus> I'm not sure what to do with "leveldb: generate lib independent of locale sort" https://github.com/bitcoin/bitcoin/pull/8575 it's a change to leveldb,and we no longer need it since 0.13.0, and we likely won't do a leveldb subtree update anymore for 0.12.x.
< GitHub197> [bitcoin] laanwj closed pull request #8459: [0.13] release-notes: Do not encourage changing blockmaxsize to blockmaxweight (0.13...0.13_relnotes_remove_bad_advice) https://github.com/bitcoin/bitcoin/pull/8459
< GitHub71> [bitcoin] laanwj closed pull request #8282: net: Feeler connections to increase online addrs in the tried table. (master...feelers3) https://github.com/bitcoin/bitcoin/pull/8282
< GitHub189> bitcoin/master 026c6ed Wladimir J. van der Laan: Merge #8282: net: Feeler connections to increase online addrs in the tried table....
< GitHub189> bitcoin/master dbb1f64 Ethan Heilman: Added feeler connections increasing good addrs in the tried table....
< GitHub189> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/62a5a8a01866...026c6edac947
< gmaxwell> wumpus: bitcoin script
< GitHub120> [bitcoin] rebroad opened pull request #8587: Provide bloom services to whitelisted nodes. (master...WhitelistedBloom) https://github.com/bitcoin/bitcoin/pull/8587
< GitHub188> bitcoin/0.13 f2306fb Wladimir J. van der Laan: doc: Clean out release notes after 0.13.0 release
< GitHub188> [bitcoin] laanwj pushed 1 new commit to 0.13: https://github.com/bitcoin/bitcoin/commit/f2306fbe01426ce11c4df6a5c1b837106bc2c702
< gmaxwell> jonasschnelli: re, service bit comment, that was my thinking too but I didn't make that argument because I didn't want to encourage another bitcoin classic like sybil attack. :)
< wumpus> does anyone happen to have the digest mail from bitcoin-dev or core-dev containing the 0.13.0 announcement?
< wumpus> people without GPG can't read it anymore, but who cares, they don't take security seriously so shouldn't be using bitcoin in the first place right :)
< paveljanik> I think we should abandon GPG and Bitcoin sign everything...
< jl2012> wumpus: please do not include any email address within a PGP signed message. The signature is invalid because the "@" is replaced by "at": https://lists.linuxfoundation.org/pipermail/bitcoin-dev/2015-June/009045.html
< GitHub124> [bitcoin] pstratem opened pull request #8585: Remove last caller of IncOrderPosNext (master...2016-08-24-cwallet-incorderposnext) https://github.com/bitcoin/bitcoin/pull/8585
< GitHub109> [bitcoin] rebroad opened pull request #8583: Show XTHIN in GUI (master...ShowXTHINinGUI) https://github.com/bitcoin/bitcoin/pull/8583

2016-08-24

< GitHub45> [bitcoin] MarcoFalke opened pull request #8581: [wallet] rpc: Drop misleading option (master...Mf1608-walletDropRpc) https://github.com/bitcoin/bitcoin/pull/8581
< GitHub18> [bitcoin] sipa opened pull request #8580: Make CTransaction actually immutable (master...pain) https://github.com/bitcoin/bitcoin/pull/8580
< GitHub185> [bitcoin] MarcoFalke opened pull request #8579: Performance: Prefer prefix operator for non-primitive types (master...Mf1608-perfIter) https://github.com/bitcoin/bitcoin/pull/8579
< GitHub13> [bitcoin] MarcoFalke opened pull request #8578: [test] Remove unused code (master...Mf1608-qaUnused) https://github.com/bitcoin/bitcoin/pull/8578
< jonasschnelli> Oh. Bitcoin-Core 0.13.0 does not run on OSX 10.7, sanity check issue
< jonasschnelli> MarcoFalke, paveljanik: interested in testing an ACK/NACK https://github.com/bitcoin/bitcoin/pull/7826?
< GitHub94> [bitcoin] jonasschnelli closed pull request #8463: [qt] Remove Priority from coincontrol dialog (master...Mf1608-qtPrio) https://github.com/bitcoin/bitcoin/pull/8463
< GitHub112> bitcoin/master 62a5a8a Jonas Schnelli: Merge #8463: [qt] Remove Priority from coincontrol dialog...
< GitHub112> bitcoin/master fa8dd78 MarcoFalke: [qt] Remove Priority from coincontrol dialog
< GitHub112> [bitcoin] jonasschnelli pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/85d4e21a6178...62a5a8a01866
< GitHub2> [bitcoin] laanwj closed pull request #8567: Add default port numbers to REST doc (master...patch-1) https://github.com/bitcoin/bitcoin/pull/8567
< GitHub145> bitcoin/master 85d4e21 Wladimir J. van der Laan: Merge #8567: Add default port numbers to REST doc...
< GitHub145> bitcoin/master c911035 djpnewton: Add default port numbers to REST doc
< GitHub145> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/21857d2bf746...85d4e21a6178
< wumpus> should probably update my key at bitcoin.org with all the new signatures though
< gmaxwell> someone in #bitcoin said it didn't, which primed me to conclude it didn't when I looked. :)
< wumpus> gmaxwell: do I need to do anything special for that? I just did gpg --export --armor 0x90C8019E36C2E964 > ../websites/bitcoin.org/laanwj-releases.asc and git sees no difference
< gmaxwell> wumpus: https://bitcoin.org/laanwj-releases.asc should probably come with the signature of that key by your personal key. Keyserver retention of keys that are leafs isn't always all that good.
< GitHub112> [bitcoin] laanwj closed pull request #8450: [Test] Replace rpc_wallet_tests.cpp with python RPC unit tests (master...2016-08-03-remove-rpc-wallet-tests) https://github.com/bitcoin/bitcoin/pull/8450
< GitHub65> bitcoin/master 21857d2 Wladimir J. van der Laan: Merge #8450: [Test] Replace rpc_wallet_tests.cpp with python RPC unit tests...
< GitHub65> bitcoin/master 9578333 Patrick Strateman: Remove rpc_wallet_tests.cpp
< GitHub65> bitcoin/master 25400c4 Patrick Strateman: Account wallet feature RPC tests.
< GitHub65> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/f12d2b5a8ac3...21857d2bf746
< GitHub158> [bitcoin] ajtowns opened pull request #8575: leveldb: generate lib independent of locale sort (master...leveldb-locale-reproducible) https://github.com/bitcoin/bitcoin/pull/8575
< paveljanik> Can anyone please explain to me, what is the purpose of this line? https://github.com/bitcoin/bitcoin/blob/master/src/qt/recentrequeststablemodel.h#L34?
< wumpus> (and do the necessary build system changes, analogous to Patrick's https://github.com/bitcoin/bitcoin/pull/7940)
< wumpus> I'll merge patches when necessary for bitcoin or general performance/correctness
< wumpus> bitcoin-core's is maintained, in the sense that it will always be the best version for our use
< jonasschnelli> IMO if jeff is not maintaining it, we should take care about bitcoin-core/UniValue
< wumpus> jonasschnelli: well, effectively, for bitcoin it is. I think we're the only client for univalue anyhow.
< MarcoFalke> No, I think we keep bitcoin-core/stuff only for bitcoin-core related patches
< jonasschnelli> But UniValue upstream is now bitcoin-core/univalue, right?
< wumpus> did that once, too https://github.com/laanwj/bitcoin/commit/431c1b987b34589f32f4c2d0ee0f2571ba70e349 I don't really know where that went, looks like I never did any high-level performance measurements, only benchmark of the CRC function itself
< gmaxwell> wumpus: fyi there was a thread on bitcointalk recently where someone profiled bitcoin core, and was asking why we weren't using faster sha2 because it was 35% in their benchmarks. I pointed them to your work with the SIMD sha2. They seemed to think that there would be 10x speedups, so I was sad to have to disappoint them. :)
< GitHub188> [bitcoin] sipa closed pull request #8560: Trivial: Fix two VarInt examples in serialize.h (master...fix_varint_examples) https://github.com/bitcoin/bitcoin/pull/8560
< GitHub15> bitcoin/master f12d2b5 Pieter Wuille: Merge #8560: Trivial: Fix two VarInt examples in serialize.h...
< GitHub15> bitcoin/master 7bd5ff4 Christian Barcenas: Trivial: Fix two VarInt examples in serialize.h
< GitHub15> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/f9167003d947...f12d2b5a8ac3
< sipa> i'd love to spend time on making bitcoin core function as an SPV client, and make the wallet changes that are necessary
< GitHub147> bitcoin/master f916700 Wladimir J. van der Laan: Merge #8445: Move CWallet::setKeyPool to private section of CWallet....
< GitHub83> [bitcoin] laanwj closed pull request #8445: Move CWallet::setKeyPool to private section of CWallet. (master...2016-07-01-cwallet-api-cleanup) https://github.com/bitcoin/bitcoin/pull/8445
< GitHub147> bitcoin/master 8680d3a Patrick Strateman: Move wallet initialization logic from AppInit2 to CWallet::InitLoadWallet
< GitHub147> bitcoin/master e86eb71 Patrick Strateman: Move CWallet::setKeyPool to private section of CWallet
< GitHub147> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/9358893518a1...f9167003d947
< jonasschnelli> I think https://github.com/bitcoin/bitcoin/pull/8445 is ready for merge
< jonasschnelli> There is a very small PR for those we care: :) https://github.com/bitcoin/bitcoin/pull/8564
< jonasschnelli> BTW: travis has failed on walletbackup.py randomly, the PR does not change anything there IMO: https://travis-ci.org/bitcoin/bitcoin/jobs/154141125#L1464
< GitHub21> [bitcoin] jonasschnelli opened pull request #8574: [Wallet] refactor CWallet/CWalletDB/CDB (master...2016/08/bdb_abstraction_2) https://github.com/bitcoin/bitcoin/pull/8574
< GitHub193> [bitcoin] jonasschnelli opened pull request #8573: Set jonasschnellis dns-seeder filter flag (master...2016/08/filter_seed) https://github.com/bitcoin/bitcoin/pull/8573
< GitHub21> [bitcoin] rebroad opened pull request #8572: Don't effectively blacklist pruned nodes. (master...Don'tBanPrunedNodes) https://github.com/bitcoin/bitcoin/pull/8572
< GitHub121> [bitcoin] rebroad opened pull request #8571: Don't disconnect just because a node's services have changed. (master...UnexpectedServicesNoDisconnect) https://github.com/bitcoin/bitcoin/pull/8571
< luke-jr> midnightmagic: it makes me more confident Bitcoin in general can gain user awareness and security needed to succeed
< GitHub117> [bitcoin] nomnombtc opened pull request #8568: new var DIST_CONTRIB adds useful things for packagers from contrib (master...DIST_CONTRIB) https://github.com/bitcoin/bitcoin/pull/8568

2016-08-23

< GitHub90> [bitcoin] djpnewton opened pull request #8567: Add default port numbers to REST doc (master...patch-1) https://github.com/bitcoin/bitcoin/pull/8567
< GitHub47> [bitcoin] achow101 opened pull request #8566: Easy to use gitian building script (master...gitian-build-script) https://github.com/bitcoin/bitcoin/pull/8566
< Chris_Stewart_5> Is the 'flags' hex representation right for this documentation on the bitcoin developer reference wrt to merkle blocks?
< luke-jr> PatBoy: #bitcoin
< GitHub45> [bitcoin] MarcoFalke closed pull request #8527: Take minRelayTxFee into account in FEEFILTER messages (master...clampfeefilter) https://github.com/bitcoin/bitcoin/pull/8527
< GitHub149> [bitcoin] roques closed pull request #8565: .gitignore TAGS and emacs lock files (master...gitignore) https://github.com/bitcoin/bitcoin/pull/8565
< wumpus> <a href="https://github.com/bitcoin/bitcoin/pull/7762"><a href="https://github.com/bitcoin/bitcoin/pull/7762">#7762</a></a> yes that definitely doesn't look like standards-compliant HTML
< MarcoFalke> ull/7762"><a href="https://github.com/bitcoin/bitcoin/pull/7762">#7762<
< sipa> wumpus: solution: next time just post a link to the bitcoin-core-dev LM archove page for the announcement mail :)
< wumpus> gmaxwell: the announce-list right? bitcoin-dev and bitcoin-core-dev messages should pass
< MarcoFalke> wumpus: Fingers crossed: https://github.com/bitcoin-dot-org/bitcoin.org/pull/1349
< btcdrak> verifying by copy paste from the ML archive works fine https://lists.linuxfoundation.org/pipermail/bitcoin-core-dev/2016-August/000018.html
< btcdrak> luke-jr: bitcoin-core-dev mailing list is another LF managed list.
< wumpus> this is the diff between the mail I sent and the one I received from bitcoin-announce: https://gist.github.com/laanwj/3807c6024c6c28ae06e567a63045fb25
< achow101> Copying the text from the bitcoin-dev mailing list archive verifies fine
< wumpus> bitcoin-core-dev is the same host and software as bitcoin-dev
< TD-Linux> actually no, it's bitcoin-core-dev. I haven't gotten the bitcoin-dev one yet
< wumpus> TD-Linux: that's the one from bitcoin-dev?
< luke-jr> bitcoin core ML is different from bitcoin-dev; I imagine btcdrak manages it
< sipa> who manages the bitcoin-dev list, actually?
< wumpus> I'm still fighting the bitcoin.org release announcement here https://github.com/bitcoin-dot-org/bitcoin.org/pull/1348
< michagogo> 18:47:51 <GitHub48> [bitcoin] laanwj pushed 1 new commit to master: https://github.com/bitcoin/bitcoin/commit/9358893518a19f38eab9186c768ad81c7a2f9cec 18:47:51 <GitHub48> bitcoin/master 9358893 Wladimir J. van der Laan: doc: Add historical release notes for 0.12.1 0.13.0
< otium> thank you for the great work and for our Bitcoin that you are nursing for all of us
< wumpus> bleh, having trouble with updating bitcoin.org https://github.com/bitcoin-dot-org/bitcoin.org/pull/1348
< GitHub48> bitcoin/master 9358893 Wladimir J. van der Laan: doc: Add historical release notes for 0.12.1 0.13.0
< GitHub48> [bitcoin] laanwj pushed 1 new commit to master: https://github.com/bitcoin/bitcoin/commit/9358893518a19f38eab9186c768ad81c7a2f9cec
< GitHub113> [bitcoin] roques opened pull request #8565: .gitignore TAGS and emacs lock files (master...gitignore) https://github.com/bitcoin/bitcoin/pull/8565
< GitHub107> [bitcoin] jonasschnelli opened pull request #8564: [Wallet] remove unused code/conditions in ReadAtCursor (master...2016/08/bdb_abstraction_1) https://github.com/bitcoin/bitcoin/pull/8564
< GitHub129> [bitcoin] ajtowns opened pull request #8563: Add configure check for -latomic (master...autoconf-latomic) https://github.com/bitcoin/bitcoin/pull/8563
< GitHub129> [bitcoin] rebroad opened pull request #8561: Show "end" instead of many zros when getheaders request received with… (master...LessGetheadersZeros) https://github.com/bitcoin/bitcoin/pull/8561
< GitHub188> [bitcoin] cbarcenas opened pull request #8560: Trivial: Fix two VarInt examples in serialize.h (master...fix_varint_examples) https://github.com/bitcoin/bitcoin/pull/8560

2016-08-22

< sipa> oh, i didn't see there was a 'bitcoin' in that URL
< aj> michagogo: there were a couple of ssl patches due to EVP_CIPHER_CTX and EVP_MD_CTX and ECDSA_SIG becoming opaque that are still needed as of rc3, https://github.com/ajtowns/bitcoin-deb/blob/master/debian/patches/3001_openssl_1_1_fixes.patch (kinda ugly since what works with openssl 1.1 doesn't work with openssl 1.0.2...)
< GitHub51> [bitcoin] laanwj closed pull request #8539: CDB: fix debug output (master...patch-1) https://github.com/bitcoin/bitcoin/pull/8539
< GitHub93> bitcoin/master 41d8e78 Wladimir J. van der Laan: Merge #8539: CDB: fix debug output...
< GitHub93> bitcoin/master fab2e26 crowning-: CDB: fix debug output...
< GitHub93> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/37e77c106cb5...41d8e78f94c5
< sipa> it's the directory name for your signatures under https://github.com/bitcoin-core/gitian.sigs/tree/master/0.13.0rc1-linux for example
< sipa> bitcoin core 0.13 ARM signature validation speed on Nexus 5X: 1.03ms/txin
< Thireus> it's on the release page of the Bitcoin github
< wumpus> b89b2d68db7fee03cf1db6331386f1a56c2d759a31f745f0283858df6ebc986c src/bitcoin-0.13.0.tar.gz
< wumpus> btcdrak: oh no, the machine conspiracy behind bitcoin is out in the open now
< GitHub40> [bitcoin] jonasschnelli opened pull request #8559: Fix maxuploadtarget recommended minimum calculation (master...2016/08/max_ut) https://github.com/bitcoin/bitcoin/pull/8559
< wumpus> oh I missed the highlight here somehow: https://github.com/bitcoin/bitcoin/issues/8518
< xinxi> We may start to prove Bitcoin soon.
< GitHub62> [bitcoin] laanwj closed pull request #8554: trivial: remove unused variable (master...remove-block-tmp) https://github.com/bitcoin/bitcoin/pull/8554
< GitHub50> bitcoin/master 37e77c1 Wladimir J. van der Laan: Merge #8554: trivial: remove unused variable...
< GitHub50> bitcoin/master 4207630 Daniel Kraft: trivial: remove unused variable...
< GitHub50> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/bb0f763a253f...37e77c106cb5
< GitHub185> [bitcoin] laanwj closed pull request #8558: Add copyright header to wallet_text_fixture.cpp (master...2016_08_wallet_test_fixture_copyright) https://github.com/bitcoin/bitcoin/pull/8558
< GitHub77> bitcoin/master bb0f763 Wladimir J. van der Laan: Merge #8558: Add copyright header to wallet_text_fixture.cpp...
< GitHub77> bitcoin/master 653bb3d Wladimir J. van der Laan: Add copyright header to wallet_text_fixture.cpp...
< GitHub77> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/760741a00833...bb0f763a253f
< GitHub173> [bitcoin] MarcoFalke closed pull request #8551: [qa] Remove unused code (master...Mf1608-qaUnused) https://github.com/bitcoin/bitcoin/pull/8551
< GitHub169> bitcoin/master 760741a MarcoFalke: Merge #8551: [qa] Remove unused code...
< GitHub169> bitcoin/master faaec13 MarcoFalke: [qa] Remove unused code
< GitHub169> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/a55a018d5f2d...760741a00833
< GitHub27> [bitcoin] laanwj closed pull request #8548: [wallet] Use __func__ to get function name for output printing (master...Mf1608-walletFunc) https://github.com/bitcoin/bitcoin/pull/8548
< GitHub157> [bitcoin] laanwj closed pull request #8376: [Wallet][Trivial] Fix exception message to reference actual thrower. (master...2016-07-19-cwallet-sethmasterkey) https://github.com/bitcoin/bitcoin/pull/8376
< GitHub30> bitcoin/master a55a018 Wladimir J. van der Laan: Merge #8548: [wallet] Use __func__ to get function name for output printing...
< GitHub30> bitcoin/master fa785d1 MarcoFalke: Use __func__ to get function name for output printing
< GitHub30> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/2468292a0353...a55a018d5f2d
< GitHub22> [bitcoin] laanwj opened pull request #8558: Add copyright header to wallet_text_fixture.cpp (master...2016_08_wallet_test_fixture_copyright) https://github.com/bitcoin/bitcoin/pull/8558

2016-08-21

< GitHub71> [bitcoin] MarcoFalke opened pull request #8557: [contrib] verifybinaries: Adjust parsing to new rc path (master...Mf1608-verifyBins) https://github.com/bitcoin/bitcoin/pull/8557
< Valer> Who is Bitcoin Core Developer?

2016-08-20

< luke-jr> BlueMatt: https://github.com/bitcoin/bips/pull/423 thoughts?
< moli> c0der0, wrong channel, ask in #bitcoin
< c0der0> so this is where the developers that sign bitcoin core reside? I saw the message at bitcoin.org and needed to verify if 01EA5486DE18A882D4C2684590C8019E36C2E964 is the right key
< jonasschnelli> Segwit test failed again on travis: https://travis-ci.org/bitcoin/bitcoin/jobs/153752648
< GitHub151> [bitcoin] domob1812 opened pull request #8554: trivial: remove unused variable (master...remove-block-tmp) https://github.com/bitcoin/bitcoin/pull/8554
< GitHub40> [bitcoin] PrinceofOrange opened pull request #8552: Bitcoin 139 d4.9 (master...bitcoin-139-d4.9) https://github.com/bitcoin/bitcoin/pull/8552

2016-08-19

< GitHub4> [bitcoin] MarcoFalke opened pull request #8551: [qa] Remove unused code (master...Mf1608-qaUnused) https://github.com/bitcoin/bitcoin/pull/8551
< GitHub95> [bitcoin] jonasschnelli opened pull request #8550: [Qt] Add interactive mempool graph (master...2016/08/stats_qt) https://github.com/bitcoin/bitcoin/pull/8550
< GitHub42> [bitcoin] jmcorgan opened pull request #8549: zmq: mempool notifications (master...zmq_mempool) https://github.com/bitcoin/bitcoin/pull/8549
< GitHub179> [bitcoin] jonasschnelli closed pull request #8517: [Qt] show wallet HD state in statusbar (master...2016/08/hd_gui) https://github.com/bitcoin/bitcoin/pull/8517
< GitHub136> bitcoin/master 914154f Jonas Schnelli: [Qt] add HD enabled/disabled icon to the status bar
< GitHub136> bitcoin/master 2468292 Jonas Schnelli: Merge #8517: [Qt] show wallet HD state in statusbar...
< GitHub136> [bitcoin] jonasschnelli pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/56ac0469609a...2468292a0353
< GitHub184> [bitcoin] jonasschnelli closed pull request #8443: [Wallet] Trivial cleanup of HD wallet changes (master...2016/08/hd_fixes) https://github.com/bitcoin/bitcoin/pull/8443