2015-12-22

< GitHub106> [bitcoin] jrmithdobbs closed pull request #7243: This community NEEDS to adopt a code of conduct. Recent events make t… (master...code_of_conduct) https://github.com/bitcoin/bitcoin/pull/7243
< jcorgan> blade: this is not the correct channel for this, please go to #bitcoin
< blade> im useig the bitcoin core wallet

2015-12-21

< GitHub71> [bitcoin] jrmithdobbs opened pull request #7243: This community NEEDS to adopt a code of conduct. Recent events make t… (master...code_of_conduct) https://github.com/bitcoin/bitcoin/pull/7243
< GitHub179> [bitcoin] bom64 opened pull request #7242: 0.12 Change the -keypool info text (master...0.12) https://github.com/bitcoin/bitcoin/pull/7242
< GitHub74> bitcoin/0.12 12c469b Suhas Daftuar: [Mempool] Fix mempool limiting and replace-by-fee for PrioritiseTransaction...
< GitHub74> [bitcoin] laanwj pushed 1 new commit to 0.12: https://github.com/bitcoin/bitcoin/commit/12c469b236aa9b31b3744e5c529b9236dda27b27
< GitHub80> [bitcoin] laanwj closed pull request #7062: [Mempool] Fix mempool limiting and replace-by-fee for PrioritiseTransaction (master...fix-mempool-limiting) https://github.com/bitcoin/bitcoin/pull/7062
< GitHub78> bitcoin/master 27fae34 Suhas Daftuar: Use fee deltas for determining mempool acceptance
< GitHub78> bitcoin/master 9ef2a25 Suhas Daftuar: Update replace-by-fee logic to use fee deltas
< GitHub78> bitcoin/master eb30666 Suhas Daftuar: Fix mempool limiting for PrioritiseTransaction...
< GitHub78> [bitcoin] laanwj pushed 5 new commits to master: https://github.com/bitcoin/bitcoin/compare/8ea5ef1d3994...c24337964f2d
< GitHub85> [bitcoin] jtimon closed pull request #6625: BLOCKING: Consensus: Move blocksize and related parameters to consensusparams ...without removing consensus/consensus.h [#6526 alternative] (master...consensus-blocksize-0.12.99) https://github.com/bitcoin/bitcoin/pull/6625
< GitHub159> [bitcoin] jtimon opened pull request #7238: Blocksize: Some small preparations for a blocksize hardfork (master...6526-6625-remainings-0.13.99) https://github.com/bitcoin/bitcoin/pull/7238

2015-12-20

< GitHub4> [bitcoin] dgenr8 opened pull request #7236: Use createrawtx locktime parm in txn_clone (master...use_rpc_locktime_clone) https://github.com/bitcoin/bitcoin/pull/7236
< maaku> that's effectively useless given how bitcoin development works...

2015-12-18

< GitHub169> [bitcoin] jtimon reopened pull request #6597: Chainparams: Don't check the genesis block (master...chainparams-genesis-no-check-0.12.99) https://github.com/bitcoin/bitcoin/pull/6597
< GitHub39> [bitcoin] jtimon reopened pull request #6625: BLOCKING: Consensus: Move blocksize and related parameters to consensusparams ...without removing consensus/consensus.h [#6526 alternative] (master...consensus-blocksize-0.12.99) https://github.com/bitcoin/bitcoin/pull/6625
< GitHub159> [bitcoin] instagibbs opened pull request #7232: Added help text for chainwork value (master...getblockhelp) https://github.com/bitcoin/bitcoin/pull/7232
< GitHub105> [bitcoin] jgarzik opened pull request #7230: BIP 202: 2mb blocksize step (master...2015_2mb_blocksize_step) https://github.com/bitcoin/bitcoin/pull/7230
< GitHub123> [bitcoin] MarcoFalke opened pull request #7229: [qa] wallet: Check if maintenance changes the balance (master...MarcoFalke-2015-rpcWalletMaintenance) https://github.com/bitcoin/bitcoin/pull/7229
< GitHub49> bitcoin/0.12 eccd671 fanquake: [Depends] Bump Boost, miniupnpc, ccache & zeromq...
< GitHub49> [bitcoin] laanwj pushed 1 new commit to 0.12: https://github.com/bitcoin/bitcoin/commit/eccd67106d7ebfcb1d0913c2036848f57ba344fb
< GitHub64> [bitcoin] laanwj closed pull request #6980: [Depends] Bump Boost, miniupnpc, ccache & zeromq (master...depends-bump-boost) https://github.com/bitcoin/bitcoin/pull/6980
< GitHub122> bitcoin/master 26f8ea5 fanquake: [depends] native ccache 3.2.4
< GitHub122> bitcoin/master 9e940fa fanquake: [depends] Boost 1.59.0
< GitHub122> bitcoin/master 17ad964 fanquake: [depends] miniupnpc 1.9.20151026
< GitHub122> [bitcoin] laanwj pushed 7 new commits to master: https://github.com/bitcoin/bitcoin/compare/cd3f12c61ca5...8ea5ef1d3994
< GitHub39> [bitcoin] jtimon closed pull request #6597: Chainparams: Don't check the genesis block (master...chainparams-genesis-no-check-0.12.99) https://github.com/bitcoin/bitcoin/pull/6597
< GitHub37> [bitcoin] jtimon closed pull request #6625: BLOCKING: Consensus: Move blocksize and related parameters to consensusparams ...without removing consensus/consensus.h [#6526 alternative] (master...consensus-blocksize-0.12.99) https://github.com/bitcoin/bitcoin/pull/6625

2015-12-17

< GitHub22> [bitcoin] sdaftuar opened pull request #7226: Tests: Add more tests to p2p-fullblocktest (master...add-more-tests) https://github.com/bitcoin/bitcoin/pull/7226
< Luke-Jr> jonasschnelli: do users expect Bitcoin Core and Bitcoin LJR to overwrite each other, is the question IMO
< jonasschnelli> Changing the whole name from Bitcoin-Qt.app to Bitcoin\ Core.app seems okay for me (with loosing the "replace" functionallity).
< jonasschnelli> Luke-Jr: the filename itself is still "Bitcoin-Qt.app",.. not ideal... but works.
< GitHub109> [bitcoin] laanwj reopened pull request #7165: WIP: build: Enable C++11 in build, require C++11 compiler (master...2015_12_c++11) https://github.com/bitcoin/bitcoin/pull/7165
< jonasschnelli> Yes. We kept bitcoin-qt because otherwise new versions would not overwrite old ones.
< jonasschnelli> cfields: Luke-Jr's name unify PR works really good. We might want to fix the osx application name ("Bitcoin-Qt").
< GitHub173> [bitcoin] sdaftuar opened pull request #7225: Eliminate unnecessary call to CheckBlock (master...eliminate-extra-checkblock) https://github.com/bitcoin/bitcoin/pull/7225
< cfields> Luke-Jr: what's missing in the bitcoin build itself? the python libs?
< wumpus> <jgarzik> benjyz1, #bitcoin-dev please
< wumpus> off topic things not related to bitcoin development, If it fits into the above category, you're a legit contributor and somehow got 'censored' feel free to send a complaint to me
< jgarzik> benjyz1, #bitcoin-dev please
< benjyz1> heavy censorship on bitcoin-dev.. what is going on
< GitHub129> [bitcoin] laanwj closed pull request #7165: WIP: build: Enable C++11 in build, require C++11 compiler (master...2015_12_c++11) https://github.com/bitcoin/bitcoin/pull/7165
< jonasschnelli> Luke-Jr: your TestyCOIN-Crazy projects still uses "Bitcoin-Qt.app" as filename (as you mentioned). But I guess the whole dmg process should also work if we would use a different filename? right?
< Luke-Jr> oh, it works because we don't actually use "Bitcoin Core" in filenames
< Luke-Jr> looking at the osx descriptor, it seems to assume bitcoin-* prefix in the filename
< GitHub197> bitcoin/0.12 f3ad812 Wladimir J. van der Laan: test: don't override BITCOIND and BITCOINCLI if they're set...
< GitHub197> [bitcoin] laanwj pushed 1 new commit to 0.12: https://github.com/bitcoin/bitcoin/commit/f3ad81220850a4158ab329f5279f7530cbb70a87
< GitHub64> [bitcoin] laanwj closed pull request #7209: test: don't override BITCOIND and BITCOINCLI if they're set (master...2015_12_rpctests_setenviron) https://github.com/bitcoin/bitcoin/pull/7209
< GitHub104> bitcoin/master cd3f12c Wladimir J. van der Laan: Merge pull request #7209...
< GitHub104> bitcoin/master 83cdcbd Wladimir J. van der Laan: test: don't override BITCOIND and BITCOINCLI if they're set...
< GitHub104> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/d22245f92375...cd3f12c61ca5
< GitHub41> bitcoin/0.12 9572e49 Elias Rohrer: Removed offline testnet DNSSeed 'alexykot.me'....
< GitHub41> [bitcoin] laanwj pushed 1 new commit to 0.12: https://github.com/bitcoin/bitcoin/commit/9572e4944a6130640477690f2158d373af8017cc
< GitHub63> [bitcoin] laanwj closed pull request #7216: Removed offline testnet DNSSeed 'alexykot.me'. (master...delete_offline_dnsseed) https://github.com/bitcoin/bitcoin/pull/7216
< GitHub34> bitcoin/master d22245f Wladimir J. van der Laan: Merge pull request #7216...
< GitHub34> bitcoin/master e18378e Elias Rohrer: Removed offline testnet DNSSeed 'alexykot.me'.
< GitHub34> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/7a5040155ed5...d22245f92375
< Luke-Jr> depends/ is probably how to do it, but it doesn't seem capable right now of installing native stuff that Bitcoin picks up on
< Luke-Jr> Bitcoin-Core is in DS_Store, so I doubt it will work with it changed
< GitHub152> [bitcoin] sidhujag closed pull request #7224: bitcoin rename to syscoin (master...btcbase) https://github.com/bitcoin/bitcoin/pull/7224
< GitHub3> [bitcoin] sidhujag opened pull request #7224: bitcoin rename to syscoin (master...btcbase) https://github.com/bitcoin/bitcoin/pull/7224

2015-12-16

< GitHub68> [bitcoin] sdaftuar opened pull request #7222: [WIP] RPC: Indicate which transactions are signaling opt-in RBF (master...add-optin-info) https://github.com/bitcoin/bitcoin/pull/7222
< GitHub4> [bitcoin] luke-jr closed pull request #7220: RBF: Allow replacements to pay for minRelayFee(replaced)+minRelayFee(replacement) rather than actualFee(replaced)+minRelayFee(replacement) (master...rbf_feecomp) https://github.com/bitcoin/bitcoin/pull/7220
< helo> someone familiar with p2pool code may want to take a look at KipIngram's messages in #bitcoin
< GitHub110> [bitcoin] parazyd closed pull request #7221: Merge pull request #1 from bitcoin/master (master...master) https://github.com/bitcoin/bitcoin/pull/7221
< GitHub62> [bitcoin] parazyd opened pull request #7221: Merge pull request #1 from bitcoin/master (master...master) https://github.com/bitcoin/bitcoin/pull/7221
< jonasschnelli> Luke-Jr: hmm... IPv6 seems to be set up coorect. dig AAAA bitcoin.jonasschnelli.ch gives me the right answer, .. i can ping6 from another machine (in the same lan although).
< GitHub143> [bitcoin] luke-jr opened pull request #7220: RBF: Allow replacements to pay for minRelayFee(replaced)+minRelayFee(replacement) rather than actualFee(replaced)+minRelayFee(replacement) (master...rbf_feecomp) https://github.com/bitcoin/bitcoin/pull/7220
< GitHub128> [bitcoin] luke-jr opened pull request #7219: Make RBF policies optional (master...rbf_opts) https://github.com/bitcoin/bitcoin/pull/7219
< GitHub32> [bitcoin] MarcoFalke opened pull request #7218: [qt] Fix misleading translation (master...MarcoFalke-2015-trivial7) https://github.com/bitcoin/bitcoin/pull/7218
< BlueMatt> (pool servers are largely bit proprietary blobs that get essentially no external review, whereas bitcoin core is....not that)

2015-12-15

< GitHub85> [bitcoin] sdaftuar opened pull request #7217: Mark blocks with too many sigops as failed (master...fix-sigops-rejection) https://github.com/bitcoin/bitcoin/pull/7217
< sipa> #bitcoin-dev is more for development of the protocol
< sipa> This is specifically about the Bitcoin Core software, and its implementation details
< desantis> honest question: How does this channel differ from bitcoin-dev?
< GitHub138> [bitcoin] tnull opened pull request #7216: Removed offline testnet DNSSeed 'alexykot.me'. (master...delete_offline_dnsseed) https://github.com/bitcoin/bitcoin/pull/7216

2015-12-14

< GitHub92> [bitcoin] MarcoFalke opened pull request #7214: qt5: Use the fixed font the system recommends (master...MarcoFalke-2015-qt5monospace) https://github.com/bitcoin/bitcoin/pull/7214
< sdaftuar> hm. does bitcoin core use the container-based environment?
< sdaftuar> is there anything special that needs to be done to make travis work well on personal repos? i enabled travis on my fork of the project so i could test branches before opening PRs, but they fail a lot for strange reasons (even when the same branch succeeds as part of a bitcoin core PR)
< GitHub24> [bitcoin] mb300sd opened pull request #7213: Rename OP_NOP2 to OP_CHECKLOCKTIMEVERIFY (master...rename-opnop2) https://github.com/bitcoin/bitcoin/pull/7213
< Luke-Jr> cfields: https://github.com/bitcoin/bitcoin/pull/7192 needs some depends attention
< GitHub156> [bitcoin] EthanHeilman opened pull request #7212: Adds unittests for CAddrMan and CAddrinfo, removes source of non-determinism. (master...unittest) https://github.com/bitcoin/bitcoin/pull/7212
< GitHub158> [bitcoin] laanwj opened pull request #7209: test: don't override BITCOIND and BITCOINCLI if they're set (master...2015_12_rpctests_setenviron) https://github.com/bitcoin/bitcoin/pull/7209
< jonasschnelli> wumpus: BITCOIND=bitcoin-qt ./qa/pull-tester/rpc-tests.py won't work.
< GitHub8> bitcoin/0.12 10b88be Pieter Wuille: Replace trickle nodes with per-node/message Poisson delays...
< GitHub8> [bitcoin] laanwj pushed 1 new commit to 0.12: https://github.com/bitcoin/bitcoin/commit/10b88be79856ee7ee66f69705c16b335941e396e
< GitHub183> [bitcoin] laanwj closed pull request #7125: Replace global trickle node with random delays (master...timetrickle) https://github.com/bitcoin/bitcoin/pull/7125
< GitHub166> bitcoin/master 5400ef6 Pieter Wuille: Replace trickle nodes with per-node/message Poisson delays...
< GitHub166> bitcoin/master 7a50401 Wladimir J. van der Laan: Merge pull request #7125...
< GitHub166> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/dc511dcfd9ea...7a5040155ed5
< GitHub12> [bitcoin] laanwj opened pull request #7208: Make max tip age an option instead of chainparam (master...2015_12_maxtipage) https://github.com/bitcoin/bitcoin/pull/7208
< jonasschnelli> Results in: /usr/bin/tiffcp -c none dpi36.background.tiff dpi72.background.tiff dist/Bitcoin-Qt.app/Contents/MacOS/Bitcoin-Qt dist/.background/background.tiff
< GitHub83> bitcoin/0.12 06c6a58 accraze: Checks for null data transaction before issuing error to debug.log...
< GitHub83> [bitcoin] laanwj pushed 1 new commit to 0.12: https://github.com/bitcoin/bitcoin/commit/06c6a584635bb42c511baf505cebd7cdf77b89e9
< GitHub92> [bitcoin] laanwj closed pull request #7200: Checks for null data transaction before issuing error to debug.log (master...null-tx-debug) https://github.com/bitcoin/bitcoin/pull/7200
< GitHub149> bitcoin/master d812daf accraze: fix logic for error log
< GitHub149> bitcoin/master c611acc accraze: wallet: check if tx scriptPubKey is unspendable
< GitHub149> bitcoin/master b6915b8 accraze: checks for null data transaction before debug.log...
< GitHub149> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/5f3c670d1222...dc511dcfd9ea
< GitHub60> [bitcoin] laanwj closed pull request #7068: [RPC-Tests] add simple way to run rpc test over QT clients (master...2015/11/rpc_tests_qt) https://github.com/bitcoin/bitcoin/pull/7068
< GitHub91> bitcoin/master 5f3c670 Wladimir J. van der Laan: Merge pull request #7068...
< GitHub91> bitcoin/master 979698c Jonas Schnelli: [RPC-Tests] add option to run rpc test over QT clients
< GitHub91> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/ea0f5a2b04bb...5f3c670d1222
< GitHub104> [bitcoin] laanwj closed pull request #7185: Note that reviewers should mention the id of the commits they reviewed. (master...2015-12-07-contributingackcommit) https://github.com/bitcoin/bitcoin/pull/7185
< GitHub69> bitcoin/master ea0f5a2 Wladimir J. van der Laan: Merge pull request #7185...
< GitHub69> bitcoin/master e1030dd Patrick Strateman: Note that reviewers should mention the commit hash of the commits they reviewed.
< GitHub69> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b7c704abab4e...ea0f5a2b04bb
< jonasschnelli> Just passing BITCOIND=bitcoin-qt won't work because of the missing -server argument. So a change is required anyways. Will rebase.
< wumpus> well if it's not useful enough (e.g. you can achieve the same with BITCOIND=bitcoin-qt ./test...) we should close it
< jonasschnelli> Luke-Jr: still building: https://bitcoin.jonasschnelli.ch/pulls/7192/
< jonasschnelli> Will do as soon as i can have a hw wallet that support bitcoin/pgp/ssh with the same device.
< jonasschnelli> MarcoFalke: i'm working on a bitcoin/pgp/ssh hardware wallet
< GitHub172> bitcoin/0.12 f43c2f9 Matt Corallo: Add "NODE_BLOOM" to guiutil so that peers don't get UNKNOWN[4]...
< GitHub172> [bitcoin] laanwj pushed 1 new commit to 0.12: https://github.com/bitcoin/bitcoin/commit/f43c2f9a8a30760c34dd4691de4ed4d7a7b9969e
< jonasschnelli> SSL2 and 3 not: openssl s_client -connect bitcoin.jonasschnelli.ch:443 -ssl2
< jonasschnelli> SSL1 handshake is okay: openssl s_client -connect bitcoin.jonasschnelli.ch:443 -ssl1
< Luke-Jr> An error occurred during a connection to bitcoin.jonasschnelli.ch. SSL received a record that exceeded the maximum permissible length. (Error code: ssl_error_rx_record_too_long)
< GitHub16> [bitcoin] jonasschnelli closed pull request #7206: Add "NODE_BLOOM" to guiutil so that peers don't get UNKNOWN[4] (master...2015_12_bloom_gui) https://github.com/bitcoin/bitcoin/pull/7206
< GitHub4> bitcoin/master b7c704a Jonas Schnelli: Merge pull request #7206...
< GitHub4> bitcoin/master daf6466 Matt Corallo: Add "NODE_BLOOM" to guiutil so that peers don't get UNKNOWN[4]
< GitHub4> [bitcoin] jonasschnelli pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/9ee02cf564d1...b7c704abab4e
< GitHub39> [bitcoin] TheBlueMatt opened pull request #7206: Add "NODE_BLOOM" to guiutil so that peers don't get UNKNOWN[4] (master...2015_12_bloom_gui) https://github.com/bitcoin/bitcoin/pull/7206

2015-12-13

< GitHub92> [bitcoin] smenglish closed pull request #7201: Update hmac_sha256.cpp (master...master) https://github.com/bitcoin/bitcoin/pull/7201
< GitHub166> [bitcoin] MarcoFalke opened pull request #7205: Update copyright headers to 2015 (master...MarcoFalke-2015-BumpHeaders-0.12) https://github.com/bitcoin/bitcoin/pull/7205
< btcdrak> Luke-Jr: I know wumpus has been planning to slowly increase the number of maintainer for specific areas. The GUI maintainer is a trial. It's part of why https://github.com/bitcoin/bitcoin/blob/master/CONTRIBUTING.md was written.
< btcdrak> Luke-Jr: Yes, we must actively say that. Bitcoin Core devs (as I have said repeatedly in private) are playing into the hands of disruption by being too politically correct and "nice". We need to be prepared to call a spade a spade.
< btcdrak> Luke-Jr: "Bitcoin Core" as a phrase is probably trademarkable, but I really dont see the benefit.
< midnightmagic> what is bitcoin's licence anyway.. mit/bsd?
< CodeShark> if phantomcircuit is specifically referring to actions taken by someone on "behalf" of Bitcoin Core, I think the policy needs to be specifically directed at that...and we need to come out with one voice saying that someone doesn't represent Bitcoin Core
< btcdrak> gavinandresen: you dont contribute to Bitcoin Core just to creating chaos. While others might be more polite, I'm going right ahead and saying this, please go away, you are not wanted here.
< GitHub38> [bitcoin] pstratem closed pull request #6943: Wallet: Store hash for encrypted keys (master...ckey_hash) https://github.com/bitcoin/bitcoin/pull/6943
< GitHub101> [bitcoin] pstratem closed pull request #6874: Net: Cork socket send writes (master...msg_more) https://github.com/bitcoin/bitcoin/pull/6874
< GitHub180> [bitcoin] pstratem closed pull request #7057: Wallet: Flush database to log files (master...2015-11-18-wallet-flush) https://github.com/bitcoin/bitcoin/pull/7057
< GitHub44> [bitcoin] pstratem closed pull request #7064: [Wallet]Perform entire CWallet::TopUpKeyPool in a transaction. (master...2015-11-19-wallet-topupkeypool) https://github.com/bitcoin/bitcoin/pull/7064
< GitHub150> [bitcoin] pstratem closed pull request #7071: Wallet: Replace TxnAbort with assert. (master...2015-11-20-wallet-activetxn) https://github.com/bitcoin/bitcoin/pull/7071
< GitHub115> [bitcoin] pstratem closed pull request #7104: [Mining] Build empty block on when chainTip changes. (master...2015-11-26-gbt-latency) https://github.com/bitcoin/bitcoin/pull/7104
< GitHub101> [bitcoin] pstratem closed pull request #7150: Print size of pcoinsTip cache in AcceptToMemoryPool (master...2015-12-1-printcachesize) https://github.com/bitcoin/bitcoin/pull/7150
< GitHub178> [bitcoin] pstratem closed pull request #7203: [WIP][Mining]Release cs_main in CreateNewBlock while selecting transactions from mempool. (master...2015-12-12-createnewblocklocks) https://github.com/bitcoin/bitcoin/pull/7203
< gmaxwell> gavinandresen: Hi. I'm concerned that I keep hearing reports of you taking actions on behalf of Bitcoin Core while not actually communicating with anyone involved in the project, or being involved yourself. I wasn't sure of how much of this was rumor or miscommunication, but this appears to be you directly confirming it:
< GitHub0> [bitcoin] pstratem opened pull request #7203: Release cs_main in CreateNewBlock while selecting transactions from mempool. (master...2015-12-12-createnewblocklocks) https://github.com/bitcoin/bitcoin/pull/7203
< gmaxwell> Basically the system currently encourages you to play block arrival time chicken. Pay massively low fees, get mined fast. hurray! ... next time, mined slowly "Boo bitcoin is broken!"
< morcos> the idea being that before the block has decent size that from a long term perspective you need to be adding at least some txs or bitcoin is useless, but once it gets of a decent size, then you need to worry more about what the profit maximizing strategy is?
< gmaxwell> Thats what Bitcoin did in 2012 and before, and I think the actual implementation wasn't a good path. It's weird to have such a huge amount of statefulness regarding when transactions come in.

2015-12-12

< GitHub166> [bitcoin] smenglish opened pull request #7201: Update hmac_sha256.cpp (master...master) https://github.com/bitcoin/bitcoin/pull/7201
< GitHub130> [bitcoin] accraze opened pull request #7200: Checks for null data transaction before issuing error to debug.log (master...null-tx-debug) https://github.com/bitcoin/bitcoin/pull/7200
< gmaxwell> So what bitcoin did pre-2012 was more reasonable; in the sense that it provided gradual back pressure, but it depended on future state and it was strangely order dependant. (e.g. first txn into a block could go in with low fees, then higher paying things were excluded later).

2015-12-11

< gmaxwell> I've been giving some though to bringing back the tiered fee behavior bitcoin had untul mid 2012. The earlier logic had a rate table where transactions were added in arrival order, with the required fee gradually going up as the non-fee-sorted candidate block became bigger. In 2012 it was changed to the sorting based selection on the basis that this was the rational behavior to miners, and the '
< GitHub94> [bitcoin] laanwj closed pull request #7156: rpc: remove cs_main lock from `createrawtransaction` (master...2015_12_createrawtransaction_nolock) https://github.com/bitcoin/bitcoin/pull/7156
< GitHub177> bitcoin/master 9ee02cf Wladimir J. van der Laan: Merge pull request #7156...
< GitHub177> bitcoin/master 6e76587 Wladimir J. van der Laan: rpc: remove cs_main lock from `createrawtransaction`...
< GitHub177> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/d1e17ff6408c...9ee02cf564d1
< GitHub141> bitcoin/0.12 8fc174a Wladimir J. van der Laan: net: Add and document network messages in protocol.h...
< GitHub141> [bitcoin] laanwj pushed 1 new commit to 0.12: https://github.com/bitcoin/bitcoin/commit/8fc174aae64882d43549ad57bece0c23805e567b
< GitHub18> bitcoin/master d1e17ff Wladimir J. van der Laan: Merge pull request #7181...
< GitHub31> [bitcoin] laanwj closed pull request #7181: net: Add and document network messages in protocol.h (master...2015_12_network_messages) https://github.com/bitcoin/bitcoin/pull/7181
< GitHub18> bitcoin/master 9bbe71b Wladimir J. van der Laan: net: Add and document network messages in protocol.h...
< GitHub18> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/f7f44b1bdd15...d1e17ff6408c
< wumpus> may be possible to split up between a descriptor that just builds the dependencies and one that builds bitcoin, or some other trick
< gmaxwell> or just set it to use 2kbit rsa, the generation time is probably not that bad on anything that can compile bitcoin.
< jonasschnelli> Luke-Jr: seems to have build issues on OSX: https://bitcoin.jonasschnelli.ch/pulls/7192/build-osx.log
< GitHub16> [bitcoin] laanwj closed pull request #7197: Set link from http:// to https:// (master...patch-1) https://github.com/bitcoin/bitcoin/pull/7197
< GitHub39> bitcoin/master 00423e1 Suriyaa Kudo: Set link from http:// to https://...
< GitHub39> bitcoin/master f7f44b1 Wladimir J. van der Laan: Merge pull request #7197...
< GitHub39> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/5dc63ed1ca8a...f7f44b1bdd15
< GitHub187> [bitcoin] laanwj closed pull request #7196: Doxygen (master...doxygen-a) https://github.com/bitcoin/bitcoin/pull/7196

2015-12-10

< gmaxwell> the whole coinselector in bitcoin core appears to have been written with a strong assumption that an address isn't used more than once. :(
< wumpus> at last I prefer not including unnecessary commits, I admit other people may have different opinions, but that's the annoying thing about having one wallet implementation bound to bitcoin core
< GitHub99> [bitcoin] SuriyaaKudoIsc opened pull request #7197: Set link from http:// to https:// (master...patch-1) https://github.com/bitcoin/bitcoin/pull/7197
< morcos> wangchun: i'm not sure, but i think it should in later versions of bitcoin core
< GitHub163> bitcoin/0.12 44fef99 Wladimir J. van der Laan: net: Fix sent reject messages for blocks and transactions...
< GitHub163> bitcoin/0.12 96e8d12 AlSzacrel: Coinselection prunes extraneous inputs from ApproximateBestSubset...
< GitHub163> [bitcoin] laanwj pushed 2 new commits to 0.12: https://github.com/bitcoin/bitcoin/compare/b2d7ada3727f...44fef99e666e
< GitHub67> [bitcoin] laanwj closed pull request #7179: net: Fix sent reject messages for blocks and transactions (master...2015_12_fix_reject_message) https://github.com/bitcoin/bitcoin/pull/7179
< GitHub121> bitcoin/master 2041190 Wladimir J. van der Laan: test: Add basic test for `reject` code...
< GitHub121> bitcoin/master 5dc63ed Wladimir J. van der Laan: Merge pull request #7179...
< GitHub121> bitcoin/master 9fc6ed6 Wladimir J. van der Laan: net: Fix sent reject messages for blocks and transactions...
< GitHub121> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/00b4b8d1c47a...5dc63ed1ca8a
< * Luke-Jr> :| "Could not connect to host bitcoin.jonasschnelli.ch: SSL negotiation failed."
< jonasschnelli> Luke-Jr: okay. Restarted the build: https://bitcoin.jonasschnelli.ch/pulls/7192/
< Luke-Jr> do you know if src/qt/res/bitcoin-qt-res.rc can use PRODUCT_NAME?
< Luke-Jr> jonasschnelli: re isolating "Bitcoin Core" to fewer places.. there's 8 instances in the .ui files; I could move those to the equivalent .cpp, but it would look not-so-nice in Designer; thoughts?

2015-12-09

< GitHub53> [bitcoin] awelch83 opened pull request #7196: Doxygen (master...doxygen-a) https://github.com/bitcoin/bitcoin/pull/7196
< GitHub111> [bitcoin] jamesob opened pull request #7194: [tests] Add RPC tests for getblockheader (master...test_getblockheader) https://github.com/bitcoin/bitcoin/pull/7194
< jonasschnelli_> <Luke-Jr>[11:56:41] let's rename to Nucleul Bitcoin in all languages. it sounds nicer.: <-- haha
< GitHub177> [bitcoin] MarcoFalke opened pull request #7193: [wallet] Cleanup tests (master...MarcoFalke-2015-WalletTests) https://github.com/bitcoin/bitcoin/pull/7193
< GitHub35> [bitcoin] luke-jr opened pull request #7192: Unify product name to as few places as possible without major changes (master...single_prodname) https://github.com/bitcoin/bitcoin/pull/7192
< Luke-Jr> let's rename to Nucleul Bitcoin in all languages. it sounds nicer.
< jonasschnelli_> nucleul sounded after a reasonable Bitcoin wallet name...:-)
< wumpus> jonasschnelli_: hehe in Dutch it's called "Bitcoin Kern"
< jonasschnelli_> Maybe my lol was inappropriate (I thought he is using a different wallet (Nucleul Bitcoin versiunea v0.11.0 (32-bit)), but looks like core. Could be that the user really face an issue/bug.
< GitHub51> [bitcoin] laanwj closed pull request #7154: [Qt] add InMempool() info to transaction details (master...2015/12/qt_conflicts) https://github.com/bitcoin/bitcoin/pull/7154
< GitHub113> bitcoin/master 00b4b8d Wladimir J. van der Laan: Merge pull request #7154...
< GitHub113> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/0800092fc252...00b4b8d1c47a
< GitHub113> bitcoin/master a3c3ddb Jonas Schnelli: [Qt] add InMempool() info to transaction details
< Luke-Jr> bet someone told him to add a bitcoin.conf option or something
< Luke-Jr> Nucleul Bitcoin is our name in Romanian?
< dcousens> those bastards, lol, yeah no worries, jsut wanted to quickly compile something and realised I had to put include "config/bitcoin-config.h" in a bunch of files just to stop it exploding
< GitHub154> [bitcoin] NicolasDorier opened pull request #7190: Performance fix for #6312 (master...sequencenumbers) https://github.com/bitcoin/bitcoin/pull/7190
< GitHub114> [bitcoin] jonasschnelli closed pull request #7188: Update sha512.cpp (master...patch-1) https://github.com/bitcoin/bitcoin/pull/7188

2015-12-08

< GitHub130> [bitcoin] smenglish opened pull request #7188: Update sha512.cpp (master...patch-1) https://github.com/bitcoin/bitcoin/pull/7188
< MarcoFalke> Only if both are installed it chooses qt5: https://github.com/bitcoin/bitcoin/pull/6938/files
< GitHub133> [bitcoin] morcos reopened pull request #7187: Keep reorgs fast for SequenceLocks checks (master...fastReorgBIP68) https://github.com/bitcoin/bitcoin/pull/7187
< GitHub65> [bitcoin] morcos closed pull request #7187: Keep reorgs fast for SequenceLocks checks (master...fastReorgBIP68) https://github.com/bitcoin/bitcoin/pull/7187
< GitHub96> [bitcoin] morcos opened pull request #7187: Keep reorgs fast for SequenceLocks checks (master...fastReorgBIP68) https://github.com/bitcoin/bitcoin/pull/7187
< wumpus> I'm seeing none of this behavior from bitcoin core nodes
< GitHub88> bitcoin/master fc0f52d Murch: Added a test for the pruning of extraneous inputs after ApproximateBestSet
< GitHub88> bitcoin/master af9510e Murch: Moved set reduction to the end of ApproximateBestSubset to reduce performance impact
< GitHub88> bitcoin/master 5c03483 AlSzacrel: Coinselection prunes extraneous inputs from ApproximateBestSubset...
< GitHub121> [bitcoin] laanwj closed pull request #4906: Issue#1643: Coinselection prunes extraneous inputs from ApproximateBestSubset (master...Fix-#1643) https://github.com/bitcoin/bitcoin/pull/4906
< GitHub88> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/16ccb74ef2e0...0800092fc252
< GitHub1> bitcoin/master 16ccb74 Wladimir J. van der Laan: Merge pull request #7180...
< GitHub1> bitcoin/master e3bc5e0 Wladimir J. van der Laan: net: Account for `sendheaders` `verack` messages...
< GitHub1> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/dc0305d15aa0...16ccb74ef2e0
< GitHub171> [bitcoin] luke-jr opened pull request #7186: Backports for 0.10.5 (updated to dc0305d) (0.10...backports-for-0.10.5) https://github.com/bitcoin/bitcoin/pull/7186
< GitHub132> [bitcoin] jtimon reopened pull request #7091: Consensus build package (master...consensus-build) https://github.com/bitcoin/bitcoin/pull/7091
< GitHub36> [bitcoin] pstratem closed pull request #6745: Net: Remove "Address refresh broadcast" logic. (master...addr_known_reset) https://github.com/bitcoin/bitcoin/pull/6745
< GitHub197> [bitcoin] pstratem closed pull request #5989: improve addr/inv trickle logic (master...trickle_fix) https://github.com/bitcoin/bitcoin/pull/5989
< GitHub148> [bitcoin] pstratem opened pull request #7185: Note that reviewers should mention the id of the commits they reviewed. (master...2015-12-07-contributingackcommit) https://github.com/bitcoin/bitcoin/pull/7185

2015-12-07

< GitHub170> [bitcoin] morcos opened pull request #7184: [WIP] Implement SequenceLocks functions for BIP 68 (master...alternate68) https://github.com/bitcoin/bitcoin/pull/7184
< GitHub173> [bitcoin] Xekyo opened pull request #7183: Removed obsolete lines of code, broke continuation of loop. (master...Fix-#7182) https://github.com/bitcoin/bitcoin/pull/7183
< GitHub25> [bitcoin] laanwj opened pull request #7181: net: Add and document network messages in protocol.h (master...2015_12_network_messages) https://github.com/bitcoin/bitcoin/pull/7181
< GitHub38> [bitcoin] laanwj opened pull request #7180: net: Account for `sendheaders` `verack` messages (master...2015_12_log_sendheaders_verack) https://github.com/bitcoin/bitcoin/pull/7180
< GitHub92> [bitcoin] laanwj opened pull request #7179: net: Fix sent reject messages for blocks and transactions (master...2015_12_fix_reject_message) https://github.com/bitcoin/bitcoin/pull/7179
< GitHub85> bitcoin/master dc0305d Wladimir J. van der Laan: Merge pull request #6589...
< GitHub194> [bitcoin] laanwj closed pull request #6589: log bytes recv/sent per command (master...2015/08/throttle_prework) https://github.com/bitcoin/bitcoin/pull/6589
< GitHub85> bitcoin/master ca188c6 Jonas Schnelli: log bytes recv/sent per command
< GitHub85> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/82bcf405f6db...dc0305d15aa0
< GitHub113> bitcoin/0.12 b2d7ada Wladimir J. van der Laan: test: remove necessity to call create_callback_map...
< GitHub113> [bitcoin] laanwj pushed 1 new commit to 0.12: https://github.com/bitcoin/bitcoin/commit/b2d7ada3727f026ccd83d3d64c75aab660d8053e
< GitHub17> [bitcoin] laanwj closed pull request #7171: tests: remove necessity to call create_callback_map (master...2015_12_p2p_test_no_cbmap) https://github.com/bitcoin/bitcoin/pull/7171
< GitHub147> bitcoin/master 82bcf40 Wladimir J. van der Laan: Merge pull request #7171...
< GitHub147> bitcoin/master 2f601d2 Wladimir J. van der Laan: test: remove necessity to call create_callback_map...
< GitHub147> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/075faaebf2e5...82bcf405f6db
< GitHub111> bitcoin/0.12 82aff88 Matt Corallo: Don't do mempool lookups for "mempool" command without a filter...
< GitHub111> [bitcoin] laanwj pushed 1 new commit to 0.12: https://github.com/bitcoin/bitcoin/commit/82aff880d32f73bae28aa2cc071348ada603159b
< dcousens> what do y'all get for "bitcoin-cli getrawtransaction 4a5e1e4baab89f3a32518a88c31bc87f618f76673e2cc77ab2127b7afdeda33b"

2015-12-06

< GitHub123> [bitcoin] MarcoFalke opened pull request #7177: [qa] Change default block priority size to 0 (master...MarcoFalke-2015-rpcNoPrio) https://github.com/bitcoin/bitcoin/pull/7177

2015-12-05

< GitHub32> [bitcoin] gmaxwell closed pull request #7174: Don't do mempool lookups for "mempool" command without a filter (master...2015-12-braindead-mempool) https://github.com/bitcoin/bitcoin/pull/7174
< GitHub64> bitcoin/master 075faae Gregory Maxwell: Merge pull request #7174...
< GitHub64> bitcoin/master 96918a2 Matt Corallo: Don't do mempool lookups for "mempool" command without a filter
< GitHub64> [bitcoin] gmaxwell pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/a2822b97cb83...075faaebf2e5
< gmaxwell> AFAIK bitcoin core is the only wallet thats ever supported priority on the send side, and it's never even optimized for using priority wisely.
< paveljanik> gmaxwell, using this argument, this calls for a lot of NEW users to bitcoin to call for "restart" of the network...
< phantomcircuit> Luke-Jr, either fees become a significant portion of the total block reward and miners acting in their own self interest ignore priority, or we have massively misjudged the economic rational for mining and bitcoin will fail
< Luke-Jr> believe it or not, most miners *do* care to some degree about Bitcoin's success
< GitHub43> [bitcoin] TheBlueMatt opened pull request #7174: Don't do mempool lookups for "mempool" command without a filter (master...2015-12-braindead-mempool) https://github.com/bitcoin/bitcoin/pull/7174
< GitHub105> [bitcoin] laanwj closed pull request #7170: tests: Disable Tor interaction (master...2015_12_tests_nolistentor) https://github.com/bitcoin/bitcoin/pull/7170
< GitHub189> bitcoin/master a2822b9 Wladimir J. van der Laan: Merge pull request #7170...
< GitHub189> bitcoin/master 4c40ec0 Wladimir J. van der Laan: tests: Disable Tor interaction...
< GitHub189> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/792259278e4f...a2822b97cb83

2015-12-04

< GitHub93> bitcoin/0.12 f31955d Gregory Maxwell: Replace setInventoryKnown with a rolling bloom filter....
< GitHub93> [bitcoin] laanwj pushed 1 new commit to 0.12: https://github.com/bitcoin/bitcoin/commit/f31955d9da152e5e849575f0297f8fe1904cbfbc
< GitHub73> [bitcoin] laanwj opened pull request #7171: test: remove necessity to call create_callback_map (master...2015_12_p2p_test_no_cbmap) https://github.com/bitcoin/bitcoin/pull/7171
< GitHub74> [bitcoin] laanwj opened pull request #7170: tests: Disable Tor interaction (master...2015_12_tests_nolistentor) https://github.com/bitcoin/bitcoin/pull/7170
< * wumpus> must have the craziest multi-stage bitcoind launch command "nice -+19 screen -d -m gdb -x run.gdbscript -args ./bitcoind -datadir=/usb/bitcoin" (run.gdbscript is http://stackoverflow.com/a/5286038/216357 )
< jonasschnelli> wumpus: Okay. Switch to your find("other") approach, but i need to pre-populate "*other*" as well (which is fine): https://github.com/bitcoin/bitcoin/pull/6589/files#diff-9a82240fe7dfe86564178691cc57f2f1R2358
< GitHub63> bitcoin/0.12 6ba25d2 Gregory Maxwell: Disconnect on mempool requests from peers when over the upload limit....
< GitHub63> [bitcoin] laanwj pushed 1 new commit to 0.12: https://github.com/bitcoin/bitcoin/commit/6ba25d28868146d5d6dbd671881db3a58f549567
< GitHub164> bitcoin/master 7922592 Wladimir J. van der Laan: Merge pull request #7166...
< GitHub147> [bitcoin] laanwj closed pull request #7166: Disconnect on mempool requests from peers when over the upload limit. (master...mempool_p2p_when_overlimit) https://github.com/bitcoin/bitcoin/pull/7166
< GitHub164> bitcoin/master 6aadc75 Gregory Maxwell: Disconnect on mempool requests from peers when over the upload limit....
< GitHub164> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/5548d9cb11c8...792259278e4f
< GitHub136> [bitcoin] paveljanik opened pull request #7169: [Trivial] Disable compiler warnings about unused functions (master...20151204_scheduler_tests_warning) https://github.com/bitcoin/bitcoin/pull/7169
< GitHub179> [bitcoin] tadasvaranavicius closed pull request #7168: Fix socket recv buffer bug (master...fix_network_buffer_bug) https://github.com/bitcoin/bitcoin/pull/7168
< GitHub149> [bitcoin] tadasvaranavicius opened pull request #7168: Fix socket recv buffer bug (master...fix_network_buffer_bug) https://github.com/bitcoin/bitcoin/pull/7168
< GitHub68> [bitcoin] jameshilliard opened pull request #7167: Move TestBlockValidity into a background thread (master...TBVBackground) https://github.com/bitcoin/bitcoin/pull/7167

2015-12-03

< GitHub29> [bitcoin] gmaxwell opened pull request #7166: Disconnect on mempool requests from peers when over the upload limit. (master...mempool_p2p_when_overlimit) https://github.com/bitcoin/bitcoin/pull/7166
< jtimon> the code is in jtimon/libconsensus-f2 (pre-bitcoin-jt) anyway
< jtimon> and I don't close it because you have objections to it, just because I'm not going to rebase it as is if it's incompatible with building only once, which I also want for bitcoin jt
< GitHub19> [bitcoin] jtimon closed pull request #7091: Consensus build package (master...consensus-build) https://github.com/bitcoin/bitcoin/pull/7091
< GitHub142> [bitcoin] laanwj opened pull request #7165: WIP: build: Enable C++11 in build, require C++11 compiler (master...2015_12_c++11) https://github.com/bitcoin/bitcoin/pull/7165
< GitHub156> [bitcoin] ptschip opened pull request #7164: Do not download transactions during initial blockchain sync (master...tx_getdata) https://github.com/bitcoin/bitcoin/pull/7164
< jtimon> MarcoFalke: yeah I suggested it https://github.com/bitcoin/bitcoin/pull/6625#issuecomment-157403741 but sipa said that wasn't necessary. As said here https://github.com/bitcoin/bitcoin/pull/6625#issuecomment-161610648 is what I'm going to do the next time that the last commit causes conflicts
< MarcoFalke> jtimon maybe you can create a separate PR for the first 3 commits of https://github.com/bitcoin/bitcoin/pull/6625/commits
< GitHub168> bitcoin/0.12 cfb44ce Matt Bogosian: Add missing automake package to deb-based UNIX install instructions....
< GitHub168> [bitcoin] laanwj pushed 1 new commit to 0.12: https://github.com/bitcoin/bitcoin/commit/cfb44ce97a939cb9b6db96f4b273c2a618e11d88
< GitHub183> [bitcoin] laanwj closed pull request #7152: Add missing automake package to deb-based UNIX install instructions. (master...posita/fix-doc-build-unix) https://github.com/bitcoin/bitcoin/pull/7152
< GitHub52> bitcoin/master 5548d9c Wladimir J. van der Laan: Merge pull request #7152...
< GitHub52> bitcoin/master b440409 Matt Bogosian: Add missing automake package to deb-based UNIX install instructions.
< GitHub52> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/54a550bef8a8...5548d9cb11c8
< GitHub154> [bitcoin] laanwj reopened pull request #6745: Net: Remove "Address refresh broadcast" logic. (master...addr_known_reset) https://github.com/bitcoin/bitcoin/pull/6745
< GitHub22> [bitcoin] laanwj closed pull request #7113: Switch to a more efficient rolling Bloom filter (master...betterrolling) https://github.com/bitcoin/bitcoin/pull/7113
< GitHub84> bitcoin/master 086ee67 Pieter Wuille: Switch to a more efficient rolling Bloom filter...
< GitHub84> bitcoin/master 54a550b Wladimir J. van der Laan: Merge pull request #7113...
< GitHub84> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/884367662155...54a550bef8a8