2016-08-19

< luke-jr> then your clients should hire people of their own to work as Bitcoin Core developers; off-topic anyway, #bitcoin
< catsAREnotSECURE> signing key. During this "unofficial" meeting in #bitcoin, there were a minimum of 123 lines of comment from Wladimir van der Laan, all of which are not included in the "official" meeting log. Given the recent advisory on the website, regarding "state actors", there is speculation amoung my clients that conflict of interest disclosure and transparency may be insufficient amoung the Bitcoin Core developers.
< da2ce7> catsAREnotSECURE, well, I'm sorry for your mistake, please move it back to #bitcoin, or elsewhere.
< catsAREnotSECURE> This question was intended for midnightmagic. The conversation began in #bitcoin, but for transparency reasons, I sought to move it to this channel.
< catsAREnotSECURE> Is there public disclosure of bitcoin holdings / wallets associated with specific developers?
< jonasschnelli> sipa, gmaxwell: about the binary ecdsa signing. We could create a standard ecdsa sig along with the GPG assets sig. Something like bitcoin-osx-0.13-build.assert.ecsig, where we just place a 64byte compact sig. ecdsa(sha256(sha256(<filecontent>)), P)
< GitHub95> [bitcoin] jonasschnelli closed pull request #8541: Trivial: Fix typos in various files (master...various-typos) https://github.com/bitcoin/bitcoin/pull/8541
< GitHub144> [bitcoin] petertodd opened pull request #8543: Use ANYONECANPAY if -spendzeroconfchange=0 (master...2016-08-anyonecanpay-if-spendzeroconfchange-disabled) https://github.com/bitcoin/bitcoin/pull/8543

2016-08-18

< gmaxwell> Someone in #bitcoin had 17 BTC stolen from them today because they came for tech support (their wallet was a couple years behind and they were wondering why they hadn't seen a payment yet) and someone wumpus and I were talking to, "moldish", pulled them into PM and got them to do a dumpprivkey. :(
< GitHub168> [bitcoin] theuni opened pull request #8542: RFC: net: Pass best block known height into net (master...pass-in-height) https://github.com/bitcoin/bitcoin/pull/8542
< BashCo> fwiw, I've attempted a PR to improve the language of the binary safety alert. https://github.com/bitcoin-dot-org/bitcoin.org/pull/1344
< gmaxwell> Kanzure, achow101. Regarding your public comments on the bitcoin.org notice.
< gmaxwell> So part of why I was deflecting on the endless rathole of the bitcoin.org stuff is because I had more to say that that I didn't fit in the scope of the meeting, and we had other things to accomplish that unfortunately we didn't have time for. :(
< btcdrak> for the record there was some discussion on the ML https://lists.linuxfoundation.org/pipermail/bitcoin-dev/2016-August/013006.html
< btcdrak> last calls for review of the 0.13.0 blog post https://github.com/bitcoin-core/bitcoincore.org/pull/199
< kanzure> wumpus: yeah, there should be a peer review process for posts to bitcoincore.org -- and bitcoin.org would be wise to adopt a similar practice.
< anchow101> Why not host binaries on GitHub and move completely off of bitcoin.orgs system
< kanzure> yes it makes sense to not use "Bitcoin Foundation" -- perhaps chaincode would be a good org to blame instead? :D (kidding- let's be nice)
< wumpus> btcdrak: we already provide torrents for people that don't want to download from bitcoin.org - it solves nothing of the verification problems ofc
< kanzure> re: posting hashes, i also suggest we consider posting hashes and maybe sigs on bitcoincore.org -- we can also ask bitcoin.org to do the same if they feel up to that
< jonasschnelli> Question: we should try to get new certificates for OSX/Win in the name of "Bitcoin Core".
< jonasschnelli> We still sign with "Bitcoin Foundation"
< kanzure> should we be complaining about hkp to the bitcoin.org people?
< jonasschnelli> Okay. I'll work on a short design and post it to bitcoin-core-dev ML
< MarcoFalke> So bitcoin-core-verify checks the release, but is part of the release... Isn't this circular?
< jonasschnelli> this would allow to build a "trusted-chain" of bitcoin-core binaries
< jonasschnelli> The only concern is – and this is why i borugh it up – the ec-pubkeys together with dev-name should be placed in bitcoin/bitcoin
< MarcoFalke> So how do you verify bitcoin-core-verify?
< wumpus> I suppose it will be a separate executable within the bitcoin core distribution, other software can also include it if they want, but that's not initially very important
< jonasschnelli> bitcoin-core-verify will list valid signatues of devs by listing devs name.
< jonasschnelli> bitcoin-core-verify <folder-or-file> -> 1) hashes file(s) 2) download gitian assets files together with ECDSA sigs 3) verify hashed against downloaded assets files 4) verify assets ECDSA sigs against in-binary-pubkeys (dev-name/pubkey)
< jonasschnelli> bitcoin-core-verify and bitcoin-core-sign
< jonasschnelli> I propose to add to cli tools to the bitcoin-core package
< luke-jr> you can gitian-build things besides Bitcoin
< 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
< GitHub128> [bitcoin] leijurv opened pull request #8541: Trivial: Fix typos in various files (master...various-typos) https://github.com/bitcoin/bitcoin/pull/8541
< GitHub103> [bitcoin] laanwj opened pull request #8540: qt: Fix random segfault when closing "Choose data directory" dialog (master...2016_08_qt_choosedatadir_crash) https://github.com/bitcoin/bitcoin/pull/8540
< GitHub165> [bitcoin] crowning- opened pull request #8539: CDB: fix debug output (master...patch-1) https://github.com/bitcoin/bitcoin/pull/8539
< jl2012> sipa: the rpc-test for https://github.com/bitcoin/bitcoin/pull/8533 seems ok now. I replaced you low_s signature hack with actually transforming the S value
< GitHub121> [bitcoin] mcccs closed pull request #8537: Trivial: little typos (master...litttle-typos) https://github.com/bitcoin/bitcoin/pull/8537
< GitHub101> [bitcoin] mcccs opened pull request #8538: Remove IP transaction check (master...Ip-check) https://github.com/bitcoin/bitcoin/pull/8538
< GitHub62> [bitcoin] sipa closed pull request #8453: Bring secp256k1 subtree up to date with master (master...2016_08_update_secp256k1) https://github.com/bitcoin/bitcoin/pull/8453
< GitHub188> bitcoin/master 8250de1 Pieter Wuille: Merge #8453: Bring secp256k1 subtree up to date with master...
< GitHub188> bitcoin/master 0237096 Wladimir J. van der Laan: Merge commit 'b2135359b3ad37cf2ac09b008079ddb237eff2c9'
< GitHub188> bitcoin/master b213535 Wladimir J. van der Laan: Squashed 'src/secp256k1/' changes from 6c527ec..7a49cac...
< GitHub188> [bitcoin] sipa pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/35f64e45c207...8250de13587e
< GitHub88> [bitcoin] mcccs opened pull request #8537: Trivial: little typos (master...litttle-typos) https://github.com/bitcoin/bitcoin/pull/8537
< GitHub32> [bitcoin] MarcoFalke opened pull request #8536: [WIP] [qa] Adjust timeouts for micro-optimization of run time (master...Mf1608-qaOptSync) https://github.com/bitcoin/bitcoin/pull/8536
< GitHub68> bitcoin/master 35f64e4 Wladimir J. van der Laan: Revert "[qa] Adjust timeouts for micro-optimization of run time"...
< GitHub68> [bitcoin] laanwj pushed 1 new commit to master: https://github.com/bitcoin/bitcoin/commit/35f64e45c207960078eef58ccc50d91e4abc2c55
< wumpus> looks like I reported the issue correct then here https://github.com/bitcoin/bitcoin/issues/8532
< btcdrak> look at jl2012 PR yesterday https://travis-ci.org/bitcoin/bitcoin/builds/153047455
< GitHub13> [bitcoin] laanwj closed pull request #8534: [travis] Drop java (master...Mf1608-qaJava) https://github.com/bitcoin/bitcoin/pull/8534
< GitHub31> bitcoin/master 671fdae Wladimir J. van der Laan: Merge #8534: [travis] Drop java...
< GitHub31> bitcoin/master fa0afde MarcoFalke: [travis] Drop java
< GitHub31> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/a78f95a976de...671fdae5f5b3
< jonasschnelli> And store ECDSA pubkeys of each gitian builder in our bitcoin/bitcoin git
< jonasschnelli> For the binary verification problem, we should have a verification process in Bitcoin-Qt/bitcoin-cli (or maybe an additional tool) that can verify a downloaded binary by downloading the gitian.sigs
< GitHub90> [bitcoin] laanwj closed pull request #8531: [qa] abandonconflict: Use assert_equal (master...Mf1608-qaAssert) https://github.com/bitcoin/bitcoin/pull/8531
< GitHub150> bitcoin/master a78f95a Wladimir J. van der Laan: Merge #8531: [qa] abandonconflict: Use assert_equal...
< GitHub150> bitcoin/master fa64306 MarcoFalke: [qa] abandonconflict: Use assert_equal
< GitHub150> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/733035bdb755...a78f95a976de

2016-08-17

< Chris_Stewart_5> In the bitcoin developer reference, when decoding a partial merkle tree it says: Fail if there are unused flag bits—except for the minimum number of bits necessary to pad up to the next full byte.
< GitHub1> [bitcoin] MarcoFalke opened pull request #8534: [travis] Drop java (master...Mf1608-qaJava) https://github.com/bitcoin/bitcoin/pull/8534
< GitHub170> [bitcoin] jl2012 closed pull request #8514: Enforce LOW_S rules on all transactions with WITNESS BIP9 parameters (master...lows) https://github.com/bitcoin/bitcoin/pull/8514
< GitHub166> [bitcoin] jl2012 opened pull request #8533: Implement LOW_S and NULLDUMMY softfork (BIP146) (master...bip146) https://github.com/bitcoin/bitcoin/pull/8533
< Anduck> bitcoin core slack has leaked my e-mail to scammers
< btcdrak> Seeking reviews of the 0.13.0 release blog post https://github.com/bitcoin-core/bitcoincore.org/pull/199
< jonasschnelli> Is there a way to locally detect unsafe iterator usage? https://travis-ci.org/bitcoin/bitcoin/jobs/152408747#L1413
< GitHub97> [bitcoin] MarcoFalke opened pull request #8531: [qa] abandonconflict: Use assert_equal (master...Mf1608-qaAssert) https://github.com/bitcoin/bitcoin/pull/8531
< GitHub95> [bitcoin] laanwj closed pull request #8504: test: Remove java comparison tool (master...2016_08_jexit) https://github.com/bitcoin/bitcoin/pull/8504
< GitHub138> bitcoin/master 733035b Wladimir J. van der Laan: Merge #8504: test: Remove java comparison tool...
< GitHub138> bitcoin/master eb0c52e Wladimir J. van der Laan: travis: Remove hostname hack...
< GitHub138> bitcoin/master 3c5251d Wladimir J. van der Laan: test: Remove java comparison tool
< GitHub138> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/e753eaeb343c...733035bdb755
< GitHub176> bitcoin/0.13 5e499e7 Luke Dashjr: doc/release-notes: Document changed bytespersigop behaviour
< GitHub176> bitcoin/0.13 40d705c Luke Dashjr: doc/release-notes: Mention the relevance of Compact Blocks on non-mining nodes' influence on network policy
< GitHub176> bitcoin/0.13 4f55293 Luke Dashjr: doc/release-notes: Misc
< GitHub55> [bitcoin] laanwj closed pull request #8490: [0.13] release notes: Mention new relevance of non-mining nodes on network policy; and misc fixes (0.13...relnotes_013_misc) https://github.com/bitcoin/bitcoin/pull/8490
< GitHub176> [bitcoin] laanwj pushed 4 new commits to 0.13: https://github.com/bitcoin/bitcoin/compare/4374f0ee35f8...a402396dce64
< GitHub132> [bitcoin] laanwj closed pull request #8505: Trivial: Fix typos in various files (master...replacable-typo) https://github.com/bitcoin/bitcoin/pull/8505
< GitHub146> bitcoin/master e753eae Wladimir J. van der Laan: Merge #8505: Trivial: Fix typos in various files...
< GitHub146> bitcoin/master 1aacfc2 leijurv: various typos
< GitHub146> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/65e644418106...e753eaeb343c
< GitHub120> [bitcoin] laanwj closed pull request #8513: Fix a type error that would not compile on OSX. (master...fix-osx-break) https://github.com/bitcoin/bitcoin/pull/8513
< GitHub110> bitcoin/master 65e6444 Wladimir J. van der Laan: Merge #8513: Fix a type error that would not compile on OSX....
< GitHub110> bitcoin/master 8194a6e Jeremy Rubin: Fix a type error that would not compile on Apple LLVM version 6.1.0 (clang-602.0.53) (based on LLVM 3.6.0svn)
< GitHub110> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/740cff5808c3...65e644418106
< GitHub83> [bitcoin] laanwj closed pull request #8482: [qa] Use single cache dir for chains (master...Mf1608-qaCache) https://github.com/bitcoin/bitcoin/pull/8482
< GitHub62> [bitcoin] laanwj closed pull request #8296: [qa] Don't overwrite setup_chain and sort scripts (master...Mf1607-qaSetupChain) https://github.com/bitcoin/bitcoin/pull/8296
< GitHub170> bitcoin/master fad8cf6 MarcoFalke: [qa] Use single cache dir for chains
< GitHub170> bitcoin/master fa2d68f MarcoFalke: [qa] Adjust timeouts for micro-optimization of run time
< GitHub170> bitcoin/master fae596f MarcoFalke: [qa] Sort scripts by time for pull_tester and don't overwrite setup_chain
< GitHub170> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/29496fc9e095...740cff5808c3
< GitHub20> [bitcoin] laanwj closed pull request #8512: Trivial: Corrected JSON typo on setban of net.cpp (master...setban) https://github.com/bitcoin/bitcoin/pull/8512
< GitHub138> bitcoin/master 29496fc Wladimir J. van der Laan: Merge #8512: Trivial: Corrected JSON typo on setban of net.cpp...
< GitHub138> bitcoin/master 6ffd996 Sev: Corrected JSON typo on setban of net.cpp
< GitHub138> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b41ac928b89f...29496fc9e095
< GitHub157> [bitcoin] laanwj closed pull request #8521: qa: Remove duplicate `hash160` implementation (master...2016_08_hash160_dupe) https://github.com/bitcoin/bitcoin/pull/8521
< GitHub99> bitcoin/master b41ac92 Wladimir J. van der Laan: Merge #8521: qa: Remove duplicate `hash160` implementation...
< GitHub99> bitcoin/master c1582d5 Wladimir J. van der Laan: qa: Remove duplicate `hash160` implementation...
< GitHub99> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b40e19c6c48f...b41ac928b89f
< GitHub114> [bitcoin] laanwj closed pull request #8520: build: Remove check for `openssl/ec.h` (master...2016_08_remove_openssl_ech_check) https://github.com/bitcoin/bitcoin/pull/8520
< GitHub26> bitcoin/master b40e19c Wladimir J. van der Laan: Merge #8520: build: Remove check for `openssl/ec.h`...
< GitHub26> bitcoin/master 936c144 Wladimir J. van der Laan: build: Remove check for `openssl/ec.h`...
< GitHub26> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/2c2d471e18f0...b40e19c6c48f
< luke-jr> jonasschnelli: used to be part of https://github.com/bitcoin/bips/pull/362, right?

2016-08-16

< instagibbs> this is referring to the reject filter, right? https://github.com/bitcoin/bitcoin/blob/master/qa/rpc-tests/p2p-segwit.py#L294
< GitHub92> [bitcoin] instagibbs opened pull request #8528: Update p2p-segwit.py to reflect correct AskFor behavior (master...rejectsw) https://github.com/bitcoin/bitcoin/pull/8528
< instagibbs> sdaftuar, this comment refering to the reject filter? https://github.com/bitcoin/bitcoin/blob/master/qa/rpc-tests/p2p-segwit.py#L949
< sipa> Chris_Stewart_5: the sighash type is something bitcoin specific
< GitHub26> [bitcoin] sipa opened pull request #8527: Take minRelayTxFee into account in FEEFILTER messages (master...clampfeefilter) https://github.com/bitcoin/bitcoin/pull/8527
< GitHub50> [bitcoin] jl2012 opened pull request #8526: Make non-minimal OP_IF/NOTIF argument non-standard for P2WSH (master...minimalif) https://github.com/bitcoin/bitcoin/pull/8526
< GitHub8> [bitcoin] sipa opened pull request #8525: Do not store witness txn in rejection cache (master...nowitnessreject) https://github.com/bitcoin/bitcoin/pull/8525
< GitHub76> [bitcoin] sipa opened pull request #8524: Precompute sighashes (master...noprecomputecachedhashes) https://github.com/bitcoin/bitcoin/pull/8524
< NicolasDorier> sipa: yep, just reviewed it, now I would prefer you make a new PR which supersede mine with https://github.com/sipa/bitcoin/commits/noprecomputecachedhashes, you can squash the two commits as well.
< GitHub136> bitcoin/0.13 2f58589 Pieter Wuille: Mention dump/import support for HD wallets
< GitHub136> bitcoin/0.13 fe20b83 Pieter Wuille: Remove refactors from list of changes
< GitHub136> bitcoin/0.13 7f84015 Pieter Wuille: Inline mempool RPCs and feefilter into misc sections
< GitHub97> [bitcoin] laanwj closed pull request #8519: [0.13] A few small improvements to the 0.13 release notes (0.13...relnotes-0.13) https://github.com/bitcoin/bitcoin/pull/8519
< GitHub136> [bitcoin] laanwj pushed 4 new commits to 0.13: https://github.com/bitcoin/bitcoin/compare/b52c67c4b188...4374f0ee35f8
< GitHub99> [bitcoin] laanwj opened pull request #8521: qa: Remove duplicate `hash160` implementation (master...2016_08_hash160_dupe) https://github.com/bitcoin/bitcoin/pull/8521
< Lightsword> sipa, ugh…I have that because sendrawtransaction doesn’t have an option like this https://github.com/bitcoin/bitcoin/pull/7533
< GitHub26> [bitcoin] laanwj closed pull request #8516: [trivial] remove no-longer-used InitError logic (master...deadiniterr) https://github.com/bitcoin/bitcoin/pull/8516
< GitHub37> bitcoin/master 2c2d471 Wladimir J. van der Laan: Merge #8516: [trivial] remove no-longer-used InitError logic...
< GitHub37> bitcoin/master edb6cf1 instagibbs: remove no-longer-used InitError logic
< GitHub37> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/bbd9740f534f...2c2d471e18f0
< GitHub58> [bitcoin] laanwj opened pull request #8520: build: Remove check for `openssl/ec.h` (master...2016_08_remove_openssl_ech_check) https://github.com/bitcoin/bitcoin/pull/8520
< GitHub169> [bitcoin] sipa opened pull request #8519: [0.13] A few small improvements to the 0.13 release notes (0.13...relnotes-0.13) https://github.com/bitcoin/bitcoin/pull/8519
< GitHub125> [bitcoin] laanwj closed pull request #8058: [Doc] Add issue template (master...docPRT) https://github.com/bitcoin/bitcoin/pull/8058
< GitHub162> bitcoin/master bbd9740 Wladimir J. van der Laan: Merge #8058: [Doc] Add issue template...
< GitHub162> bitcoin/master 3897668 CryptoVote: Adds issue template. [skip ci]
< GitHub162> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/6e5e5abba6f8...bbd9740f534f

2016-08-15

< GitHub176> [bitcoin] jonasschnelli opened pull request #8517: [Qt] show wallet version and HD state in debugwindow (master...2016/08/hd_gui) https://github.com/bitcoin/bitcoin/pull/8517
< GitHub30> [bitcoin] instagibbs opened pull request #8516: [trivial] remove no-longer-used InitError logic (master...deadiniterr) https://github.com/bitcoin/bitcoin/pull/8516
< GitHub68> [bitcoin] MarcoFalke closed pull request #8270: Tests: Use portable #! in python scripts (/usr/bin/env) (master...master) https://github.com/bitcoin/bitcoin/pull/8270
< GitHub119> bitcoin/master 6e5e5ab MarcoFalke: Merge #8270: Tests: Use portable #! in python scripts (/usr/bin/env)...
< GitHub119> bitcoin/master 7b01ce2 Matthew King: Favour python over python2 as per PR #7723
< GitHub119> bitcoin/master 873e81f Matthew King: Use portable #! in python scripts (/usr/bin/env)
< GitHub119> [bitcoin] MarcoFalke pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/c50386315093...6e5e5abba6f8
< GitHub137> [bitcoin] jonasschnelli closed pull request #8192: [trivial] Remove URLs from About dialog translations (master...source-code-link) https://github.com/bitcoin/bitcoin/pull/8192
< GitHub161> bitcoin/master c503863 Jonas Schnelli: Merge #8192: [trivial] Remove URLs from About dialog translations...
< GitHub161> bitcoin/master 208d37f fanquake: [trivial] Remove URLs from About dialog translations
< GitHub161> [bitcoin] jonasschnelli pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/1030fa718c59...c50386315093
< MarcoFalke> sipa: src/qt/bitcoin-qt -testnet -disablewallet -addnode=bitcoin.sipa.be -printtoconsole
< sipa> try -addnode=bitcoin.sipa.be ?
< GitHub11> [bitcoin] laanwj closed pull request #8128: Net: Turn net structures into dumb storage classes (master...split-resolve) https://github.com/bitcoin/bitcoin/pull/8128
< GitHub78> bitcoin/master b6c3ff3 Cory Fields: net: Split resolving out of CSubNet
< GitHub78> bitcoin/master f96c7c4 Cory Fields: net: Split resolving out of CService
< GitHub78> [bitcoin] laanwj pushed 9 new commits to master: https://github.com/bitcoin/bitcoin/compare/d727f77e3904...1030fa718c59
< GitHub78> bitcoin/master 31d6b1d Cory Fields: net: Split resolving out of CNetAddr
< GitHub109> [bitcoin] sipa opened pull request #8515: A few mempool removal optimizations (master...moresharedmem) https://github.com/bitcoin/bitcoin/pull/8515
< GitHub95> [bitcoin] jl2012 opened pull request #8514: Enforce LOW_S rules on all transactions with WITNESS BIP9 parameters (master...lows) https://github.com/bitcoin/bitcoin/pull/8514
< GitHub151> [bitcoin] sipa closed pull request #7946: Reduce cs_main locks during ConnectTip/SyncWithWallets (master...2016/04/cs_main_wallet) https://github.com/bitcoin/bitcoin/pull/7946
< GitHub92> bitcoin/master d727f77 Pieter Wuille: Merge #7946: Reduce cs_main locks during ConnectTip/SyncWithWallets...
< GitHub92> bitcoin/master b3b3c2a Jonas Schnelli: Reduce cs_main locks during ConnectTip/SyncWithWallets
< GitHub92> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/38590729631c...d727f77e3904
< GitHub134> [bitcoin] JeremyRubin opened pull request #8513: Fix a type error that would not compile on OSX. (master...fix-osx-break) https://github.com/bitcoin/bitcoin/pull/8513

2016-08-14

< GitHub85> [bitcoin] sevastos opened pull request #8512: Trivial: Corrected JSON typo on setban of net.cpp (master...setban) https://github.com/bitcoin/bitcoin/pull/8512
< GitHub143> [bitcoin] laanwj closed pull request #8509: Master (master...master) https://github.com/bitcoin/bitcoin/pull/8509
< GitHub5> [bitcoin] laanwj closed pull request #8510: Bitcoin 139 d4.9 (master...bitcoin-139-d4.9) https://github.com/bitcoin/bitcoin/pull/8510
< GitHub93> [bitcoin] PrinceofOrange opened pull request #8510: Bitcoin 139 d4.9 (master...bitcoin-139-d4.9) https://github.com/bitcoin/bitcoin/pull/8510
< GitHub125> [bitcoin] PrinceofOrange opened pull request #8509: Master (master...master) https://github.com/bitcoin/bitcoin/pull/8509

2016-08-13

< instagibbs> so I realized that Core on Windows has a "Verified Publisher" of "The Bitcoin Foundation Inc"
< btcdrak> Maybe we could host the RCs at https://github.com/bitcoin/bitcoin/releases/tag/v0.13.0rc3
< GitHub140> [bitcoin] leijurv closed pull request #8506: Trivial: Standardize spelling of "serialize" and "optimize" in mining.cpp and main.cpp (master...serialise-typo) https://github.com/bitcoin/bitcoin/pull/8506
< GitHub47> [bitcoin] leijurv closed pull request #8507: Trivial: Standardize spelling of "optimization" vs "optimisation" in main.cpp (master...optimization-typo) https://github.com/bitcoin/bitcoin/pull/8507
< GitHub93> [bitcoin] leijurv opened pull request #8507: Trivial: Standardize spelling of "optimization" vs "optimisation" in main.cpp (master...optimization-typo) https://github.com/bitcoin/bitcoin/pull/8507
< GitHub107> [bitcoin] leijurv opened pull request #8506: Trivial: Fix typo in mining.cpp (master...serialise-typo) https://github.com/bitcoin/bitcoin/pull/8506
< achow101> instagibbs: I have it up on github https://github.com/achow101/bitcoin/releases/tag/v0.13.0rc3
< GitHub16> [bitcoin] leijurv opened pull request #8505: Trivial: Fix typo in main.cpp (master...replacable-typo) https://github.com/bitcoin/bitcoin/pull/8505
< instagibbs> bitcoin.org is down, someone have rc3 binary link
< GitHub138> [bitcoin] laanwj opened pull request #8504: test: Remove java comparison tool (master...2016_08_jexit) https://github.com/bitcoin/bitcoin/pull/8504
< GitHub41> [bitcoin] laanwj closed pull request #8353: Trivial: tiny c++11 refactors (master...cpp11) https://github.com/bitcoin/bitcoin/pull/8353
< GitHub32> bitcoin/master c784086 whythat: use std::map::emplace() instead of std::map::insert()
< GitHub32> bitcoin/master 5e187e7 whythat: use c++11 std::unique_ptr instead of boost::shared_ptr
< GitHub32> bitcoin/master 947913f whythat: use std::map::erase(const_iterator, const_iterator) to get non-constant iterator
< GitHub32> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/efce84d888a6...38590729631c
< GitHub52> [bitcoin] laanwj closed pull request #8293: Bugfix: Allow building libbitcoinconsensus without any univalue (master...sys_univalue_opt) https://github.com/bitcoin/bitcoin/pull/8293
< GitHub49> bitcoin/master efce84d Wladimir J. van der Laan: Merge #8293: Bugfix: Allow building libbitcoinconsensus without any univalue...
< GitHub49> bitcoin/master 8a270b2 Luke Dashjr: Bugfix: Allow building libbitcoinconsensus without any univalue
< GitHub49> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/fe1780ed3eca...efce84d888a6
< GitHub130> [bitcoin] laanwj closed pull request #8492: configure: Allow building bench_bitcoin by itself (master...conf_only_bench) https://github.com/bitcoin/bitcoin/pull/8492
< GitHub117> bitcoin/master fe1780e Wladimir J. van der Laan: Merge #8492: configure: Allow building bench_bitcoin by itself...
< GitHub117> bitcoin/master 216d796 Luke Dashjr: configure: Allow building bench_bitcoin by itself
< GitHub117> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/94368d643473...fe1780ed3eca
< GitHub142> [bitcoin] laanwj closed pull request #8503: [doc] typos, text errors (master...Test-branch) https://github.com/bitcoin/bitcoin/pull/8503
< GitHub166> bitcoin/master 94368d6 Wladimir J. van der Laan: Merge #8503: [doc] typos, text errors...
< GitHub166> bitcoin/master e11f9a2 Justin Camarena: [doc] typos, READMEs, comments...
< GitHub166> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/0d0abcac1efe...94368d643473
< midnightmagic> cjcj: But this isn't the place for it. Please ask in #bitcoin.

2016-08-12

< GitHub137> [bitcoin] juscamarena opened pull request #8503: [doc] typos, text errors (master...Test-branch) https://github.com/bitcoin/bitcoin/pull/8503
< wumpus> to be clear I really like this funcitonality in the GUI, and possible in bitcoin-cli, but let's not conflate this with hypothetical JSON-RPC nested commanding
< jonasschnelli> Anyone interested in testing this: https://github.com/bitcoin/bitcoin/pull/7783?
< wumpus> but even more low-traffic. Most bitcoin developer don't like mailling lists a lot, it seems, most discussion happens on github and IRC
< jonasschnelli> There is also a bitcoin-core lf mailing list,... this would be for Bitcoin-Core topics
< jonasschnelli> JZA: bitcoin-dev is more for protocol discussion (must not be directly related to Bitcoin-Core)
< JZA> is the linux foundation bitcoin-dev the official mailing list for bitcoin-core development?
< GitHub4> [bitcoin] jonasschnelli opened pull request #8501: Add mempool statistics collector (master...2016/08/stats_rpc) https://github.com/bitcoin/bitcoin/pull/8501
< murch> jonasschnelli: I'm currently working on a Master thesis on Coin Selection. You might recognize my nickname from Bitcoin.Stackexchange.com ;)
< jonasschnelli> which bitcoin definitively isn't
< sipa> that should be on bitcoin.org even
< Lightsword> sipa, so that won’t conflict with other messages unless something else used “sendcmpct” in NetMsgType here right? https://github.com/bitcoin/bitcoin/blob/d612837814020ae832499d18e6ee5eb919a87907/src/protocol.cpp#L38

2016-08-11

< luke-jr> actually, tests could use review independently: https://github.com/bitcoin/libblkmaker/pull/7
< GitHub177> bitcoin/0.13 b52c67c Wladimir J. van der Laan: doc: Update changelog for rc3
< GitHub177> [bitcoin] laanwj pushed 1 new commit to 0.13: https://github.com/bitcoin/bitcoin/commit/b52c67c4b188c274de60fbd5e26441e9a644dba6
< GitHub73> bitcoin/0.13 9058617 Wladimir J. van der Laan: qt: translations update pre-rc3
< GitHub73> [bitcoin] laanwj pushed 1 new commit to 0.13: https://github.com/bitcoin/bitcoin/commit/9058617afb24594f09848d2a53403760bd64470a
< TD-Linux> they submitted a BIP draft to bitcoin-dev which only reserved the bit with no documentation of what it did
< murch> Lightsword: Don't let yourself quote like that. You'll face endless harrassment on Reddit. :p I'm having a hard time finding the Bitcoin Classic release notes.
< murch> By the way, while there are still lots of well-informed people reading this. Does anyone else have another list of realistic spending values for Bitcoin Nodes? I'd love to have a second scenario to test my coin selection simulation on.
< pigeons> i asked before too, got answers about gavin making changes for CIA, "only real bitcoin address starts with a 1", etc
< luke-jr> jtimon: to answer your earlier question best I can: I think they perceive 0.5.3 as the last release before the Bitcoin Foundation "got involved" (I know the BCF was never involved, but they're conspiracy nuts)
< gmaxwell> e.g. if you update your bitcoin node and then days/weeks later it starts doing thing with segwit commitments that breaks your miners or pooling software, that is preferable. You would prefer the break to happen at upgrade time.
< btcdrak> There's also a blog post being written about 0.13.0 if some eyes could review and comment https://github.com/bitcoin-core/bitcoincore.org/pull/199
< wumpus> not seeing anything here https://github.com/bitcoin/bitcoin/milestone/20
< luke-jr> on the off-chance there's no objection, I'd like to PR https://github.com/bitcoin/bitcoin/commit/5a716a3bc6621e4d2e2c1de5b6b5596d6877d589 for 0.13.0 mostly just to make #8459 uncontroversial
< wumpus> they probably write raw transactions in their head instead of using the bitcoin core wallet
< wumpus> they're not really coupled the same way in other software than bitcoin core
< jl2012> this is a conceptual PR: https://github.com/bitcoin/bitcoin/pull/8499
< 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
< GitHub118> [bitcoin] jl2012 opened pull request #8499: [Untested] Check bad witness (master...badwitnesscheck) https://github.com/bitcoin/bitcoin/pull/8499
< GitHub196> [bitcoin] laanwj closed pull request #8481: Qt: Fix minimize and close bugs (master...fix-minimize-and-close-bugs) https://github.com/bitcoin/bitcoin/pull/8481
< GitHub65> bitcoin/master 0d0abca Wladimir J. van der Laan: Merge #8481: Qt: Fix minimize and close bugs...
< GitHub65> bitcoin/master 05242e9 adlawren: Fix minimize and close bugs...
< GitHub65> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/edebf425a2df...0d0abcac1efe
< wumpus> the wallet utxo index may be helpful here, https://github.com/bitcoin/bitcoin/pull/7823
< GitHub104> [bitcoin] jtimon opened pull request #8498: Optimization: Minimize the number of times it is checked that no money... (master...0.13-consensus-inputs) https://github.com/bitcoin/bitcoin/pull/8498

2016-08-10

< jtimon> kanzure: https://github.com/bitcoin/bitcoin/pull/8493 looks like a long branch (because it has many tiny steps commits that could be squashed) but it's only +373 −94 , please check it out
< GitHub7> [bitcoin] laanwj closed pull request #8495: [0.13] Bugfix: Use pre-BIP141 sigops until segwit activates (GBT) (0.13...bugfix_gbt_sigops_presegwit-0.13.x) https://github.com/bitcoin/bitcoin/pull/8495
< GitHub52> bitcoin/0.13 8b0eee6 Luke Dashjr: Bugfix: Use pre-BIP141 sigops until segwit activates...
< GitHub52> [bitcoin] laanwj pushed 1 new commit to 0.13: https://github.com/bitcoin/bitcoin/commit/8b0eee66e9c9057b6e53bb1f4a0a3821083e5df0
< GitHub177> [bitcoin] MarcoFalke closed pull request #8477: [qa] Temporarily disable ipv6 in rpcbind test (master...Mf1608-qaIpv6) https://github.com/bitcoin/bitcoin/pull/8477
< GitHub120> [bitcoin] luke-jr opened pull request #8495: [0.13] Bugfix: Use pre-BIP141 sigops until segwit activates (GBT) (0.13...bugfix_gbt_sigops_presegwit-0.13.x) https://github.com/bitcoin/bitcoin/pull/8495
< GitHub185> bitcoin/0.13 45c656b Wladimir J. van der Laan: Merge #8465: [0.13] Document reindexing changes...
< GitHub185> bitcoin/0.13 b49d963 Pieter Wuille: Document reindexing changes
< GitHub185> [bitcoin] laanwj pushed 2 new commits to 0.13: https://github.com/bitcoin/bitcoin/compare/edc2c700a75c...45c656b914f0
< GitHub30> [bitcoin] laanwj closed pull request #8465: [0.13] Document reindexing changes (0.13...docreindex) https://github.com/bitcoin/bitcoin/pull/8465
< jonasschnelli> gmaxwell: So using tor hidden service over 443 would probably allow bypassing firewalls (assume some large company firewalls where only 80,443 is open) to connect to the bitcoin network?
< GitHub28> bitcoin/0.13 edc2c70 Wladimir J. van der Laan: Merge #8438: [0.13] backport: Treat high-sigop transactions as larger rather than rejecting them...
< GitHub79> [bitcoin] laanwj closed pull request #8438: [0.13] backport: Treat high-sigop transactions as larger rather than rejecting them (0.13...btc-13-sigops) https://github.com/bitcoin/bitcoin/pull/8438
< GitHub28> bitcoin/0.13 3f65ba2 Pieter Wuille: Treat high-sigop transactions as larger rather than rejecting them
< GitHub28> [bitcoin] laanwj pushed 2 new commits to 0.13: https://github.com/bitcoin/bitcoin/compare/114f7e944b1c...edc2c700a75c
< GitHub114> [bitcoin] laanwj closed pull request #8489: Bugfix: Use pre-BIP141 sigops until segwit activates (GBT) (master...bugfix_gbt_sigops_presegwit) https://github.com/bitcoin/bitcoin/pull/8489
< GitHub4> bitcoin/master edebf42 Wladimir J. van der Laan: Merge #8489: Bugfix: Use pre-BIP141 sigops until segwit activates (GBT)...
< GitHub4> bitcoin/master 239cbd2 Luke Dashjr: qa/rpc-tests/segwit: Test GBT sigops before and after activation
< GitHub4> bitcoin/master 160f895 Luke Dashjr: Bugfix: Use pre-BIP141 sigops until segwit activates
< GitHub4> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/484312bda2d4...edebf425a2df
< jonasschnelli> Would ICE works for Bitcoin? Or would that require UDP? (maybe off-topic here)
< jonasschnelli> Does bitcoin-cores UPNP (or UPNP in general) uses nat traversal or ICE? Or is the only way to get connection from the outside if your router supports uPNP and opens the port?
< GitHub189> [bitcoin] MarcoFalke opened pull request #8494: [init, wallet] ParameterInteraction() iff wallet enabled (master...Mf1608-initWalletParamInt) https://github.com/bitcoin/bitcoin/pull/8494
< GitHub86> [bitcoin] sipa closed pull request #8467: [Trivial] Do not shadow members in dbwrapper (master...20160805_Wshadow_dbwrapper) https://github.com/bitcoin/bitcoin/pull/8467
< GitHub128> bitcoin/master 484312b Pieter Wuille: Merge #8467: [Trivial] Do not shadow members in dbwrapper...
< GitHub128> bitcoin/master 4a35e0f Pavel Janík: Do not shadow members in dbwrapper
< GitHub128> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/6e6ab2c32382...484312bda2d4

2016-08-09

< GitHub124> [bitcoin] jtimon opened pull request #8493: Untested: libcosnensus: Expose VerifyHeader (master...0.13-consensus-header) https://github.com/bitcoin/bitcoin/pull/8493
< Chris_Stewart_5> What this documenation is describing is what really happens inside of `TraverseAndBuild` right? https://bitcoin.org/en/developer-reference#creating-a-merkleblock-message
< GitHub190> [bitcoin] jonasschnelli closed pull request #5916: add key generation type (master...2015/03/keygentype) https://github.com/bitcoin/bitcoin/pull/5916
< GitHub32> [bitcoin] luke-jr opened pull request #8492: configure: Allow building bench_bitcoin by itself (master...conf_only_bench) https://github.com/bitcoin/bitcoin/pull/8492

2016-08-08

< GitHub30> [bitcoin] JeremyRubin reopened pull request #8464: "Lockfree" Checkqueue Implementation (master...lockfree-checkqueue-squashed) https://github.com/bitcoin/bitcoin/pull/8464
< GitHub63> [bitcoin] JeremyRubin closed pull request #8464: "Lockfree" Checkqueue Implementation (master...lockfree-checkqueue-squashed) https://github.com/bitcoin/bitcoin/pull/8464
< GitHub37> [bitcoin] luke-jr opened pull request #8490: [0.13] release notes: Mention new relevance of non-mining nodes on network policy; s/HD/D/; and misc fixes (0.13...relnotes_013_misc) https://github.com/bitcoin/bitcoin/pull/8490
< GitHub3> [bitcoin] luke-jr opened pull request #8489: Bugfix: Use pre-BIP141 sigops until segwit activates (GBT) (master...bugfix_gbt_sigops_presegwit) https://github.com/bitcoin/bitcoin/pull/8489
< jonasschnelli> achow101: in case you want to compare the build log: https://bitcoin.jonasschnelli.ch/nightlybuilds/2016-08-08/build-win.log
< GitHub143> [bitcoin] maaku opened pull request #8488: Add deleteprivkey and forgetaddress RPC calls (master...forgetaddress-1) https://github.com/bitcoin/bitcoin/pull/8488
< GitHub104> [bitcoin] achow101 opened pull request #8487: Persist the datadir after option reset (master...persist-datadir) https://github.com/bitcoin/bitcoin/pull/8487
< btcdrak> There is a website PR for the release, please can you review it https://github.com/bitcoin-core/bitcoincore.org/pull/199
< GitHub19> [bitcoin] MarcoFalke opened pull request #8486: [wallet] Add high transaction fee warnings (master...Mf1607-walletHighFeeWarn) https://github.com/bitcoin/bitcoin/pull/8486