2016-08-24

< 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
< GitHub187> bitcoin/master 7e5d94d Jonas Schnelli: [Wallet] Trivial cleanup of HD wallet changes
< GitHub187> bitcoin/master 56ac046 Jonas Schnelli: Merge #8443: [Wallet] Trivial cleanup of HD wallet changes...
< GitHub187> [bitcoin] jonasschnelli pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/f4e777819c54...56ac0469609a
< GitHub196> [bitcoin] MarcoFalke opened pull request #8548: [wallet] Use __func__ to get function name for output printing (master...Mf1608-walletFunc) https://github.com/bitcoin/bitcoin/pull/8548
< GitHub20> [bitcoin] btcdrak opened pull request #8547: Update btcdrak key with new expiry dates (master...btcdrak-key) https://github.com/bitcoin/bitcoin/pull/8547
< jonasschnelli> For creating the signatures, I guess it can be a third party tool.. don't need to be part of Bitcoin-Core
< jonasschnelli> we provide the verification-feature in Bitcoin-Qt and in the new cli only verify tool
< gmaxwell> Yes, but using a web wallet because bitcoin core install required an extra, foreign, step would not be. :)
< gmaxwell> I mean the bitcoin core update and the signatures for it should be a single file. The download/verifying tool can both be included in it, and available seperately (for the first time user).
< gmaxwell> By two files I mean "bitcoin" and "the witness" if there are two files someone watching the network can tell which users check and which don't, and someone checking on an offline computer has two files to download and copy across, so they won't.
< gmaxwell> it's important that its also easy to use the tool without it making network connections, so that it can be used on offline hosts, and also not "phone home" every time bitcoin is installed, which would make it easy to track who is using it. (both who is using bitcoin and who is using it with/without verifying-- we want a network attacker to not be able to tell users who verify from users who don'
< jonasschnelli> gmaxwell, sipa: what do you think about the following approach: https://bitcoin.jonasschnelli.ch/ecdsa_sig.txt
< GitHub142> [bitcoin] mcccs closed pull request #8538: Remove IP transaction check (master...Ip-check) https://github.com/bitcoin/bitcoin/pull/8538
< GitHub140> [bitcoin] mcccs opened pull request #8546: Remove IP transaction check [CLEAN] (master...abc123) https://github.com/bitcoin/bitcoin/pull/8546
< GitHub3> [bitcoin] laanwj closed pull request #8461: document return value of networkhashps for getmininginfo RPC endpoint (master...rpcMiningHelp) https://github.com/bitcoin/bitcoin/pull/8461
< GitHub39> bitcoin/master f4e7778 Wladimir J. van der Laan: Merge #8461: document return value of networkhashps for getmininginfo RPC endpoint...
< GitHub39> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/36404aeec8c1...f4e777819c54
< GitHub39> bitcoin/master 65f4532 Jameson Lopp: document return value of networkhashps for getmininginfo RPC endpoint
< GitHub3> [bitcoin] MarcoFalke opened pull request #8545: [doc] Update git-subtree-check.sh README (master...Mf1608-doc) https://github.com/bitcoin/bitcoin/pull/8545
< GitHub158> [bitcoin] laanwj closed 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
< GitHub82> bitcoin/master 36404ae Wladimir J. van der Laan: Merge #8540: qt: Fix random segfault when closing "Choose data directory" dialog...
< GitHub82> bitcoin/master b4a9aa5 Wladimir J. van der Laan: qt: Fix random segfault when closing "Choose data directory" dialog...
< GitHub82> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/8250de13587e...36404aeec8c1
< jonasschnelli> (for core, except your using -server/d with bitcoin-cli
< jouke> They should wake up indeed. But I lost a bit of confidence in "people" to be able to do so since I started with Bitcoin. On the other hand, a police officer once told me: "These people are allowed to vote".
< wumpus> that's a much bigger problem than just bitcoin
< wumpus> stupid fuckers, sometimes I'm about to believe the story that bitcoin is just making it easy for people to be scammed out of their money
< LeMiner> And perhaps a topic message in #bitcoin that states to be cautious when people try to "help" you through pm's
<@sipa> of course, then people may get users to run bitcoin-qt with -server...
< wumpus> <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. :(
< da2ce7> jonasschnelli we need a constant 'Project CN', so that somebody doesn't release 'bitcotn-core' and project will ignore all the signing policies. - The verify app should also spit out warnings: "bitcotn-core" is very similar to "bitcoin-core".
< 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