2016-03-01

< GitHub73> [bitcoin] laanwj pushed 4 new commits to 0.11: https://github.com/bitcoin/bitcoin/compare/c40ec1421048...a0cfe3a9e6c5
< GitHub130> [bitcoin] luke-jr opened pull request #7625: Bugfix: Check for bench_bitcoin being enabled where needed, and skip UniValue dependency when unused (master...bugfix_bench_checks) https://github.com/bitcoin/bitcoin/pull/7625
< GitHub168> bitcoin/0.12 ca8f160 Luke Dashjr: Bugfix: gitian: Add curl to packages (now needed for depends)...
< GitHub168> [bitcoin] laanwj pushed 1 new commit to 0.12: https://github.com/bitcoin/bitcoin/commit/ca8f160af5a54d08f8dc73acd959b0a73a7b427c
< GitHub64> bitcoin/0.10 4e1134b Luke Dashjr: Bugfix: gitian: Add curl to packages (now needed for depends)...
< GitHub64> [bitcoin] laanwj pushed 1 new commit to 0.10: https://github.com/bitcoin/bitcoin/commit/4e1134bdf1acff669c0f489934ac5f919c634d69
< GitHub49> [bitcoin] laanwj closed pull request #7614: Bugfix: gitian: Add curl to packages (now needed for depends) (master...depends_curl) https://github.com/bitcoin/bitcoin/pull/7614
< GitHub172> bitcoin/master 732c010 Wladimir J. van der Laan: Merge #7614: Bugfix: gitian: Add curl to packages (now needed for depends)...
< GitHub172> bitcoin/master e5daa2e Luke Dashjr: Merge branch 'master' into depends_curl
< GitHub172> bitcoin/master 5c70a6d Luke Dashjr: Bugfix: gitian: Add curl to packages (now needed for depends)
< GitHub172> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/f5ecd0737130...732c01089601
< GitHub127> bitcoin/0.12 a10da9a MarcoFalke: [depends] builders: No need to set -L and --location for curl...
< GitHub127> bitcoin/0.12 00d57b4 Luke Dashjr: Workaround Travis-side CI issues...
< GitHub127> [bitcoin] laanwj pushed 2 new commits to 0.12: https://github.com/bitcoin/bitcoin/compare/35af157641dd...a10da9aa4933
< btcdrak> so we could change the fallback URL to https://github.com/bitcoin-core/depends-fallback/releases/download/src/ but the best would be to setup a permanent redirect from dev.bitcoincore.org -> https://github.com/bitcoin-core/depends-fallbacks/releases/download/src/ then if for any reason the github repo changed you can just update the redirect and now we dont
< wumpus> btcdrak: dev.bitcoin.org may go away any time
< GitHub142> bitcoin/0.12 35af157 Wladimir J. van der Laan: doc: Clean out release notes...
< GitHub142> [bitcoin] laanwj pushed 1 new commit to 0.12: https://github.com/bitcoin/bitcoin/commit/35af157641ddbf6090e86edff7533d45ee4fb990
< GitHub104> [bitcoin] laanwj closed pull request #7624: [doc] Missing credit added (0.12...patch-2) https://github.com/bitcoin/bitcoin/pull/7624
< GitHub140> bitcoin/master f5ecd07 Wladimir J. van der Laan: doc: Add missing credit to 0.12.0 release notes...
< GitHub140> [bitcoin] laanwj pushed 1 new commit to master: https://github.com/bitcoin/bitcoin/commit/f5ecd0737130eed8daf9d76c5232dce7e40b7150
< GitHub148> [bitcoin] crowning- opened pull request #7624: [doc] Missing credit added (0.12...patch-2) https://github.com/bitcoin/bitcoin/pull/7624
< GitHub34> [bitcoin] jonasschnelli closed pull request #7560: [OSX] fix brew openssl detection (master...2016/02/osx_openssl) https://github.com/bitcoin/bitcoin/pull/7560
< gmaxwell> mostly unchanged over the last several years" --- kristov claimed this to me in #bitcoin several weeks ago, and I refuted it at list listing pages of features.
< gmaxwell> I wonder if anyone connected to OBPP ever ran bitcoin core, their screenshot is four years old. It's especially disappointing that it claims "the Qt client has remained
< btcdrak> This channel is for Bitcoin Core development discussion. please move to #bitcoin.
< randy-waterhouse> except the report is NOT for experts ... I might be the most paranoid bitcoin user out there (of the ones i know of), I use only core and that report seems to come to highly erroneous conclusions
< btcdrak> shouldnt this conversation be in #bitcoin ?
< gmaxwell> catlasshrugged: some people are. I am, and I had one person in here contact me privately and suggest that the report attacked bitcoin core specifically for political reasons related to blocksize drama (citing comments by you and other OBPP) folks. For what its worth, I told them that I didn't think that was the case here.
< catlasshrugged> I really appreciate any help people can put forth, especially with improvements to the threat model, the way that different attacks are weighted, and the correct modeling of full node clients like Bitcoin-Qt
< gmaxwell> I think bitcoin.org wallet criteria did some disclosure requirement stuff, I'm looking for it now.
< gmaxwell> bitcoin.org has disclosure requirements, IIRC.
< gmaxwell> warren: why not go work on improving Bitcoin Core's privacy documentation? :) it's scattered in many places.
< petertodd> catlasshrugged: btw, I'll note that the 'expose' made it clear that it was visible via network queries in the first line: https://www.reddit.com/r/Bitcoin/comments/447s7c/samourai_is_the_most_private_and_anonymous/
< gmaxwell> in any case, catlasshrugged offered to go break out some of the analysis on Bitcoin Core. I think that would be super useful to us--- and much better a use of time then the circular argument now.
< gmaxwell> Luke-Jr: and electrum is accordingly ranked lower than bitcoin core for privacy. I'd agree. (I think the armory developers would agree too)
< gmaxwell> Yes, I think most people would agree that electrum has significantly worse privacy than bitcoin core due to the server model.
< gmaxwell> catlasshrugged: many people have been basically begging you to substantiate placing many of these wallets that phone home the users addresses over bitcoin core. You haven't. Perhaps it will take you some research to do that-- I understand you didn't make this report all yourself--... probably you should go do that and we can continue discussions later.
< catlasshrugged> NEWS FLASH: Everyone has access to a massive information that could be used to denanonymize a large % of all bitcoin addresses -- it's called the blockchain and Google
< petertodd> catlasshrugged: right now bc.i as an example has access to a massive amount of information that could be used to deanonymise a large % of all bitcoin addresses - lets be clear on that
< catlasshrugged> what bitcoin privacy engineering did you do during those 6 years?
< catlasshrugged> aknix: can you remind me about your expertise in bitcoin privacy engineering?
< petertodd> here's an example of Samourai claiming they are an SPV wallet: https://www.reddit.com/r/Bitcoin/comments/35bynz/samurai_wallet_some_interesting_privacy_features/cr32w75
< catlasshrugged> as you've pointed out a billion times in public, it's easy to sybil the bitcoin network anyway, so it's not like SPV wallets are a lot better off
< sipa> catlasshrugged: in what way is Bitcoin-Qt's privacy worse than Samurai's?
< dirtynewshoes> I do not believe the goal of bitcoin-qt is not to be easy privacy for the average user.. but to be a solid foundation that works well
< warren> Your report would be better to exclude Bitcoin-Qt entirely, then you're at least comparing apples to apples, or "of the lite wallets which is least bad for privacy".
< sipa> catlasshrugged: then you should at least be able to give one aspect at which bitcoin-qt ranks lower than your #1
< catlasshrugged> bitcoin-qt is a wallet. I'm not going to exclude it.
< catlasshrugged> petertodd: that's relatively true, though when I wrote about the use of Bitcoin-Qt in a book, I found that people desperately needed the directions. Especially setting connecting it to Tor, etc.
< petertodd> sipa: Bitcoin Core is something non-experts can and do run mind you
< sipa> If expert bitcoin users is not the audience, perhaps you should exclude Bitcoin Core and say you consider it out of scope, instead of ranking it lower than others without any argument for doing so
< petertodd> catlasshrugged: you don't need to be an expert to run bitcoin core I'll note
< catlasshrugged> I don't think there's anything in the report that suggests that its primary audience is expert bitcoin users.
< randy-waterhouse> "If they are expert users who know their way around Bitcoin-Qt, they don't need the report." ... perhaps this caveat needs to be displayed prominently somewhere in the front of the report?
< catlasshrugged> pigeons: no. the best information that I have available (unresolved objections notwithstanding) is that users would be well served to make decisions based on the report. If they are expert users who know their way around Bitcoin-Qt, they don't need the report.
< dirtynewshoes> catlasshrugged: Would it be used by the average bitcoin user? (Stable enough?)
< gmaxwell> I'm still hoping to hear of _any_ blockchain analysis resistance features implemented by, say, ledger that are lacking in Bitcoin Core. I may not agree with the relative ranking of these two critical areas, but ... I don't think bitcoin-core should fair poorly vs the other available wallets even when blockchain analysis is ranked much more highly than network facing privacy.
< aknix> I mean cmon you recoomend a fake UI for a bitcoin wallet
< aknix> what are you gonna do open tinder to use bitcoin?
< pigeons> catlasshrugged: i didnt ask about the "average bitcoin user" I asked about "new users seeking privacy with privacy needs"
< gmaxwell> I think you do _devistating_ harm to the bitcoin ecosystem when you present privacy disaster personal data phone-homing lite wallets as _superior_.
< gmaxwell> catlasshrugged: I would arugue, and I think I would bury you in a public debate that in terms of practical privacy Bitcoin QT (or other FN wallets like armory) are strictly superior in actually delivered privacy; and-- their vastyly superior privacy is a primary reason why a user should want to use them.
< gmaxwell> I think you're underemphasicing network; esp since the primary purpose of the current analysis companies are tying addresses to identities and geographies which cannot be done without a network component; but .. I don't think we need to resolve that weighing disagreement, because I think that Bitcoin Core does at least as well as almost everyone else in both. (ignoring e.g. coinjoin functionalit
< catlasshrugged> pigeons: practically speaking, I am not concerned at all about the average bitcoin user deciding not to use Qt based on the report. The average Bitcoin user simply isn't using Qt, full stop. That's not a knock on Qt, just a statement of fact about market share.
< gmaxwell> catlasshrugged: great, now, for all the things in the list that aren't doing coinjoin; what network analytics protection do they do better than Bitcoin Core?
< gmaxwell> Not reusing addresses? Bitcoin core does as much as is possible there short of actually forbidding the users from doing it.
< gmaxwell> catlasshrugged: yes, virtually all of the wallets you've reviewed send deanonmizing data to third parties. Bitcoin Core does not. Too bad it's ranked way down on your list.
< gmaxwell> You just recommended to end users that to preserve their privacy they should run https://www.reddit.com/r/Bitcoin/comments/447s7c/samourai_is_the_most_private_and_anonymous/ If you can't acknoweldge that something is _seriously_ wrong that your process caused you to do that, or even make an _attempt_ to convince someone here that this was justfied than I don't see how futher progress is possib
< gmaxwell> If you're saying the user is the target audience then you're currently telling them to run samouri wallet over bitcoin core; a closed source wallet that phones home the users addresses.
< gmaxwell> catlasshrugged: if your message is that they all suck than your audience isn't an actual bitcoin user, it's the industry.
< catlasshrugged> how do I make that meaningful to the average Bitcoin user?
< catlasshrugged> again, I'll remind you that #bitcoin-core-dev is not the primary target audience for these reports.
< catlasshrugged> take-aways I am not looking to express: Ledger is better than Bitcoin-Qt; Bitcoin-Qt sucks!
< gmaxwell> I do think the comparison is a false dichotomy though; your highest rated wallet _I think_ does nothing better than bitcoin core for blockchain analysis resistance, but it's far weaker to 'network' and 'server' analysis resistance.
< catlasshrugged> I mention the blockchain analysis vs network stuff as probably the primary determinant of Bitcoin-Qt's rank relative to other wallets. Getting the weights is the hardest part, but I want it to be the absolute best we can make it.
< gmaxwell> petertodd: it doesn't help that many of the names on the project have been very adversarial towards Bitcoin Core in the past, work for wallets included in the report etc.
< gmaxwell> petertodd: it's not like anyone involved failed to know that if you rank "doesn't send your address info to third parties" very highly the result will be to put armory and bitcoin core very highly.
< gmaxwell> petertodd: I don't know that this has much value though; after all, I was arguing with catlasshrugged in #bitcoin a few weeks ao about the importance of not sending your address information to third parties.
< catlasshrugged> primarily what I think the bitcoin core team would want to nitpick on would be our weights.wiki page (which unfortunately does not give good detail about why we chose the weights, I'd like to improve that next edition)
< aknix> hmm thats not very bitcoin like.
< catlasshrugged> I'd be happy to send you a copy of the ratings for Bitcoin-Qt
< gmaxwell> catlasshrugged: where are the actual ratings for the wallet? I'd like to make for myself a list of things that your process rated other wallets higher than bitcoin core... both for a mixture of nitpicking and improvement.
< petertodd> catlasshrugged: yes, my way of using Bitcoin Core is definitely not an average user's way - but all the same, it's not a *negative* for privacy
< gmaxwell> catlasshrugged: well it would be, I'm not saying that I would be surprised that bitcoin core wasn't rated top or whatever, rather that things that were rated about it were ones that phone home the users addresses... thats horrifying to me. And I'd like to know what in particular you think these half dozen other wallets did better to justify the higher rating than core while they had that fundime
< petertodd> catlasshrugged: e.g. I personally use bitcoin core as a day-to-day spending wallet, and then delete my wallet.dat files regularly to prevent making mistakes that accidentally combine inputs I don't want combined
< catlasshrugged> N.B. ***no one in this room is an average Bitcoin user.***
< catlasshrugged> I would consider doing a point-by-point comparison between Ledger and Bitcoin-Qt, although I am seriously wondering whether this will be an invitation to be nitpicked to death
< gmaxwell> catlasshrugged: users cannot escape that by reusing addresses in bitcoin core, due to change.
< catlasshrugged> gmaxwell: oops, you're right about the clicks -- Bitcoin-Qt got 0 clicks for that = full score
< gmaxwell> E.g. Bitcoin core displays no static "wallet adresse", to get an address you hit a button which always gives you a fresh one.
< catlasshrugged> An expert Bitcoin user can probably do everything and more with Bitcoin-Qt that he can do with Ledger.
< gmaxwell> Bitcoin Core almost forces users to not reuse addresses, to get an old address you need several clicks (I think one is a right click).
< catlasshrugged> HD wallets are hugely useful for incentivizing users to not reuse addresses. Keep in mind, this review is focused on the average Bitcoin user largely, and not expert users.
< gmaxwell> catlasshrugged: uh.. you know that it's externally undetectable to users if bitcoin core uses HD wallets; right?
< gmaxwell> catlasshrugged: what does ledger (for example) do for resistance to blockchain analysis that Bitcoin Core does not.
< kanzure> catlasshrugged: i think you should include disclaimers like "review by bitcoin core developers has said x about this rating scheme"
< gmaxwell> catlasshrugged: well bitcoin core has "PIR" in the naieve form: send everyone the whole database is PIR. :)
< catlasshrugged> I would love to get some input from Bitcoin-Qt devs on this, but at the moment I'm not really clear on how to form a working relationship here
< * aknix> would prefer being told the truth than anything else. I have always come to some bitcoin channel for that over the years ;)
< gmaxwell> I mean, you should be embarssed that your process resulted in rating this https://www.reddit.com/r/Bitcoin/comments/447s7c/samourai_is_the_most_private_and_anonymous/ over bitcoin core.
< catlasshrugged> "... like are we being negatively marked here for doing the _only_ thing known to provide strong privacy?" nope, only Bitcoin-Qt and armory got points for this criteria, all other wallets got 0
< aknix> Also is providing a pseudo UI for a bitcoin wallet a good idea either?
< aknix> "Unless the user explicitly requests for them to be displayed, do not display Bitcoin addresses in text or QR code form, or transaction hashes" - This seems a bit cumbersome, I dont think people would treat this much differently than a check anyway..
< catlasshrugged> "onsidering those questionare answers which were mostly privacy positive (BIP 69 where kristov gave a misleading answer-- I don't think we should implement BIP69, it doesn't improve privacy and could hurt it); it's remarkable to me that they rated Bitcoin-QT poorly if thats where their information was coming from." the only criteria relevant Bitcoin-Qt received full marks for for random sorting of outputs.
< catlasshrugged> "Likewise, they rated Bitcoin-Qt at 0 for physical security;" nope, Bitcoin-Qt got a zero for physical privacy. I know there is overlap, but this wasn't a security-focused assessment. It's all 100% driven by our threat model.
< * aknix> hehheheehe bitcoin so hawt righ naow
< sipa> i don't believe that CT in its current form is acceptable for Bitcoin, due to its huge transactions and processing requirement
< gmaxwell> Considering those questionare answers which were mostly privacy positive (BIP 69 where kristov gave a misleading answer-- I don't think we should implement BIP69, it doesn't improve privacy and could hurt it); it's remarkable to me that they rated Bitcoin-QT poorly if thats where their information was coming from.

2016-02-29

< gmaxwell> Likewise, they rated Bitcoin-Qt at 0 for physical security; when I believe it's the only wallet software in their test which is hardned against timing and RF sidechannels (some of the hardware wallets, like ledger are). Most of their wallets in their test do not implement meaningful KDFs for wallet encryption, thouh Bitcoin Core does.
< gmaxwell> The reports is quite ... remarkable. For example, it places "Samourai" wallet ahead of Bitcoin-QT.... this wallet was recently in some reddit controversy when someone reverse engineered their closed source binaries and found that they were, without disclosure, sending all the user's addresses to BC.i.
< gmaxwell> Was anyone here ever contacted by a group called the "Bitcoin Privacy Project"? They just put out a report that said that we did not respond to repeated contact attempts.
< GitHub133> [bitcoin] vlamer closed pull request #7623: newcomer can think bitcoincore.org is Bitcoin homepage. (master...master) https://github.com/bitcoin/bitcoin/pull/7623
< GitHub41> [bitcoin] vlamer opened pull request #7623: newcomer can think bitcoincore.org is Bitcoin homepage. (master...master) https://github.com/bitcoin/bitcoin/pull/7623
< GitHub192> [bitcoin] TazeTSchnitzel opened pull request #7622: Increase DEFAULT_BLOCK_MAX_SIZE to 1MB (master...increaseDefaultBlockSize) https://github.com/bitcoin/bitcoin/pull/7622
< GitHub49> [bitcoin] mrbandrews opened pull request #7621: Fixes ZMQ startup with bad arguments. (master...ba-fix-zmq) https://github.com/bitcoin/bitcoin/pull/7621
< GitHub91> [bitcoin] MarcoFalke opened pull request #7620: [travis] Only run check-doc.py once (master...patch-1) https://github.com/bitcoin/bitcoin/pull/7620
< jonasschnelli> IMO it has nothing to do with the build itself... i think it should go here: https://github.com/bitcoin/bitcoin/blob/master/qa/rpc-tests/README.md
< MarcoFalke> You could create a pull to disable it instead: https://github.com/bitcoin/bitcoin/pull/6103/files#r38071295
< go1111111> a bit of friction for new devs that it might be worth it to avoid: on linux if i follow the build instructions all tests pass except for zmq_test.py, because python-zmq isn't on the build dependency list. on one hand this is fine, because it's not needed for non-devs. on the other hand, it'd be nice if tests passed after following build instructions. worth it to submit a PR like this? https://github.com/elliotolds/bitcoin/commit/bc48a5b89e1
< GitHub72> [bitcoin] laanwj closed pull request #7537: wallet: Warn on unexpected EOF while salvaging wallet (master...2016_02_salvage_unexpected_eof) https://github.com/bitcoin/bitcoin/pull/7537
< GitHub183> bitcoin/master 78e81b0 Wladimir J. van der Laan: Merge #7537: wallet: Warn on unexpected EOF while salvaging wallet...
< GitHub183> bitcoin/master ca8fb59 Wladimir J. van der Laan: wallet: Warn on unexpected EOF while salvaging wallet...
< GitHub183> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/f06af574fbb8...78e81b0bc554
< GitHub31> [bitcoin] laanwj closed pull request #7590: Improving wording related to Boost library requirements [updated] (master...patch-3) https://github.com/bitcoin/bitcoin/pull/7590
< GitHub17> bitcoin/master f06af57 Wladimir J. van der Laan: Merge #7590: Improving wording related to Boost library requirements [updated]...
< GitHub17> bitcoin/master 8c5a5fb Jonathan Cross: Improving wording related to Boost library requirements [updated]...
< GitHub17> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b53d201eab6d...f06af574fbb8
< GitHub173> [bitcoin] laanwj closed pull request #7606: [depends] builders: No need to set -L and --location for curl (master...Mf1602-curl) https://github.com/bitcoin/bitcoin/pull/7606
< GitHub24> bitcoin/master b53d201 Wladimir J. van der Laan: Merge #7606: [depends] builders: No need to set -L and --location for curl...
< GitHub24> bitcoin/master fa7a5c5 MarcoFalke: [depends] builders: No need to set -L and --location for curl
< GitHub24> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/354b03dee188...b53d201eab6d
< GitHub134> [bitcoin] laanwj closed pull request #7604: build: Remove spurious dollar sign. Fixes #7189. (master...fix_qt4_failback) https://github.com/bitcoin/bitcoin/pull/7604
< GitHub61> bitcoin/master 354b03d Wladimir J. van der Laan: Merge #7604: build: Remove spurious dollar sign. Fixes #7189....
< GitHub61> bitcoin/master 3d19193 Chris Moore: Remove spurious dollar sign. Fixes #7189.
< GitHub61> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/f39819140c30...354b03dee188
< GitHub129> [bitcoin] laanwj closed pull request #7612: [doc] Typo fix (master...master) https://github.com/bitcoin/bitcoin/pull/7612
< GitHub63> bitcoin/master f398191 Wladimir J. van der Laan: Merge #7612: [doc] Typo fix...
< GitHub63> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/18b3f1b7f625...f39819140c30
< GitHub63> bitcoin/master ff2be40 Alfie John: [doc] Typo fix...
< GitHub44> bitcoin/0.10 6164639 MarcoFalke: [depends] builders: No need to set -L and --location for curl...
< GitHub44> bitcoin/0.10 6bf4884 Luke Dashjr: Workaround Travis-side CI issues...
< GitHub44> bitcoin/0.10 dc9ae4c MarcoFalke: Fix url in .travis.yml...
< GitHub44> [bitcoin] laanwj pushed 4 new commits to 0.10: https://github.com/bitcoin/bitcoin/compare/b0c97ce31a93...12a0c0b3aac4
< GitHub147> [bitcoin] laanwj closed pull request #7607: [0.10] Fix .travis.yml (0.10...Mf1602-010travis) https://github.com/bitcoin/bitcoin/pull/7607
< GitHub133> [bitcoin] laanwj closed pull request #7619: Add missing sudo entry in gitian VM setup. (master...gitian-sudo) https://github.com/bitcoin/bitcoin/pull/7619
< GitHub168> bitcoin/master 18b3f1b Wladimir J. van der Laan: Merge #7619: Add missing sudo entry in gitian VM setup....
< GitHub168> bitcoin/master f41927e BtcDrak: Add missing sudo entry in gitian VM setup....
< GitHub168> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/317462123f8e...18b3f1b7f625

2016-02-27

< GitHub34> [bitcoin] btcdrak opened pull request #7619: Add missing sudo entry in gitian VM setup. (master...gitian-sudo) https://github.com/bitcoin/bitcoin/pull/7619
< GitHub116> [bitcoin] MarcoFalke opened pull request #7617: [doc] Minor corrections to release notes and links (master...Mf1602-trivial9) https://github.com/bitcoin/bitcoin/pull/7617
< GitHub56> [bitcoin] MarcoFalke opened pull request #7616: Revert "depends: fix Boost 1.55 build on GCC 5" (master...Mf1602-boost155) https://github.com/bitcoin/bitcoin/pull/7616
< GitHub93> [bitcoin] MarcoFalke opened pull request #7615: [wallet] Couple MIN_CHANGE with minimum relay fee (master...Mf1602-walletMinChange) https://github.com/bitcoin/bitcoin/pull/7615
< GitHub146> [bitcoin] luke-jr opened pull request #7614: Bugfix: gitian: Add curl to packages (now needed for depends) (master...depends_curl) https://github.com/bitcoin/bitcoin/pull/7614
< PRab_> gotcha. I had thought about it previously, but when I looked at https://github.com/bitcoin/bitcoin/blob/master/contrib/bitcoind.bash-completion I ran away.
< gamersg> its a pretty small change, first time looking into bitcoin code
< GitHub42> [bitcoin] GamerSg opened pull request #7613: Add autocomplete to bitcoin-qt's console window. (master...autocomplete) https://github.com/bitcoin/bitcoin/pull/7613
< GitHub88> [bitcoin] alfiedotwtf opened pull request #7612: [doc] Typo fix (master...master) https://github.com/bitcoin/bitcoin/pull/7612

2016-02-26

< GitHub104> [bitcoin] jonasschnelli opened pull request #7610: Fix memleak in TorController (master...2016/02/torctrl_memleak) https://github.com/bitcoin/bitcoin/pull/7610
< GitHub148> [bitcoin] AliceWonderMiscreations opened pull request #7609: All files related to my RPM spec file project in one commit (master...master) https://github.com/bitcoin/bitcoin/pull/7609
< GitHub80> [bitcoin] AliceWonderMiscreations closed pull request #7588: Sample RPM spec file for Bitcoin 0.12.0 (master...master) https://github.com/bitcoin/bitcoin/pull/7588
< GitHub191> [bitcoin] AliceWonderMiscreations reopened pull request #7588: Sample RPM spec file for Bitcoin 0.12.0 (master...master) https://github.com/bitcoin/bitcoin/pull/7588
< GitHub131> [bitcoin] MarcoFalke opened pull request #7608: [wallet] Move hardcoded file name out of log messages (master...Mf1602-walletFileName) https://github.com/bitcoin/bitcoin/pull/7608
< GitHub138> [bitcoin] MarcoFalke opened pull request #7607: [0.10] Fix .travis.yml (0.10...Mf1602-010travis) https://github.com/bitcoin/bitcoin/pull/7607
< GitHub138> [bitcoin] MarcoFalke opened pull request #7606: [depends] builders: No need to set -L and --location for curl (master...Mf1602-curl) https://github.com/bitcoin/bitcoin/pull/7606
< GitHub172> [bitcoin] jonasschnelli opened pull request #7605: Remove openssl info from init/log and from Qt debug window (master...2016/02/rm_openssl_log) https://github.com/bitcoin/bitcoin/pull/7605
< jtimon> petertodd: in any case, github/bitcoin/jtimon/bip9-0.12.99-bip113-just-in-case should be easy to rebase on top of both #7575 and #7566
< GitHub182> [bitcoin] dooglus opened pull request #7604: Remove spurious dollar sign. Fixes #7189. (master...fix_qt4_failback) https://github.com/bitcoin/bitcoin/pull/7604
< GitHub38> [bitcoin] jtimon closed pull request #7564: libconsensus-p2a: Preparations to decouple libconsensus from coins.o (master...libconsensus-p2a-coins-cpp-interface-0.12.99) https://github.com/bitcoin/bitcoin/pull/7564
< GitHub96> [bitcoin] JeremyRand opened pull request #7603: Build System: Use PACKAGE_TARNAME in NSIS script (master...nsis-tarname) https://github.com/bitcoin/bitcoin/pull/7603
< aj> gmaxwell: lightningbot is back on #bitcoin-dev; it was blocked from joining because it wasn't registered with nickserv fwiw

2016-02-25

< jtimon> the difference between my implementation and yours that worries me the most is https://github.com/bitcoin/bitcoin/pull/7575/files#r54179238
< jtimon> sipa btcdrak I was going to comment on https://github.com/sipa/bitcoin/commit/99f66325e83f8da3b5dfe38cad4f5fdc60bca05a#commitcomment-16313065 but I thought IRC could be more appropriate:
< GitHub22> [bitcoin] instagibbs opened pull request #7602: [WIP] [RPC] Add call zaptransaction to delete transaction from wallet (master...zaponetx) https://github.com/bitcoin/bitcoin/pull/7602
< GitHub141> [bitcoin] ebfull opened pull request #7601: [WIP] HTLC implementation in the wallet (master...zkcp) https://github.com/bitcoin/bitcoin/pull/7601
< GitHub136> [bitcoin] sdaftuar opened pull request #7600: [WIP] Mining: Select transactions using feerate-with-ancestors (master...ancestor-mining) https://github.com/bitcoin/bitcoin/pull/7600
< zooko> In current Bitcoin core.
< zooko> Great, so no matter how badly screwed up this new openssl announcement is, openssl isn't being given the opportunity to touch Bitcoin signing keys.
< jtimon> btcdrak: fwiw all my consensus PRs merged since 0.12 's fork are already backported (with anything that would make the rebase less clean) in https://github.com/jtimon/bitcoin/tree/backports-0.12 in case they were necessary for backporting SF functionality
< zooko> Does the payment protocol use Bitcoin private (signing) keys?
< jtimon> petertodd: lol, actually you guys need the time machine to review my code in the past, most of what's in https://github.com/jtimon/bitcoin/tree/libconsensus-p3 / https://github.com/jtimon/bitcoin/tree/jt I had coded in some way or another for long (although I constantly rewrite history to try to have the "most mergeable things" as the first commits)
< petertodd> gmaxwell: interesting question re: PP - does the average install of Bitcoin Core's qt wallet actually let people click on a PP link and have their wallet pop up? if not, strongly suggests it isn't being used much
< petertodd> btcdrak: well, so long as we can co-ordinate that with the bitcoin core release schedule - which admittedly is much easier if we continue to do that in minor version releases
< CodeShark> https://github.com/bitcoin/bips/pull/219 didn't even get merged over this whole starttime crap :(
< btcdrak> We also have a sort of chicken and egg situation for changing the default tx version, so I proposed this https://github.com/btcdrak/bitcoin/commit/957d59043b1eb3a2525eae6cae6a2a15b2eab401 so it can be done in two steps
< btcdrak> sipa started a BIP9 implementation in https://github.com/bitcoin/bitcoin/pull/7575
< btcdrak> Yes. I have written an ISM rollout in https://github.com/bitcoin/bitcoin/pull/7561 but we may need to consider BIP9 now
< gmaxwell> Hello. Welcome to today's meeting (bot is broken in #bitcoin-dev. Topics?
< GitHub178> [bitcoin] morcos opened pull request #7598: Refactor CreateNewBlock to be a method of the BlockAssembler class (master...BlockAssembler) https://github.com/bitcoin/bitcoin/pull/7598
< GitHub199> [bitcoin] luke-jr closed pull request #7529: Bugfix: Rename descendantfees to descendantmodfees (master...bugfix_descendantfees) https://github.com/bitcoin/bitcoin/pull/7529
< GitHub23> [bitcoin] AliceWonderMiscreations closed pull request #7588: Sample RPM spec file for Bitcoin 0.12.0 (master...master) https://github.com/bitcoin/bitcoin/pull/7588
< GitHub184> [bitcoin] promag closed pull request #7498: Add createtransaction (master...feature/rpc-createtransaction) https://github.com/bitcoin/bitcoin/pull/7498
< GitHub107> [bitcoin] promag closed pull request #6570: Add option to specify rescan starting timestamp to RPC import calls (master...feature/import-rescan-from-block-index) https://github.com/bitcoin/bitcoin/pull/6570

2016-02-24

< GitHub122> [bitcoin] sdaftuar opened pull request #7594: Mempool: Add tracking of ancestor packages (master...ancestor-tracking) https://github.com/bitcoin/bitcoin/pull/7594
< petertodd> it's right in the BIP for it too: https://github.com/bitcoin/bips/blob/master/bip-0125.mediawiki#summary
< gmaxwell> Aside: is it a known issue that bitcoin-cli will get disconnected during a rescan for importprivkey?
< GitHub99> [bitcoin] laanwj opened pull request #7592: mempool: Reduce ERROR logging for mempool rejects (master...2016_02_mempool_error_spam) https://github.com/bitcoin/bitcoin/pull/7592
< GitHub23> [bitcoin] jonathancross closed pull request #7589: Improving wording related to Boost library requirements (master...patch-2) https://github.com/bitcoin/bitcoin/pull/7589
< GitHub160> [bitcoin] jonathancross opened pull request #7590: Improving wording related to Boost library requirements [updated] (master...patch-3) https://github.com/bitcoin/bitcoin/pull/7590
< GitHub162> [bitcoin] jonathancross opened pull request #7589: Improving wording related to required Boost library requirement (master...patch-2) https://github.com/bitcoin/bitcoin/pull/7589
< GitHub199> [bitcoin] AliceWonderMiscreations opened pull request #7588: Sample RPM spec file for Bitcoin 0.12.0 (master...master) https://github.com/bitcoin/bitcoin/pull/7588
< GitHub112> [bitcoin] laanwj closed pull request #7559: [build-aux] Correct AC_PACKAGE_NAME brackets in bitcoin m4 scripts (master...correct-m4-brackets) https://github.com/bitcoin/bitcoin/pull/7559
< GitHub65> [bitcoin] laanwj closed pull request #7583: [doc] Fix typos (master...fix-typo) https://github.com/bitcoin/bitcoin/pull/7583
< GitHub51> bitcoin/master 6e4dfa1 Cédric Félizard: [doc] Fix typos
< GitHub51> bitcoin/master 3174621 Wladimir J. van der Laan: Merge #7583: [doc] Fix typos...
< GitHub51> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/8b958ab15b8c...317462123f8e
< GitHub28> [bitcoin] laanwj closed pull request #7550: rpc: Input-from-stdin mode for bitcoin-cli (master...2016_02_cli_stdin2) https://github.com/bitcoin/bitcoin/pull/7550
< GitHub84> bitcoin/master 8b958ab Wladimir J. van der Laan: Merge #7550: rpc: Input-from-stdin mode for bitcoin-cli...
< GitHub84> bitcoin/master f22f14c Wladimir J. van der Laan: doc: mention bitcoin-cli -stdin in release notes
< GitHub84> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/a08c41dfc232...8b958ab15b8c
< GitHub84> bitcoin/master 92bcca3 Wladimir J. van der Laan: rpc: Input-from-stdin mode for bitcoin-cli...
< GitHub164> [bitcoin] laanwj closed pull request #7581: Corrections of 0.12 release notes (master...patch-2) https://github.com/bitcoin/bitcoin/pull/7581
< GitHub125> [bitcoin] laanwj closed pull request #7585: Update rpcconsole.cpp (master...patch-1) https://github.com/bitcoin/bitcoin/pull/7585
< GitHub53> [bitcoin] jonasschnelli opened pull request #7586: [Qt] fix for building against LibreSSL (master...2016/02/fix_openssl_libressl) https://github.com/bitcoin/bitcoin/pull/7586
< wumpus> but yes, qt4 detection seems to be broken, see https://github.com/bitcoin/bitcoin/issues/7189
< BlueMatt> configure: WARNING: Qt dependencies not found; bitcoin-qt frontend will not be built
< btcdrak> baldur: this is correct. but you should ask these questions in #bitcoin
< jonasschnelli> BlueMatt: https://github.com/bitcoin/bitcoin/commit/9ef7c5 (unsigned) was merged by wumpus in https://github.com/bitcoin/bitcoin/pull/7153
< GitHub78> [bitcoin] AliceWonderMiscreations opened pull request #7585: Update rpcconsole.cpp (master...patch-1) https://github.com/bitcoin/bitcoin/pull/7585
< BlueMatt> jonasschnelli: oops, i meant its parent - https://github.com/bitcoin/bitcoin/commit/9ef7c5 is unsigned

2016-02-23

< GitHub47> [bitcoin] infertux opened pull request #7583: [doc] Fix typos (master...fix-typo) https://github.com/bitcoin/bitcoin/pull/7583
< kanzure> instagibbs: slack notifications for bitcoin irc mentions is pretty cool, it works.
< kanzure> zooko: it's manual, sadly. the mailing list that moderated emails are sent to is configured to only allow email from bitcoin-dev-owner@lists.linuxfoundation.org, that's the only configuration detail. when rejecting email, the moderator has to type bitcoin-dev-moderation@lists.ozlabs.org into the bitcoin-dev moderation queue interface.
< zooko> How was the bitcoin-dev mailing list GNU Mailman configured to hold all postings for moderation?
< GitHub108> [bitcoin] instagibbs opened pull request #7581: Corrections of 0.12 release notes (master...patch-2) https://github.com/bitcoin/bitcoin/pull/7581
< wumpus> instagibbs: sure, although the longer you wait the less sense it makes (and you'll need to apply to both bitcoin.org and bitcoin now)
< GitHub107> bitcoin/master a08c41d Wladimir J. van der Laan: doc: include post-mortem fixes to 0.12.0 release notes...
< GitHub107> [bitcoin] laanwj pushed 1 new commit to master: https://github.com/bitcoin/bitcoin/commit/a08c41dfc23234064da322159652b44684df9375
< jonasschnelli> 3xgitian sigs available now: https://github.com/bitcoin/gitian.sigs/pull/322
< jtimon> s/looking for/ expecting about zmq in bitcoin core
< wumpus> (zmq in bitcoin core, I mean)
< JackH> yes I read about it, but how exactly do we use it together with bitcoin? do you bind it to 8333?
< jonasschnelli> Or, you can take a risk and download my 0.12.0 build: https://bitcoin.jonasschnelli.ch/releasebuilds/v0.12.0/
< jonasschnelli> The one you referred is https://github.com/jonasschnelli/bitcoin/commit/453c567?

2016-02-22

< GitHub72> [bitcoin] jonasschnelli opened pull request #7579: [Qt] show network/chain errors in the GUI (master...2016/02/gui_alert) https://github.com/bitcoin/bitcoin/pull/7579
< GitHub133> [bitcoin] jonasschnelli opened pull request #7577: [Wallet] move "load wallet phase" to CWallet (master...2016/02/wallet_ref2) https://github.com/bitcoin/bitcoin/pull/7577
< GitHub124> [bitcoin] jonasschnelli opened pull request #7576: [Wallet] move wallet help string creation to CWallet (master...2016/02/wallet_helpstr) https://github.com/bitcoin/bitcoin/pull/7576

2016-02-21

< GitHub60> [bitcoin] sipa opened pull request #7575: Minimal BIP9 implementation (master...bip9) https://github.com/bitcoin/bitcoin/pull/7575
< GitHub13> [bitcoin] NicolasDorier opened pull request #7574: Remove STANDARD_LOCKTIME_VERIFY_FLAGS and mempool policy's flags (master...removeFlag) https://github.com/bitcoin/bitcoin/pull/7574
< GitHub77> [bitcoin] mruddy opened pull request #7573: P2P: add trustsystemclock command line option (master...trust-system-clock) https://github.com/bitcoin/bitcoin/pull/7573
< GitHub54> [bitcoin] mruddy opened pull request #7570: Net: Add IPv6 Scoped Address Support (master...ipv6-link-local) https://github.com/bitcoin/bitcoin/pull/7570
< GitHub98> [bitcoin] raedah opened pull request #7569: show coinjoins clearly in qt transaction list (master...fixtxdisplay) https://github.com/bitcoin/bitcoin/pull/7569

2016-02-20

< GitHub149> [bitcoin] dgenr8 opened pull request #7568: Corrections to bad-chain alert triggering (master...fix_bad_chain_alert_trigger) https://github.com/bitcoin/bitcoin/pull/7568
< GitHub136> [bitcoin] vlamer closed pull request #7567: Merge pull request #3 from bitcoin/0.12 (0.12...0.12) https://github.com/bitcoin/bitcoin/pull/7567
< GitHub93> [bitcoin] vlamer opened pull request #7567: Merge pull request #3 from bitcoin/0.12 (0.12...0.12) https://github.com/bitcoin/bitcoin/pull/7567
< GitHub137> [bitcoin] jtimon opened pull request #7566: WIP: Implement BIP9 and get BIP113 to be deployed with it as an example (master...bip9-0.12.99) https://github.com/bitcoin/bitcoin/pull/7566
< GitHub176> [bitcoin] jtimon opened pull request #7565: bip9/bip113/libconsensus-p2a: Deployment preparations forBIP113 + #7552 + Introduce Consensus::VerifyTx() (master...libconsensus-p2a-verifytx-bip113-0.12.99) https://github.com/bitcoin/bitcoin/pull/7565
< GitHub137> [bitcoin] jtimon opened pull request #7564: libconsensus-p2a: Preparations to decouple libconsensus from coins.o (master...libconsensus-p2a-coins-cpp-interface-0.12.99) https://github.com/bitcoin/bitcoin/pull/7564
< GitHub61> [bitcoin] jtimon opened pull request #7563: libconsensus-p2a: Decouple pow.o from chain.o and move it to the consensus package (master...libconsensus-p2a-chain-cpp-interface-0.12.99) https://github.com/bitcoin/bitcoin/pull/7563

2016-02-19

< GitHub189> [bitcoin] btcdrak opened pull request #7562: Bump transaction version default to 2 (master...txversionbump) https://github.com/bitcoin/bitcoin/pull/7562
< GitHub87> [bitcoin] btcdrak opened pull request #7561: IsSuperMajority() softfork for BIPs 68,112 and 113 (master...softfork) https://github.com/bitcoin/bitcoin/pull/7561
< Chris_Stewart_5> Documentation oriented pull requests for the bitcoin core code base is more than welcome right?
< GitHub113> [bitcoin] fanquake reopened pull request #7559: [build-aux] Correct AC_PACKAGE_NAME brackets in bitcoin m4 scripts (master...correct-m4-brackets) https://github.com/bitcoin/bitcoin/pull/7559
< GitHub85> [bitcoin] fanquake closed pull request #7559: [build-aux] Correct AC_PACKAGE_NAME brackets in bitcoin m4 scripts (master...correct-m4-brackets) https://github.com/bitcoin/bitcoin/pull/7559
< GitHub134> [bitcoin] jonasschnelli opened pull request #7560: [OSX] fix brew openssl detection (master...2016/02/osx_openssl) https://github.com/bitcoin/bitcoin/pull/7560
< GitHub98> [bitcoin] fanquake opened pull request #7559: [build-aux] Correct AC_PACKAGE_NAME brackets in bitcoin m4 scripts (master...correct-m4-brackets) https://github.com/bitcoin/bitcoin/pull/7559
< wumpus> cfields: btw: I think you introduced the SCRIPT_ERR codes, does checking them in a test, as in https://github.com/bitcoin/bitcoin/pull/7517, make sense?
< wumpus> not sure it's fair to talk of a bus factor here. In principle someone else could sign, he doesn't have the one golden key that blocks bitcoin releases forever
< wumpus> cfields signs, I upload the bins to bitcoin.org, good to not have one person do both
< wumpus> michagogo: not much - libc is linked dynamically, so in every case a distro upgrade of libc will also fix bitcoin core
< GitHub176> [bitcoin] laanwj closed pull request #7541: Clarify description of blockindex (master...master) https://github.com/bitcoin/bitcoin/pull/7541
< GitHub62> bitcoin/master b6e00af Wladimir J. van der Laan: Merge #7541: Clarify description of blockindex...
< GitHub62> bitcoin/master 7eef1d0 Matthew Zipkin: Clarify description of blockindex...
< GitHub62> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/0e3ccbfb26b1...b6e00af8193f
< GitHub92> [bitcoin] instagibbs opened pull request #7558: Add importprunedfunds rpc call (master...prunedforport) https://github.com/bitcoin/bitcoin/pull/7558
< raedah> what is (fAllFromMe > mine) checking? and where does ISMINE_SPENDABLE come from? https://github.com/bitcoin/bitcoin/blob/master/src/qt/transactionrecord.cpp#L84-L98

2016-02-18

< midnightmagic> (just for tor though, the bitcoin stuff is unchanged)
< midnightmagic> yeah, the tor stuff also verifies tag signatures, so as part of my modifications to get it co-existing with bitcoin gitian I have to maintain a minor fork. gitian doesn't play nice with itself.
< btcdrak> If anyone wants to review the 0.12 blog post, most welcome https://github.com/bitcoin-core/bitcoincore.org/pull/116
< GitHub51> [bitcoin] morcos opened pull request #7557: Encapsulate options for mempool policy (master...policyOptions) https://github.com/bitcoin/bitcoin/pull/7557
< GitHub54> [bitcoin] morcos opened pull request #7556: Some cleanup work for mempool and policy estimator (master...cleanupMempool) https://github.com/bitcoin/bitcoin/pull/7556
< GitHub125> [bitcoin] laanwj closed pull request #7555: Merge pull request (master...master) https://github.com/bitcoin/bitcoin/pull/7555
< GitHub21> [bitcoin] vlamer opened pull request #7555: Merge pull request (master...master) https://github.com/bitcoin/bitcoin/pull/7555
< GitHub159> bitcoin/master 0e3ccbf Wladimir J. van der Laan: doc: Add historical release notes for 0.10.4 0.11.2 and 0.12.0
< GitHub159> [bitcoin] laanwj pushed 1 new commit to master: https://github.com/bitcoin/bitcoin/commit/0e3ccbfb26b11ea3d9ed7dfd39886d69097286e1
< GitHub107> [bitcoin] laanwj closed pull request #7429: 0.12 release notes: Sort by importance to usability (0.12...0.12-sort-release-notes) https://github.com/bitcoin/bitcoin/pull/7429