2015-12-02

< Luke-Jr> and at least my Bitcoin LJR wallet does (although I understand 0.12 won't)
< Luke-Jr> gmaxwell: "Bitcoin only works for users with lots of bitcoins during spam attacks" is much better than "Bitcoin doesn't work during spam attacks period"
< Luke-Jr> gmaxwell: specifically, https://github.com/bitcoin/bitcoin/pull/7149 fixes the priority calculation without adding any such loop to CNB
< gmaxwell> Luke-Jr: slow block generation is clearly harmful to Bitcoin, in a way which I think is worse than loss of priority.
< Luke-Jr> gmaxwell: I see no benefit whatsoever to changing the default policy in a way that is clearly harmful to Bitcoin.
< GitHub159> [bitcoin] gmaxwell closed pull request #7037: Move the blocknotify callback ahead of peer announcement. (master...notify_early) https://github.com/bitcoin/bitcoin/pull/7037
< GitHub81> [bitcoin] gmaxwell closed pull request #7100: Replace setInventoryKnown with a rolling bloom filter. (master...known_bloom) https://github.com/bitcoin/bitcoin/pull/7100

2015-12-01

< GitHub150> [bitcoin] luke-jr opened pull request #7151: Revert default block priority size to 50k (master...revert_priodef) https://github.com/bitcoin/bitcoin/pull/7151
< GitHub39> [bitcoin] pstratem opened pull request #7150: Print size of pcoinsTip cache in AcceptToMemoryPool (master...2015-12-1-printcachesize) https://github.com/bitcoin/bitcoin/pull/7150
< GitHub68> [bitcoin] luke-jr opened pull request #7149: Bugfix: Correctly calculate priority when inputs are mined after dependent transactions enter the memory pool (master...bugfix_priority) https://github.com/bitcoin/bitcoin/pull/7149
< Luke-Jr> anything else is harmful to Bitcoin
< sipa> morcos: if everything in bitcoin was rational, nobody would have started using it
< GitHub151> [bitcoin] MarcoFalke reopened pull request #7084: mempool: Replace maxFeeRate of 10000*minRelayTxFee with maxTxFee (master...MarcoFalke-2015-mempoolMaxTxFee) https://github.com/bitcoin/bitcoin/pull/7084
< GitHub90> bitcoin/master 7230187 Alex Morcos: Add TxPriority class and comparator
< GitHub194> [bitcoin] sipa closed pull request #6898: Rewrite CreateNewBlock (master...fasterCNB) https://github.com/bitcoin/bitcoin/pull/6898
< GitHub90> bitcoin/master c49d5bc Alex Morcos: Store the total sig op count of a tx....
< GitHub90> bitcoin/master f3fe836 Alex Morcos: Add a score index to the mempool....
< GitHub90> [bitcoin] sipa pushed 7 new commits to master: https://github.com/bitcoin/bitcoin/compare/16f4a6e0fe26...4077ad20d03f
< GitHub22> [bitcoin] MarcoFalke closed pull request #7084: mempool: Replace maxFeeRate of 10000*minRelayTxFee with maxTxFee (master...MarcoFalke-2015-mempoolMaxTxFee) https://github.com/bitcoin/bitcoin/pull/7084
< GitHub13> [bitcoin] MarcoFalke opened pull request #7148: [WIP] Run extended nightly builds in travis (master...travis/nightly) https://github.com/bitcoin/bitcoin/pull/7148
< GitHub119> [bitcoin] jtimon closed pull request #7145: Mempool: Improve mempool's concurrency (master...mempool-estimator) https://github.com/bitcoin/bitcoin/pull/7145
< GitHub29> [bitcoin] jtimon closed pull request #7115: Mempool: Decouple CBlockPolicyEstimator from CTxMemPool (fix #6134) (master...6134-nits) https://github.com/bitcoin/bitcoin/pull/7115
< GitHub38> [bitcoin] MarcoFalke opened pull request #7147: Univalue: Pull subtree (master...MarcoFalke-2015-univalueSync) https://github.com/bitcoin/bitcoin/pull/7147
< GitHub55> [bitcoin] jtimon reopened pull request #7115: Mempool: Decouple CBlockPolicyEstimator from CTxMemPool (fix #6134) (master...6134-nits) https://github.com/bitcoin/bitcoin/pull/7115
< GitHub54> [bitcoin] paveljanik opened pull request #7146: Name union to prevent compiler warning (master...20151201_prevector_name_union) https://github.com/bitcoin/bitcoin/pull/7146
< jtimon> morcos: sipa BlueMatt I hope that if we can't agree on https://github.com/jtimon/bitcoin/commit/3c30cea7245a2fd44bbd9cf8844c6730855f63e4 , at least we can agree on #7145
< GitHub54> [bitcoin] jtimon opened pull request #7145: Mempool: Improve mempool's concurrency (master...mempool-estimator) https://github.com/bitcoin/bitcoin/pull/7145
< GitHub68> [bitcoin] laanwj opened pull request #7144: test: Disable scheduler test manythreads (master...2015_12_disable_schedulertest) https://github.com/bitcoin/bitcoin/pull/7144
< GitHub180> [bitcoin] laanwj closed pull request #7063: [Tests] Add prioritisetransaction RPC test (master...add-prioritisetransaction-rpctest) https://github.com/bitcoin/bitcoin/pull/7063
< jonasschnelli> sipa asked about adding a option in Qt ("provide X MB of ram for bitcoin-qt")...
< GitHub145> [bitcoin] laanwj closed pull request #7137: Tests: Explicitly set chain limits in replace-by-fee test (master...fix-rbf-test) https://github.com/bitcoin/bitcoin/pull/7137
< GitHub167> bitcoin/master 16f4a6e Wladimir J. van der Laan: Merge pull request #7137...
< GitHub167> bitcoin/master cfdc662 Suhas Daftuar: Explicitly set chain limits in replace-by-fee test
< GitHub167> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/bc1f4275705a...16f4a6e0fe26
< GitHub159> bitcoin/master bc1f427 Wladimir J. van der Laan: qt: periodic translations update
< GitHub159> [bitcoin] laanwj pushed 1 new commit to master: https://github.com/bitcoin/bitcoin/commit/bc1f4275705a6aae03ce439cd317ec4166075c08
< GitHub63> [bitcoin] laanwj closed pull request #7141: rpc: Don't translate warning messages (master...2015_12_warnings_translations) https://github.com/bitcoin/bitcoin/pull/7141
< GitHub161> bitcoin/master c0c08c7 Wladimir J. van der Laan: Merge pull request #7141...
< GitHub161> bitcoin/master aabc897 Wladimir J. van der Laan: rpc: Don't translate warning messages...
< GitHub161> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/a60538bc456c...c0c08c7c68d0
< GitHub146> [bitcoin] laanwj closed pull request #7143: qt: use QMetaObject::invokeMethod for cross-thread signaling in clientmodel (master...2015_12_qt4_build) https://github.com/bitcoin/bitcoin/pull/7143
< GitHub117> bitcoin/master a60538b Wladimir J. van der Laan: Merge pull request #7143...
< GitHub117> bitcoin/master 6da12df Wladimir J. van der Laan: qt: use QMetaObject::invokeMethod for cross-thread signaling in clientmodel...
< GitHub117> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/2ef5ffa59afa...a60538bc456c
< GitHub180> [bitcoin] laanwj closed pull request #6915: [Mempool] Improve removal of invalid transactions after reorgs (master...fix-reorg-handling) https://github.com/bitcoin/bitcoin/pull/6915
< GitHub85> bitcoin/master b0a064c Matt Corallo: Fix comment in removeForReorg
< GitHub85> bitcoin/master 9b060e5 Matt Corallo: Fix removal of time-locked transactions during reorg
< GitHub85> bitcoin/master 0c9959a Matt Corallo: Add failing test checking timelocked-txn removal during reorg
< GitHub85> [bitcoin] laanwj pushed 9 new commits to master: https://github.com/bitcoin/bitcoin/compare/9afbd96919af...2ef5ffa59afa
< GitHub47> bitcoin/master 9afbd96 Wladimir J. van der Laan: Merge pull request #7022...
< GitHub77> [bitcoin] laanwj closed pull request #7022: Change default block priority size to 0 (master...defaultPrioritySize) https://github.com/bitcoin/bitcoin/pull/7022
< GitHub47> bitcoin/master 50947ef Alex Morcos: Change default block priority size to 0...
< GitHub47> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/6abf6eb7bb77...9afbd96919af
< GitHub181> [bitcoin] laanwj opened pull request #7143: qt: use QMetaObject::invokeMethod for cross-thread signaling in clientmodel (master...2015_12_qt4_build) https://github.com/bitcoin/bitcoin/pull/7143
< GitHub93> [bitcoin] luke-jr opened pull request #7142: [WIP] Travis: Test build against system Qt4 (master...travis_qt4) https://github.com/bitcoin/bitcoin/pull/7142
< BlueMatt> wumpus: https://github.com/bitcoin/bitcoin/pull/6872 <-- review away
< Luke-Jr> https://github.com/bitcoin/bitcoin/pull/7128 has a bunch of utACKs, so I just rebased it
< Luke-Jr> probably should close https://github.com/bitcoin/bitcoin/pull/6451 since it's past Nov 11
< GitHub69> bitcoin/master 6abf6eb Wladimir J. van der Laan: Merge pull request #7063...
< GitHub69> bitcoin/master 2b31ab9 Suhas Daftuar: Add rpc test for prioritisetransaction
< GitHub69> bitcoin/master 6e8b07f Suhas Daftuar: Add rounding helper function to util.py
< GitHub69> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/8f761e87c3fd...6abf6eb7bb77
< BlueMatt> so https://github.com/bitcoin/bitcoin/pull/7037 plus a "I want no txn" option would be my vote
< BlueMatt> no, we'd need https://github.com/bitcoin/bitcoin/pull/7037 too
< BlueMatt> wumpus: if we want to do a fix for gbt for 0.12, i think its https://github.com/bitcoin/bitcoin/pull/7104
< wumpus> Luke-Jr: you could help testing https://github.com/bitcoin/bitcoin/pull/6898 :)
< wumpus> https://github.com/bitcoin/bitcoin/pull/6872 needs rebase, and BlueMatt has no time to do it until tomorrow
< wumpus> https://github.com/bitcoin/bitcoin/pull/6898 has no ACKs at all, not even untested or concept
< wumpus> gmaxwell: final ack on https://github.com/bitcoin/bitcoin/pull/6915 would be nice
< GitHub11> [bitcoin] laanwj closed pull request #7136: [trivial] Fix markdown, links and help messages (master...MarcoFalke-2015-trivial5) https://github.com/bitcoin/bitcoin/pull/7136
< GitHub24> bitcoin/master 9999cb0 MarcoFalke: Fix url in .travis.yml
< GitHub24> bitcoin/master fad3035 MarcoFalke: [doc] Minor markdown fixes
< GitHub24> bitcoin/master fa22a10 MarcoFalke: contrib: Del. gitian downloader config and update gitian README
< GitHub24> [bitcoin] laanwj pushed 6 new commits to master: https://github.com/bitcoin/bitcoin/compare/327291af02d0...8f761e87c3fd
< GitHub22> [bitcoin] laanwj closed pull request #6914: Add pre-allocated vector type and use it for CScript (master...prevector) https://github.com/bitcoin/bitcoin/pull/6914
< GitHub104> bitcoin/master 327291a Wladimir J. van der Laan: Merge pull request #6914...
< GitHub104> bitcoin/master 114b581 Pieter Wuille: Prevector type
< GitHub104> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/9490bd71bdb4...327291af02d0
< GitHub99> [bitcoin] laanwj closed pull request #7096: [Wallet] Improve minimum absolute fee GUI options (master...2015/11/feefix) https://github.com/bitcoin/bitcoin/pull/7096
< GitHub129> bitcoin/master 31b508a Jonas Schnelli: [Qt] make use of the nMinimumTotalFee (absolute) in coincontrols fee calculation
< GitHub129> bitcoin/master 80462dd Jonas Schnelli: [Qt] use ASYMP_UTF8 (≈) whenever we show a fee that is not absolute
< GitHub129> bitcoin/master ecc7c82 Pieter Wuille: Move fPayAtLeastCustomFee function to CC
< GitHub129> [bitcoin] laanwj pushed 5 new commits to master: https://github.com/bitcoin/bitcoin/compare/eb3d1b348773...9490bd71bdb4
< GitHub146> [bitcoin] laanwj closed pull request #7135: [trivial] pull-tester cleanup: Run keypool, Tidy stdout (master...MarcoFalke-2015-pullTester) https://github.com/bitcoin/bitcoin/pull/7135
< GitHub101> bitcoin/master eb3d1b3 Wladimir J. van der Laan: Merge pull request #7135...
< GitHub101> bitcoin/master fa3a38a MarcoFalke: [qa] pull-tester: Cleanup (run keypool, tidy stdout)...
< GitHub101> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/30c2d8c635c4...eb3d1b348773
< GitHub34> [bitcoin] laanwj closed pull request #7134: [Qt] Don't translate warning messages (master...2015/11/qt_getwarnings) https://github.com/bitcoin/bitcoin/pull/7134
< GitHub130> [bitcoin] laanwj opened pull request #7141: rpc: Don't translate warning messages (master...2015_12_warnings_translations) https://github.com/bitcoin/bitcoin/pull/7141
< GitHub98> [bitcoin] laanwj closed pull request #7105: Keep track of explicit wallet conflicts instead of using mempool (master...realconflicts) https://github.com/bitcoin/bitcoin/pull/7105
< GitHub170> bitcoin/master 30c2d8c Wladimir J. van der Laan: Merge pull request #7105...
< GitHub170> bitcoin/master 9ac63d6 Pieter Wuille: Keep track of explicit wallet conflicts instead of using mempool
< GitHub170> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/1b5118bfa0d9...30c2d8c635c4
< GitHub11> [bitcoin] laanwj closed pull request #7079: Prevent peer flooding inv request queue (redux) (redux) (master...setAskFor) https://github.com/bitcoin/bitcoin/pull/7079
< GitHub104> bitcoin/master 1b5118b Wladimir J. van der Laan: Merge pull request #7079...
< GitHub104> bitcoin/master 5029698 kazcw: prevent peer flooding request queue for an inv...
< GitHub104> bitcoin/master ebb25f4 Gregory Maxwell: Limit setAskFor and retire requested entries only when a getdata returns....
< GitHub104> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/c143c499c85b...1b5118bfa0d9
< GitHub134> [bitcoin] gmaxwell closed pull request #7072: [RPC] Add transaction size to JSON output (master...master) https://github.com/bitcoin/bitcoin/pull/7072
< GitHub18> bitcoin/master c143c49 Gregory Maxwell: Merge pull request #7072...
< GitHub18> bitcoin/master 996d311 Nick: [RPC] Add transaction size to JSON output...
< GitHub18> [bitcoin] gmaxwell pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/438ee59839ad...c143c499c85b
< GitHub150> [bitcoin] jtimon closed pull request #7115: Mempool: Decouple CBlockPolicyEstimator from CTxMemPool (fix #6134) (master...6134-nits) https://github.com/bitcoin/bitcoin/pull/7115
< jtimon> morcos: if you can't agree with https://github.com/jtimon/bitcoin/commit/3c30cea7245a2fd44bbd9cf8844c6730855f63e4 you will have to explain why we need to introduce an unecessary dependency "right now"

2015-11-30

< jtimon> I specially would like to remove that nMempoolSizeMax local variable that's only used for a check but doesn't actually gurantee anything about the future contents of "GetArg("-maxmempool", DEFAULT_MAX_MEMPOOL_SIZE) * 1000000" line https://github.com/bitcoin/bitcoin/compare/master...jtimon:mempool-circular-dependency#diff-c865a8939105e6350a50af02766291b7L902
< jtimon> sipa: thus https://github.com/bitcoin/bitcoin/compare/master...jtimon:mempool-circular-dependency I thought you would like it more than the initial one
< jcorgan> rook520: take it to #bitcoin
< rook520> Why is it wrong to get involved in the bitcoin trade?? Wouldnt it be smart to buy some shares and just let it sit and not touch it...and just let it evolve with bitcoin itself??
< BlueMatt> jtimon: java wrapper in bitcoin core for libbitcoinconsensus?
< morcos> when we started working on bitcoin
< sipa> going to run master + memory-usage-impacting-PRs-i-expect-to-be-merged on bitcoin.sipa.be, with empty bitcoin.conf
< GitHub15> [bitcoin] gmaxwell closed pull request #7044: RPC: Added additional config option for multiple RPC users. (master...multrpc) https://github.com/bitcoin/bitcoin/pull/7044
< GitHub26> bitcoin/master d52fbf0 Gregory Sanders: Added additional config option for multiple RPC users.
< GitHub26> bitcoin/master 438ee59 Gregory Maxwell: Merge pull request #7044...
< GitHub26> [bitcoin] gmaxwell pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/34e02e014718...438ee59839ad
< sipa> jonasschnelli: opinion about asking on bitcoin-qt first startup how much RAM the user wants to make available for chainstate?
< jgarzik> (picocoin thing just an anecdote noting Fedora people working on the overall ecosystem; Harald also works on bitcoin)
< jgarzik> Vaguely related - Fedora keeps inching along towards shipping bitcoin - I got a patch from Harald Hoyer to picocoin, adding bitcoin's curve to an otherwise functional openssl lib. Once libsecp256k1 is packaged into Fedora, that makes a nice end run around the patent/copyright annoyances blocking Fedora deployment.
< GitHub22> [bitcoin] sdaftuar opened pull request #7137: Tests: Explicitly set chain limits in replace-by-fee test (master...fix-rbf-test) https://github.com/bitcoin/bitcoin/pull/7137
< jonasschnelli> sipa: I think you could pull https://github.com/jonasschnelli/bitcoin/commit/965a1e6d9c5c1af5555665c0a94cde85dfe19a7b into your #7105
< GitHub142> bitcoin/master 9b8fc6c Wladimir J. van der Laan: Merge pull request #7035...
< GitHub142> bitcoin/master 4531fc4 Daniel Cousens: torcontrol: only output disconnect if -debug=tor
< GitHub142> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/96b802510da0...9b8fc6c89a08
< GitHub20> [bitcoin] sipa opened pull request #7133: Replace setInventoryKnown with a rolling bloom filter (rebase of #7100) (master...known_bloom) https://github.com/bitcoin/bitcoin/pull/7133
< GitHub88> [bitcoin] laanwj closed pull request #7112: [Qt] reduce cs_main locks during tip update, more fluently update UI (master...2015/11/qt_tipupdate) https://github.com/bitcoin/bitcoin/pull/7112
< GitHub77> bitcoin/master 947d20b Jonas Schnelli: [Qt] reduce cs_main in getVerificationProgress()
< GitHub77> bitcoin/master e6d50fc Jonas Schnelli: [Qt] update block tip (height and date) without locking cs_main, update always (each block)
< GitHub77> bitcoin/master 012fc91 Jonas Schnelli: NotifyBlockTip signal: switch from hash (uint256) to CBlockIndex*...
< GitHub77> [bitcoin] laanwj pushed 6 new commits to master: https://github.com/bitcoin/bitcoin/compare/a7751824ce8a...96b802510da0
< wumpus> more generally: please help moving anything with 0.12 milestone along https://github.com/bitcoin/bitcoin/pulls?q=is%3Aopen+is%3Apr+milestone%3A0.12.0 deadline is tomorrow
< GitHub37> bitcoin/0.11 5f09cda MarcoFalke: [wallet] Add rpc tests to verify fee calculations...
< GitHub37> [bitcoin] laanwj pushed 2 new commits to 0.11: https://github.com/bitcoin/bitcoin/compare/595c8d6301cf...5f09cda0bf4c
< GitHub37> bitcoin/0.11 7d0a05f Ryan Havar: Default fPayAtLeastCustomFee to false...
< GitHub124> [bitcoin] laanwj closed pull request #7103: [wallet, rpc tests] Fix settxfee, paytxfee (master...FixSettxfee) https://github.com/bitcoin/bitcoin/pull/7103
< GitHub24> bitcoin/master a775182 Wladimir J. van der Laan: Merge pull request #7103...
< GitHub24> bitcoin/master fa506c0 MarcoFalke: [wallet] Add rpc tests to verify fee calculations
< GitHub24> bitcoin/master 4b89f01 Ryan Havar: Default fPayAtLeastCustomFee to false...
< GitHub24> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/6fc287f2df54...a7751824ce8a
< GitHub101> [bitcoin] laanwj closed pull request #7027: [qa] rpc-tests: Properly use test framework (master...MarcoFalke-2015-rpcTestCleanup) https://github.com/bitcoin/bitcoin/pull/7027
< GitHub144> bitcoin/master 6fc287f Wladimir J. van der Laan: Merge pull request #7027...
< GitHub144> bitcoin/master 4ea1790 MarcoFalke: [qa] keypool: DRY: Use test framework
< GitHub144> bitcoin/master c6973ca MarcoFalke: [qa] keypool: Fix white space to prepare transition to test framework
< GitHub144> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/9ebedc1756e3...6fc287f2df54
< sipa> CodeShark: this is bitcoin core dev
< wumpus> bitcoin core's wallet makes the explicit assumption that we're the only one with the keys in this wallet.dat, they cannot be shared, so that should never happen
< jonasschnelli> sipa: the change results in a ugly diff. But i think it makes sense: https://github.com/jonasschnelli/bitcoin/commit/9af5f9cb8773da2904aa3819234aaebd2efb5d15
< btcdrak> gmaxwell: I thoroughly agree with you. In fact, we sort of need a PR side to act as a communication bridge with the general public. The weekly IRC meetings have been really well received, but because they are posted on social media they get buried. This is why I discussed with harding to make a new section on bitcoin.org so we can collate these kind of
< wumpus> gmaxwell: this is another example of how something only gains interest as soon as it is merged. RBF discussions have been going on for as long bitcoin exists, and still they manage to pose it as if this is something new and controversial instead of the compromise of years of argument
< GitHub118> [bitcoin] jonasschnelli closed pull request #7127: Bugfix: Default -uiplatform is not actually the platform this build was compiled on (master...bugfix_uiplatform) https://github.com/bitcoin/bitcoin/pull/7127
< GitHub198> bitcoin/master fa93174 Jonas Schnelli: Merge pull request #7127...
< GitHub198> bitcoin/master a6cbc02 Luke Dashjr: Bugfix: Default -uiplatform is not actually the platform this build was compiled on
< GitHub198> [bitcoin] jonasschnelli pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/c28d3937b095...fa93174a7c06
< gmaxwell> Just so people aren't caught surprised, the addtion of Opt-in RBF to Bitcoin Core has inspired a seemingly organized misinformation campaign that has been spiraling all over the place. I've been maintaining a FAQ on reddit: https://www.reddit.com/r/Bitcoin/comments/3urm8o/optin_rbf_is_misunderstood_ask_questions_about_it/
< sipa> andytoshi: good thing bitcoin core uses a reasonable policy :)

2015-11-29

< GitHub105> [bitcoin] sipa closed pull request #6494: Allow block announcements with headers (master...direct-headers-announcement) https://github.com/bitcoin/bitcoin/pull/6494
< GitHub38> [bitcoin] sipa closed pull request #7129: Direct headers announcement (rebase of #6494) (master...direct-headers-announcement) https://github.com/bitcoin/bitcoin/pull/7129
< GitHub182> bitcoin/master 49fb8e8 Pieter Wuille: Documentation updates for BIP 130
< GitHub182> bitcoin/master 5d5ef3a Pieter Wuille: Merge pull request #7129...
< GitHub182> bitcoin/master 50262d8 Suhas Daftuar: Allow block announcements with headers...
< GitHub182> [bitcoin] sipa pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/c894fbbb1dc0...5d5ef3a4cf8e
< GitHub47> [bitcoin] sipa opened pull request #7129: Direct headers announcement (rebase of #6494) (master...direct-headers-announcement) https://github.com/bitcoin/bitcoin/pull/7129
< GitHub12> [bitcoin] gmaxwell closed pull request #7119: Add option to opt into full-RBF when sending funds (master...2015-11-opt-into-full-rbf-option) https://github.com/bitcoin/bitcoin/pull/7119
< GitHub27> bitcoin/master c894fbb Gregory Maxwell: Merge pull request #7106...
< GitHub63> [bitcoin] gmaxwell closed pull request #7106: Fix and improve relay from whitelisted peers (master...realwhiterelay) https://github.com/bitcoin/bitcoin/pull/7106
< GitHub27> bitcoin/master a9f3d3d Pieter Wuille: Fix and improve relay from whitelisted peers...
< GitHub27> [bitcoin] gmaxwell pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/61457c29d735...c894fbbb1dc0
< GitHub107> [bitcoin] gmaxwell closed pull request #7123: [WIP] Make trickle logic useful again, delay trickle when past upload limit. (master...actually_trickle) https://github.com/bitcoin/bitcoin/pull/7123

2015-11-28

< GitHub170> [bitcoin] luke-jr opened pull request #7128: Constrain rpcport default values to a single location in code (master...const_rpcport) https://github.com/bitcoin/bitcoin/pull/7128
< GitHub26> [bitcoin] luke-jr opened pull request #7127: Bugfix: Default -uiplatform is not actually the platform this build was compiled on (master...bugfix_uiplatform) https://github.com/bitcoin/bitcoin/pull/7127
< MarcoFalke> Also I haven't looked at chainparams https://github.com/bitcoin/bitcoin/pull/6349/files#diff-dbba8d0e1257cc14ce18de8fd6c9a0b1 Not sure if still relevant
< GitHub79> [bitcoin] MarcoFalke opened pull request #7126: [trivial] Move blocksonly parameter interaction to InitParameterInteraction() (master...MarcoFalke-2015-initParam) https://github.com/bitcoin/bitcoin/pull/7126
< GitHub168> [bitcoin] sipa closed pull request #6508: Switch to a constant-space Merkle root/branch algorithm. (master...constmerkle) https://github.com/bitcoin/bitcoin/pull/6508
< GitHub109> bitcoin/master 61457c2 Pieter Wuille: Merge pull request #6508...
< GitHub109> bitcoin/master eece63f Pieter Wuille: Switch blocks to a constant-space Merkle root/branch algorithm....
< GitHub109> bitcoin/master ee60e56 Pieter Wuille: Add merkle.{h,cpp}, generic merkle root/branch algorithm
< GitHub109> [bitcoin] sipa pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/8d26289c9a7c...61457c29d735
< GitHub101> [bitcoin] sipa closed pull request #6349: Constrain constant values to a single location in code (master...opt_defaults) https://github.com/bitcoin/bitcoin/pull/6349
< GitHub37> [bitcoin] sipa closed pull request #6961: luke-jr constants (master...luke-jr-const) https://github.com/bitcoin/bitcoin/pull/6961
< GitHub17> bitcoin/master 3307bdb Luke Dashjr: Bugfix: Omit wallet-related options from -help when wallet is not supported
< GitHub17> bitcoin/master faf93f3 MarcoFalke: [trivial] Reuse translation and cleanup DEFAULT_* values...
< GitHub17> bitcoin/master b966aa8 Luke Dashjr: Constrain constant values to a single location in code
< GitHub17> [bitcoin] sipa pushed 5 new commits to master: https://github.com/bitcoin/bitcoin/compare/92aa7311d64c...8d26289c9a7c
< sipa> perhaps you'll even find things to improve and start contributing to bitcoin
< SomeT> hi, is there a tutorial on how to modify bitcoin from the source code? so many files I have no idea where to start
< GitHub147> [bitcoin] sipa opened pull request #7125: Replace global trickle node with random delays (master...timetrickle) https://github.com/bitcoin/bitcoin/pull/7125
< GitHub142> [bitcoin] jonasschnelli closed pull request #7067: [Wallet] improve detection of conflicted transactions (master...2015/11/mempool_wallet) https://github.com/bitcoin/bitcoin/pull/7067
< GitHub0> bitcoin/master 92aa731 Pieter Wuille: Merge pull request #6942...
< GitHub105> [bitcoin] sipa closed pull request #6942: Fix CCoins serialization documentation (master...docfix) https://github.com/bitcoin/bitcoin/pull/6942
< GitHub0> bitcoin/master e482a7f Peter Josling: Fix CCoins serialization documentation...
< GitHub0> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/8332457e95c9...92aa7311d64c
< GitHub104> [bitcoin] sipa closed pull request #7003: doc: Add non-style-related development guidelines (master...2015_11_development_guidelines) https://github.com/bitcoin/bitcoin/pull/7003
< GitHub38> bitcoin/master b8c06ef Wladimir J. van der Laan: doc: Add non-style-related development guidelines...
< GitHub38> bitcoin/master 8332457 Pieter Wuille: Merge pull request #7003...
< GitHub38> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/8284feb3c91f...8332457e95c9
< GitHub170> [bitcoin] gmaxwell opened pull request #7123: [WIP] Make trickle logic useful again, delay trickle when past upload limit. (master...actually_trickle) https://github.com/bitcoin/bitcoin/pull/7123
< GitHub62> [bitcoin] robegan21 opened pull request #7122: simple fix to mask command line arguments after they are copied (master...MaskCommandLineArgs) https://github.com/bitcoin/bitcoin/pull/7122

2015-11-27

< GitHub102> [bitcoin] petertodd opened pull request #7119: Add option to opt into full-RBF when sending funds (master...2015-11-opt-into-full-rbf-option) https://github.com/bitcoin/bitcoin/pull/7119
< jtimon> anyway, I'll rewrite my policy stuff on bitcoin jt 0.12, just like my consensus stuff and then decide, worse case I can just rebase a giant branch once per major release
< GitHub131> [bitcoin] laanwj closed pull request #7117: Print correct minimum mempool size in MB (master...patch-14) https://github.com/bitcoin/bitcoin/pull/7117
< GitHub146> [bitcoin] paveljanik opened pull request #7118: Describe maxmempool and mempoolminfee in the getmempoolinfo RPC help. (master...20151127_getmempoolinfo_fixes) https://github.com/bitcoin/bitcoin/pull/7118
< GitHub85> [bitcoin] paveljanik opened pull request #7117: Print correct minimum mempool size in MB (master...patch-14) https://github.com/bitcoin/bitcoin/pull/7117
< GitHub112> [bitcoin] laanwj closed pull request #7078: uint256::GetCheapHash bigendian compatibility (master...ppc) https://github.com/bitcoin/bitcoin/pull/7078
< GitHub60> bitcoin/master 93e0514 Wladimir J. van der Laan: Merge pull request #7078...
< GitHub60> bitcoin/master c434940 daniel: uint256::GetCheapHash bigendian compatibility
< GitHub60> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/f3d0fdd3924f...93e0514fd083
< GitHub1> [bitcoin] jtimon opened pull request #7116: Trivial: Fix warning introduced by #7053 by casting to uint64_t (master...fix-7053) https://github.com/bitcoin/bitcoin/pull/7116
< GitHub29> [bitcoin] laanwj closed pull request #7114: util: Don't set strMiscWarning on every exception (master...2015_11_exception_nomiscwarning) https://github.com/bitcoin/bitcoin/pull/7114
< GitHub43> bitcoin/master d6454f6 Wladimir J. van der Laan: Merge pull request #7090...
< GitHub43> bitcoin/master cde857f Peter Todd: Connect to Tor hidden services by default...
< GitHub43> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/d8368a07baf2...d6454f6cb23e
< GitHub46> [bitcoin] jtimon opened pull request #7115: Mempool: Decouple CBlockPolicyEstimator from CTxMemPool (fix #6134) (master...6134-nits) https://github.com/bitcoin/bitcoin/pull/7115
< GitHub192> bitcoin/master d8368a0 Wladimir J. van der Laan: Merge pull request #7095...
< GitHub140> [bitcoin] laanwj closed pull request #7095: Replace scriptnum_test's normative ScriptNum implementation (master...2015_11_remove_openssl_consensus_checks) https://github.com/bitcoin/bitcoin/pull/7095
< GitHub192> bitcoin/master 4ec3561 Wladimir J. van der Laan: Replace scriptnum_test's normative ScriptNum implementation...
< GitHub192> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/9502b7f634a7...d8368a07baf2
< GitHub136> [bitcoin] laanwj closed pull request #7083: [init] Print OpenSSL version fix (master...MarcoFalke-2015-initOpenSSL) https://github.com/bitcoin/bitcoin/pull/7083
< GitHub162> bitcoin/master 9502b7f Wladimir J. van der Laan: Merge pull request #7083...
< GitHub162> bitcoin/master faf12bc MarcoFalke: OpenSSL 1.1.0: Fix text variant of the version number
< GitHub162> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/2a94cd67e805...9502b7f634a7
< GitHub75> [bitcoin] sipa opened pull request #7113: Switch to a more efficient rolling Bloom filter (master...betterrolling) https://github.com/bitcoin/bitcoin/pull/7113
< GitHub174> [bitcoin] laanwj closed pull request #7058: Fix ZMQ docs - Improve logging (master...zmqdoc) https://github.com/bitcoin/bitcoin/pull/7058
< GitHub74> bitcoin/master ffacd27 Daniel Cousens: zmq: prepend zmq to debug messages
< GitHub74> bitcoin/master 14075b1 Daniel Cousens: init: add zmq to debug categories
< GitHub74> bitcoin/master cdcd816 Daniel Cousens: init: amend ZMQ flag names
< GitHub74> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/f8a8e27a6a19...9a3e1a59dfed
< wumpus> otherwise you're creating a bitcoin fork bomb
< wumpus> could add a mention about to https://bitcoin.org/en/full-node
< wumpus> in the end could just as well have used our own, hand-rolled binary packet format, every language needs a special library to interface to bitcoin anyway ...

2015-11-26

< GitHub102> [bitcoin] Cocosoft opened pull request #7107: Qt: Add network port input box to GUI settings (master...qtnetworkport) https://github.com/bitcoin/bitcoin/pull/7107
< GitHub77> [bitcoin] sipa opened pull request #7106: Fix and improve relay from whitelisted peers (master...realwhiterelay) https://github.com/bitcoin/bitcoin/pull/7106
< GitHub6> [bitcoin] sipa opened pull request #7105: Keep track of explicit wallet conflicts instead of using mempool (master...realconflicts) https://github.com/bitcoin/bitcoin/pull/7105
< jtimon> cfields_: I can't undesrtand why these commits break the build, it would be great if you could explain. why does the order matter in bitcoind_LDADD? why crypto/libbitcoin_crypto.a works but consensus/libbitcoin_consensus.a doesn't (inside of the consensus dir)? https://github.com/jtimon/bitcoin/compare/consensus-build...jtimon:consensus-build-full
< GitHub0> [bitcoin] pstratem opened pull request #7104: [Mining] Build empty block on when chainTip changes. (master...2015-11-26-gbt-latency) https://github.com/bitcoin/bitcoin/pull/7104
< MarcoFalke> Can you cancel https://travis-ci.org/bitcoin/bitcoin/builds/92213684 as well
< jonasschnelli> I think MarcoFalke is right: https://travis-ci.org/bitcoin/bitcoin/builds/92025728
< GitHub194> [bitcoin] MarcoFalke opened pull request #7103: [wallet, rpc tests] Fix settxfee, paytxfee (master...FixSettxfee) https://github.com/bitcoin/bitcoin/pull/7103
< GitHub188> [bitcoin] laanwj closed pull request #7069: [trivial] Fix -maxmempool InitError (master...MarcoFalke-2015-maxmempoolInitError) https://github.com/bitcoin/bitcoin/pull/7069
< GitHub56> bitcoin/master f8a8e27 Wladimir J. van der Laan: Merge pull request #7069...
< GitHub56> bitcoin/master fa472f3 MarcoFalke: [trivial] Fix -maxmempool InitError
< GitHub56> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/be281d8a83ca...f8a8e27a6a19
< jonasschnelli> I think https://github.com/bitcoin/bitcoin/pull/7096/files is minimal if you hide the RPC tests.
< jonasschnelli> we could try to get https://github.com/bitcoin/bitcoin/pull/7096 reviewed and merged, and backported.... then remove the absolute fee from master (which is trivial)
< gmaxwell> jonasschnelli: AFAIK nothing about the bitcoin system uses absolute fees, and it would be irrational for miners to do so.
< wumpus> e.g. the tests here hardcode the hash function used even https://github.com/bitcoin/bitcoin/pull/7078
< jonasschnelli> the problem is, you can set the tx fee over rpc, but because of https://github.com/bitcoin/bitcoin/pull/5200/files#diff-d7618bdc04db23aa74d6a5a4198c58fdR1637, it will always take the feePerKB value as absolute fee.
< wump> so re: https://github.com/bitcoin/bitcoin/pull/7096 was this change in RPC settxfee behavior ever planned, or noticed before? I think it's strange that this is in 0.11, but no one reported the change from per-kB to absolute fee?
< GitHub110> [bitcoin] laanwj closed pull request #7087: [Net]Add -enforcenodebloom option (master...2015-11-23-bloom-disable) https://github.com/bitcoin/bitcoin/pull/7087
< GitHub96> bitcoin/master 9cf6688 Patick Strateman: Document both the peerbloomfilters and enforcenodebloom options.
< GitHub96> bitcoin/master 0f4dc53 Patick Strateman: Add enforcenodebloom option....
< GitHub96> bitcoin/master b3caa9b Patick Strateman: Move bloom filter filtering logic outside of command "switch" (giant if/else)....
< GitHub96> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/2b2ddc558e1c...be281d8a83ca
< GitHub92> [bitcoin] gmaxwell opened pull request #7100: Replace setInventoryKnown with a rolling bloom filter. (master...known_bloom) https://github.com/bitcoin/bitcoin/pull/7100
< tulip> dnsseed.bitcoin.DASHjr.org
< sipa> cfields_: i think it's running github.com/sipa/bitcoin-seeder
< gmaxwell> sipa: do you have an opinions about jtimons's comment: https://github.com/bitcoin/bitcoin/pull/6508
< gmaxwell> sipa: gonna address my comment on https://github.com/bitcoin/bitcoin/pull/6996 ?

2015-11-25

< GitHub82> [bitcoin] gmaxwell opened pull request #7099: Add whitelistforcerelay setting and default to off. (master...control_relay_force) https://github.com/bitcoin/bitcoin/pull/7099
< GitHub147> [bitcoin] jonasschnelli closed pull request #6708: [wallet] Default fPayAtLeastCustomFee to false (master...MarcoFalke-2015-walletFixPayTxFee) https://github.com/bitcoin/bitcoin/pull/6708
< GitHub180> [bitcoin] jonasschnelli opened pull request #7096: [Wallet] fix settxfee and improve minimum absolute fee GUI options (master...2015/11/feefix) https://github.com/bitcoin/bitcoin/pull/7096
< wumpus> it never was; gitian-downloader was meant to be a downloader for bitcoin that checks the gitian signatures, but AFAIK it was never completed or used
< GitHub91> [bitcoin] laanwj opened pull request #7095: Replace scriptnum_test's normative ScriptNum implementation (master...2015_11_remove_openssl_consensus_checks) https://github.com/bitcoin/bitcoin/pull/7095
< sipa> so nobody ever noticed that this makes bitcoin core's transaction pay around 4x too much fee by default?
< MarcoFalke> jonasschnelli, are you working on the https://github.com/bitcoin/bitcoin/issues/6749#issuecomment-157746758 thing?
< jonasschnelli> If we drop absolute fees (which would be the right step), rpc tests need this update: https://github.com/bitcoin/bitcoin/pull/6708/files#diff-51c9989cea15f3cac744183b78cb0688
< jonasschnelli> i think we should review https://github.com/bitcoin/bitcoin/pull/6708
< jonasschnelli> If it would be, that assert would not be true: https://github.com/bitcoin/bitcoin/blob/master/qa/rpc-tests/wallet.py#L111
< GitHub130> [bitcoin] laanwj closed pull request #7088: [trivial] pull secp256k1 subtree (master...MarcoFalke-2015-syncSecp256k1) https://github.com/bitcoin/bitcoin/pull/7088
< GitHub53> bitcoin/master 2b2ddc5 Wladimir J. van der Laan: Merge pull request #7088...
< GitHub53> bitcoin/master fa63e49 MarcoFalke: Merge commit '5ad54630935d1f340666de7bc9ffef9b8a1df296' into HEAD
< GitHub53> bitcoin/master 5ad5463 MarcoFalke: Squashed 'src/secp256k1/' changes from 2bfb82b..6c527ec...
< GitHub53> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/348b281f8a67...2b2ddc558e1c
< wumpus> jonasschnelli: good call on closing #7089 - indeed, bitcoin's core github is not the place to request changes to consensus behavior
< GitHub54> [bitcoin] laanwj closed pull request #7092: build: Set osx permissions in the dmg to make Gatekeeper happy (master...osx-perm-fix) https://github.com/bitcoin/bitcoin/pull/7092
< GitHub102> bitcoin/master 348b281 Wladimir J. van der Laan: Merge pull request #7092...
< GitHub102> bitcoin/master 392d3c5 Cory Fields: build: Set osx permissions in the dmg to make Gatekeeper happy
< GitHub102> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/26af1ac7cbce...348b281f8a67
< wumpus> was that the 'link yourself rich' or something transaction from bitcoin talk?
< GitHub116> [bitcoin] jonasschnelli closed pull request #7006: [Qt] add startup option to reset Qt settings (master...2015/11/qt_resetsettings) https://github.com/bitcoin/bitcoin/pull/7006
< GitHub144> bitcoin/master 26af1ac Jonas Schnelli: Merge pull request #7006...