2016-04-08

< wumpus> "The only drawback that I see is that bitcoin-cli (and other RPC command-line clients) would effectively have to do two roundtrips to the server. Once to get instructions on how to convert the command-line parameters to JSON, and once to do the actual command." apparently I forgot about a thing called caching :p
< jonasschnelli> Okay... and maybe also auto-detecting JSON object in bitcoin-cli to get rid of the static conversion table.
< jonasschnelli> Implementation details and things there where to much "Core-only" for bitcoin-dev list can now be discussed on bitcoin-core-dev
< jonasschnelli> CHANELINFO: we have now a bitcoin-core-dev mailing list @ linux foundation. Please subscribe! https://lists.linuxfoundation.org/mailman/listinfo/bitcoin-core-dev

2016-04-07

< wumpus> #action review and ACK https://github.com/bitcoin/bitcoin/pull/7835/files asap
< wumpus> heh true sipa, we made a step forward with bitcoin-qt :-)
< michagogo> Sorry I'm late, but: Bitcoin Core is still in beta, no?
< wumpus> (or maybe jonasschnelli and me should start a fork of bitcoin core *ducks*)
< GitHub118> [bitcoin] sdaftuar opened pull request #7835: Version 2 transactions remain non-standard until CSV activates (master...CSV-relay-after-softfork) https://github.com/bitcoin/bitcoin/pull/7835
< wumpus> instagibbs: probably not, it's a common intermittent error: https://github.com/bitcoin/bitcoin/issues/7470
< GitHub136> [bitcoin] laanwj opened pull request #7833: tests: Check Content-Type header returned from RPC server (master...2016_04_rpc_tests_check_content_type) https://github.com/bitcoin/bitcoin/pull/7833
< wumpus> (at least the one in bitcoin)
< wumpus> make[5]: Entering directory `/home/travis/build/bitcoin/bitcoin/bitcoin-i686-w64-mingw32/src/secp256k1' random seed = 7810890d4968c3f148a3cdd563177c92 err:seh:setup_exception_record stack overflow 1040 bytes in thread 0009 eip 7bc46e38 esp 00540f20 stack 0x540000-0x541000-0x740000 FAIL: tests.exe
< wumpus> that's exactly what happens, JSON RPC errors are mapped to HTTP errors here: https://github.com/bitcoin/bitcoin/blob/master/src/httprpc.cpp#L75 I don't know what the JSON RPC spec says I think it's a bit of a level violation
< sipa> ah, it writes a bitcoin.conf file
< GitHub32> bitcoin/0.12 cada7c2 Wladimir J. van der Laan: Fill in rest of release notes
< GitHub32> bitcoin/0.12 4c3a00d Wladimir J. van der Laan: Reduce block timeout to 10 minutes...
< GitHub32> [bitcoin] laanwj pushed 2 new commits to 0.12: https://github.com/bitcoin/bitcoin/compare/90f1d246d388...cada7c2418ef
< GitHub3> bitcoin/master 5851915 Wladimir J. van der Laan: Merge #7832: Reduce block timeout to 10 minutes...
< GitHub61> [bitcoin] laanwj closed pull request #7832: Reduce block timeout to 10 minutes (master...2016_04_reduce_block_timeout) https://github.com/bitcoin/bitcoin/pull/7832
< GitHub3> bitcoin/master 62b9a55 Wladimir J. van der Laan: Reduce block timeout to 10 minutes...
< GitHub3> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/1ddf0cee679d...5851915a006a
< GitHub161> [bitcoin] laanwj opened pull request #7832: Reduce block timeout to 10 minutes (master...2016_04_reduce_block_timeout) https://github.com/bitcoin/bitcoin/pull/7832
< GitHub39> bitcoin/0.12 90f1d24 Pieter Wuille: Track block download times per individual block...
< GitHub39> [bitcoin] laanwj pushed 1 new commit to 0.12: https://github.com/bitcoin/bitcoin/commit/90f1d246d38803eb546c6652ddce5ebea55eec98
< GitHub85> [bitcoin] laanwj closed pull request #7804: Track block download times per individual block (master...betterkicktimeout) https://github.com/bitcoin/bitcoin/pull/7804
< GitHub95> bitcoin/master 1ddf0ce Wladimir J. van der Laan: Merge #7804: Track block download times per individual block...
< GitHub95> bitcoin/master 0e24bbf Pieter Wuille: Self check after the last peer is removed
< GitHub95> bitcoin/master 2d1d658 Pieter Wuille: Track block download times per individual block...
< GitHub95> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/bbaf5976af84...1ddf0cee679d
< GitHub0> bitcoin/0.12 4226aac Wladimir J. van der Laan: init: allow shutdown during 'Activating best chain...'...
< GitHub0> [bitcoin] laanwj pushed 1 new commit to 0.12: https://github.com/bitcoin/bitcoin/commit/4226aacdba7d0e1e22555dac69363b3b460a166b
< GitHub122> [bitcoin] laanwj closed pull request #7821: init: allow shutdown during 'Activating best chain...' (master...2016_04_shutdown_during_activate_best_chain) https://github.com/bitcoin/bitcoin/pull/7821
< GitHub82> bitcoin/master bbaf597 Wladimir J. van der Laan: Merge #7821: init: allow shutdown during 'Activating best chain...'...
< GitHub82> bitcoin/master 07398e8 Wladimir J. van der Laan: init: allow shutdown during 'Activating best chain...'...
< GitHub82> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/3bc71e1572cb...bbaf5976af84
< wumpus> for anyone that wants to do perf measurements of their own I've open sourced the lmdb experiment, be sure to read README.md https://github.com/laanwj/bitcoin/tree/2016_04_mdb
< GitHub72> [bitcoin] jonasschnelli opened pull request #7831: [CLI] refactor wallets RPC JSON conversions (master...2016/04/cli_conversion) https://github.com/bitcoin/bitcoin/pull/7831
< GitHub110> [bitcoin] jonasschnelli opened pull request #7830: [Wallet] Add cloned wallet, remove accounts, reset version (master...2016/04/wallet2) https://github.com/bitcoin/bitcoin/pull/7830
< jonasschnelli> wumpus: does "getlabeladdress" would still make sense for labels only? https://github.com/bitcoin/bitcoin/pull/7729/files#diff-df7d84ff2f53fcb2a0dc15a3a51e55ceR2506

2016-04-06

< jtimon> before bitcoin
< kanzure> bitcoin was your first introduction to git?
< kanzure> ... doing other tasks, making this an excellent chance for you to get feedback from me on basic contributions to Bitcoin Core. Hopefully this will help you streamline submissions to Core, please let me know how I can help and I'm also offering some promises/commitments (see below)."
< kanzure> i would start the PR text with something like: "I opened this PR so that newcomers to the Bitcoin Core project could make simple changes to get familiar with contributing to Bitcoin Core. I have marked a number of TODO comments in the source code of this pull request. The changes listed here are not critical to the project but they are good introductory material. Each TODO is relatively simple, and more experienced developers are busy ...
< kanzure> one other minor point of feedback is that if you want to attract novices to your pull request, saying "Probably nobody with bitcoin development experience will think this is a priority. I don't either." will not make it clear to novices that you think these changes are good or worth doing at all :)
< GitHub99> [bitcoin] jtimon opened pull request #7829: TODO: Kill "Params()" (master...0.12.99-todo-globals-chainparams) https://github.com/bitcoin/bitcoin/pull/7829
< GitHub50> [bitcoin] jtimon opened pull request #7828: Trivial: Globals: Explicitly pass const CChainParams& to ProcessMessage() (master...0.12.99-chainparams-trivial) https://github.com/bitcoin/bitcoin/pull/7828
< GitHub93> [bitcoin] mrbandrews opened pull request #7827: Speed up getchaintips. (master...ba-fix-chaintips) https://github.com/bitcoin/bitcoin/pull/7827
< GitHub37> [bitcoin] jonasschnelli opened pull request #7826: [Qt] show conflicts of unconfirmed transactions in the UI (master...2016/04/ui_mem_cflct) https://github.com/bitcoin/bitcoin/pull/7826
< Chris_Stewart_5> specifically looking at these lines of code https://github.com/bitcoin/bitcoin/blob/master/src/script/interpreter.cpp#L191-L197
< GitHub105> [bitcoin] laanwj closed pull request #6774: IsSuperMajority() moved to separate soft forks unit (master...ISM_to_softforks_unit) https://github.com/bitcoin/bitcoin/pull/6774
< gmaxwell> $ ./bitcoin-cli stop
< jtimon> paveljanik: thank you, see the 0bin link above. Now it is something in https://github.com/bitcoin/bitcoin/pull/7820/commits and I'm not cleaning up properly between tests
< GitHub144> [bitcoin] laanwj closed pull request #7802: [qa] httpbasics: Actually test second connection (master...Mf1604-qaTestHttp) https://github.com/bitcoin/bitcoin/pull/7802
< GitHub54> bitcoin/master 3bc71e1 Wladimir J. van der Laan: Merge #7802: [qa] httpbasics: Actually test second connection...
< GitHub54> bitcoin/master fa24456 MarcoFalke: [qa] httpbasics: Actually test second connection
< GitHub54> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/401c65c6b3e2...3bc71e1572cb
< GitHub183> [bitcoin] laanwj closed pull request #7803: [qa] maxblocksinflight: Actually enable test (master...Mf1604-qaTestMaxBlocks) https://github.com/bitcoin/bitcoin/pull/7803
< GitHub191> bitcoin/master 401c65c Wladimir J. van der Laan: Merge #7803: [qa] maxblocksinflight: Actually enable test...
< GitHub191> bitcoin/master fac724c MarcoFalke: [qa] maxblocksinflight: Actually enable test
< GitHub191> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/70ac71b877f3...401c65c6b3e2
< GitHub158> [bitcoin] laanwj closed pull request #7801: [qa] Remove misleading "errorString syntax" (master...Mf1604-qaTestErrorString) https://github.com/bitcoin/bitcoin/pull/7801
< GitHub168> bitcoin/master 70ac71b Wladimir J. van der Laan: Merge #7801: [qa] Remove misleading "errorString syntax"...
< GitHub168> bitcoin/master ffff866 MarcoFalke: [qa] Remove misleading "errorString syntax"
< GitHub168> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/1b2460bd5824...70ac71b877f3
< GitHub56> [bitcoin] pedrobranco opened pull request #7825: Prevent multiple calls to ExtractDestination (master...enhancement/prevent-multiple-calls-extractdestination) https://github.com/bitcoin/bitcoin/pull/7825
< GitHub62> [bitcoin] jonasschnelli opened pull request #7824: Add uncontroversial RBF base features (master...2016/04/rbf_uncontroversial) https://github.com/bitcoin/bitcoin/pull/7824
< GitHub154> [bitcoin] jpdffonseca opened pull request #7823: [Wallet] Add index of unspent transaction outputs (UTXOs) (master...enhancement/cache-unspent-outputs) https://github.com/bitcoin/bitcoin/pull/7823
< GitHub118> [bitcoin] jpdffonseca opened pull request #7822: [qa] Add test to RPC listunspent (master...support/add-test-listunspent) https://github.com/bitcoin/bitcoin/pull/7822
< wumpus> I've heard about bitcoin over nfc, but no never over IRyet :-)
< sipa> We need a bitcoin-over-ir protocol...
< GitHub9> [bitcoin] laanwj opened pull request #7821: init: allow shutdown during 'Activating best chain...' (master...2016_04_shutdown_during_activate_best_chain) https://github.com/bitcoin/bitcoin/pull/7821

2016-04-05

< PRab> sipa: It should. Maybe it doesn't scan old commits. I believe https://github.com/bitcoin/gitian.sigs/commits/master/0.12.0rc5-linux/prab/bitcoin-linux-0.12-build.assert should have been signed with that key.
< GitHub127> [bitcoin] jtimon opened pull request #7820: Consensus: Policy: Move CFeeRate out of consensus module and create CPolicy interface (master...0.12.99-consensus-dust-out) https://github.com/bitcoin/bitcoin/pull/7820
< MarcoFalke> Note: Please don't retrigger https://github.com/bitcoin/bitcoin/pull/7817 (just reported the issue to travis)
< GitHub174> [bitcoin] laanwj closed pull request #7654: Add net2 debug option (master...DebugNet2) https://github.com/bitcoin/bitcoin/pull/7654
< GitHub166> bitcoin/0.12 c210654 Wladimir J. van der Laan: pre-rc1 translations update...
< GitHub166> [bitcoin] laanwj pushed 1 new commit to 0.12: https://github.com/bitcoin/bitcoin/commit/c2106543fe017d443c2e50daf3dd1d42e6ec35a2
< sipa> that's how Bitcoin was in 2009, and it had massive security risks
< GitHub171> [bitcoin] laanwj closed pull request #7793: [doxygen] Fix member comments (master...Mf1604-doxygenMembers) https://github.com/bitcoin/bitcoin/pull/7793
< GitHub120> bitcoin/master 1b2460b Wladimir J. van der Laan: Merge #7793: [doxygen] Fix member comments...
< GitHub120> bitcoin/master fada0c4 MarcoFalke: [doc] Fix doxygen comments for members
< GitHub120> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/4dc1b3a29693...1b2460bd5824
< Chris_Stewart_5> sipa: If it was evaluated at as one big concatenated list though the next byte could be in the scriptPubKey. Where does the validation fail in bitcoin core?
< GitHub118> [bitcoin] laanwj closed pull request #7788: Use relative paths instead of absolute paths in protoc calls (master...20160402_protoc_use_relpath) https://github.com/bitcoin/bitcoin/pull/7788
< GitHub31> bitcoin/master 4dc1b3a Wladimir J. van der Laan: Merge #7788: Use relative paths instead of absolute paths in protoc calls...
< GitHub31> bitcoin/master 0087f26 Pavel Janík: Use relative paths instead of absolute paths
< GitHub31> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/e30a5b0aaaa9...4dc1b3a29693
< GitHub68> [bitcoin] jonasschnelli opened pull request #7819: [Qt] Simple opt-in-RBF checkbox (master...2016/04/qt_rbf_set) https://github.com/bitcoin/bitcoin/pull/7819
< Chris_Stewart_5> I'm looking at this test case from script_invalid.json inside of bitcoin core, shouldn't this have a MINIMALDATA flag?
< GitHub42> [bitcoin] laanwj closed pull request #7791: Doc: change Precise to Trusty in gitian-building.md (master...patch-1) https://github.com/bitcoin/bitcoin/pull/7791
< GitHub48> bitcoin/master e30a5b0 Wladimir J. van der Laan: Merge #7791: Doc: change Precise to Trusty in gitian-building.md...
< GitHub48> bitcoin/master 190c1e2 JeremyRand: Doc: change Precise to Trusty in gitian-building.md...
< GitHub48> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/916b15a87a1f...e30a5b0aaaa9
< GitHub175> [bitcoin] sipa opened pull request #7818: Refactor script tests (master...refactorscriptests) https://github.com/bitcoin/bitcoin/pull/7818
< GitHub49> [bitcoin] jonasschnelli opened pull request #7817: [Qt] attribute replaceable (RBF) transactions (master...2016/04/qt_rbf) https://github.com/bitcoin/bitcoin/pull/7817
< GitHub102> [bitcoin] laanwj closed pull request #7774: RPC: add versionHex in getblock and getblockheader JSON results (master...hexver) https://github.com/bitcoin/bitcoin/pull/7774
< GitHub91> bitcoin/master 916b15a Wladimir J. van der Laan: Merge #7774: RPC: add versionHex in getblock and getblockheader JSON results...
< GitHub91> bitcoin/master 92107d5 mruddy: RPC: add versionHex in getblock and getblockheader JSON results; expand data in getblockchaininfo bip9_softforks field.
< GitHub91> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/3cc0fb3a23e5...916b15a87a1f
< GitHub158> [bitcoin] jonasschnelli opened pull request #7816: [Wallet] slighly refactor GetOldestKeyPoolTime() (master...2016/04/wallet_oldest_key) https://github.com/bitcoin/bitcoin/pull/7816
< GitHub41> bitcoin/0.12 a784675 Wladimir J. van der Laan: build: Remove unnecessary executables from gitian release...
< GitHub41> [bitcoin] laanwj pushed 1 new commit to 0.12: https://github.com/bitcoin/bitcoin/commit/a784675a329d6206af125d997381221dd1e99d11
< GitHub28> [bitcoin] laanwj closed pull request #7776: build: Remove unnecessary executables from gitian release (master...2016_03_gitian_release_cleanup) https://github.com/bitcoin/bitcoin/pull/7776
< GitHub129> bitcoin/master 3cc0fb3 Wladimir J. van der Laan: Merge #7776: build: Remove unnecessary executables from gitian release...
< GitHub129> bitcoin/master f063863 Wladimir J. van der Laan: build: Remove unnecessary executables from gitian release...
< GitHub129> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/55db5f07b1c4...3cc0fb3a23e5
< GitHub91> [bitcoin] laanwj closed pull request #7781: devtools: Auto-set branch to merge to in github-merge (master...2016_04_github_merge_autobranch) https://github.com/bitcoin/bitcoin/pull/7781
< GitHub48> bitcoin/master 55db5f0 Wladimir J. van der Laan: Merge #7781: devtools: Auto-set branch to merge to in github-merge...
< GitHub48> bitcoin/master 10d3ae1 Wladimir J. van der Laan: devtools: Auto-set branch to merge to in github-merge...
< GitHub48> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/214ec0b5e8b2...55db5f07b1c4
< GitHub21> [bitcoin] laanwj closed pull request #7813: [doc] Update port in tor.md (master...Mf1604-docTor) https://github.com/bitcoin/bitcoin/pull/7813
< GitHub15> bitcoin/master 214ec0b Wladimir J. van der Laan: Merge #7813: [doc] Update port in tor.md...
< GitHub15> bitcoin/master 3373c43 Adam Brown: [doc] Update port in tor.md...
< GitHub15> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/a9149688f87c...214ec0b5e8b2
< btcdrak> kinlo: yes, it's only one way though, so you can read on the Bitcoin Core Slack
< GitHub79> [bitcoin] laanwj opened pull request #7815: Break circular dependency main ↔ txdb (master...2016_04_break_txdb_main_dep) https://github.com/bitcoin/bitcoin/pull/7815
< GitHub153> [bitcoin] MarcoFalke opened pull request #7814: [qa] Switch to py3 (master...Mf1604-qaPy3) https://github.com/bitcoin/bitcoin/pull/7814
< GitHub91> [bitcoin] MarcoFalke opened pull request #7813: [doc] Update port in tor.md (master...Mf1604-docTor) https://github.com/bitcoin/bitcoin/pull/7813
< GitHub178> [bitcoin] jonasschnelli opened pull request #7812: Tiny refactor of `IsRBFOptIn`, avoid exception (master...2016/04/rbf_refact) https://github.com/bitcoin/bitcoin/pull/7812
< jonasschnelli> sipa: I think this check is more appropriate: https://github.com/bitcoin/bitcoin/pull/7222/files#diff-d0eca4d0f80c5b045d2aa64609e811ecR28?
< jonasschnelli> Or anyone else... check https://github.com/bitcoin/bitcoin/pull/7222/files#diff-df7d84ff2f53fcb2a0dc15a3a51e55ceR85 I think this is wrong.
< GitHub37> [bitcoin] MarcoFalke opened pull request #7811: [0.12.2] qa Backports (0.12...Mf1604-qa012) https://github.com/bitcoin/bitcoin/pull/7811
< jonasschnelli> This looks ready: https://github.com/bitcoin/bitcoin/pull/7753
< GitHub111> [bitcoin] paveljanik opened pull request #7810: Refactor AlertNotifyOnce out of UpdateTip (master...20160405_AlertNotifyOnce) https://github.com/bitcoin/bitcoin/pull/7810

2016-04-04

< GitHub124> [bitcoin] theuni opened pull request #7809: depends: some base fixes/changes (master...depends-updates) https://github.com/bitcoin/bitcoin/pull/7809
< GitHub6> [bitcoin] instagibbs opened pull request #7807: Fixed miner test values, gave constants for less error-prone values. (master...mintest) https://github.com/bitcoin/bitcoin/pull/7807
< GitHub163> [bitcoin] instagibbs closed pull request #7784: miner_tests number clarification (master...patch-4) https://github.com/bitcoin/bitcoin/pull/7784
< GitHub28> bitcoin/0.12 e3341aa Wladimir J. van der Laan: Merge #7800: [0.12] Update release notes...
< GitHub158> [bitcoin] laanwj closed pull request #7800: [0.12] Update release notes (0.12...docs) https://github.com/bitcoin/bitcoin/pull/7800
< GitHub28> bitcoin/0.12 e10c044 BtcDrak: [0.12] Update release notes
< GitHub28> [bitcoin] laanwj pushed 2 new commits to 0.12: https://github.com/bitcoin/bitcoin/compare/834aaef7bd37...e3341aa94e1f
< sipa> we could use bitcoin script for the signatures, so you can do multisig auth :p
< GitHub144> bitcoin/0.12 0a79c04 Alex Morcos: Bug fix to RPC test
< GitHub144> bitcoin/0.12 0d09af7 Suhas Daftuar: Add RPC test exercising BIP68 (mempool only)
< GitHub144> bitcoin/0.12 15ba08c Alex Morcos: Implement SequenceLocks functions...
< GitHub144> [bitcoin] laanwj pushed 24 new commits to 0.12: https://github.com/bitcoin/bitcoin/compare/c5f94f6584cb...834aaef7bd37
< GitHub1> [bitcoin] laanwj closed pull request #7543: [0.12] Backport BIP9, BIP68 and BIP112 with softfork (0.12...dot12_backport_bip68) https://github.com/bitcoin/bitcoin/pull/7543
< GitHub148> [bitcoin] gmaxwell opened pull request #7805: Eliminate TX trickle bypass, sort TX invs for privacy and priority. (master...sorted_invs) https://github.com/bitcoin/bitcoin/pull/7805
< gmaxwell> on auth, this protocol looks like it would result in a bitcoin daemon announcing a persistant identity to everyone that connects to it?
< gmaxwell> average message size in bitcoin p2p is someting like 100 bytes right now.
< gmaxwell> Many of the messages in bitcoin are quite small, so the extra lengths do make a meaningful impact.
< jonasschnelli> gmaxwell: the rekeying is local policy, but should not be under 600 seconds to avoid dos (https://github.com/bitcoin/bips/pull/362/files#diff-0bd7a3b80179294f4bcb38cae13c8534R142)
< jonasschnelli> gmaxwell, sipa: If you interested to review the enc/auth BIP again, a new version is here https://github.com/bitcoin/bips/pull/362/files
< sipa> in the bitcoin-tx source code they are called mutate
< sipa> gmaxwell: ah, reimplementing bitcoin-tx as an rpc? ;)
< gmaxwell> positional arguments are sadness. but the json style of arguments is sadness squared. Thats a place where I think bitcoin-cli translating unix style arguments to json might be OK, as long as it was done consistently so someone could just learn the cli to rpc mapping.
< jonasschnelli> bitcoin-cli could combine stuff
< jonasschnelli> gmaxwell: imo RPC != commandline, .. bitcoin-cli == commandline
< GitHub144> [bitcoin] jonasschnelli closed pull request #7707: [RPC][QT] UI support for abandoned transactions (master...2016/03/abandon_ui) https://github.com/bitcoin/bitcoin/pull/7707
< GitHub77> bitcoin/master a914968 Jonas Schnelli: Merge #7707: [RPC][QT] UI support for abandoned transactions...
< GitHub77> [bitcoin] jonasschnelli pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/e662a7628801...a9149688f87c
< GitHub77> bitcoin/master 8efed3b Jonas Schnelli: [Qt] Support for abandoned/abandoning transactions
< jonasschnelli> I'm also not sure if passing a CREATE_TX_RBF_OPT_IN - flag to FundTransaction is the way to go: https://github.com/bitcoin/bitcoin/pull/7159/files#diff-12635a58447c65585f51d32b7e04075bR747
< gmaxwell> it bothers me that rawtxn are trivially distinguishable from txn created by bitcoin core itself for no good reason... and it's weird that there is _less_ functionality via rawtxn now.
< jonasschnelli> Any final feelings about RBF for createrawtx (https://github.com/bitcoin/bitcoin/pull/7159/files)?

2016-04-03

< GitHub135> [bitcoin] sipa opened pull request #7804: Track block download times per individual block (master...betterkicktimeout) https://github.com/bitcoin/bitcoin/pull/7804
< GitHub61> [bitcoin] MarcoFalke opened pull request #7802: [qa] httpbasics: Actually test second connection (master...Mf1604-qaTestHttp) https://github.com/bitcoin/bitcoin/pull/7802
< GitHub101> [bitcoin] MarcoFalke opened pull request #7803: [qa] maxblocksinflight: Actually enable test (master...Mf1604-qaTestMaxBlocks) https://github.com/bitcoin/bitcoin/pull/7803
< GitHub11> [bitcoin] MarcoFalke opened pull request #7801: [qa] Remove misleading "errorString syntax" (master...Mf1604-qaTestErrorString) https://github.com/bitcoin/bitcoin/pull/7801
< btcdrak> gmaxwell: I took a stab at 0.12 release notes for the softfork, if you have time, could you check for accuracy/completeness? https://github.com/bitcoin/bitcoin/pull/7800
< GitHub126> [bitcoin] btcdrak opened pull request #7800: [0.12] Update release notes (0.12...docs) https://github.com/bitcoin/bitcoin/pull/7800
< GitHub143> [bitcoin] laanwj closed pull request #7797: RPC: fix generatetoaddress failing to parse address (master...generatetoaddress-fix) https://github.com/bitcoin/bitcoin/pull/7797
< GitHub64> bitcoin/master e662a76 Wladimir J. van der Laan: Merge #7797: RPC: fix generatetoaddress failing to parse address...
< GitHub64> bitcoin/master 60361ca mruddy: RPC: fix generatetoaddress failing to parse address and add unit test
< GitHub64> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/ff5874bcf7e3...e662a7628801
< GitHub197> [bitcoin] laanwj closed pull request #7778: [qa] Bug fixes and refactor (master...Mf1604-qaFixesRefactor) https://github.com/bitcoin/bitcoin/pull/7778
< GitHub60> bitcoin/master faaa3c9 MarcoFalke: [qa] mininode: Catch exceptions in got_data
< GitHub60> bitcoin/master fa2cea1 MarcoFalke: [qa] rpc-tests: Properly use integers, floats
< GitHub60> bitcoin/master fa524d9 MarcoFalke: [qa] Use python2/3 syntax
< GitHub60> [bitcoin] laanwj pushed 5 new commits to master: https://github.com/bitcoin/bitcoin/compare/e0b3e19816b6...ff5874bcf7e3
< GitHub31> [bitcoin] laanwj closed pull request #7789: [Doc] Add note about using the Qt official binary installer. (master...20160402_Qt56_official_installer_note) https://github.com/bitcoin/bitcoin/pull/7789
< GitHub142> bitcoin/master e0b3e19 Wladimir J. van der Laan: Merge #7789: [Doc] Add note about using the Qt official binary installer....
< GitHub142> bitcoin/master 4b9dfd6 Pavel Janík: Add note about using the Qt official binary installer.
< GitHub142> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/30c2dd8d0598...e0b3e19816b6
< GitHub108> [bitcoin] MarcoFalke opened pull request #7798: [travis] Print the commit which was evaluated (master...Mf1604-travisCommitLog) https://github.com/bitcoin/bitcoin/pull/7798
< GitHub173> [bitcoin] mruddy opened pull request #7797: RPC: fix generatetoaddress failing to parse address (master...generatetoaddress-fix) https://github.com/bitcoin/bitcoin/pull/7797
< GitHub86> [bitcoin] MarcoFalke opened pull request #7796: [amount] Add support for negative fee rates (master...Mf1604-amountNeg64) https://github.com/bitcoin/bitcoin/pull/7796
< GitHub92> [bitcoin] laanwj opened pull request #7795: UpdateTip: log only one line at most per block (master...2016_04_block_warning_logging_inline) https://github.com/bitcoin/bitcoin/pull/7795
< GitHub117> [bitcoin] MarcoFalke opened pull request #7793: [doxygen] Fix member comments (master...Mf1604-doxygenMembers) https://github.com/bitcoin/bitcoin/pull/7793
< GitHub45> [bitcoin] laanwj opened pull request #7792: depends: mac deploy Py3 compatibility (master...2016_04_fix_macosx_gitian) https://github.com/bitcoin/bitcoin/pull/7792

2016-04-02

< GitHub25> [bitcoin] JeremyRand opened pull request #7791: Doc: change Precise to Trusty in gitian-building.md (master...patch-1) https://github.com/bitcoin/bitcoin/pull/7791
< GitHub23> [bitcoin] paveljanik opened pull request #7789: [Doc] Add note about using the Qt official binary installer. (master...20160402_Qt56_official_installer_note) https://github.com/bitcoin/bitcoin/pull/7789
< GitHub88> [bitcoin] paveljanik opened pull request #7788: Use relative paths instead of absolute paths in protoc calls (master...20160402_protoc_use_relpath) https://github.com/bitcoin/bitcoin/pull/7788
< GitHub12> [bitcoin] MarcoFalke opened pull request #7787: [Moveonly] Create ui_interface.cpp (master...Mf1604-move-ui-helpers) https://github.com/bitcoin/bitcoin/pull/7787
< GitHub71> bitcoin/master 30c2dd8 Wladimir J. van der Laan: Merge #7691: [Wallet] refactor wallet/init interaction...
< GitHub162> [bitcoin] laanwj closed pull request #7691: [Wallet] refactor wallet/init interaction (master...2016/03/wallet_mod) https://github.com/bitcoin/bitcoin/pull/7691
< GitHub71> bitcoin/master 25340b7 Jonas Schnelli: [Wallet] refactor wallet/init interaction
< GitHub71> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/a67bc5e12854...30c2dd8d0598
< GitHub9> [bitcoin] laanwj closed pull request #7711: [build-aux] Update Boost & check macros to latest serials (master...build-aux-change) https://github.com/bitcoin/bitcoin/pull/7711
< GitHub52> bitcoin/master a67bc5e Wladimir J. van der Laan: Merge #7711: [build-aux] Update Boost & check macros to latest serials...
< GitHub52> bitcoin/master cf5c786 fanquake: [build-aux] Update Boost & check macros to latest serials
< GitHub52> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/c4df72bafcca...a67bc5e12854
< GitHub135> [bitcoin] laanwj closed pull request #7785: Trivial: Fix typo: Optimizaton -> Optimization [skip ci] (master...patch-16) https://github.com/bitcoin/bitcoin/pull/7785
< GitHub187> bitcoin/master c4df72b Wladimir J. van der Laan: Merge #7785: Trivial: Fix typo: Optimizaton -> Optimization [skip ci]...
< GitHub187> bitcoin/master a22bde9 paveljanik: Fix typo: Optimizaton -> Optimization
< GitHub187> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/e9723cb27384...c4df72bafcca
< GitHub174> [bitcoin] laanwj closed pull request #7651: Only need to subtract 1 (originally nNow was in seconds). (master...AskforTime) https://github.com/bitcoin/bitcoin/pull/7651
< GitHub175> [bitcoin] laanwj closed pull request #7652: Node id when misbehaving (master...NodeIdWhenMisbehaving) https://github.com/bitcoin/bitcoin/pull/7652

2016-04-01

< GitHub5> [bitcoin] s-matthew-english opened pull request #7786: Update policy.cpp (master...patch-2) https://github.com/bitcoin/bitcoin/pull/7786
< jonasschnelli> kanzure: If you have time, I need a native english to fix the enc/auth BIPs. https://github.com/bitcoin/bips/pull/362/files
< gmaxwell> Arnavion: correct. Some projects, like bitcoin core forbid webpage merges by procedure.
< GitHub151> [bitcoin] paveljanik opened pull request #7785: Trivial: Fix typo: Optimizaton -> Optimization [skip ci] (master...patch-16) https://github.com/bitcoin/bitcoin/pull/7785
< GitHub128> [bitcoin] instagibbs opened pull request #7784: miner_tests number clarification (master...patch-4) https://github.com/bitcoin/bitcoin/pull/7784
< GitHub75> [bitcoin] laanwj closed pull request #7464: Opt-in RBF must be strictly enabled or disabled before GBT can be called (master...master) https://github.com/bitcoin/bitcoin/pull/7464
< GitHub44> [bitcoin] laanwj closed pull request #7485: Use system univalue by default (master...sys_univalue_def) https://github.com/bitcoin/bitcoin/pull/7485
< GitHub3> [bitcoin] jonasschnelli opened pull request #7783: [Qt] RPC-Console: support nested commands and simple value queries (master...2016/04/qt_console_nested) https://github.com/bitcoin/bitcoin/pull/7783
< GitHub124> bitcoin/master e9723cb Wladimir J. van der Laan: Merge #7489: tests: Make proxy_test work on travis servers without IPv6...
< GitHub179> [bitcoin] laanwj closed pull request #7489: tests: Make proxy_test work on travis servers without IPv6 (master...2016_02_ipv6_tests_conditional) https://github.com/bitcoin/bitcoin/pull/7489
< GitHub124> bitcoin/master 7539f1a Wladimir J. van der Laan: tests: Make proxy_test work on travis servers without IPv6
< GitHub124> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/28ad4d9fc2be...e9723cb27384
< GitHub181> [bitcoin] laanwj opened pull request #7781: devtools: Auto-set branch to merge to in github-merge (master...2016_04_github_merge_autobranch) https://github.com/bitcoin/bitcoin/pull/7781
< GitHub52> bitcoin/0.12 c5f94f6 Wladimir J. van der Laan: Merge #7780: [0.12] Disable bad-chain alert...
< GitHub66> [bitcoin] laanwj closed pull request #7780: [0.12] Disable bad-chain alert (0.12...spurious) https://github.com/bitcoin/bitcoin/pull/7780
< GitHub52> bitcoin/0.12 8692626 BtcDrak: Disable bad chain alerts...
< GitHub52> [bitcoin] laanwj pushed 2 new commits to 0.12: https://github.com/bitcoin/bitcoin/compare/4d035bcc9aa3...c5f94f6584cb
< Luke-Jr> maybe some day when Bitcoin is 1.0 and we're all down to <=40 hour weeks with nothing to do, it will make sense to reconsider XD
< GitHub66> [bitcoin] morcos closed pull request #7556: Some cleanup work for mempool and policy estimator (master...cleanupMempool) https://github.com/bitcoin/bitcoin/pull/7556
< GitHub137> [bitcoin] morcos closed pull request #7557: Encapsulate options for mempool policy (master...policyOptions) https://github.com/bitcoin/bitcoin/pull/7557
< ebfull> the bitcoin core contributing guidelines are vague on one point; is the author of a pull request free to rebase as their changes become unmergeable?

2016-03-31

< GitHub123> [bitcoin] btcdrak opened pull request #7780: [0.12] Disable bad-chain alert (0.12...spurious) https://github.com/bitcoin/bitcoin/pull/7780
< petertodd> I recently spoke to a very confused user who thought the alerts meant bitcoin was fending off attack :/
< btcdrak> morcos: this is my answer to back chain alerts for 0.12.1 https://github.com/bitcoin/bitcoin/compare/0.12...btcdrak:spurious
< wumpus> action points from last meeting: ACTION: review more at https://github.com/bitcoin/bitcoin/pull/7648 (wumpus, 19:05:42) done and merged
< GitHub28> [bitcoin] NicolasDorier closed pull request #7574: Remove STANDARD_LOCKTIME_VERIFY_FLAGS and mempool policy's flags (master...removeFlag) https://github.com/bitcoin/bitcoin/pull/7574
< GitHub32> [bitcoin] jtimon opened pull request #7779: Discussion: Consensus: There's only one type of consensus flags (master...0.12.99-consensus-unify-flags) https://github.com/bitcoin/bitcoin/pull/7779
< GitHub176> [bitcoin] MarcoFalke closed pull request #7722: [qa] Refactor python2 syntax (master...Mf1603-qaPy2/3) https://github.com/bitcoin/bitcoin/pull/7722
< GitHub136> [bitcoin] MarcoFalke opened pull request #7778: [qa] Bug fixes and refactor (master...Mf1604-qaFixesRefactor) https://github.com/bitcoin/bitcoin/pull/7778
< GitHub142> bitcoin/0.11 d626faa Luke Dashjr: Merge commit '5c0b357' into backports-for-0.11.3
< GitHub142> bitcoin/0.11 5c0b357 Cory Fields: build: Use fPIC rather than fPIE for qt objects....
< GitHub142> bitcoin/0.11 90de0e1 Cory Fields: build: Split hardening/fPIE options out...
< GitHub25> [bitcoin] laanwj closed pull request #7743: [0.11] Important backports for 0.11.3 (updated to v0.12.0) (0.11...backports-for-0.11.3) https://github.com/bitcoin/bitcoin/pull/7743
< GitHub142> [bitcoin] laanwj pushed 5 new commits to 0.11: https://github.com/bitcoin/bitcoin/compare/c251f46bea8f...ecaa178a235a
< GitHub57> [bitcoin] jtimon closed pull request #7731: Discussion: By "more precision", I don't mean using rational numbers in CFeeRate (master...0.12.99-feerate) https://github.com/bitcoin/bitcoin/pull/7731
< GitHub104> [bitcoin] laanwj opened pull request #7776: build: Remove unnecessary executables from gitian release (master...2016_03_gitian_release_cleanup) https://github.com/bitcoin/bitcoin/pull/7776
< GitHub199> [bitcoin] laanwj closed pull request #7511: [WIP] New ax_pthread.m4 from upstream - draft 3 (not final), for testing on all platforms (master...20160211_WIP_test_new_ax_pthread) https://github.com/bitcoin/bitcoin/pull/7511
< GitHub97> [bitcoin] laanwj closed pull request #7477: Fix quoting of copyright holders in configure.ac. (master...configure-quoting) https://github.com/bitcoin/bitcoin/pull/7477
< GitHub124> bitcoin/master 28ad4d9 Wladimir J. van der Laan: Merge #7477: Fix quoting of copyright holders in configure.ac....
< GitHub124> bitcoin/master 72fd008 Daniel Kraft: Fix quoting of copyright holders in configure.ac....
< GitHub124> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/63832688939f...28ad4d9fc2be
< GitHub125> bitcoin/master 6383268 Jonas Schnelli: Merge #7772: Clear the input line after activating autocomplete...
< GitHub14> [bitcoin] jonasschnelli closed pull request #7772: Clear the input line after activating autocomplete (master...20160330_completer_clean_input_line) https://github.com/bitcoin/bitcoin/pull/7772
< GitHub125> bitcoin/master ae2156f Pavel Janík: Clear the input line after activating autocomplete
< GitHub125> [bitcoin] jonasschnelli pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/209dbeb05f88...63832688939f
< GitHub59> bitcoin/0.12 4d035bc accraze: [doc] added depends cross compile info...
< GitHub59> [bitcoin] laanwj pushed 1 new commit to 0.12: https://github.com/bitcoin/bitcoin/commit/4d035bcc9aa3388dc7f37cf81451e55f0b6270ee
< GitHub89> [bitcoin] laanwj closed pull request #7747: [docs] added depends cross compile info (master...depends-build-docs) https://github.com/bitcoin/bitcoin/pull/7747
< GitHub99> bitcoin/master 209dbeb Wladimir J. van der Laan: Merge #7747: [docs] added depends cross compile info...
< GitHub99> bitcoin/master 3e55b3a accraze: [doc] added depends cross compile info
< GitHub99> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/3081fb9a3105...209dbeb05f88
< GitHub95> [bitcoin] laanwj closed pull request #7763: Put hex-encoded version in UpdateTip (master...hexver) https://github.com/bitcoin/bitcoin/pull/7763
< GitHub59> bitcoin/master 3081fb9 Wladimir J. van der Laan: Merge #7763: Put hex-encoded version in UpdateTip...
< GitHub59> bitcoin/master eff736e Pieter Wuille: Reformat version in UpdateTip and other messages...
< GitHub59> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/7c80e720402d...3081fb9a3105
< GitHub51> [bitcoin] laanwj closed pull request #7773: Fix comments in tests (master...csv-comments) https://github.com/bitcoin/bitcoin/pull/7773
< GitHub195> bitcoin/master 7c80e72 Wladimir J. van der Laan: Merge #7773: Fix comments in tests...
< GitHub195> bitcoin/master 40234ba BtcDrak: Fix comments in tests
< GitHub195> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/16555b658f5b...7c80e720402d
< GitHub45> [bitcoin] laanwj closed pull request #7766: rpc: Register calls where they are defined (master...2016_03_separate_rpc_registration) https://github.com/bitcoin/bitcoin/pull/7766
< GitHub33> bitcoin/master 16555b6 Wladimir J. van der Laan: Merge #7766: rpc: Register calls where they are defined...
< GitHub33> bitcoin/master fb8a8cf Wladimir J. van der Laan: rpc: Register calls where they are defined...
< GitHub33> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/e8a8f3d4b22b...16555b658f5b
< GitHub136> [bitcoin] laanwj closed pull request #7775: 0.12 (master...0.12) https://github.com/bitcoin/bitcoin/pull/7775
< GitHub72> [bitcoin] maneotrix opened pull request #7775: 0.12 (master...0.12) https://github.com/bitcoin/bitcoin/pull/7775

2016-03-30

< GitHub77> [bitcoin] mruddy opened pull request #7774: RPC: BIP9 version bits related, format version as hex in getblock and getblockheader (master...hexver) https://github.com/bitcoin/bitcoin/pull/7774
< GitHub155> [bitcoin] btcdrak opened pull request #7773: Fix comments in tests (master...csv-comments) https://github.com/bitcoin/bitcoin/pull/7773
< GitHub0> [bitcoin] paveljanik opened pull request #7772: Clear the input line after activating autocomplete (master...20160330_completer_clean_input_line) https://github.com/bitcoin/bitcoin/pull/7772
< btcdrak> trolling for review/commentary on https://github.com/bitcoin/bitcoin/pull/7568
< btcdrak> Trolling for review of https://github.com/bitcoin/bitcoin/pull/7707 - we'd like this for 0.12.1
< GitHub76> bitcoin/0.12 ba80cee Wladimir J. van der Laan: bump version to 0.12.1
< GitHub76> [bitcoin] laanwj pushed 1 new commit to 0.12: https://github.com/bitcoin/bitcoin/commit/ba80ceef59bdfb7e0a42da4df81335698047fbce