2016-03-01

< 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
< GitHub102> [bitcoin] pstratem opened pull request #7553: Remove vfReachable and modify IsReachable to only use vfLimited. (master...2016-02-17-reachable) https://github.com/bitcoin/bitcoin/pull/7553

2016-02-17

< GitHub6> [bitcoin] laanwj opened pull request #7552: rpc: Add `verifyrawtransactions` call (master...2016_02_verifytransaction) https://github.com/bitcoin/bitcoin/pull/7552
< GitHub144> [bitcoin] pedrobranco opened pull request #7551: Add importmulti RPC call (master...feature/rpc-import-multi) https://github.com/bitcoin/bitcoin/pull/7551
< GitHub33> [bitcoin] laanwj opened pull request #7550: rpc: Input-from-stdin mode for bitcoin-cli (master...2016_02_cli_stdin2) https://github.com/bitcoin/bitcoin/pull/7550
< GitHub100> [bitcoin] ptschip closed pull request #7549: Fix for locking issue and compile warning (master...BUIP010_pt3) https://github.com/bitcoin/bitcoin/pull/7549
< GitHub5> [bitcoin] ptschip opened pull request #7549: Fix for locking issue and compile warning (master...BUIP010_pt3) https://github.com/bitcoin/bitcoin/pull/7549
< wumpus> phantomcircuit: yes, sandboxing that code would be nice. Although arguably, against libc exploits, no one stands a chance. If bitcoin-qt doesn't get exploited itself some other service or process will, and they'll pwn the box anyway.
< GitHub159> [bitcoin] laanwj closed pull request #7548: Correct duplicate names in release notes (0.12...fix-release-note-names) https://github.com/bitcoin/bitcoin/pull/7548
< GitHub85> bitcoin/0.12 188ca9c Wladimir J. van der Laan: Merge #7548: Correct duplicate names in release notes...
< GitHub85> bitcoin/0.12 ea52530 fanquake: Fix duplicate names in release notes...
< GitHub85> [bitcoin] laanwj pushed 2 new commits to 0.12: https://github.com/bitcoin/bitcoin/compare/b4662646352d...188ca9c305d3
< GitHub39> [bitcoin] fanquake opened pull request #7548: Correct duplicate names in release notes (0.12...fix-release-note-names) https://github.com/bitcoin/bitcoin/pull/7548
< phantomcircuit> it's attack surface alone is larger than the entire rest of bitcoin-qt

2016-02-16

< GitHub148> [bitcoin] btcdrak opened pull request #7544: Backport BIP112 implementation for 0.12 (0.12...dot12_backport_bip112) https://github.com/bitcoin/bitcoin/pull/7544
< GitHub24> [bitcoin] btcdrak opened pull request #7543: Backport BIP68 implementation for 0.12 (0.12...dot12_backport_bip68) https://github.com/bitcoin/bitcoin/pull/7543
< GitHub8> [bitcoin] morcos opened pull request #7542: Implement "feefilter" P2P message (master...feefilter) https://github.com/bitcoin/bitcoin/pull/7542
< GitHub178> [bitcoin] pinheadmz opened pull request #7541: Clarify description of blockindex (master...master) https://github.com/bitcoin/bitcoin/pull/7541
< GitHub175> [bitcoin] btcdrak opened pull request #7540: Rename OP_NOP3 to OP_CHECKSEQUENCEVERIFY (master...rename_nop3) https://github.com/bitcoin/bitcoin/pull/7540
< GitHub130> [bitcoin] sdaftuar opened pull request #7539: Add tags to mempool's mapTx indices (master...use-tags) https://github.com/bitcoin/bitcoin/pull/7539
< GitHub121> bitcoin/0.12 b466264 Wladimir J. van der Laan: doc: Remove another duplicate author name from release notes...
< GitHub121> [bitcoin] laanwj pushed 1 new commit to 0.12: https://github.com/bitcoin/bitcoin/commit/b4662646352dfe3d3f9bc46682a38973a119d0d8
< GitHub21> bitcoin/master 8b70a64 Wladimir J. van der Laan: Merge #7526: fix spelling of advertise (shows up in the debug log)...
< GitHub124> [bitcoin] laanwj closed pull request #7526: fix spelling of advertise (shows up in the debug log) (master...advertize-advertise) https://github.com/bitcoin/bitcoin/pull/7526
< GitHub21> bitcoin/master 37767fd jloughry: fix spelling of advertise in src and doc
< GitHub21> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/93c85d458ac3...8b70a64d62c6
< GitHub17> bitcoin/master a381076 BtcDrak: Code style fix....
< GitHub115> [bitcoin] laanwj closed pull request #7524: BIP-112: Mempool-only CHECKSEQUENCEVERIFY (master...checksequenceverify) https://github.com/bitcoin/bitcoin/pull/7524
< GitHub17> bitcoin/master c3c3752 BtcDrak: Separate CheckLockTime() and CheckSequence() logic...
< GitHub17> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/2d4f73f47e52...93c85d458ac3
< GitHub17> bitcoin/master 53e53a3 Mark Friedenbach: BIP112: Implement CHECKSEQUENCEVERIFY...
< GitHub162> [bitcoin] laanwj closed pull request #7509: Common argument defaults for NODE_BLOOM stuff and -wallet (master...common_defaults_0.12) https://github.com/bitcoin/bitcoin/pull/7509
< GitHub9> bitcoin/master 2d4f73f Wladimir J. van der Laan: Merge #7509: Common argument defaults for NODE_BLOOM stuff and -wallet...
< GitHub9> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/ce5fc02e61c8...2d4f73f47e52
< GitHub9> bitcoin/master 1fb91b3 Luke Dashjr: Common argument defaults for NODE_BLOOM stuff and -wallet
< GitHub56> [bitcoin] laanwj closed pull request #7536: test: test leading spaces for ParseHex (master...2016_02_test_parsehex_leading_space) https://github.com/bitcoin/bitcoin/pull/7536
< GitHub14> bitcoin/master ce5fc02 Wladimir J. van der Laan: Merge #7536: test: test leading spaces for ParseHex...
< GitHub14> bitcoin/master f31b6b8 Wladimir J. van der Laan: test: test leading space for ParseHex...
< GitHub14> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/19324cf09cad...ce5fc02e61c8
< wumpus> cfields: kleetus added a autotools build system to leveldb so we can do out-of-tree builds, which would be very nice to have, can you take a look to see if everything is ok? https://btcdrakgithub.com/bitcoin/leveldb/pull/11
< btcdrak> khaNH: support is in #bitcoin
< khaNH> ANY1 KNOW HOW TO USE BITCOIN CORE?

2016-02-15

< GitHub99> [bitcoin] laanwj closed pull request #7504: Crystal clean make clean (master...20160210_crystal_clean_make_clean) https://github.com/bitcoin/bitcoin/pull/7504
< GitHub176> bitcoin/master ae6eca0 Pavel Janík: make clean should clean .a files
< GitHub176> bitcoin/master 19324cf Wladimir J. van der Laan: Merge #7504: Crystal clean make clean...
< GitHub176> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/18cb2d509055...19324cf09cad
< GitHub35> [bitcoin] laanwj closed pull request #7531: Add bip68-sequence.py to extended rpc tests (master...bip68-rpc) https://github.com/bitcoin/bitcoin/pull/7531
< GitHub136> bitcoin/master 18cb2d5 Wladimir J. van der Laan: Merge #7531: Add bip68-sequence.py to extended rpc tests...
< GitHub136> bitcoin/master 6ba8b2a BtcDrak: Add bip68-sequence.py to extended rpc tests
< GitHub136> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/bf1e113311fe...18cb2d509055
< morcos> Here is the basic idea for a feefilter: https://gist.github.com/morcos/9aab223c443c9258c979 , implemented here: https://github.com/morcos/bitcoin/tree/feefilter
< cypherBlock> for segwit transactions types that are nested in P2SH (p2sh-p2wsh & p2sh-p2wpkh), is the intent that those are there primarily for un upgraded wallets to send bitcoin to upgraded wallets? Any other usage?
< GitHub131> [bitcoin] laanwj opened pull request #7537: wallet: Warn on unexpected EOF while salvaging wallet (master...2016_02_salvage_unexpected_eof) https://github.com/bitcoin/bitcoin/pull/7537
< GitHub119> [bitcoin] laanwj opened pull request #7536: test: test leading spaces for ParseHex (master...2016_02_test_parsehex_leading_space) https://github.com/bitcoin/bitcoin/pull/7536
< GitHub85> [bitcoin] laanwj closed pull request #7101: Display coinjoins properly in QT (master...master) https://github.com/bitcoin/bitcoin/pull/7101
< wumpus> anyone willing to pick this up? https://github.com/bitcoin/bitcoin/issues/7516 "Missing new RPC calls in 0.12 release notes"
< GitHub26> [bitcoin] laanwj closed pull request #7487: Workaround Travis-side CI issues (master...depends_curl) https://github.com/bitcoin/bitcoin/pull/7487
< GitHub166> bitcoin/master c01f08d Luke Dashjr: Bugfix: depends/Travis: Use --location (follow redirects) and --fail [on HTTP error response] with curl
< GitHub166> bitcoin/master 1ecbb3b Luke Dashjr: depends: Use curl for fetching on Linux...
< GitHub166> bitcoin/master 5d1148c Luke Dashjr: Travis: Use curl rather than wget for Mac SDK
< GitHub166> [bitcoin] laanwj pushed 5 new commits to master: https://github.com/bitcoin/bitcoin/compare/9b9bfcec5af8...bf1e113311fe
< GitHub20> [bitcoin] laanwj closed pull request #7528: autogen.sh: warn about needing autoconf if autoreconf is not found (master...warn-autoconf) https://github.com/bitcoin/bitcoin/pull/7528
< GitHub181> bitcoin/master 9b9bfce Wladimir J. van der Laan: Merge #7528: autogen.sh: warn about needing autoconf if autoreconf is not found...
< GitHub181> bitcoin/master 889426d Andrés G. Aragoneses: autogen.sh: warn about needing autoconf if autoreconf is not found...
< GitHub181> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/db2e1c0650c9...9b9bfcec5af8
< GitHub26> [bitcoin] laanwj closed pull request #7527: [Trivial, RPC] Fix and cleanup listreceivedbyX documentation (master...listfix) https://github.com/bitcoin/bitcoin/pull/7527
< GitHub103> bitcoin/master db2e1c0 Wladimir J. van der Laan: Merge #7527: [Trivial, RPC] Fix and cleanup listreceivedbyX documentation...
< GitHub103> bitcoin/master c372572 instagibbs: Fix and cleanup listreceivedbyX documentation
< GitHub103> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/80d1f2e48364...db2e1c0650c9

2016-02-14

< GitHub156> [bitcoin] luke-jr opened pull request #7534: Minimal RPC & wallet support for CLTV-enabled multisig addresses (master...cltv_address) https://github.com/bitcoin/bitcoin/pull/7534
< GitHub189> [bitcoin] luke-jr opened pull request #7533: RPC: sendrawtransaction: Allow the user to ignore/override specific rejections (master...sendraw_force) https://github.com/bitcoin/bitcoin/pull/7533

2016-02-13

< MarcoFalke> Looks fine because we did not change the packet: https://github.com/bitcoin/bitcoin/blob/master/doc/gitian-building.md#installing-gitian
< MarcoFalke> I can't remember updating gitian. This happened right after win-rc5 : https://github.com/bitcoin/gitian.sigs/pull/304
< GitHub134> [bitcoin] btcdrak opened pull request #7531: Add bip68-sequence.py to extended rpc tests (master...bip68-rpc) https://github.com/bitcoin/bitcoin/pull/7531
< GitHub39> [bitcoin] knocte opened pull request #7530: autogen.sh: check for libtool before automake fails to find it (master...libtool-check) https://github.com/bitcoin/bitcoin/pull/7530
< cfields> 46ef9d35b4fbd89e74065647173ae0bbe8e04c6b47413a421576b0800a04ef37 bitcoin-0.12.0-linux32.tar.gz
< cfields> wumpus: c713f82859a27e9e0f9e7fb926f074bef855e255bfbb1207ce6987534233137a bitcoin-0.12.0-linux64.tar.gz
< wumpus> cfields: Good news: for osx, the dmg and -osx64.tar.gz stayed the same, just bitcoin-0.12.0-osx-unsigned.tar.gz changed. Same as you!
< wumpus> c713f82859a27e9e0f9e7fb926f074bef855e255bfbb1207ce6987534233137a for bitcoin-0.12.0-linux64.tar.gz
< warren> wumpus: cfields and I have been talking for a few months about an eventual replacement, basic idea is to be able to build a deterministic toolchain on any Linux distro, then use that to build things like Bitcoin.
< GitHub67> [bitcoin] luke-jr opened pull request #7529: Bugfix: Rename descendantfees to descendantmodfees (master...bugfix_descendantfees) https://github.com/bitcoin/bitcoin/pull/7529

2016-02-12

< Luke-Jr> wumpus: GreenBits apparently runs Bitcoin Core Daemon on Android these days
< wumpus> there's not even an android bitcoin core release, let alone blackberry
< wumpus> I know most of the bitcoin core codebase in detail, though I sometimes get confused by the wallet and some of the mempool workings
< Luke-Jr> since when is ~/.bitcoin/testnet3/bitcoin.conf ignored and why?
< GitHub136> [bitcoin] knocte opened pull request #7528: autogen.sh: warn about needing autoconf if autoreconf is not found (master...warn-autoconf) https://github.com/bitcoin/bitcoin/pull/7528
< wumpus> f153f1555b60edd61fa57a9b2f6b43d8d6449fd5c0e6a4ef0f4ba745c6376566 bitcoin-0.12.0-osx-unsigned.tar.gz didn't change for me cfields, osx output is the same
< cfields> wumpus: c427d4076e1827d5fd3d645d26f3e1504ffe8b53c33c559c5dc49605d0b10cbd bitcoin-0.12.0-osx-unsigned.tar.gz
< GitHub167> [bitcoin] instagibbs opened pull request #7527: [RPC] Fix and listreceivedbyX documentation (master...listfix) https://github.com/bitcoin/bitcoin/pull/7527
< cfields> 4098c47e08a882892b2a2b88761cf688ae11d6a5a1d02270a154d60c0393e7fb bitcoin-0.12.0-win-unsigned.tar.gz
< wumpus> + b785149cc0cc56dfb1c28626300f85ff8ea1b9658f8016bd9b202e32180a2bd9 bitcoin-0.12.0-win64.zip
< wumpus> - b13f362e4efbcdcc539398010ef3b287209dc497a057f1f86805cc610c1e6796 bitcoin-0.12.0-win64.zip
< GitHub81> [bitcoin] jloughry opened pull request #7526: fix spelling of advertise (shows up in the debug log) (master...advertize-advertise) https://github.com/bitcoin/bitcoin/pull/7526
< GitHub164> [bitcoin] btcdrak opened pull request #7524: BIP-112: Mempool-only CHECKSEQUENCEVERIFY (master...checksequenceverify) https://github.com/bitcoin/bitcoin/pull/7524
< wumpus> cfields: so bench_bitcoin, bitcoin-cli, bitcoin-tx match - bitcoind, bitcoin-qt, test-bitcoin and test-bitcoin-qt differ
< wumpus> more gitian problems, this time building macosx. What is happening? https://github.com/bitcoin/gitian.sigs/pull/304
< GitHub95> [bitcoin] laanwj closed pull request #7184: Implement SequenceLocks functions for BIP 68 (master...alternate68) https://github.com/bitcoin/bitcoin/pull/7184
< GitHub174> bitcoin/master a51c79b Alex Morcos: Bug fix to RPC test
< GitHub174> bitcoin/master da6ad5f Suhas Daftuar: Add RPC test exercising BIP68 (mempool only)
< GitHub174> bitcoin/master c6c2f0f Alex Morcos: Implement SequenceLocks functions...
< GitHub174> [bitcoin] laanwj pushed 5 new commits to master: https://github.com/bitcoin/bitcoin/compare/621940e04090...80d1f2e48364
< GitHub160> [bitcoin] laanwj closed pull request #7520: LibreSSL doesn't define OPENSSL_VERSION, use LIBRESSL_VERSION_TEXT instead (master...20160211_LibreSSL_compile_fix) https://github.com/bitcoin/bitcoin/pull/7520
< GitHub105> bitcoin/master a0a17b3 Pavel Janík: LibreSSL doesn't define OPENSSL_VERSION, use LIBRESSL_VERSION_TEXT instead
< GitHub105> bitcoin/master 621940e Wladimir J. van der Laan: Merge #7520: LibreSSL doesn't define OPENSSL_VERSION, use LIBRESSL_VERSION_TEXT instead...
< GitHub105> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/2f3f4af4cc2b...621940e04090
< GitHub26> bitcoin/0.12 02d707f Wladimir J. van der Laan: Merge #7523: Fix of semantic failure "meet pay"...
< GitHub26> [bitcoin] laanwj pushed 2 new commits to 0.12: https://github.com/bitcoin/bitcoin/compare/04503f78c750...02d707ff37a1
< GitHub26> bitcoin/0.12 e473c2d wodry: Fix of semantic failure "meet pay"...