2017-05-23

< bitcoin-git> bitcoin/master 5844609 practicalswift: [net] Avoid initialization to a value that is never read...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/ce8176d0389c...7e96ecf075e8
< bitcoin-git> [bitcoin] laanwj closed pull request #10295: [qt] Move some WalletModel functions into CWallet (master...pr/ipc-move) https://github.com/bitcoin/bitcoin/pull/10295
< bitcoin-git> bitcoin/master 429aa9e Russell Yanofsky: [test] Move some tests from qt -> wallet...
< bitcoin-git> bitcoin/master d944bd7 Russell Yanofsky: [qt] Move some WalletModel functions into CWallet...
< bitcoin-git> bitcoin/master ef8ca17 Russell Yanofsky: [test] Add tests for some walletmodel functions...
< bitcoin-git> [bitcoin] laanwj pushed 5 new commits to master: https://github.com/bitcoin/bitcoin/compare/46771514fa86...ce8176d0389c
< bitcoin-git> bitcoin/master 4677151 Wladimir J. van der Laan: Merge #9571: RPC: getblockchaininfo returns BIP signaling statistics...
< bitcoin-git> bitcoin/master 557c9a6 Matthew Zipkin: RPC: getblockchaininfo: BIP9 stats...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/433c57aa6f30...46771514fa86
< bitcoin-git> [bitcoin] laanwj closed pull request #10336: Get actual path for EUID instead of HOME dir (master...contrib) https://github.com/bitcoin/bitcoin/pull/10336
< bitcoin-git> [bitcoin] laanwj closed pull request #10421: [qt] Remove excess logic: Prefer "return foo;" to "if (foo) { return true; } else { return false; }" (master...if-expr-return-true-else-return-false) https://github.com/bitcoin/bitcoin/pull/10421
< bitcoin-git> bitcoin/master 433c57a Wladimir J. van der Laan: Merge #10421: [qt] Remove excess logic: Prefer "return foo;" to "if (foo) { return true; } else { return false; }"...
< bitcoin-git> bitcoin/master e49b868 practicalswift: [qt] Remove excess logic...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/15254e907e8c...433c57aa6f30
< bitcoin-git> [bitcoin] laanwj closed pull request #10372: Add perf counter data to GetStrongRandBytes state in scheduler (master...2017-05-scheduler-rng) https://github.com/bitcoin/bitcoin/pull/10372
< bitcoin-git> bitcoin/master 15254e9 Wladimir J. van der Laan: Merge #10372: Add perf counter data to GetStrongRandBytes state in scheduler...
< bitcoin-git> bitcoin/master 888cce5 Matt Corallo: Add perf counter data to GetStrongRandBytes state in scheduler
< bitcoin-git> bitcoin/master 399fb8f Matt Corallo: Add internal method to add new random data to our internal RNG state
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/e76a3927c3b0...15254e907e8c
< bitcoin-git> bitcoin/0.14 321419b Russell Yanofsky: Fix importwallet edge case rescan bug...
< bitcoin-git> [bitcoin] laanwj pushed 1 new commit to 0.14: https://github.com/bitcoin/bitcoin/commit/321419bc06fdc19e3713b2bcfc10c3c9bbbb8b62
< bitcoin-git> [bitcoin] laanwj closed pull request #10410: Fix importwallet edge case rescan bug (master...pr/scanimp) https://github.com/bitcoin/bitcoin/pull/10410
< bitcoin-git> bitcoin/master e76a392 Wladimir J. van der Laan: Merge #10410: Fix importwallet edge case rescan bug...
< bitcoin-git> bitcoin/master 2a8e35a Russell Yanofsky: Fix importwallet edge case rescan bug...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/e4775167cb4b...e76a3927c3b0
< luke-jr> jtimon: here is where I explain why BIP 149 is less safe (among other issues) vs BIP 148: https://www.reddit.com/r/Bitcoin/comments/69dm8e/what_is_segregated_witness_explanation_for/dh6dg3z/
< da2ce7> One of the critical security properties of bitcoin is a well-distributed set of miners. AsicBoost dramatically breaks this assumtion.
< wumpus> "I think there is a dramatic under estimation the damage of asicboost to Bitcoin" in what way? because they overrepresent themselves, resulting in more mining centralization than would otherwise be the case?
< da2ce7> wumpus, I agree: BIP148 may be, even then, too rushed (but I personally believe BIP148 is the safest option available, as I think there is a dramatic under estimation the damage of asicboost to Bitcoin). However, I haven't seen this considered in any of the significant anti-BIP148 responses to the mailing list. Centrally not an analysis of the tradeoffs available.
< da2ce7> Many developers have stated that they think that BIP148 is rushed, but remain silent on the fact that Bitcoin is under active exploit.
< da2ce7> Is the fact that Bitcoin is under active exploit by the asicboost vulnerably a important consideration for the urgency of BIP148?
< jonasschnelli> The protocol wiki "https://en.bitcoin.it/wiki/Protocol_documentation" says:

2017-05-22

< jtimon> BlueMatt: I can't remember which of your PRs contained something somilar to https://github.com/bitcoin/bitcoin/pull/10427 but it's basically what you wanted with my 2 smaller but more important for me nits
< bitcoin-git> [bitcoin] ryanofsky opened pull request #10443: Add fee_est tool for debugging fee estimation code (master...pr/fee) https://github.com/bitcoin/bitcoin/pull/10443
< bitcoin-git> [bitcoin] earonesty closed pull request #10428: -bip148 option (0.14...0.14) https://github.com/bitcoin/bitcoin/pull/10428
< bitcoin-git> [bitcoin] earonesty opened pull request #10442: add a -bip148 option (master...master) https://github.com/bitcoin/bitcoin/pull/10442
< bitcoin-git> [bitcoin] laanwj closed pull request #10390: [wallet] remove minimum total fee option (master...killminfee) https://github.com/bitcoin/bitcoin/pull/10390
< bitcoin-git> [bitcoin] laanwj closed pull request #10415: [tests] Speed up fuzzing by ~200x when using afl-fuzz (master...fast-afl-fuzzing) https://github.com/bitcoin/bitcoin/pull/10415
< bitcoin-git> bitcoin/master 217b416 Wladimir J. van der Laan: Merge #10415: [tests] Speed up fuzzing by ~200x when using afl-fuzz...
< bitcoin-git> bitcoin/master 693247b practicalswift: [test] Speed up fuzzing by ~200x when using afl-fuzz...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/651291383047...217b416c727a
< bitcoin-git> [bitcoin] practicalswift opened pull request #10440: [tests] Add libFuzzer support. (master...libfuzzer) https://github.com/bitcoin/bitcoin/pull/10440
< bitcoin-git> bitcoin/0.13 b654842 Alex Morcos: Populate services in GetLocalAddress...
< bitcoin-git> [bitcoin] laanwj pushed 1 new commit to 0.13: https://github.com/bitcoin/bitcoin/commit/b6548420291de0e077b02de71b4f2cea3dac0f8c
< bitcoin-git> bitcoin/0.14 37a8fc5 Alex Morcos: Populate services in GetLocalAddress...
< bitcoin-git> [bitcoin] laanwj pushed 1 new commit to 0.14: https://github.com/bitcoin/bitcoin/commit/37a8fc54d480788a06ff17415128d815ac55eaed
< bitcoin-git> [bitcoin] laanwj closed pull request #10424: Populate services in GetLocalAddress (master...notnodenone) https://github.com/bitcoin/bitcoin/pull/10424
< bitcoin-git> bitcoin/master 6512913 Wladimir J. van der Laan: Merge #10424: Populate services in GetLocalAddress...
< bitcoin-git> bitcoin/master 3070134 Alex Morcos: Populate services in GetLocalAddress...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/8e5725666b51...651291383047
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #10433: [tests] improve tmpdir structure (master...test_runner_tmpdir) https://github.com/bitcoin/bitcoin/pull/10433
< bitcoin-git> bitcoin/master 8e57256 MarcoFalke: Merge #10433: [tests] improve tmpdir structure...
< bitcoin-git> bitcoin/master b040243 John Newbery: [tests] improve tmpdir structure
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/a4ca0b042365...8e5725666b51
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #10431: Prevent shadowing the global dustRelayFee (master...20170519_policy_shadowing) https://github.com/bitcoin/bitcoin/pull/10431
< bitcoin-git> bitcoin/master 43c5877 Pavel Janík: Prevent shadowing the global dustRelayFee.
< bitcoin-git> bitcoin/master a4ca0b0 MarcoFalke: Merge #10431: Prevent shadowing the global dustRelayFee...
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/6b99dafe4204...a4ca0b042365

2017-05-21

< bitcoin-git> [bitcoin] earonesty reopened pull request #10428: -bip148 option (0.14...0.14) https://github.com/bitcoin/bitcoin/pull/10428
< bitcoin-git> [bitcoin] earonesty closed pull request #10428: -bip148 option (0.14...0.14) https://github.com/bitcoin/bitcoin/pull/10428
< bitcoin-git> [bitcoin] sanch0panza opened pull request #10437: Implement BIP135 (generalized versionbits) (master...bip135-core-dev-clean1) https://github.com/bitcoin/bitcoin/pull/10437
< luke-jr> https://github.com/bitcoin/bitcoin/compare/0.12…luke-jr:0.12.1+bip148 for a github diff..

2017-05-20

< murchandamus> gmaxwell, sipa, instagibbs: I've been able to put a few hours into this, this morning. This is my work in progress, where I'm at now: https://github.com/Xekyo/bitcoin/commit/9fa0bb5bc5c3db97972ff55403eed0ce992a8750
< gmaxwell> sipa: https://github.com/bitcoin/bitcoin/pull/10195/ tests failed with a no output timeout.
< bitcoin-git> [bitcoin] sipa opened pull request #10434: [WIP] 3072-bit MuHash based hash_serialized (master...muhash) https://github.com/bitcoin/bitcoin/pull/10434

2017-05-19

< gribble> https://github.com/bitcoin/bitcoin/issues/10426 | Replace bytes_serialized with bogosize by sipa · Pull Request #10426 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] jnewbery opened pull request #10433: [tests] improve tmpdir structure (master...test_runner_tmpdir) https://github.com/bitcoin/bitcoin/pull/10433
< bitcoin-git> [bitcoin] laanwj closed pull request #10429: tests: fix spurious addrman test failure (master...fix-addrman-test) https://github.com/bitcoin/bitcoin/pull/10429
< bitcoin-git> bitcoin/master 6b99daf Wladimir J. van der Laan: Merge #10429: tests: fix spurious addrman test failure...
< bitcoin-git> bitcoin/master 6b51b0a Cory Fields: tests: fix spurious addrman test failure...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/993a1d710e7e...6b99dafe4204
< bitcoin-git> [bitcoin] laanwj closed pull request #10432: [Trivial] Add BITCOIN_FS_H endif footer in fs.h (master...patch-1) https://github.com/bitcoin/bitcoin/pull/10432
< bitcoin-git> bitcoin/master 2300a5e Ahmad Kazi: [Trivial] Add BITCOIN_FS_H endif footer in fs.h
< bitcoin-git> bitcoin/master 993a1d7 Wladimir J. van der Laan: Merge #10432: [Trivial] Add BITCOIN_FS_H endif footer in fs.h...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/ea6fde3f1d26...993a1d710e7e
< bitcoin-git> [bitcoin] plaxton opened pull request #10432: [Trivial] Add BITCOIN_FS_H endif footer in fs.h (master...patch-1) https://github.com/bitcoin/bitcoin/pull/10432
< bitcoin-git> [bitcoin] paveljanik opened pull request #10431: Prevent shadowing the global dustRelayFee (master...20170519_policy_shadowing) https://github.com/bitcoin/bitcoin/pull/10431
< sipa> any idea what's going on here: https://travis-ci.org/bitcoin/bitcoin/jobs/233833539
< sipa> SopaXorzTaker: you've been told repeatedly to not make proposals about changing bitcoin here
< ajd_> maybe better to run the proposal past the people in #bitcoin first
< ajd_> - Generally discouraged: shower thoughts, wild speculation, jokes, +1s, non-technical bitcoin issues, rehashing settled topics without new data, moderation concerns.
< bitcoin-git> [bitcoin] theuni opened pull request #10429: tests: fix spurious addrman test failure (master...fix-addrman-test) https://github.com/bitcoin/bitcoin/pull/10429
< bitcoin-git> [bitcoin] earonesty opened pull request #10428: -bip148 option (0.14...0.14) https://github.com/bitcoin/bitcoin/pull/10428
< bitcoin-git> [bitcoin] jtimon opened pull request #10427: Consensus: Introduce static GetScriptFlags (mostly MOVEONLY) (master...b15-consensus-script-flags-min) https://github.com/bitcoin/bitcoin/pull/10427

2017-05-18

< bitcoin-git> [bitcoin] MarcoFalke closed pull request #10241: Allow tests to pass even when stderr got populated (master...2017/04/test_stderr) https://github.com/bitcoin/bitcoin/pull/10241
< bitcoin-git> [bitcoin] sipa opened pull request #10426: Replace bytes_serialized with bogosize (master...bogosize) https://github.com/bitcoin/bitcoin/pull/10426
< bitcoin-git> [bitcoin] earonesty opened pull request #10425: 0.14 (0.14...0.14) https://github.com/bitcoin/bitcoin/pull/10425
< jtimon> travis tests seem to be stuck for https://github.com/bitcoin/bitcoin/pull/9176
< bitcoin-git> [bitcoin] morcos opened pull request #10424: Populate services in GetLocalAddress (master...notnodenone) https://github.com/bitcoin/bitcoin/pull/10424
< bitcoin-git> [bitcoin] jnewbery opened pull request #10423: [tests] skipped tests should clean up after themselves (master...cleanup_skipped) https://github.com/bitcoin/bitcoin/pull/10423
< gribble> https://github.com/bitcoin/bitcoin/issues/10396 | Report LevelDB estimate for chainstate size in gettxoutsetinfo by sipa · Pull Request #10396 · bitcoin/bitcoin · GitHub
< sipa> short point: wrt the pruned-node-serving, see http://bitcoin.sipa.be/depths.png
< gribble> https://github.com/bitcoin/bitcoin/issues/9494 | Introduce an ArgsManager class encapsulating cs_args, mapArgs and mapMultiArgs by jtimon · Pull Request #9494 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10240 | Add HD wallet auto-restore functionality by jonasschnelli · Pull Request #10240 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10295 | [qt] Move some WalletModel functions into CWallet by ryanofsky · Pull Request #10295 · bitcoin/bitcoin · GitHub
< jtimon> there's not many explicit acks on https://github.com/bitcoin/bitcoin/pull/10339
< gribble> https://github.com/bitcoin/bitcoin/issues/10195 | Switch chainstate db and cache to per-txout model by sipa · Pull Request #10195 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10195 | Switch chainstate db and cache to per-txout model by sipa · Pull Request #10195 · bitcoin/bitcoin · GitHub
< jonasschnelli> BTW: kallewoof has an draft impl. on serving filters over the p2p (though bloom): https://github.com/kallewoof/bitcoin/pull/1/files (in case someone wants to drive this further)
< jonasschnelli> Filter for BDF, read gmaxwell's reply: https://lists.linuxfoundation.org/pipermail/bitcoin-dev/2016-May/012637.html
< bitcoin-git> bitcoin/master ea6fde3 Wladimir J. van der Laan: Merge #8329: Consensus: MOVEONLY: Move functions for tx verification...
< bitcoin-git> bitcoin/master 618d07f Jorge Timón: MOVEONLY: tx functions to consensus/tx_verify.o...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/28c6e8d71b3a...ea6fde3f1d26
< gmaxwell> #bitcoin-core-dev Meeting: wumpus sipa gmaxwell jonasschnelli morcos luke-jr btcdrak sdaftuar jtimon cfields petertodd kanzure bluematt instagibbs phantomcircuit codeshark michagogo marcofalke paveljanik NicolasDorier
< bitcoin-git> [bitcoin] instagibbs closed pull request #9102: Really don't validate genesis block (master...dontvalidategenesis) https://github.com/bitcoin/bitcoin/pull/9102
< bitcoin-git> [bitcoin] laanwj closed pull request #10408: Net: Improvements to Tor control port parser (master...torcontrol-parser-patches) https://github.com/bitcoin/bitcoin/pull/10408
< bitcoin-git> bitcoin/master d63677b Jack Grigg: torcontrol: Fix ParseTorReplyMapping...
< bitcoin-git> bitcoin/master d8e03c0 Jack Grigg: torcontrol: Improve comments
< bitcoin-git> bitcoin/master 29f3c20 Jack Grigg: torcontrol: Add unit tests for Tor reply parsers
< bitcoin-git> [bitcoin] laanwj pushed 7 new commits to master: https://github.com/bitcoin/bitcoin/compare/962cd3f0587e...28c6e8d71b3a
< bitcoin-git> [bitcoin] morcos opened pull request #10422: Fix timestamp in fee estimate debug message (master...fixtimeunits) https://github.com/bitcoin/bitcoin/pull/10422
< bitcoin-git> [bitcoin] practicalswift opened pull request #10421: [qt] Remove excess logic (master...if-expr-return-true-else-return-false) https://github.com/bitcoin/bitcoin/pull/10421
< bitcoin-git> [bitcoin] ryanofsky opened pull request #10420: Add Qt tests for wallet spends & bumpfee (master...pr/btest) https://github.com/bitcoin/bitcoin/pull/10420
< bitcoin-git> [bitcoin] fanquake closed pull request #9427: Use compact blocks for blocks that have equal work to our active tip (master...UseCmpctBlockForCompetingBlocks) https://github.com/bitcoin/bitcoin/pull/9427
< jonasschnelli> Added https://github.com/bitcoin/bitcoin/pull/10240 to the "High-priority for review" project
< bitcoin-git> [bitcoin] jonasschnelli closed pull request #9697: [Qt] simple fee bumper with user verification (master...2017/02/qt_bumpfee) https://github.com/bitcoin/bitcoin/pull/9697
< bitcoin-git> bitcoin/master 2ec911f Jonas Schnelli: Add cs_wallet lock assertion to SignTransaction()
< bitcoin-git> bitcoin/master 2678d3d Jonas Schnelli: Show old-fee, increase a new-fee in Qt fee bumper confirmation dialog
< bitcoin-git> bitcoin/master fbf385c Jonas Schnelli: [Qt] simple fee bumper with user verification
< bitcoin-git> [bitcoin] jonasschnelli pushed 8 new commits to master: https://github.com/bitcoin/bitcoin/compare/2acface32aba...962cd3f0587e
< bitcoin-git> [bitcoin] laanwj closed pull request #9750: Bloomfilter: parameter variables made constant (master...bloomVar) https://github.com/bitcoin/bitcoin/pull/9750
< bitcoin-git> bitcoin/master 2acface Wladimir J. van der Laan: Merge #9750: Bloomfilter: parameter variables made constant...
< bitcoin-git> bitcoin/master d60d54d ロハン ダル: merge with bitcoin core
< bitcoin-git> bitcoin/master 64aa36e ロハン ダル: param variables made const
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/ae786098bc58...2acface32aba
< bitcoin-git> [bitcoin] practicalswift opened pull request #10419: [trivial] Fix two recently introduced typos (master...typos-201705) https://github.com/bitcoin/bitcoin/pull/10419
< bitcoin-git> [bitcoin] laanwj closed pull request #10417: BIP 148 support (master...master-BIP148) https://github.com/bitcoin/bitcoin/pull/10417
< bitcoin-git> [bitcoin] theuni closed pull request #10285: net: refactor the connection process. moving towards async connections. (master...connman-events6) https://github.com/bitcoin/bitcoin/pull/10285
< bitcoin-git> [bitcoin] sipa closed pull request #8952: Add query options to listunspent RPC call (master...enhancement/improve-rpc-listunspent) https://github.com/bitcoin/bitcoin/pull/8952
< bitcoin-git> [bitcoin] sipa closed pull request #10061: [net] Added SetSocketNoDelay() utility function (master...tjps_nodelay) https://github.com/bitcoin/bitcoin/pull/10061
< bitcoin-git> bitcoin/master ad415bc Thomas Snider: [net] Added SetSocketNoDelay() utility function
< bitcoin-git> bitcoin/master ae78609 Pieter Wuille: Merge #10061: [net] Added SetSocketNoDelay() utility function...
< bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/c33652576ce2...ae786098bc58
< bitcoin-git> [bitcoin] sipa closed pull request #10395: Replace boost::function with std::function (C++11) (master...replace-boost-function) https://github.com/bitcoin/bitcoin/pull/10395
< bitcoin-git> bitcoin/master c336525 Pieter Wuille: Merge #10395: Replace boost::function with std::function (C++11)...
< bitcoin-git> bitcoin/master 1b936f5 practicalswift: Replace boost::function with std::function (C++11)
< bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/e317c0d19201...c33652576ce2
< bitcoin-git> bitcoin/master e317c0d Pieter Wuille: Merge #8384: Add witness data output to TxInError messages...
< bitcoin-git> bitcoin/master 6e9e026 Gregory Sanders: Expand signrawtransaction.py to cover error witness checking
< bitcoin-git> [bitcoin] sipa pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/bee35299716c...e317c0d19201
< bitcoin-git> bitcoin/master 9f7341b Gregory Sanders: Add witness data output to TxInError messages

2017-05-17

< Anduck> not only in bitcoin scene, that is
< lichtamberg_> And you are also removing the possibility to switch to the correct chain, if BIP148 really gets much traction at the end. With the toggle option, people can switch more or less easily.. without => you are forcing the users to unofficial versions.. most people dont compile bitcoin themself
< lichtamberg_> On one side you are asking for more community participation, and on the other side you are dismissing one of the biggest community movements which bitcoin ever had..
< Anduck> anyway, even while bip148 may be stupidly risky, there's no point in trying to slow it down. it happens if it happens. running bitcoin node will increasingly require more knowledge of what actual rules does the code follow. there's no point in trying to oppose this from realising
< Anduck> bgp is not comparable to bitcoin because there's no competitor. nobody gains anything if they get bgp replaced by something else
< BlueMatt> bitcoin even is
< BlueMatt> i assume you can picture the "how to install bitcoin core" guides where someone tells you to use that flag and then they'll tell you you've been paid
< BlueMatt> sdf_0010: maybe a better comparison is a flag to run bitcoin core on testnet but have everything pretend to be mainnet
< BlueMatt> if they're, otoh, expecting "Core" to decide consensus rules for the bitcoin ecosystem, then they can also clearly fuck off
< BlueMatt> well then they're expecting us to do the job of figuring out what the community's consensus on bitcoin's consensus rules is, and it is clearly not bip 148
< BlueMatt> on top of people doing physical outreach and talking to various groups at meetups, plus discussions with long-time bitcoin holders and investors
< BlueMatt> it is not bitcoin core's job to merge something *until* that exists, irrespective of whether individual contributors are pushing for it
< BlueMatt> businesses are also users of the system, especially the many that do not expose bitcoin to their users, and there is clearly not vast, broad support for bip 148 in that community even *if* core merges it
< BlueMatt> luke-jr: this is absurd, do you seriously believe the vast, vast majority of bitcoin users (incl businesses) support bip 148?
< gribble> https://github.com/bitcoin/bitcoin/issues/10417 | BIP 148 support by achow101 · Pull Request #10417 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] sipa closed pull request #10196: Bugfix: PrioritiseTransaction updates the mempool tx counter (master...2017-04-prioritise-transaction) https://github.com/bitcoin/bitcoin/pull/10196
< bitcoin-git> bitcoin/master bee3529 Pieter Wuille: Merge #10196: Bugfix: PrioritiseTransaction updates the mempool tx counter...
< bitcoin-git> bitcoin/master 6c2e25c Suhas Daftuar: [qa] Test prioritise_transaction / getblocktemplate interaction
< bitcoin-git> bitcoin/master acc2e4b Suhas Daftuar: Bugfix: PrioritiseTransaction updates the mempool tx counter...
< bitcoin-git> [bitcoin] sipa pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/318ea50a1c2f...bee35299716c
< bitcoin-git> [bitcoin] sipa closed pull request #10199: Better fee estimates (master...smarterfee) https://github.com/bitcoin/bitcoin/pull/10199
< bitcoin-git> bitcoin/master d3e30bc Alex Morcos: Refactor to update moving average on fly
< bitcoin-git> bitcoin/master e5007ba Alex Morcos: Change parameters for fee estimation and estimates on all 3 time horizons....
< bitcoin-git> bitcoin/master c0a273f Alex Morcos: Change file format for fee estimates....
< bitcoin-git> [bitcoin] sipa pushed 16 new commits to master: https://github.com/bitcoin/bitcoin/compare/e61fc2d7cb4f...318ea50a1c2f
< bitcoin-git> [bitcoin] laanwj pushed 1 new commit to 0.13: https://github.com/bitcoin/bitcoin/commit/8adf75e6a124267da1ae46be1eee50c52ce6082b
< bitcoin-git> bitcoin/0.13 8adf75e fanquake: [depends] miniupnpc 2.0.20170509...
< bitcoin-git> [bitcoin] laanwj closed pull request #10414: [depends] miniupnpc 2.0.20170509 (master...depends-miniupnpc-2-0-20170509) https://github.com/bitcoin/bitcoin/pull/10414
< bitcoin-git> bitcoin/master e61fc2d Wladimir J. van der Laan: Merge #10414: [depends] miniupnpc 2.0.20170509...
< bitcoin-git> bitcoin/master af5d48c fanquake: [depends] miniupnpc 2.0.20170509
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/ea1fd43bb9a3...e61fc2d7cb4f
< bitcoin-git> [bitcoin] achow101 opened pull request #10417: BIP 148 support (master...master-BIP148) https://github.com/bitcoin/bitcoin/pull/10417
< bitcoin-git> [bitcoin] laanwj closed pull request #10257: [test] Add test for getmemoryinfo (master...test_getmemoryinfo) https://github.com/bitcoin/bitcoin/pull/10257
< bitcoin-git> bitcoin/master d4668f3 Jimmy Song: [test] Add test for getmemoryinfo...
< bitcoin-git> bitcoin/master ea1fd43 Wladimir J. van der Laan: Merge #10257: [test] Add test for getmemoryinfo...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/526e8390e6be...ea1fd43bb9a3
< cypherbit> join bitcoin
< SopaXorzTaker> banned from #bitcoin for spreading bullsh*t
< wumpus> again, #bitcoin, this is not the place to discuss the protocol in general
< wumpus> please, #bitcoin
< bitcoin-git> [bitcoin] practicalswift opened pull request #10415: [fuzz] Speed up fuzzing by ~200x when using afl-fuzz (master...fast-afl-fuzzing) https://github.com/bitcoin/bitcoin/pull/10415
< bitcoin-git> [bitcoin] fanquake opened pull request #10414: [depends] miniupnpc 2.0.20170509 (master...depends-miniupnpc-2-0-20170509) https://github.com/bitcoin/bitcoin/pull/10414
< bitcoin-git> [bitcoin] fanquake closed pull request #9325: Allow first cmpctblock/blocktxn received to be processed (rather than first requested) (master...ProcessFirstCmpct) https://github.com/bitcoin/bitcoin/pull/9325
< bitcoin-git> [bitcoin] laanwj closed pull request #10388: Output line to debug.log when IsInitialBlockDownload latches to false (master...printIBD) https://github.com/bitcoin/bitcoin/pull/10388
< bitcoin-git> bitcoin/master 526e839 Wladimir J. van der Laan: Merge #10388: Output line to debug.log when IsInitialBlockDownload latches to false...
< bitcoin-git> bitcoin/master 65d484a Alex Morcos: Output line to debug.log when IsInitialBlockDownload latches to false
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/32f671b14107...526e8390e6be
< bitcoin-git> [bitcoin] laanwj closed pull request #10319: Remove unused argument from MarkBlockAsInFlight(...) (master...remove-unused-argument) https://github.com/bitcoin/bitcoin/pull/10319
< bitcoin-git> bitcoin/master 32f671b Wladimir J. van der Laan: Merge #10319: Remove unused argument from MarkBlockAsInFlight(...)...
< bitcoin-git> bitcoin/master 6345f0b practicalswift: Remove unused argument from MarkBlockAsInFlight(...)
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/d25449f85862...32f671b14107
< gribble> https://github.com/bitcoin/bitcoin/issues/10388 | Output line to debug.log when IsInitialBlockDownload latches to false by morcos · Pull Request #10388 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10319 | Remove unused argument from MarkBlockAsInFlight(...) by practicalswift · Pull Request #10319 · bitcoin/bitcoin · GitHub
< bitcoin-git> bitcoin/master ed1fcdc Luke Dashjr: Bugfix: Detect genbuild.sh in repo correctly
< bitcoin-git> bitcoin/master df63490 Luke Dashjr: Merge tag 'branch-0.13' into bugfix_gitdir
< bitcoin-git> bitcoin/master e98e3dd Luke Dashjr: Bugfix: Only use git for build info if the repository is actually the right one...
< bitcoin-git> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/0542978aae12...d25449f85862
< gribble> https://github.com/bitcoin/bitcoin/issues/7522 | Bugfix: Only use git for build info if the repository is actually the right one by luke-jr · Pull Request #7522 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] fanquake closed pull request #9091: Optimize sending of getheaders when pindexLast is an ancestor of pindexBestHeader (master...OptimizeMoreGetheaders) https://github.com/bitcoin/bitcoin/pull/9091
< bitcoin-git> [bitcoin] laanwj closed pull request #10405: tests: Correct testcase in script_tests.json for large number OP_EQUAL (master...2017_05_scriptnum_test) https://github.com/bitcoin/bitcoin/pull/10405
< bitcoin-git> bitcoin/master 2f84cf6 Wladimir J. van der Laan: tests: Correct testcase in script_tests.json for large number OP_EQUAL...
< bitcoin-git> bitcoin/master 0542978 Wladimir J. van der Laan: Merge #10405: tests: Correct testcase in script_tests.json for large number OP_EQUAL...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/08ac35a7e3ef...0542978aae12
< bitcoin-git> [bitcoin] fanquake closed pull request #8958: Improve logic for advertising blocks (master...BetterBroadcastLogic) https://github.com/bitcoin/bitcoin/pull/8958
< bitcoin-git> [bitcoin] fanquake closed pull request #9704: Store downloaded blocks prior to shutdown. (master...StoreBlocksAtShutdown) https://github.com/bitcoin/bitcoin/pull/9704
< bitcoin-git> bitcoin/master 08ac35a Wladimir J. van der Laan: Merge #10413: Fix docs (there's no rpc command setpaytxfee)...
< bitcoin-git> bitcoin/master 0f1b26a Ryan Havar: Fix docs (there's no rpc command setpaytxfee)
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/9390845c53e3...08ac35a7e3ef
< bitcoin-git> [bitcoin] fanquake closed pull request #9403: Don't ask for TX relay from feeler connections (master...NoRelayForFeelers) https://github.com/bitcoin/bitcoin/pull/9403
< bitcoin-git> bitcoin/master 9390845 Wladimir J. van der Laan: Merge #8952: Add query options to listunspent RPC call...
< bitcoin-git> bitcoin/master bc63d0e Pedro Branco: Add query options to listunspent rpc call
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/541199788c10...9390845c53e3
< bitcoin-git> [bitcoin] laanwj closed pull request #10374: qa: Warn when specified test is not found (master...Mf1705-qaWarnTestNotFound) https://github.com/bitcoin/bitcoin/pull/10374
< bitcoin-git> bitcoin/master 5411997 Wladimir J. van der Laan: Merge #10374: qa: Warn when specified test is not found...
< bitcoin-git> bitcoin/master fac79e4 MarcoFalke: qa: Warn when specified test is not found
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b45a52aeff0a...541199788c10
< bitcoin-git> [bitcoin] laanwj closed pull request #10404: doc: Add logging to FinalizeNode() (master...2017-05-log-finalize-node) https://github.com/bitcoin/bitcoin/pull/10404
< bitcoin-git> bitcoin/master b45a52a Wladimir J. van der Laan: Merge #10404: doc: Add logging to FinalizeNode()...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/95546c859b71...b45a52aeff0a
< bitcoin-git> bitcoin/master 1530bfc Suhas Daftuar: Add logging to FinalizeNode()
< gribble> https://github.com/bitcoin/bitcoin/issues/177650 | HTTP Error 404: Not Found
< gmaxwell> 2017-05-17 05:43:35 /home/gmaxwell/.bitcoin/blocks/index/177650.log: dropping 30275 bytes; Corruption: checksum mismatch
< bitcoin-git> [bitcoin] RHavar opened pull request #10413: Fix docs (there's no rpc command setpaytxfee) (master...patch-2) https://github.com/bitcoin/bitcoin/pull/10413

2017-05-16

< bitcoin-git> [bitcoin] ryanofsky opened pull request #10412: Improve wallet rescan API (master...pr/scanclean) https://github.com/bitcoin/bitcoin/pull/10412
< Chris_Stewart_5> If I have a transaction that spends a raw witness spk and the input is already signed on the spending tx, and then I'm trying to call fundrawtransaction to add a fee. Is this currently unsupported in bitcoin core? Do I need to use P2SH(P2WSH)?
< bitcoin-git> [bitcoin] ryanofsky opened pull request #10410: Fix importwallet edge case rescan bug (master...pr/scanimp) https://github.com/bitcoin/bitcoin/pull/10410
< bitcoin-git> [bitcoin] practicalswift opened pull request #10409: [tests] Add fuzz testing for BlockTransactions (BLOCKTXN) and BlockTransactionsRequest (GETBLOCKTXN) deserialization (master...fuzz-blocktransactions) https://github.com/bitcoin/bitcoin/pull/10409
< jonasschnelli> Can someone working on the net-code give a final ACK/NACK for https://github.com/bitcoin/bitcoin/pull/9502? sipa, cfields?
< bitcoin-git> [bitcoin] jonasschnelli closed pull request #10362: [GUI] Add OSX keystroke to RPCConsole info (master...RPC-OSX-Keystroke) https://github.com/bitcoin/bitcoin/pull/10362
< bitcoin-git> bitcoin/master 95546c8 Jonas Schnelli: Merge #10362: [GUI] Add OSX keystroke to RPCConsole info...
< bitcoin-git> bitcoin/master 012fa9b Spencer Lievens: Add OSX keystroke to clear RPCConsole...
< bitcoin-git> [bitcoin] jonasschnelli pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/d0c37ee78984...95546c859b71
< bitcoin-git> [bitcoin] str4d opened pull request #10408: Net: Improvements to Tor control port parser (master...torcontrol-parser-patches) https://github.com/bitcoin/bitcoin/pull/10408
< bitcoin-git> [bitcoin] laanwj closed pull request #10383: [logging] log system time and mock time (master...log_mocktime) https://github.com/bitcoin/bitcoin/pull/10383
< bitcoin-git> bitcoin/master d0c37ee Wladimir J. van der Laan: Merge #10383: [logging] log system time and mock time...
< bitcoin-git> bitcoin/master 761392d John Newbery: [logging] log system time and mock time
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b6ee855b411e...d0c37ee78984

2017-05-15

< bitcoin-git> [bitcoin] fanquake closed pull request #10407: Trivial: fixed spelling, "Bitcion" -> "Bitcoin" (master...metacoin-fix-translation-spelling) https://github.com/bitcoin/bitcoin/pull/10407
< bitcoin-git> [bitcoin] sipa closed pull request #10380: [doc] Removing comments about dirty entries on txmempool (master...master) https://github.com/bitcoin/bitcoin/pull/10380
< bitcoin-git> bitcoin/master b6ee855 Pieter Wuille: Merge #10380: [doc] Removing comments about dirty entries on txmempool...
< bitcoin-git> bitcoin/master 4f3782e Simone Madeo: [doc] Removing comments about dirty entries on txmempool
< bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/8bd16ee12fc8...b6ee855b411e
< bitcoin-git> [bitcoin] metacoin opened pull request #10407: Trivial: fixed spelling, "Bitcion" -> "Bitcoin" (master...metacoin-fix-translation-spelling) https://github.com/bitcoin/bitcoin/pull/10407
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #10376: [tests] fix disconnect_ban intermittency (master...disconnect_ban_flakiness) https://github.com/bitcoin/bitcoin/pull/10376
< bitcoin-git> bitcoin/master 8bd16ee MarcoFalke: Merge #10376: [tests] fix disconnect_ban intermittency...
< bitcoin-git> bitcoin/master 3ba2c08 John Newbery: [tests] fix disconnect_ban intermittency
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/96c850c20913...8bd16ee12fc8
< windsok> #bitcoin-wizards
< sipa> this channel is about implementation and development of the bitcoin core software
< dermoth> #bitcoin-dev would be more appropriate?
< sipa> well this channel is about the developement of bitcoin core
< dermoth> sipa, yes possibly OT - hoe many bitcoin "dev" channels there is?? :)
< gmaxwell> sipa: well you can filter the inputs that were spent, there was a point on bitcoin-dev about this a year ago.
< sipa> perhaps this is more for #bitcoin-wizards then
< dermoth> Hey there... I was thinking about the issue of block size vs. propagation time... If I'm not mistaken Bitcoin already sends header only first so miners can start mining quickly on tip, but they cannot validate/include any transaction until they get the full block...
< bitcoin-git> [bitcoin] laanwj closed pull request #8704: [RPC] Transaction details in getblock (master...getblock-extraverbose) https://github.com/bitcoin/bitcoin/pull/8704
< bitcoin-git> bitcoin/master 96c850c Wladimir J. van der Laan: Merge #8704: [RPC] Transaction details in getblock...
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/41987aa92f0d...96c850c20913
< bitcoin-git> bitcoin/master c99ab3c Luke Dashjr: RPC: Allow multiple names for parameters
< bitcoin-git> bitcoin/master e3c9f2d Andrew Chow: Use a verbosity instead of two verbose parameters...
< bitcoin-git> [bitcoin] laanwj opened pull request #10405: tests: Correct testcase in script_tests.json for large number OP_EQUAL (master...2017_05_scriptnum_test) https://github.com/bitcoin/bitcoin/pull/10405
< bitcoin-git> [bitcoin] sdaftuar opened pull request #10404: Add logging to FinalizeNode() (master...2017-05-log-finalize-node) https://github.com/bitcoin/bitcoin/pull/10404
< gribble> https://github.com/bitcoin/bitcoin/issues/10359 | [tests] functional tests should call BitcoinTestFramework start/stop node methods by jnewbery · Pull Request #10359 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] ryanofsky opened pull request #10403: Fix importmulti failure to return rescan errors (master...pr/scansame) https://github.com/bitcoin/bitcoin/pull/10403
< bitcoin-git> [bitcoin] laanwj closed pull request #9494: Introduce an ArgsManager class encapsulating cs_args, mapArgs and mapMultiArgs (master...0.13-args-class) https://github.com/bitcoin/bitcoin/pull/9494
< bitcoin-git> bitcoin/master 5292245 Jorge Timón: Util: Put mapMultiArgs inside ArgsManager...
< bitcoin-git> bitcoin/master b3cbd55 Jorge Timón: scripted-diff: Util: Encapsulate mapMultiArgs behind gArgs...
< bitcoin-git> bitcoin/master f2957ce Jorge Timón: Util: Create ArgsManager class......
< bitcoin-git> [bitcoin] laanwj pushed 5 new commits to master: https://github.com/bitcoin/bitcoin/compare/87abe20fc118...41987aa92f0d

2017-05-14

< bitcoin-git> [bitcoin] rvelhote opened pull request #10400: [RPC] Add an uptime command that displays the amount of time (in seconds) bitcoind has been running (master...rpc-uptime) https://github.com/bitcoin/bitcoin/pull/10400
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #10253: [test] Add test for getnetworkhashps (master...test_getnetworkhashps) https://github.com/bitcoin/bitcoin/pull/10253
< bitcoin-git> bitcoin/master 87abe20 MarcoFalke: Merge #10253: [test] Add test for getnetworkhashps...
< bitcoin-git> bitcoin/master de487b7 Jimmy Song: Tests: Add test for getnetworkhashps
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/94e52273f30f...87abe20fc118
< bitcoin-git> [bitcoin] sipa opened pull request #10396: Report LevelDB estimate for chainstate size in gettxoutsetinfo (master...diskdbsize) https://github.com/bitcoin/bitcoin/pull/10396

2017-05-13

< sipa> if that seems like an arbitrary distinction, please realize that many people here are not interested in the political drama surrounding bitcoin and just want to develop software
< sipa> this channel is not about discussing proposed changes to the bitcoin consensus rules
< bitcoin-git> [bitcoin] practicalswift opened pull request #10395: Replace boost::function with std::function (C++11) (master...replace-boost-function) https://github.com/bitcoin/bitcoin/pull/10395
< bitcoin-git> [bitcoin] practicalswift closed pull request #10378: Rename TxConfirmStats to CTxConfirmStats to achieve class naming consistency (master...TxConfirmStats) https://github.com/bitcoin/bitcoin/pull/10378

2017-05-12

< sipa> ok, so in the rpc signing code and in bitcoin-tx, there used to be a Modifier that was both used check whether an output already existed, and then add it
< kanzure> i added a comment here explaining that perhaps a better design would be to let user specify the final transaction size (if the user knows it) or allowing feeRate=0 if the user plans to delete an output later https://github.com/bitcoin/bitcoin/pull/10202#issuecomment-301134038
< Chris_Stewart_5> No, it returns a bool indicating if it is fully signed: https://bitcoin.org/en/developer-reference#signrawtransaction
< instagibbs> possibly related: https://github.com/bitcoin/bitcoin/pull/10202
< kanzure> this claims to override minimum estimated fee but i get error "Transaction too large for fee policy" with feeRate=0 https://github.com/bitcoin/bitcoin/pull/7967
< kanzure> this seems related https://github.com/bitcoin/bitcoin/pull/9965
< cchadwicka> i am banned from bitcoin