2015-11-16

< GitHub193> [bitcoin] laanwj pushed 2 new commits to 0.11: https://github.com/bitcoin/bitcoin/compare/7e278929df53...595c8d6301cf
< GitHub34> [bitcoin] laanwj closed pull request #7021: add bip65 tests to rpc-tests.sh -extended (in 0.11 branch) (0.11...11rpcfixups) https://github.com/bitcoin/bitcoin/pull/7021
< GitHub101> [bitcoin] gmaxwell closed pull request #6991: Minor: Clarify 'fee' field in fundrawtransaction help text (master...clarify-fundrawtransaction-help) https://github.com/bitcoin/bitcoin/pull/6991
< GitHub194> bitcoin/master 9bd3f03 Peter Todd: Clarify 'fee' field in fundrawtransaction help text...
< GitHub194> bitcoin/master 6876a78 Gregory Maxwell: Merge pull request #6991...
< GitHub194> [bitcoin] gmaxwell pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/814697c5569c...6876a78b863e
< jonasschnelli> and this matched that: https://github.com/bitcoin/bitcoin/pull/7004/files
< jonasschnelli> master needs to have https://github.com/bitcoin/bitcoin/pull/7004
< GitHub19> [bitcoin] jonasschnelli closed pull request #7000: [Qt] add shortcurts for debug-/console-window (master...2015/11/qt_shortcuts) https://github.com/bitcoin/bitcoin/pull/7000
< GitHub184> bitcoin/master 773ae46 Jonas Schnelli: [Qt] add shortcurts for debug-/console-window
< GitHub184> bitcoin/master 814697c Jonas Schnelli: Merge pull request #7000...
< GitHub184> [bitcoin] jonasschnelli pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b632145edeb3...814697c5569c
< gmaxwell> for me bitcoin core sync runs at around 8000tx/s (with secp256k1), even though every tx is involving a non-cachable missing record check for bip30 (in addition to all the cached stuff)... thats really quite remarkable IMO.
< GitHub18> [bitcoin] CodeShark opened pull request #7023: Fixed integer comparison warning. (master...signed_int_comparison_fix) https://github.com/bitcoin/bitcoin/pull/7023

2015-11-15

< GitHub84> [bitcoin] morcos opened pull request #7022: Change default block priority size to 0 (master...defaultPrioritySize) https://github.com/bitcoin/bitcoin/pull/7022
< GitHub64> [bitcoin] morcos opened pull request #7021: add bip65 tests to rpc-tests.sh -extended (in 0.11 branch) (0.11...11rpcfixups) https://github.com/bitcoin/bitcoin/pull/7021
< Luke-Jr> FWIW, I figured out why the CLTV tests failed on 0.11.2: https://github.com/bitcoin/bitcoin/pull/6523#issuecomment-156782660
< gmaxwell> FWIW, I've unsubscribed from bitcoin-dev mailing list.
< gmaxwell> yes, not a risk for us but we'll probably get asked to update when people notice bitcoin-qt links it.
< GitHub184> [bitcoin] morcos opened pull request #7020: Implement helper class for CTxMemPoolEntry constructor (master...EntryHelper) https://github.com/bitcoin/bitcoin/pull/7020
< gmaxwell> Well it's also a bit #bitcoin-core-dev too, in that I think it would be useful if work were done in the GUI to make mining fun, ... but probably more than that is just speculation :)
< gmaxwell> also it suggests a framework for setting minimum feerates which are independant of bitcoin's price-- though dependant on communications efficiency, which is perhaps no better. :)

2015-11-14

< GitHub136> [bitcoin] MarcoFalke opened pull request #7019: [trivial] travis: cover *receivedby* rpcs (master...MarcoFalke-2015-receivedby) https://github.com/bitcoin/bitcoin/pull/7019
< GitHub50> [bitcoin] morcos closed pull request #6292: Rename and comment priority calculation in TxMemPoolEntry (master...PriorityComment) https://github.com/bitcoin/bitcoin/pull/6292
< GitHub101> [bitcoin] gmaxwell opened pull request #7016: Avoid a compile error on hosts with libevent too old for EVENT_LOG_WARN. (master...without_EVENT_LOG_WARN) https://github.com/bitcoin/bitcoin/pull/7016
< gmaxwell> morcos: the relay network has its own protocol, and a local proxy you run that speaks bitcoin p2p.
< GitHub168> [bitcoin] gmaxwell closed pull request #6993: Add -blocksonly option (master...blocksonly) https://github.com/bitcoin/bitcoin/pull/6993
< GitHub86> bitcoin/master 3a96497 Patick Strateman: Add whitelistalwaysrelay option
< GitHub86> bitcoin/master 420fa81 Patick Strateman: Do not process tx inv's in blocksonly mode
< GitHub86> bitcoin/master 4044f07 Patick Strateman: Add blocksonly mode
< GitHub86> [bitcoin] gmaxwell pushed 9 new commits to master: https://github.com/bitcoin/bitcoin/compare/9ffc687288dd...b632145edeb3
< tulip> morcos: there's also stratum proxies like bfgminer and stratum-mining-proxy which present work to downstream miners, based on either an upstream work server or a Bitcoin node. the latter was meant for use with early hardware miners which supported getwork but not stratum.
< gmaxwell> (Forrestv seems to have flamed out due to a mixture of bitcoin drama, and then donations to support p2pool being relatively low (compared to life changing amounts of income centeralized pool operators were getting), and then he got ripped off ... twice, I think, by mining hardware makers.)
< gmaxwell> Also through this time the developer burned out on Bitcoin, and only does life support maintaince on p2pool now.
< gmaxwell> P2pool is nice idea with cute features, but has always struggled with the high startup cost of having to run a bitcoin node with it; and then high latency asics showed up (in particular, some of the early BFL products had 10%+ hashrate loss from 10 second retasking)
< gmaxwell> morcos: well it's armored by bitcoin nodes in and out; not that there haven't been problems.
< gmaxwell> Yes. It is. I tried to do it for bitcoin core eons ago, but the utility library was GPLed... they since changed the licensing.
< gmaxwell> sipa: someone was seeking feedback from you here: https://github.com/bitcoin/bitcoin/pull/6844
< GitHub18> bitcoin/master e495ed5 Jonas Schnelli: add documentation for exluding whitelistes peer from maxuploadtarget
< GitHub44> [bitcoin] gmaxwell closed pull request #6984: don't enforce maxuploadtarget's disconnect for whitelisted peers (master...2015/11/maxupload_whitebind) https://github.com/bitcoin/bitcoin/pull/6984
< GitHub18> bitcoin/master 5760749 Jonas Schnelli: [docs] rename reducetraffic.md to reduce-traffic.md
< GitHub18> bitcoin/master d61fcff Jonas Schnelli: don't enforce maxuploadtargets disconnect for whitelisted peers
< GitHub18> [bitcoin] gmaxwell pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/44ac42e50d56...9ffc687288dd
< GitHub79> [bitcoin] laanwj closed pull request #7004: update jonasschnellis gpg key (master...2015/11/signing_key_jonasschnelli) https://github.com/bitcoin/bitcoin/pull/7004
< GitHub151> bitcoin/master 44ac42e Wladimir J. van der Laan: Merge pull request #7004...
< GitHub151> [bitcoin] laanwj pushed 1 new commit to master: https://github.com/bitcoin/bitcoin/commit/44ac42e50d567b08d3cb3f3c5766588468ce5bbf
< GitHub81> [bitcoin] laanwj closed pull request #7013: Remove LOCK(cs_main) from decodescript (master...2015-11-remove-cs-main-from-decodescript) https://github.com/bitcoin/bitcoin/pull/7013
< GitHub73> bitcoin/master 36baa9f Wladimir J. van der Laan: Merge pull request #7013...
< GitHub73> bitcoin/master b3ae384 Peter Todd: Remove LOCK(cs_main) from decodescript...
< GitHub73> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/e0a5ef84272b...36baa9f47587
< wumpus> sure that's why I dont give "integrate it into bitcoin core" as an alternative, but say communicate through a pipe or socket
< gmaxwell> But the biggest reason for the seperate process is to tear down development barriers. E.g. so someone can build a module without the scarryness in bitcoin core. :)
< GitHub67> [bitcoin] gmaxwell closed pull request #7010: Fix fundrawtransaction handling of includeWatching (master...2015-11-fix-fundrawtransaction-bugs) https://github.com/bitcoin/bitcoin/pull/7010
< GitHub147> bitcoin/master e0a5ef8 Gregory Maxwell: Merge pull request #7010...
< GitHub147> bitcoin/master 10953a7 Peter Todd: Better error message for fundrawtransaction w/ empty vout...
< GitHub147> bitcoin/master 61e1eb2 Peter Todd: Actually use includeWatching value in fundrawtransaction...
< GitHub147> [bitcoin] gmaxwell pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/dbd2c135ddb9...e0a5ef84272b
< gmaxwell> wumpus: jonasschnelli: Care to give me a quick design sniff test? I'm thinking of having someone implement functionality for external signers in Bitcoin core along these lines, http://0bin.net/paste/7HCkLWNJBDqJB7Up#hPgdwfs06m7FJw-WQZUCBNTNfCZnrix2K0Mo++6jpbH

2015-11-13

< GitHub74> [bitcoin] petertodd opened pull request #7013: Remove LOCK(cs_main) from decodescript (master...2015-11-remove-cs-main-from-decodescript) https://github.com/bitcoin/bitcoin/pull/7013
< GitHub38> [bitcoin] petertodd opened pull request #7011: Add mediantime to getblockchaininfo (master...2015-11-add-mediantime-to-getblockchaininfo) https://github.com/bitcoin/bitcoin/pull/7011
< GitHub157> [bitcoin] petertodd opened pull request #7010: Fix fundrawtransaction handling of includeWatching (master...2015-11-fix-fundrawtransaction-bugs) https://github.com/bitcoin/bitcoin/pull/7010
< GitHub117> [bitcoin] gmaxwell closed pull request #6990: http: speed up shutdown (master...2015_11_threadexit) https://github.com/bitcoin/bitcoin/pull/6990
< GitHub138> bitcoin/master dbd2c13 Gregory Maxwell: Merge pull request #6990...
< GitHub138> [bitcoin] gmaxwell pushed 1 new commit to master: https://github.com/bitcoin/bitcoin/commit/dbd2c135ddb96bdc3a4e870c2371cb1fac227135
< Luke-Jr> sdaftuar: I don't have any env vars set related to Bitcoin
< GitHub155> [bitcoin] laanwj closed pull request #7007: Fix bug in mempool_tests unit test (master...unitTestBugFix) https://github.com/bitcoin/bitcoin/pull/7007
< GitHub165> bitcoin/master d356560 Wladimir J. van der Laan: Merge pull request #7007...
< GitHub165> bitcoin/master a78e6ea Alex Morcos: Fix bug in mempool_tests unit test
< GitHub165> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/4f09b77c7fa5...d3565604e3d9
< wumpus> jonasschnelli: you can set the 'GUI' label yourself now :) say on https://github.com/bitcoin/bitcoin/pull/7006
< GitHub121> [bitcoin] morcos opened pull request #7008: Lower bound priority (master...lowerBoundPriority) https://github.com/bitcoin/bitcoin/pull/7008
< GitHub75> [bitcoin] morcos opened pull request #7007: Fix bug in unit mempool_tests unit test (master...unitTestBugFix) https://github.com/bitcoin/bitcoin/pull/7007
< GitHub8> [bitcoin] sipa closed pull request #6983: Update libsecp256k1 (master...secp256k1new) https://github.com/bitcoin/bitcoin/pull/6983
< GitHub104> bitcoin/master 9e475d5 Pieter Wuille: Update libsecp256k1
< GitHub104> bitcoin/master 48edf57 Pieter Wuille: Update key.cpp to new secp256k1 API
< GitHub104> bitcoin/master 1d84107 Pieter Wuille: Squashed 'src/secp256k1/' changes from 22f60a6..2bfb82b...
< GitHub104> [bitcoin] sipa pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/d2e987aa1929...4f09b77c7fa5
< GitHub21> [bitcoin] jonasschnelli opened pull request #7006: [Qt] add startup option to reset Qt settings (master...2015/11/qt_resetsettings) https://github.com/bitcoin/bitcoin/pull/7006
< GitHub147> [bitcoin] jgarzik opened pull request #7005: Add MAINTAINERS file, a user guide to code subsystems (master...2015_maintainer) https://github.com/bitcoin/bitcoin/pull/7005
< jonasschnelli> [12:41:14] <Luke-Jr>jonasschnelli: why a new key? :/ <-- the new key is not really new, It's just the one that I use for bitcoin-dev posts, etc. The "old" key is used for non-bitcoin software-projects (and sadly, i have used it for gitian signatures)
< michagogo> Are you opped in #bitcoin?
< GitHub8> [bitcoin] laanwj closed pull request #6958: [trivial] Cleanup maxuploadtarget (doc & log) (master...MarcoFalke-2015-maxupload) https://github.com/bitcoin/bitcoin/pull/6958
< GitHub22> bitcoin/master d2e987a Wladimir J. van der Laan: Merge pull request #6958...
< GitHub22> bitcoin/master 9c3ee3b MarcoFalke: [doc] Add -maxuploadtarget release notes
< GitHub22> bitcoin/master b27e81f MarcoFalke: [net] Cleanup maxuploadtarget...
< GitHub22> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/24c4841d1686...d2e987aa1929
< wumpus> 0.11.2 binaries are live: https://bitcoin.org/bin/bitcoin-core-0.11.2/
< michagogo> bitcoin-osx-unsigned.tar.gz
< Luke-Jr> the file gitian generates is bitcoin-0.11.2-osx-unsigned.tar.gz
< GitHub106> [bitcoin] jonasschnelli opened pull request #7004: update jonasschnellis gpg key (master...2015/11/signing_key_jonasschnelli) https://github.com/bitcoin/bitcoin/pull/7004
< GitHub28> [bitcoin] laanwj opened pull request #7003: doc: Add non-style-related development guidelines (master...2015_11_development_guidelines) https://github.com/bitcoin/bitcoin/pull/7003
< GitHub86> bitcoin/master 24c4841 Wladimir J. van der Laan: Merge pull request #6998...
< GitHub153> [bitcoin] laanwj closed pull request #6998: doc: Remove mention of pulltester from README.md (master...2015_11_readme_pull_pulltester) https://github.com/bitcoin/bitcoin/pull/6998
< GitHub86> bitcoin/master 01afa80 Wladimir J. van der Laan: doc: Remove mention of pulltester from README.md...
< GitHub86> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/38ed190eefcc...24c4841d1686
< gmaxwell> So sipa updated this PR https://github.com/bitcoin/bitcoin/pull/6983 but I can't figure out what changed
< CodeShark> if I whitelist a node, other than network/transport issues or misbehaving, is there ever any conceivable situation where the node could get disconnected by bitcoin core
< dcousens> sorry, will go to bitcoin-dev

2015-11-12

< GitHub188> [bitcoin] jgarzik closed pull request #6771: Policy: Lower default limits for tx chains (master...lowerLimits) https://github.com/bitcoin/bitcoin/pull/6771
< GitHub107> bitcoin/master 38ed190 Jeff Garzik: Merge #6771 from branch 'lowerLimits' of git://github.com/morcos/bitcoin
< GitHub107> bitcoin/master 971a4e6 Alex Morcos: Lower default policy limits...
< GitHub107> [bitcoin] jgarzik pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/bd629d77edbe...38ed190eefcc
< jgarzik> morcos, the latter - I always travis in jgarzik/bitcoin on the merged tree before pushing
< gmaxwell> I mean, I'm probably at fault for making people think that its the only way to get a fraud proof for spending non-existing coins, since in the very first writeup I suggest that https://en.bitcoin.it/wiki/User:Gmaxwell/features#Proofs
< gmaxwell> it's been discussed in #bitcoin-wizards.
< Luke-Jr> wangchun: let's chat in #bitcoin since there is conversation here already
< GitHub56> [bitcoin] sdaftuar closed pull request #6992: [WIP] [Mempool] Add space for priority transactions (master...add-priority-to-mempool) https://github.com/bitcoin/bitcoin/pull/6992
< GitHub51> bitcoin/master 09c1ae1 Wladimir J. van der Laan: torcontrol improvements and fixes...
< GitHub51> bitcoin/master 2f796e5 Peter Todd: Better error message if Tor version too old
< GitHub51> bitcoin/master 8f4e67f Wladimir J. van der Laan: net: Automatically create hidden service, listen on Tor...
< GitHub33> [bitcoin] laanwj closed pull request #6639: net: Automatically create hidden service, listen on Tor (master...2015_08_tor_hs_v2) https://github.com/bitcoin/bitcoin/pull/6639
< GitHub51> [bitcoin] laanwj pushed 6 new commits to master: https://github.com/bitcoin/bitcoin/compare/eb6172a8ca7e...bd629d77edbe
< GitHub101> [bitcoin] laanwj closed pull request #6918: Make sigcache faster, more efficient, larger (master...smallsigcache) https://github.com/bitcoin/bitcoin/pull/6918
< GitHub77> bitcoin/master 0b9e9dc Pieter Wuille: Evict sigcache entries that are seen in a block
< GitHub77> bitcoin/master 69d373f Pieter Wuille: Don't wipe the sigcache in TestBlockValidity
< GitHub77> bitcoin/master 830e3f3 Pieter Wuille: Make sigcache faster and more efficient
< GitHub77> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/54e8bfec8387...eb6172a8ca7e
< wumpus> optimizing the disk component of getdata doesn't rank anywhere in the 'concerns that slow down bitcoin core in practice' I'm sure
< GitHub166> [bitcoin] sipa closed pull request #6931: Skip BIP 30 verification where not necessary (master...skipBIP30) https://github.com/bitcoin/bitcoin/pull/6931
< GitHub129> bitcoin/master 54e8bfe Pieter Wuille: Merge pull request #6931...
< GitHub129> bitcoin/master 33c90cf Alex Morcos: Make skipping BIP30 check chain agnostic
< GitHub129> bitcoin/master 06d81ad Alex Morcos: Skip BIP30 check after BIP34 activation
< GitHub129> [bitcoin] sipa pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/5fcc14ee0562...54e8bfec8387
< GitHub29> [bitcoin] jonasschnelli opened pull request #7000: [Qt] add shortcurts for debug-/console-window (master...2015/11/qt_shortcuts) https://github.com/bitcoin/bitcoin/pull/7000
< GitHub86> [bitcoin] jonasschnelli opened pull request #6999: add (max)uploadtarget infos to getnetinfos RPC help (master...2015/11/maxuploadtarget_rpc_help) https://github.com/bitcoin/bitcoin/pull/6999
< GitHub55> [bitcoin] laanwj opened pull request #6998: doc: Remove mention of pulltester from README.md (master...2015_11_readme_pull_pulltester) https://github.com/bitcoin/bitcoin/pull/6998
< GitHub93> [bitcoin] sturles opened pull request #6997: Don't use hard-coded AllowFree, as this is usually far too low. (master...no-default-free-priority) https://github.com/bitcoin/bitcoin/pull/6997
< GitHub139> [bitcoin] sipa opened pull request #6996: Add preciousblock RPC (master...preciousblock) https://github.com/bitcoin/bitcoin/pull/6996
< wumpus> as for (b) I know this issue all too well: https://github.com/bitcoin/bitcoin/issues/6971
< gmaxwell> prior to bitcoin they were hard(er) because of flooding attacks, but we have a one-two punch here: we have highly valuable tiny messages AND mechenisms to stop flooding attacks.
< wumpus> a tor bitcoin bundle would be pretty nice
< GitHub198> [bitcoin] laanwj closed pull request #6804: [tests] Add basic coverage reporting for RPC tests (master...rpc_coverage) https://github.com/bitcoin/bitcoin/pull/6804
< GitHub183> bitcoin/master 5fcc14e Wladimir J. van der Laan: Merge pull request #6804...
< GitHub183> bitcoin/master b5cbd39 James O'Beirne: Add basic coverage reporting for RPC tests...
< GitHub183> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/cbf9609c71c6...5fcc14ee0562
< GitHub16> [bitcoin] pstratem opened pull request #6993: Add -blocksonly option (master...blocksonly) https://github.com/bitcoin/bitcoin/pull/6993

2015-11-11

< GitHub131> [bitcoin] sdaftuar opened pull request #6992: [WIP] [Mempool] Add space for priority transactions (master...add-priority-to-mempool) https://github.com/bitcoin/bitcoin/pull/6992
< jgarzik> btcdrak, release manager != project lead... bitcoin core is intentionally decentralized ever since Satoshi turned things over to Gavin.
< GitHub3> [bitcoin] petertodd opened pull request #6991: Minor: Clarify 'fee' field in fundrawtransaction help text (master...clarify-fundrawtransaction-help) https://github.com/bitcoin/bitcoin/pull/6991
< GitHub167> [bitcoin] laanwj closed pull request #6969: doc: there is no libboost-base-dev, add missing sudo in release notes (master...2015_11_docfix) https://github.com/bitcoin/bitcoin/pull/6969
< GitHub2> bitcoin/master cbf9609 Wladimir J. van der Laan: Merge pull request #6969...
< GitHub2> bitcoin/master 7085728 Wladimir J. van der Laan: doc: there is no libboost-base-dev, add missing sudo...
< GitHub2> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/2f71b07d5ed6...cbf9609c71c6
< GitHub183> [bitcoin] laanwj closed pull request #6974: Always allow getheaders from whitelisted peers (master...2015_11_whitelisted_allow_headers) https://github.com/bitcoin/bitcoin/pull/6974
< GitHub59> bitcoin/master 2f71b07 Wladimir J. van der Laan: Merge pull request #6974...
< GitHub59> bitcoin/master 40b77d4 Wladimir J. van der Laan: Always allow getheaders from whitelisted peers...
< GitHub59> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/c9743062a37c...2f71b07d5ed6
< GitHub82> [bitcoin] laanwj closed pull request #6987: [doc] add documentation about reducing traffic (master...2015/11/doc_traffic) https://github.com/bitcoin/bitcoin/pull/6987
< GitHub144> bitcoin/master c974306 Wladimir J. van der Laan: Merge pull request #6987...
< GitHub144> bitcoin/master 42bae94 fanquake: [doc] Improve lanaguge in reducetraffic.md
< GitHub144> bitcoin/master 6a776fa Jonas Schnelli: [doc] add documentation how to reduce traffic
< GitHub144> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/44d7b561bc75...c9743062a37c
< GitHub109> [bitcoin] laanwj closed pull request #6300: getblockhash: Return RPC_NOT_FOUND when nHeight > chainActive.Height() (master...master) https://github.com/bitcoin/bitcoin/pull/6300
< GitHub175> [bitcoin] laanwj closed pull request #6988: [doc] Fix FIXME for libblkmaker in release-notes.md (master...MarcoFalke-2015-docFixme) https://github.com/bitcoin/bitcoin/pull/6988
< GitHub188> bitcoin/master 44d7b56 Wladimir J. van der Laan: Merge pull request #6988...
< GitHub188> bitcoin/master 0817bf7 MarcoFalke: [doc] Fix FIXME for libblkmaker in release-notes.md
< GitHub188> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/3ac70609345a...44d7b561bc75
< GitHub96> [bitcoin] laanwj opened pull request #6990: http: speed up shutdown (master...2015_11_threadexit) https://github.com/bitcoin/bitcoin/pull/6990
< GitHub18> [bitcoin] MarcoFalke opened pull request #6988: [doc] Fix FIXME for libblkmaker in release-notes.md (master...MarcoFalke-2015-docFixme) https://github.com/bitcoin/bitcoin/pull/6988
< jtimon> wumpus previously main.h was only included for CheckTransaction so I removed it in https://github.com/jtimon/bitcoin/commit/7ec074dca2ec60d7f1c0711be31d165121ead600#diff-4f6d60b6976522cec2974a0aea9a5ab3L14 but in the future (rebased version) it will also be needed for minRelayTxFee and DEFAULT_MIN_RELAY_TX_FEE and cannot be removed (just change the comment instead)
< harding> jonasschnelli: I do think that we could intergrate 6987 to useful effect into https://bitcoin.org/en/full-node
< harding> wumpus: ah. We do link to the Bitcoin Core documentation directory ( https://bitcoin.org/en/bitcoin-core/help#documentation ), although not prominently.
< wumpus> yes, was about to mention that, there is also overlap with the documentation on bitcoin.org (att least the bitcoin core section)
< jonasschnelli> harding: https://github.com/bitcoin/bitcoin/pull/6987 could also be something for bitcoin.org?
< jtimon> phantomcircuit: anyway I don't know of any library with functions from bitcoin core besides libbitcoinconsensus with its VerifyScript()
< GitHub19> [bitcoin] jonasschnelli opened pull request #6987: [doc] add documentation about reducing traffic (master...2015/11/doc_traffic) https://github.com/bitcoin/bitcoin/pull/6987
< GitHub104> [bitcoin] jtimon opened pull request #6986: Globals: Don't call Params() from miner.cpp (master...global-chainparams-miner) https://github.com/bitcoin/bitcoin/pull/6986
< jtimon> phantomcircuit: a library with all the pure functions in bitcoin? what do you mean by pure functions?
< GitHub127> [bitcoin] laanwj closed pull request #6978: Alternative fix for #6248 (qt+fPIE) (master...qt-pie) https://github.com/bitcoin/bitcoin/pull/6978
< GitHub40> bitcoin/master 3ac7060 Wladimir J. van der Laan: Merge pull request #6978...
< GitHub40> bitcoin/master 69d0513 Cory Fields: build: Use fPIC rather than fPIE for qt objects....
< GitHub40> bitcoin/master 17c4d9d Cory Fields: build: Split hardening/fPIE options out...
< GitHub40> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/cb841e75135e...3ac70609345a
< GitHub113> bitcoin/master 7267843 Jorge Timón: Globals: Make AcceptBlockHeader static (Fix #6163)...
< GitHub113> bitcoin/master cb841e7 Wladimir J. van der Laan: Merge pull request #6982...
< GitHub58> [bitcoin] laanwj closed pull request #6982: Globals: Make AcceptBlockHeader static (Fix #6163) (AcceptBlockHeader) (master...fix-6163) https://github.com/bitcoin/bitcoin/pull/6982
< GitHub113> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/38a4f267e82a...cb841e75135e
< GitHub105> [bitcoin] laanwj closed pull request #6985: [gitian] Use latest vm-builder ( 0.12.4+bzr494 ) on Debian (master...patch-3) https://github.com/bitcoin/bitcoin/pull/6985
< GitHub144> bitcoin/master 7791395 Michael: [gitian] Use vm-builder_0.12.4+bzr494 on Debian
< GitHub144> bitcoin/master 38a4f26 Wladimir J. van der Laan: Merge pull request #6985...
< GitHub144> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/de7d4591a7ce...38a4f267e82a
< fanquake> jonasschnelli You can grab the commit from https://github.com/fanquake/bitcoin/commits/fixupdoc
< GitHub21> [bitcoin] fanquake opened pull request #6985: [gitian] Use latest vm-builder ( 0.12.4+bzr494 ) on Debian (master...patch-3) https://github.com/bitcoin/bitcoin/pull/6985
< wumpus> for https://github.com/bitcoin/bitcoin/issues//2368 "Consider reducing 'getblocks' limit below 500" I have no idea whether it's still relevant, but the limit is still hardcoded to 500
< sipa> warren: set a large dbcache, and start closing bitcoin-qt right before the crash
< jonasschnelli> cool. You could do a commit on top of my branch https://github.com/jonasschnelli/bitcoin/commits/2015/11/MarcoFalke-2015-maxupload
< jonasschnelli> would this additional doc make sense? https://github.com/bitcoin/bitcoin/commit/f6b96ce09ec827bb9be0fd926b199009dd08bfd4
< GitHub126> [bitcoin] jonasschnelli opened pull request #6984: don't enforce maxuploadtarget's disconnect for whitelisted peers (master...2015/11/maxupload_whitebind) https://github.com/bitcoin/bitcoin/pull/6984
< wumpus> Luke-Jr: not just the return value, in multiple ways it doesn't behave the same on all platforms. we had issues due to that was well, one of the reasons why bitcoin core switched to a typesafe library for formatting that doesn't rely on libc
< warren> Running Bitcoin Core v0.11.2rc1 with blank data dir on Windows 10 ... it connected to 9 peers including a local peer. Stuck at 0 blocks for the past 10 minutes. what should I try?
< GitHub119> [bitcoin] sipa opened pull request #6983: Update libsecp256k1 (master...secp256k1new) https://github.com/bitcoin/bitcoin/pull/6983
< phantomcircuit> jtimon, there wouldn't happen to be a library somewhere of all the pure functions in bitcoin core would there?

2015-11-10

< sipa> i think it makes sense for bitcoin-qt to do something smartish to guess how much memory to use for caches
< wumpus> jgarzik: in case of bitcoin core that's noto the case, only still-unset settings get overridden
< GitHub60> [bitcoin] laanwj closed pull request #6981: doc: Add note about SI units to developer notes (master...2015_11_si_units) https://github.com/bitcoin/bitcoin/pull/6981
< jgarzik> There's a reason why I pushed Gavin hard for multiple committers when Satoshi disappeared - Gavin was release manager, not Bitcoin Leader
< Luke-Jr> as long as sipa hates 1024 KB, and I prefer 1024 KB, there's no way to satisfy both without an option. If you all decide to make it 1000-based only, I can just add an option to Bitcoin LJR if I ever care enough (unlikely)
< GitHub176> [bitcoin] jtimon opened pull request #6982: Fix #6163 (AcceptBlockHeader) (master...fix-6163) https://github.com/bitcoin/bitcoin/pull/6982
< GitHub111> bitcoin/master de7d459 Wladimir J. van der Laan: Merge pull request #6965...
< GitHub111> bitcoin/master 77f1f59 Matt Corallo: Benchmark sanity checks and fork checks in ConnectBlock
< GitHub111> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b56953e9bb5a...de7d4591a7ce
< GitHub108> [bitcoin] laanwj closed pull request #6965: Benchmark sanity checks and fork checks in ConnectBlock (master...bench) https://github.com/bitcoin/bitcoin/pull/6965
< GitHub159> [bitcoin] laanwj opened pull request #6981: doc: Add note about SI units to developer notes (master...2015_11_si_units) https://github.com/bitcoin/bitcoin/pull/6981
< btcdrak> sipa: I just added some commits which I think address your concerns https://github.com/bitcoin/bitcoin/pull/6312#issuecomment-154652895
< jgarzik> wangchun, Scaling Bitcoin Pt 2 is the next step in figuring out uncontroversial hard fork
< sipa> Bitcoin Core will implement any uncontroversial hard fork.
< wumpus> but for bitcoin core which is a high-level application there shoulod be no reason to not just use SI units
< sipa> bitcoin traditionally uses 1000-based units (feerate is per 1000 bytes, block limit is 1000000 bytes), but I can't say I've payed much attention to it myself
< GitHub4> bitcoin/master b56953e Wladimir J. van der Laan: qt: Periodic translations update
< GitHub4> [bitcoin] laanwj pushed 1 new commit to master: https://github.com/bitcoin/bitcoin/commit/b56953e9bb5a32bc35365d1f0c5de5528c0650dd
< GitHub102> [bitcoin] laanwj closed pull request #6940: Improving labels for Sent / Received "Bytes" (master...patch-1) https://github.com/bitcoin/bitcoin/pull/6940
< GitHub85> bitcoin/master 32d8b15 Wladimir J. van der Laan: Merge pull request #6940...
< GitHub85> bitcoin/master 7ca73dc Jonathan Cross: Improving labels for Sent / Received "Bytes"...
< GitHub85> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/9fa54a1b0c1a...32d8b1570cb0
< GitHub99> [bitcoin] laanwj closed pull request #6937: Fix Boost 1.58.0 build for mips arch (master...mips-options-fix) https://github.com/bitcoin/bitcoin/pull/6937
< GitHub111> [bitcoin] fanquake opened pull request #6980: [Depends] Bump Boost, miniupnpc & ccache (master...depends-bump-boost) https://github.com/bitcoin/bitcoin/pull/6980
< GitHub65> [bitcoin] laanwj closed pull request #6822: [tests] Be more strict checking dust (master...MarcoFalke-2015-minRelayTxFeeCleanup) https://github.com/bitcoin/bitcoin/pull/6822
< GitHub78> bitcoin/master 536766c MarcoFalke: [trivial] New DEFAULT_MIN_RELAY_TX_FEE = 1000
< GitHub78> bitcoin/master e20d924 MarcoFalke: [trivial] init: Use defaults MIN_RELAY_TX_FEE & TRANSACTION_MAXFEE
< GitHub78> bitcoin/master 5f46a7d MarcoFalke: transaction_tests: Be more strict checking dust...
< GitHub78> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/755b4ba848bc...9fa54a1b0c1a
< GitHub120> [bitcoin] laanwj closed pull request #6693: Set Windows TCP buffers to 64KB to match OSX and Unix (master...issue_6554) https://github.com/bitcoin/bitcoin/pull/6693
< GitHub35> bitcoin/master 755b4ba Wladimir J. van der Laan: Merge pull request #5574...
< GitHub43> [bitcoin] laanwj closed pull request #5574: Expose GUI labels in RPC as comments (master...rpc_labels) https://github.com/bitcoin/bitcoin/pull/5574
< GitHub35> bitcoin/master fd55571 Luke Dashjr: wallet: Expose GUI labels in RPC
< GitHub35> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/77beab70deae...755b4ba848bc
< GitHub170> [bitcoin] laanwj closed pull request #6248: Fix Qt build on arch by setting -fPIC (master...archbuild) https://github.com/bitcoin/bitcoin/pull/6248
< GitHub104> [bitcoin] laanwj closed pull request #6163: Chainparams: Explicit Consensus::Params arg for almost all remaining functions (master...chainparams-remaining-consensus) https://github.com/bitcoin/bitcoin/pull/6163
< GitHub179> bitcoin/master 87cbdb8 Jorge Timón: Globals: Explicit Consensus::Params arg for main:...
< GitHub179> bitcoin/master 77beab7 Wladimir J. van der Laan: Merge pull request #6163...
< GitHub179> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/503ff6e1ae69...77beab70deae
< wumpus> 0.10.4rc1 executables live https://bitcoin.org/bin/bitcoin-core-0.10.4/test/
< GitHub50> bitcoin/0.11 7e27892 Wladimir J. van der Laan: Merge pull request #6975...
< GitHub175> [bitcoin] laanwj closed pull request #6975: [doc] 0.11.2 release notes: use original pull numbers (0.11...note-0.11.2-orig-prs) https://github.com/bitcoin/bitcoin/pull/6975
< GitHub50> bitcoin/0.11 ab6ff12 David A. Harding: [doc] 0.11.2 release notes: use original pull numbers...
< GitHub50> [bitcoin] laanwj pushed 2 new commits to 0.11: https://github.com/bitcoin/bitcoin/compare/3dcb390fe9e2...7e278929df53
< jonasschnelli> dcousens: check: https://bitcoin.jonasschnelli.ch/secp/stuck_debug.log, search after last UpdateTip
< GitHub175> [bitcoin] jonasschnelli opened pull request #6979: [Qt] simple mempool info in debug window (master...2015/11/qt_mempool_easyinfo) https://github.com/bitcoin/bitcoin/pull/6979
< jonasschnelli> the strange thing is, as you can see here (https://bitcoin.jonasschnelli.ch/secp/stuck_debug.log), no block was rejected.
< GitHub106> [bitcoin] theuni opened pull request #6978: Alternative fix for #6248 (qt+fPIE) (master...qt-pie) https://github.com/bitcoin/bitcoin/pull/6978

2015-11-09

< cfields> gitian signers: osx detached sig for 0.10.4rc1: https://bitcoincore.org/cfields/bitcoin-0.10.4rc1/signature.tar.gz
< jonasschnelli> git log says: 4ee149a6db25cde31432f83369b40c92be13021c + Merge branch 'secp256k1' of https://github.com/sipa/bitcoin
< jonasschnelli> sipa: nothing in the debug log: check here if you wan't: https://bitcoin.jonasschnelli.ch/secp/stuck_debug.log
< jonasschnelli> theres a second headers-only chain which i accidentally cut of in the comment: now re-added: https://github.com/bitcoin/bitcoin/pull/6954#issuecomment-155093019
< jonasschnelli> last 1000 lines of my debug log: https://bitcoin.jonasschnelli.ch/secp/stuck_debug.log
< GitHub75> [bitcoin] harding opened pull request #6975: [doc] 0.11.2 release notes: use original pull numbers (0.11...note-0.11.2-orig-prs) https://github.com/bitcoin/bitcoin/pull/6975
< GitHub135> bitcoin/master 503ff6e Wladimir J. van der Laan: Merge pull request #6967...
< GitHub148> [bitcoin] laanwj closed pull request #6967: Use Pieter's signing subkey instead of his primary key (master...verify-commits-fixes) https://github.com/bitcoin/bitcoin/pull/6967
< GitHub135> bitcoin/master 9ea7762 Matt Corallo: Use Pieter's signing subkey instead of his primary key...
< GitHub135> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/f24880b13213...503ff6e1ae69
< GitHub127> bitcoin/0.10 c2e7baf Wladimir J. van der Laan: Bump version to 0.10.4, add release notes
< GitHub127> [bitcoin] laanwj pushed 1 new commit to 0.10: https://github.com/bitcoin/bitcoin/commit/c2e7baf2bdb2e2c163bcadaebe68ee00c9a27e7c
< GitHub72> [bitcoin] laanwj opened pull request #6974: Always allow getheaders from whitelisted peers (master...2015_11_whitelisted_allow_headers) https://github.com/bitcoin/bitcoin/pull/6974
< GitHub120> [bitcoin] ptschip opened pull request #6973: Zlib Block Compression for block relay (master...compress) https://github.com/bitcoin/bitcoin/pull/6973
< GitHub98> bitcoin/0.10 3ad96bd Alex Morcos: Fix locking in GetTransaction....
< GitHub98> bitcoin/0.10 612efe8 MarcoFalke: [Qt] Raise debug window when requested...
< GitHub98> bitcoin/0.10 9c81005 Diego Viola: Fix spelling of Qt
< GitHub98> [bitcoin] laanwj pushed 15 new commits to 0.10: https://github.com/bitcoin/bitcoin/compare/cbc4e3bd37da...3b89bf643896
< GitHub173> [bitcoin] laanwj closed pull request #6953: Backport bugfixes to 0.10 (2015-10-22 / f2c869a) (0.10...backport-bugfixes-to-0.10-20151014) https://github.com/bitcoin/bitcoin/pull/6953
< GitHub130> [bitcoin] laanwj closed pull request #6908: Chainparams: DRY: Make qt/guiutil.cpp fit BIP70 chain name strings (master...chainparams-bip70-0.12.99) https://github.com/bitcoin/bitcoin/pull/6908
< GitHub67> bitcoin/master f24880b Wladimir J. van der Laan: Merge pull request #6908...
< GitHub67> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/6176e9bf3d55...f24880b13213
< GitHub67> bitcoin/master c53d48a Jorge Timón: BIP70: Chainparams: DRY: Make qt/guiutil.cpp fit BIP70 chain name strings...
< gmaxwell> wumpus: I heard from andytoshi that you've been keeping him honest on rust-bitcoin. :)
< wumpus> yes that solves one problem but causes many others. We tried forcing the locale to C in bitcoin for a while but this messed with file name encodings on windows.
< GitHub138> bitcoin/0.11 40941d9 David A. Harding: [Docs] First-draft release notes for 0.11.2RC1
< GitHub138> bitcoin/0.11 929b2c7 David A. Harding: [docs] Minor revisions to 0.11.2RC1 release notes...
< GitHub138> bitcoin/0.11 9149589 David A. Harding: [docs] 0.11.2 release notes: add sipa graphs & leveldb note...
< GitHub138> [bitcoin] laanwj pushed 4 new commits to 0.11: https://github.com/bitcoin/bitcoin/compare/984587ac5d3e...3dcb390fe9e2
< GitHub133> [bitcoin] laanwj closed pull request #6968: [Docs] First-draft release notes for 0.11.2RC1 (0.11...release-notes-0.11.2) https://github.com/bitcoin/bitcoin/pull/6968
< wumpus> executables for 0.11.2rc1 are live: https://bitcoin.org/bin/bitcoin-core-0.11.2/test/ sorry for the delay
< wumpus> not saying that is not the way forward to 'find' the same datadir in bitcoin-cli and bitcoind, as well as other software, but it doesn't quite make it easier
< GitHub126> [bitcoin] laanwj opened pull request #6970: Fix crash in validateaddress with -disablewallet (master...2015_11_rpc_validateaddress_crash) https://github.com/bitcoin/bitcoin/pull/6970
< Luke-Jr> ~/.config/Bitcoin/Bitcoin-Qt.conf
< wumpus> bitcoin, by design, only has one datadir. If you move it, everything moves with it
< Luke-Jr> does bitcoin-qt use strDataDir for the config location too? :/
< wumpus> indeed it's the same for finding bitcoin.conf
< gmaxwell> wumpus: I'd still suggest adding (hopefully) 1/2 LOC: code to use RegQuery to find the same location in bitcoind/bitcoin-cli.
< gmaxwell> Luke-Jr: I think we could probably just add code that reads the registry in the same way RandAddSeedPerfmon and looks to the same place QT looks, and call that from bitcoind/bitcoin-cli on windows, and document the location so other programs can look. No?
< gmaxwell> wumpus: sometimes we have to... there are dozens of windows ifdefs in the code. Unless you think linking bitcoind/bitcoin-cli to QT is a better plan. :)
< wumpus> gmaxwell: qsettings is the qt equivalent of adding command line arguments, you can pass a datadir to bitcoin-cli and it will work
< Luke-Jr> wumpus: my main concern is BFGMiner, not bitcoin-cli
< gmaxwell> wumpus: it is a bit broken that changing your datadir breaks bitcoind / bitcoin-cli from finding it.
< wumpus> most people that want to use RPC with bitcoin-qt will use the debug console
< wumpus> maybe 1% of users actually use bitcoin-qt with bitcoin-cli, 0.1% uses it with a changed datadir through qt
< wumpus> bitcoin.conf is usually in the same directory as the cookie so it won't avoid *any* problems
< wumpus> otherwise I'd just have made it write a rpcuser and rpcpassword to bitcoin.conf
< dgenr8> writing to bitcoin.conf should be controlled by a setting in bitcoin.conf
< gmaxwell> then you have the "how do you find bitcoin.conf" problem.
< wumpus> don't write to bitcoin.conf
< Luke-Jr> it's tempting to change this to write to bitcoin.conf instead of QSettings then
< gmaxwell> using something QT specific for handling finding the datadir on windows was probably what should have been avoided, since bitcoind and bitcoin-cli also need to find the data directory.
< Luke-Jr> if it's overridden via bitcoin.conf or cmdline, it uses that; otherwise the platform default