2016-10-03

< GitHub108> [bitcoin] MarcoFalke closed pull request #8860: [qa] util: Move wait_bitcoinds() into stop_nodes() (master...Mf1610-qaUtilWait) https://github.com/bitcoin/bitcoin/pull/8860
< GitHub76> bitcoin/master 76f3c02 MarcoFalke: Merge #8860: [qa] util: Move wait_bitcoinds() into stop_nodes()...
< GitHub76> bitcoin/master fa7c35c MarcoFalke: [qa] util: Move wait_bitcoinds() into stop_nodes()
< GitHub76> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/eafc5f4fae52...76f3c02fb01a
< da2ce7> hello, I have a build error on macOS; bitcoin/master: https://paste.fedoraproject.org/442517/75481833/
< wumpus> https://github.com/bitcoin/bitcoin/issues/8864 we just have to add a targeting mark to them and it'd be a fun whack-a-mole minigame
< GitHub123> [bitcoin] laanwj closed pull request #8863: univalue: Pull subtree (master...Mf1610-univalue) https://github.com/bitcoin/bitcoin/pull/8863
< GitHub182> bitcoin/master eafc5f4 Wladimir J. van der Laan: Merge #8863: univalue: Pull subtree...
< GitHub182> bitcoin/master e757115 MarcoFalke: Merge commit '2ca7faab4205822b06dc2ab2bbda0a9a70fce7e0' into HEAD
< GitHub182> bitcoin/master 2ca7faa MarcoFalke: Squashed 'src/univalue/' changes from daf1285..16a1f7f...
< GitHub182> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/6f3ef551fa0f...eafc5f4fae52
< wumpus> gmaxwell: hah, already preempted that days ago: https://github.com/bitcoin/bitcoin/pull/8813#issuecomment-250788185 :)
< GitHub183> [bitcoin] laanwj closed pull request #8858: rpc: Generate auth cookie in hex instead of base64 (master...2016_10_01_moar_cookies) https://github.com/bitcoin/bitcoin/pull/8858
< GitHub64> bitcoin/master 6f3ef55 Wladimir J. van der Laan: Merge #8858: rpc: Generate auth cookie in hex instead of base64...
< GitHub64> bitcoin/master 1c80386 Wladimir J. van der Laan: rpc: Generate auth cookie in hex instead of base64...
< GitHub64> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/6013c73b3312...6f3ef551fa0f

2016-10-02

< GitHub84> [bitcoin] MarcoFalke closed pull request #8857: [qa] mininode: Only allow named args in wait_until (master...Mf1610-qaMininodeWaitUntil) https://github.com/bitcoin/bitcoin/pull/8857
< GitHub168> bitcoin/master 6013c73 MarcoFalke: Merge #8857: [qa] mininode: Only allow named args in wait_until...
< GitHub168> bitcoin/master fa66609 MarcoFalke: [qa] mininode: Only allow named args in wait_until
< GitHub168> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/6faffb8a83db...6013c73b3312
< GitHub35> [bitcoin] TheBlueMatt opened pull request #8865: Decouple peer-processing-logic from block-connection-logic (master...net_processing_1) https://github.com/bitcoin/bitcoin/pull/8865
< wumpus> achow101: people keep adding new pulls for 0.13.1, but by far the largest change that still has to go in is https://github.com/bitcoin/bitcoin/pull/8393
< BlueMatt> grrr, damn github...whatever, pulls out the CNodeState management from the top of main.cpp to the end as a part of https://github.com/TheBlueMatt/bitcoin/commits/net_processing_file, which disentangles that stuff somewhat and is like 2 functions away from splitting main.cpp in two
< GitHub119> [bitcoin] MarcoFalke opened pull request #8863: univalue: Pull subtree (master...Mf1610-univalue) https://github.com/bitcoin/bitcoin/pull/8863

2016-10-01

< GitHub19> [bitcoin] theuni opened pull request #8862: Fix a few cases where messages were sent after requested disconnect (master...fix-disconnect-send) https://github.com/bitcoin/bitcoin/pull/8862
< GitHub144> [bitcoin] CryptAxe closed pull request #8763: Optionally sweep funds from private key. (master...2751PullRequest) https://github.com/bitcoin/bitcoin/pull/8763
< GitHub163> [bitcoin] MarcoFalke opened pull request #8860: [qa] util: Move wait_bitcoinds() into stop_nodes() (master...Mf1610-qaUtilWait) https://github.com/bitcoin/bitcoin/pull/8860
< GitHub70> [bitcoin] MarcoFalke opened pull request #8859: WIP: [qa] Add script to check for datadir compatibility between versions (master...Mf1610-qaCompat) https://github.com/bitcoin/bitcoin/pull/8859
< * midnightmagic> contemplates how to block *outgoing* connections to only bitcoin, when the other side is using a non-standard port. :(
< wumpus> midnightmagic: here's the list of 52.X nodes https://dev.visucore.com/bitcoin/tmp/list.json
< GitHub161> [bitcoin] laanwj opened pull request #8858: rpc: Generate auth cookie in hex instead of base64 (master...2016_10_01_moar_cookies) https://github.com/bitcoin/bitcoin/pull/8858
< luke-jr> are you subscribed to bitcoin-dev? :|
< MarcoFalke> Sure, but for BIP1 it seems particularly weird. (Given that the (Github)user is not active in any other bitcoin related project)
< GitHub47> [bitcoin] MarcoFalke opened pull request #8857: [qa] mininode: Fix order of positional args in wait_until (master...Mf1610-qaMininodeWaitUntil) https://github.com/bitcoin/bitcoin/pull/8857
< GitHub189> bitcoin/master 6faffb8 MarcoFalke: Merge #8854: [qa] Fix race condition in p2p-compactblocks test...
< GitHub39> [bitcoin] MarcoFalke closed pull request #8854: [qa] Fix race condition in p2p-compactblocks test (master...fix-p2p-sync) https://github.com/bitcoin/bitcoin/pull/8854
< GitHub189> bitcoin/master b5fd666 Suhas Daftuar: [qa] Fix race condition in p2p-compactblocks test...
< GitHub189> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/7b784cc2bbcd...6faffb8a83db
< GitHub196> [bitcoin] jtimon opened pull request #8856: Globals: Decouple GetConfigFile and ReadConfigFile from global mapArgs (master...0.13-globals-utils-configfile) https://github.com/bitcoin/bitcoin/pull/8856
< GitHub93> [bitcoin] jtimon opened pull request #8855: Use a proper factory for creating chainparams (master...0.13-chainparams-factory) https://github.com/bitcoin/bitcoin/pull/8855
< GitHub16> [bitcoin] sdaftuar opened pull request #8854: [qa] Fix race condition in p2p-compactblocks test (master...fix-p2p-sync) https://github.com/bitcoin/bitcoin/pull/8854
< GitHub165> [bitcoin] dcousens closed pull request #7436: AcceptToMempool: extract various policy functions (master...expolicy) https://github.com/bitcoin/bitcoin/pull/7436

2016-09-30

< BlueMatt> wumpus: same guy, meant to get private review before pr'ing to bitcoin core directly
< GitHub180> [bitcoin] czzarr closed pull request #8853: add p2sh and segwit options to bitcoin-tx outscript command (master...add-p2sh-segwit-options-to-bitcoin-tx) https://github.com/bitcoin/bitcoin/pull/8853
< GitHub161> [bitcoin] czzarr opened pull request #8853: add p2sh and segwit options to bitcoin-tx outscript command (master...add-p2sh-segwit-options-to-bitcoin-tx) https://github.com/bitcoin/bitcoin/pull/8853
< GitHub118> [bitcoin] laanwj closed pull request #8852: Mention Gitian building script in doc (Laudaa) (master...2016_09_laudaa_master) https://github.com/bitcoin/bitcoin/pull/8852
< GitHub151> bitcoin/master 7b784cc Wladimir J. van der Laan: Merge #8852: Mention Gitian building script in doc (Laudaa)...
< GitHub151> bitcoin/master 203e2dd Lauda: Mention Gitian building script in doc.
< GitHub151> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/940748b4b087...7b784cc2bbcd
< GitHub95> [bitcoin] laanwj opened pull request #8852: Mention Gitian building script in doc (master...2016_09_laudaa_master) https://github.com/bitcoin/bitcoin/pull/8852
< GitHub16> [bitcoin] laanwj closed pull request #8795: [doc] Mention Gitian building script in documents. (master...master) https://github.com/bitcoin/bitcoin/pull/8795
< GitHub44> [bitcoin] laanwj closed pull request #8851: [wallet] Move key derivation logic from GenerateNewKey to DeriveNewChildKey (pstratem) (master...Mf1610-walletDeriveNewChildPStratem) https://github.com/bitcoin/bitcoin/pull/8851
< GitHub187> [bitcoin] laanwj closed pull request #8375: [Wallet] Move key derivation logic from GenerateNewKey to DeriveNewChildKey (master...2016-07-19-cwallet-derivenewkey) https://github.com/bitcoin/bitcoin/pull/8375
< GitHub129> bitcoin/master 940748b Wladimir J. van der Laan: Merge #8851: [wallet] Move key derivation logic from GenerateNewKey to DeriveNewChildKey (pstratem)...
< GitHub129> bitcoin/master e198c52 Patrick Strateman: Move key derivation logic from GenerateNewKey to DeriveNewChildKey
< GitHub129> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/fb24d7eeb47e...940748b4b087
< GitHub192> [bitcoin] MarcoFalke opened pull request #8851: [wallet] Move key derivation logic from GenerateNewKey to DeriveNewChildKey (pstratem) (master...Mf1610-walletDeriveNewChildPStratem) https://github.com/bitcoin/bitcoin/pull/8851
< GitHub100> [bitcoin] MarcoFalke closed pull request #8633: Ugly hack to print out tests as they are run to mitigate travis timeouts (master...test-driver-hack) https://github.com/bitcoin/bitcoin/pull/8633
< jtimon> in https://github.com/bitcoin/bitcoin/projects/5 please move #8526 to done
< GitHub61> [bitcoin] laanwj closed pull request #8813: bitcoind: Daemonize using daemon(3) (master...2016_09_daemonize) https://github.com/bitcoin/bitcoin/pull/8813
< GitHub54> bitcoin/master fb24d7e Wladimir J. van der Laan: Merge #8813: bitcoind: Daemonize using daemon(3)...
< GitHub54> bitcoin/master a92bf4a Matthew King: bitcoind: Daemonize using daemon(3)...
< GitHub54> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/9bc6a6bd7b0d...fb24d7eeb47e
< GitHub8> [bitcoin] laanwj opened pull request #8850: Implement (begin|end)_ptr in C++11 and add deprecation comment (master...2016_09_beginptr_deprecation) https://github.com/bitcoin/bitcoin/pull/8850
< GitHub8> [bitcoin] czzarr opened pull request #8849: print P2WSH redeemScript in getrawtransaction if it s not a pubkey (master...print-p2wsh-redeemscript-in-getrawtransaction) https://github.com/bitcoin/bitcoin/pull/8849
< GitHub165> [bitcoin] laanwj closed pull request #8822: net: Consistent checksum handling (master...2016_09_normalize_checksum_handling) https://github.com/bitcoin/bitcoin/pull/8822
< GitHub121> bitcoin/master 9bc6a6b Wladimir J. van der Laan: Merge #8822: net: Consistent checksum handling...
< GitHub171> [bitcoin] jl2012 opened pull request #8848: Add NULLDUMMY verify flag in bitcoinconsensus.h (master...consensusnulldummy) https://github.com/bitcoin/bitcoin/pull/8848
< GitHub121> bitcoin/master 305087b Wladimir J. van der Laan: net: Hardcode protocol sizes and use fixed-size types...
< GitHub121> bitcoin/master 41e58fa Wladimir J. van der Laan: net: Consistent checksum handling...
< GitHub121> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/ef0801bd1381...9bc6a6bd7b0d
< GitHub48> [bitcoin] laanwj closed pull request #8830: [test] Add option to run bitcoin-util-test.py manually (master...test-bitcoin-util-manually) https://github.com/bitcoin/bitcoin/pull/8830
< GitHub8> bitcoin/master b82f493 jnewbery: Add option to run bitcoin-util-test.py manually
< GitHub8> bitcoin/master ef0801b Wladimir J. van der Laan: Merge #8830: [test] Add option to run bitcoin-util-test.py manually...
< GitHub8> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/90adfabd5daa...ef0801bd1381
< GitHub19> [bitcoin] laanwj closed pull request #8836: bitcoin-util-test.py should fail if the output file is empty (master...bitcoin-tx-no-empty-outputs) https://github.com/bitcoin/bitcoin/pull/8836
< GitHub160> bitcoin/master 90adfab Wladimir J. van der Laan: Merge #8836: bitcoin-util-test.py should fail if the output file is empty...
< GitHub160> bitcoin/master da94697 jnewbery: bitcoin-util-test.py should fail if the output file is empty
< GitHub160> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/0572acd63bc8...90adfabd5daa
< GitHub182> [bitcoin] laanwj closed pull request #8839: test: Avoid ConnectionResetErrors during RPC tests (master...2016_09_freebsd_rpctest_fix) https://github.com/bitcoin/bitcoin/pull/8839
< GitHub40> bitcoin/master 0572acd Wladimir J. van der Laan: Merge #8839: test: Avoid ConnectionResetErrors during RPC tests...
< GitHub40> bitcoin/master 1d28faf Wladimir J. van der Laan: test: Avoid ConnectionResetErrors during RPC tests...
< GitHub40> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/929860106f5a...0572acd63bc8
< GitHub152> [bitcoin] laanwj closed pull request #8840: test: Explicitly set encoding to utf8 when opening text files (master...2016_09_textfiles_locale) https://github.com/bitcoin/bitcoin/pull/8840
< GitHub193> bitcoin/master 9298601 Wladimir J. van der Laan: Merge #8840: test: Explicitly set encoding to utf8 when opening text files...
< GitHub193> bitcoin/master 30930e8 Wladimir J. van der Laan: test: Explicitly set encoding to utf8 when opening text files...
< GitHub193> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/83998b52d07b...929860106f5a
< GitHub138> [bitcoin] laanwj closed pull request #8841: [qa] fix nulldummy test (master...nulldummytest) https://github.com/bitcoin/bitcoin/pull/8841
< GitHub114> bitcoin/master 83998b5 Wladimir J. van der Laan: Merge #8841: [qa] fix nulldummy test...
< GitHub114> bitcoin/master 46a4774 Johnson Lau: Fix nulldummy.py test
< GitHub114> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/f560d9564f74...83998b52d07b
< luke-jr> I haven't even rebuild bitcoin-qt yet
< wumpus> although as an idea for bitcoin it's new, I think. THere has been a patch to add a button to disable networking completely but that kind of misses the point. That means a full catch-up has to be done when re-enabling

2016-09-29

< wumpus> I did an experiment once with storing the UTXOs in serialized form in memory: https://github.com/laanwj/bitcoin/tree/2016_04_dummy_db
< wumpus> bitcoin interrupts video playback? even in steady state mode?
< gmaxwell> so every time I go to use bitcoin I'm stuck waiting for it to catch up.
< gmaxwell> more obvious to me just by chance of hering more people complain about it, also I've stopped running a node 24/7 on my laptop because I've been watching the battlestar galactica series in evenings and bitcoin interupts video playback. :)
< GitHub72> [bitcoin] jnewbery opened pull request #8845: Don't return the address of a P2SH of a P2SH (master...trivial-P2SH-P2SH) https://github.com/bitcoin/bitcoin/pull/8845
< GitHub131> [bitcoin] jnewbery opened pull request #8844: change sigops cost to sigops weight (master...sigops_weight) https://github.com/bitcoin/bitcoin/pull/8844
< gmaxwell> Part of why I haven't given this that much more thought is because I think bitcoin will need to move to the commit state and forget history model; the ever growing sync time is too big a tide to stand against.
< gmaxwell> well if the users of bitcoin accepted that kind of security model change, what I would suggest is something like every 26280 blocks the block is required to have a commitment to the utxo set (could be a linear hash) as of 2016 blocks prior. and then six months of work after that, that commitment becomes usable for initial sync. and so then no one need process more than a year of blocks at sync.
< sipa> brainwave: bitcoin core does not and cannot know exchange rates
< sipa> S=0; fgrep DEEP ~/.bitcoin/debug.log | cut -d ' ' -f 4 | sort -g | uniq -c | tac | while read C D; do S=$(($S+$C)); echo "$D $C $S"; done | tac | less
< GitHub159> [bitcoin] laanwj opened pull request #8843: rpc: Handle `getinfo` client-side in bitcoin-cli w/ `-getinfo` (master...2016_09_getinfo_clientside) https://github.com/bitcoin/bitcoin/pull/8843
< luke-jr> achow101_: meh, should just collapse that into #bitcoin-dev :p
< achow101_> #bitcoin-armory
< gmaxwell> for those thinking that we have to verify all the old stuff for all time, that might be true for bitcoin core, but in the future I could imagine some implementations just not bothering to verify old stuff.
< gmaxwell> wumpus: just the "you got a faithful bitcoin core download but the attacker controls your network"... but that doesn't need a checkpoint to fix, a simple partitioning detction that knows the total work of the best chain at releast time is sufficient.
< Chris_Stewart_5> gmaxwell: How have we solved the problem that checkpoints were originally created for? You have an excerpt in here: https://en.bitcoin.it/wiki/Bitcoin_Core_0.11_(ch_5):_Initial_Block_Download#Checkpoints
< gmaxwell> (for example Bitcoin Classic's current behavior simply looks at block header timestamps and ignores signatures when they're more than 24 hours (*par) old by the local clock. It's easily exploited and makes me sad.
< wumpus> 288 is the minimum pruning amount in bitcoin core already so it'd be a valid choice
< petertodd> so a social consideration here, is we can think in terms of recent history as "if there's a flaw, how much would we ever reorg w/o just saying bitcoin has failed?"
< wumpus> s/bitcoin/bittorrent/
< petertodd> sipa: so the nice thing about bitcoin, is just randomly try will probably work fairly often due to the low number of ranges out there
< wumpus> I doubt bitcoin has 'service flags' in its tracker what blocks nodes have
< petertodd> luke-jr: I don't think so - bittorrent is a very different problem than bitcoin
< petertodd> sipa: see, given that the bitcoin protocol can't be safely sharded right now, I think we can safely say that we don't need to support sharding in block relay yet
< wumpus> #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
< wumpus> that would "just" require solving https://github.com/bitcoin/bitcoin/issues/7965
< GitHub57> [bitcoin] jl2012 opened pull request #8841: [qa] fix nulldummy test (master...nulldummytest) https://github.com/bitcoin/bitcoin/pull/8841
< GitHub51> [bitcoin] laanwj opened pull request #8840: test: Explicitly set encoding to utf8 when opening text files (master...2016_09_textfiles_locale) https://github.com/bitcoin/bitcoin/pull/8840
< GitHub75> [bitcoin] laanwj closed pull request #8826: Do not include env_win.cc on non-Windows systems (master...20160928_leveldb_no_win) https://github.com/bitcoin/bitcoin/pull/8826
< GitHub17> bitcoin/master 7e5fd71 Pavel Janík: Do not include env_win.cc on non-Windows systems
< GitHub17> bitcoin/master f560d95 Wladimir J. van der Laan: Merge #8826: Do not include env_win.cc on non-Windows systems...
< GitHub17> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/c9d7b0de2fc9...f560d9564f74
< GitHub36> [bitcoin] laanwj closed pull request #8834: [qa] blockstore: Switch to dumb dbm (master...Mf1610-qaBlockstoreDumb) https://github.com/bitcoin/bitcoin/pull/8834
< GitHub53> bitcoin/master c9d7b0d Wladimir J. van der Laan: Merge #8834: [qa] blockstore: Switch to dumb dbm...
< GitHub53> bitcoin/master fa9cd25 MarcoFalke: [qa] blockstore: Switch to dumb dbm
< GitHub53> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/c84181665f34...c9d7b0de2fc9
< GitHub98> [bitcoin] laanwj closed pull request #8819: [depends] Boost 1.61.0 (master...depends-boost-1-61-0) https://github.com/bitcoin/bitcoin/pull/8819
< GitHub42> bitcoin/master 16f8823 fanquake: [depends] Boost 1.61.0
< GitHub42> bitcoin/master c841816 Wladimir J. van der Laan: Merge #8819: [depends] Boost 1.61.0...
< GitHub42> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/2dd57e4f9f58...c84181665f34
< GitHub44> [bitcoin] laanwj opened pull request #8839: test: Avoid ConnectionResetErrors during RPC tests (master...2016_09_freebsd_rpctest_fix) https://github.com/bitcoin/bitcoin/pull/8839
< GitHub194> [bitcoin] laanwj closed pull request #8835: [qa] nulldummy.py: Don't run unused code (master...Mf1610-qaNulldummyUnused) https://github.com/bitcoin/bitcoin/pull/8835
< GitHub177> bitcoin/master 2dd57e4 Wladimir J. van der Laan: Merge #8835: [qa] nulldummy.py: Don't run unused code...
< GitHub177> bitcoin/master fa156c6 MarcoFalke: [qa] nulldummy: Don't run unused code
< GitHub177> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/9b94cca41f3e...2dd57e4f9f58
< GitHub16> [bitcoin] jnewbery opened pull request #8838: Only log block size if block size is being accounted (master...dont_log_size) https://github.com/bitcoin/bitcoin/pull/8838
< GitHub100> [bitcoin] jnewbery opened pull request #8837: allow bitcoin-tx to parse partial transactions (master...bitcoin-tx-partial-transactions) https://github.com/bitcoin/bitcoin/pull/8837
< GitHub8> [bitcoin] jnewbery opened pull request #8836: bitcoin-util-test.py should fail if the output file is empty (master...bitcoin-tx-no-empty-outputs) https://github.com/bitcoin/bitcoin/pull/8836
< GitHub180> [bitcoin] MarcoFalke opened pull request #8835: [qa] nulldummy.py: Don't run unused code (master...Mf1610-qaNulldummyUnused) https://github.com/bitcoin/bitcoin/pull/8835
< GitHub166> [bitcoin] jgarzik closed pull request #6451: BIP 102: Increase block size limit to 2MB (master...2015_2mb_blocksize) https://github.com/bitcoin/bitcoin/pull/6451
< GitHub41> [bitcoin] laanwj closed pull request #8658: Remove unused statements in serialization (master...20160902_nVersion_serialization_cleanup) https://github.com/bitcoin/bitcoin/pull/8658
< GitHub52> bitcoin/master 9b94cca Wladimir J. van der Laan: Merge #8658: Remove unused statements in serialization...
< GitHub52> bitcoin/master 64d9507 Pavel Janík: [WIP] Remove unused statement in serialization
< GitHub52> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/cc9e8aca5f95...9b94cca41f3e
< GitHub122> [bitcoin] MarcoFalke closed pull request #8827: [qa] Split up slow RPC calls to avoid pruning test timeouts (master...fix-pruning-timeout) https://github.com/bitcoin/bitcoin/pull/8827
< GitHub26> bitcoin/master cc9e8ac MarcoFalke: Merge #8827: [qa] Split up slow RPC calls to avoid pruning test timeouts...
< GitHub26> bitcoin/master a0f8482 Suhas Daftuar: [qa] Split up slow RPC calls to avoid pruning test timeouts
< GitHub26> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/8ca69a2a88a7...cc9e8aca5f95
< GitHub156> [bitcoin] MarcoFalke opened pull request #8834: [qa] blockstore: Switch to dumb dbm (master...Mf1610-qaBlockstoreDumb) https://github.com/bitcoin/bitcoin/pull/8834
< sipa> the bitcoin dissector is in c currently?
< wumpus> I'm pleasantly surprised how well the wireshark dissector for the bitcoin protocol works, can just do "tcp port 18333" then use a display filter of "bitcoin" and gets a running list of bitcoin packets easy to inspect using the tree structure
< wumpus> we should start selling a hardware quality certificates: has synced the bitcoin block chain succesfully
< GitHub21> [bitcoin] MarcoFalke closed pull request #8829: Add bitcoin-tx JSON tests (master...test-bitcoin-tx-json) https://github.com/bitcoin/bitcoin/pull/8829
< GitHub87> bitcoin/master 8ca69a2 MarcoFalke: Merge #8829: Add bitcoin-tx JSON tests...
< GitHub87> bitcoin/master 54e5d7c jnewbery: Add bitcoin-tx JSON tests
< GitHub87> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/489a6ab5073c...8ca69a2a88a7
< GitHub162> [bitcoin] laanwj closed pull request #7522: Bugfix: Only use git for build info if the repository is actually the right one (master...bugfix_gitdir) https://github.com/bitcoin/bitcoin/pull/7522
< GitHub165> [bitcoin] laanwj closed pull request #8730: depends: Add libevent compatibility patch for windows (master...2016_09_libevent_windows_gcc_531) https://github.com/bitcoin/bitcoin/pull/8730
< GitHub3> bitcoin/master 489a6ab Wladimir J. van der Laan: Merge #8730: depends: Add libevent compatibility patch for windows...
< GitHub3> bitcoin/master 64047f8 Wladimir J. van der Laan: depends: Add libevent compatibility patch for windows...
< GitHub3> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/7d563cc16d64...489a6ab5073c
< GitHub93> [bitcoin] laanwj closed pull request #8832: [rpc] throw JSONRPCError when utxo set can not be read (master...Mf1610-rpcUtxoFail) https://github.com/bitcoin/bitcoin/pull/8832
< GitHub131> bitcoin/master 7d563cc Wladimir J. van der Laan: Merge #8832: [rpc] throw JSONRPCError when utxo set can not be read...
< GitHub131> bitcoin/master fa05cfd MarcoFalke: [rpc] throw JSONRPCError when utxo set can not be read
< GitHub131> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/d675984fdfa4...7d563cc16d64
< wumpus> this is strange, https://github.com/bitcoin/bitcoin/pull/8832 "waxmihs2902 approved these changes 10 hours ago" clicking on that users name gives a 404 page
< GitHub190> bitcoin/master 3f8a5d8 Luke Dashjr: Trivial: build-aux/m4/l_atomic: Fix typo
< GitHub162> [bitcoin] laanwj closed pull request #8784: Copyright headers for build scripts (master...license_build) https://github.com/bitcoin/bitcoin/pull/8784
< GitHub190> bitcoin/master 3b4b6dc Luke Dashjr: Add MIT license to autogen.sh and share/genbuild.sh
< GitHub190> bitcoin/master f4dffdd Luke Dashjr: Add MIT license to Makefiles
< GitHub190> [bitcoin] laanwj pushed 5 new commits to master: https://github.com/bitcoin/bitcoin/compare/dc641415e75e...d675984fdfa4

2016-09-28

< GitHub123> [bitcoin] MarcoFalke opened pull request #8832: [rpc] throw JSONRPCError when utxo set can not be read (master...Mf1610-rpcUtxoFail) https://github.com/bitcoin/bitcoin/pull/8832
< wumpus> can you try the solution of https://github.com/bitcoin/bitcoin/issues/8605 ? e.g. replace dbm.ndbm with dbm.dumb in qa/rpc-tests/test-framework/blockstore.py
< wumpus> BTW bitcoin core on FreeBSD nearly works out of the box with ports-provided development packages, not nearly the struggle it is on OpenBSD. Though I don't think I've run the RPC tests yet :)
< wumpus> pedrobranco: see https://github.com/bitcoin/bitcoin/issues/8605 , that's my openbsd problem, if you replace dbm.ndbm with dbm.dumb it will probably work fine
< GitHub87> [bitcoin] pstratem opened pull request #8831: Replace CWalletDB::ReadKeyValue with CWallet::LoadKeyValue (master...2016-09-28-cwallet-loadkeyvalue) https://github.com/bitcoin/bitcoin/pull/8831
< pedrobranco> Hi, anyone is experiencing high disk space usage when running bitcoin tests (env. is osx)?
< GitHub21> [bitcoin] jnewbery opened pull request #8830: [test] Add option to run bitcoin-util-test.py manually (master...test-bitcoin-util-manually) https://github.com/bitcoin/bitcoin/pull/8830
< GitHub184> [bitcoin] jnewbery opened pull request #8829: Add bitcoin-tx JSON tests (master...test-bitcoin-tx-json) https://github.com/bitcoin/bitcoin/pull/8829
< GitHub163> [bitcoin] pstratem opened pull request #8828: Move CWalletDB::ReorderTransactions to CWallet (master...2016-09-28-cwallet-reordertransactions) https://github.com/bitcoin/bitcoin/pull/8828
< GitHub167> [bitcoin] sdaftuar opened pull request #8827: [qa] Split up slow RPC calls to avoid pruning test timeouts (master...fix-pruning-timeout) https://github.com/bitcoin/bitcoin/pull/8827
< jonasschnelli> wumpus: IMO this is ready: https://github.com/bitcoin/bitcoin/pull/7551
< GitHub163> [bitcoin] paveljanik opened pull request #8826: Do not include env_win.cc on non-Windows systems (master...20160928_leveldb_no_win) https://github.com/bitcoin/bitcoin/pull/8826
< GitHub22> [bitcoin] laanwj closed pull request #8820: [depends] Fix Qt compilation with Xcode 8 (master...depends-qt-xcoderun) https://github.com/bitcoin/bitcoin/pull/8820
< GitHub190> bitcoin/master dc64141 Wladimir J. van der Laan: Merge #8820: [depends] Fix Qt compilation with Xcode 8...
< GitHub190> bitcoin/master 5776e8a fanquake: [depends] Fix Qt compilation with Xcode 8
< GitHub190> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/7b05af634be4...dc641415e75e
< GitHub121> [bitcoin] laanwj closed pull request #8823: doc: Add privacy recommendation when running hidden service (master...2016_09_tor_recommendation) https://github.com/bitcoin/bitcoin/pull/8823
< GitHub61> bitcoin/master 7b05af6 Wladimir J. van der Laan: Merge #8823: doc: Add privacy recommendation when running hidden service...
< GitHub61> bitcoin/master ac01ff2 Wladimir J. van der Laan: doc: Add privacy recommendation when running hidden service
< GitHub61> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/14b7b3fb9f79...7b05af634be4
< jonasschnelli> Maybe I missed something: whats the reason why importmulti requires a scriptPubKey https://github.com/bitcoin/bitcoin/pull/7551/files#diff-522490d83dce5375d423b23886e4125eR979?
< sipa> no objection to moving it to bitcoin-core bte
< wumpus> it isn't part of the bitcoin core release cycle either
< wumpus> I have some other scripts too, a few other code analysis tools, and to build release notes, author lists etc. All can be useful, but it's pretty much a meta-project to bitcoin itself.
< wumpus> should I move/clone https://github.com/laanwj/bitcoin-maintainer-tools into bitcoin-core?
< GitHub50> [bitcoin] laanwj closed pull request #8814: [wallet, policy] ParameterInteraction: Don't allow 0 fee (master...Mf1607-walletHighFeeWarn) https://github.com/bitcoin/bitcoin/pull/8814
< GitHub190> bitcoin/master 14b7b3f Wladimir J. van der Laan: Merge #8814: [wallet, policy] ParameterInteraction: Don't allow 0 fee...
< GitHub190> bitcoin/master fa4bfb4 MarcoFalke: [wallet, policy] ParameterInteraction: Don't allow 0 fee
< GitHub190> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/fc4f4547b7f6...14b7b3fb9f79

2016-09-27

< sipa> and changing it would introduce replay attacks to bitcoin
< sipa> bitcoin core uses some heuristics when parsing input for fundrawtransaction and decoderawtransaction to determine whether it's an incomplete non-segwit transaction instead
< GitHub86> [bitcoin] MarcoFalke closed pull request #8809: WIP: [qt] rework sync-overlay (master...Mf1609-qtSyncInf) https://github.com/bitcoin/bitcoin/pull/8809
< wumpus> cfields_: an example, let's say I'm cross-compiling for ARM. I have my own toolchain which I've built myself specifically for the device I'm cross-compiling to. But I use the depends to build bitcoin's dependencies.
< wumpus> cfields_: I think it makes sense, although it has to be deterministically reproducible over a longer time than typical bitcoin releases I guess
< d4de> I'm a bitcoin user... why, because I believe bitcoind should do one thing and one thing only instead of having the binary be bigger?
< d4de> wumpus, shouldn't the whole logic of exposing bitcoin as a hidden service be done outside of bitcoin?
< GitHub147> [bitcoin] laanwj closed pull request #8634: Add policy: null signature for failed CHECK(MULTI)SIG (master...nullfail) https://github.com/bitcoin/bitcoin/pull/8634
< GitHub3> bitcoin/master fc4f454 Wladimir J. van der Laan: Merge #8634: Add policy: null signature for failed CHECK(MULTI)SIG...
< GitHub3> bitcoin/master e41bd44 Johnson Lau: Add policy: null signature for failed CHECK(MULTI)SIG
< GitHub3> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/5a4f6d72e615...fc4f4547b7f6
< MarcoFalke> btcdrak: https://github.com/bitcoin-core/bitcoincore.org/pull/217 (website is broken)
< wumpus> only three 0.13.1-backport pulls open, we're getting there... https://github.com/bitcoin/bitcoin/pulls?q=is%3Aopen+is%3Apr+label%3A%22Needs+backport%22
< GitHub171> [bitcoin] laanwj closed pull request #8526: Make non-minimal OP_IF/NOTIF argument non-standard for P2WSH (master...minimalif) https://github.com/bitcoin/bitcoin/pull/8526
< GitHub181> bitcoin/master c72c5b1 Johnson Lau: Make non-minimal OP_IF/NOTIF argument non-standard for P2WSH
< GitHub181> bitcoin/master 5a4f6d7 Wladimir J. van der Laan: Merge #8526: Make non-minimal OP_IF/NOTIF argument non-standard for P2WSH...
< GitHub181> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/e9d5f6fec8fc...5a4f6d72e615
< GitHub169> [bitcoin] laanwj closed pull request #8357: [mempool] Fix relaypriority calculation error (master...maiiz-patch-1) https://github.com/bitcoin/bitcoin/pull/8357
< GitHub37> bitcoin/master 94a34a5 maiiz: Fix relaypriority calculation error
< GitHub37> bitcoin/master e9d5f6f Wladimir J. van der Laan: Merge #8357: [mempool] Fix relaypriority calculation error...
< GitHub37> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/14e8f9916beb...e9d5f6fec8fc
< GitHub171> [bitcoin] jnewbery opened pull request #8824: refactor TxToJSON() and ScriptPubKeyToJSON() (master...JSON_refactor) https://github.com/bitcoin/bitcoin/pull/8824
< GitHub61> [bitcoin] laanwj closed pull request #8810: tests: Add exception error message for JSONRPCException (master...2016_09_tests_rpc_error) https://github.com/bitcoin/bitcoin/pull/8810
< GitHub142> bitcoin/master 14e8f99 Wladimir J. van der Laan: Merge #8810: tests: Add exception error message for JSONRPCException...
< GitHub142> bitcoin/master 42f6aed Wladimir J. van der Laan: tests: Add exception error message for JSONRPCException...
< GitHub142> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/920ca1f0bf0b...14e8f9916beb
< GitHub65> bitcoin/0.13 1672225 Pieter Wuille: Do not store witness txn in rejection cache...
< GitHub65> bitcoin/0.13 b394a96 instagibbs: Add basic test for IsStandard witness transaction blinding...
< GitHub65> bitcoin/0.13 a5ec248 Johnson Lau: Remove createwitnessaddress...
< GitHub65> [bitcoin] laanwj pushed 12 new commits to 0.13: https://github.com/bitcoin/bitcoin/compare/254e990ce5c3...a916677ace1c
< GitHub7> [bitcoin] laanwj closed pull request #8815: Backports for 0.13.1 (0.13...2016_09_backports_0_13_1) https://github.com/bitcoin/bitcoin/pull/8815
< GitHub120> [bitcoin] laanwj opened pull request #8823: doc: Add privacy recommendation when running hidden service (master...2016_09_tor_recommendation) https://github.com/bitcoin/bitcoin/pull/8823
< GitHub29> [bitcoin] laanwj opened pull request #8822: net: Consistent checksum handling (master...2016_09_normalize_checksum_handling) https://github.com/bitcoin/bitcoin/pull/8822
< GitHub42> [bitcoin] laanwj closed pull request #8655: Do not shadow variables (trivials) (master...20160902_Wshadow_trivials) https://github.com/bitcoin/bitcoin/pull/8655
< GitHub49> bitcoin/master 920ca1f Wladimir J. van der Laan: Merge #8655: Do not shadow variables (trivials)...
< GitHub49> bitcoin/master 4731cab Pavel Janík: Do not shadow variables
< GitHub49> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/6e54c85439b6...920ca1f0bf0b
< GitHub65> [bitcoin] laanwj closed pull request #8803: Ping regularly in p2p-segwit.py to keep connection alive (master...patch-17) https://github.com/bitcoin/bitcoin/pull/8803
< GitHub136> bitcoin/master 6e54c85 Wladimir J. van der Laan: Merge #8803: Ping regularly in p2p-segwit.py to keep connection alive...
< GitHub136> bitcoin/master 0637b02 Johnson Lau: Ping regularly in p2p-segwit.py to keep connection alive...
< GitHub136> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/2f71490d2179...6e54c85439b6
< wumpus> always the highest total work, also this is a general #bitcoin question
< GitHub131> [bitcoin] MarcoFalke opened pull request #8821: [qt] sync-overlay: Don't block during reindex (master...Mf1609-qtSyncReindex) https://github.com/bitcoin/bitcoin/pull/8821
< luke-jr> what's holding back https://github.com/bitcoin/bitcoin/pull/8357 ?
< GitHub138> [bitcoin] fanquake opened pull request #8820: [depends] Fix Qt compilation with Xcode 8 (master...depends-qt-xcoderun) https://github.com/bitcoin/bitcoin/pull/8820
< GitHub23> [bitcoin] fanquake opened pull request #8819: [depends] Boost 1.61.0 (master...depends-boost-1-61-0) https://github.com/bitcoin/bitcoin/pull/8819

2016-09-26

< GitHub106> [bitcoin] jnewbery opened pull request #8817: update bitcoin-tx to output witness data (master...bitcoin-tx-witness) https://github.com/bitcoin/bitcoin/pull/8817
< GitHub129> [bitcoin] MarcoFalke closed pull request #8805: Trivial: Grammar and capitalization (master...master) https://github.com/bitcoin/bitcoin/pull/8805
< GitHub18> bitcoin/master c9ce17b Derek Miller: Trivial: Grammar and capitalization
< GitHub18> bitcoin/master 2f71490 MarcoFalke: Merge #8805: Trivial: Grammar and capitalization...
< GitHub18> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/8f1fbf36a769...2f71490d2179
< GitHub154> [bitcoin] czzarr closed pull request #8816: print P2WSH redeemScript in getrawtransaction if it s not a pubkey (master...print-p2wsh-redeemscript-in-getrawtransaction) https://github.com/bitcoin/bitcoin/pull/8816
< GitHub176> [bitcoin] czzarr opened pull request #8816: print P2WSH redeemScript in getrawtransaction if it s not a pubkey (master...print-p2wsh-redeemscript-in-getrawtransaction) https://github.com/bitcoin/bitcoin/pull/8816
< GitHub120> [bitcoin] laanwj opened pull request #8815: Backports for 0.13.1 (0.13...2016_09_backports_0_13_1) https://github.com/bitcoin/bitcoin/pull/8815
< MarcoFalke> wumpus: https://github.com/bitcoin/bitcoin/pull/8712 is a trivial cherry-pick, which was missed in ^
< GitHub162> bitcoin/0.13 3606b6b instagibbs: Update p2p-segwit.py to reflect correct AskFor behavior...
< GitHub162> bitcoin/0.13 733760a BtcDrak: Update btcdrak signing key...
< GitHub162> bitcoin/0.13 c6a6291 instagibbs: add witness address to address book...
< GitHub173> [bitcoin] laanwj closed pull request #8772: [0.13] Backports (0.13...backports-0.13) https://github.com/bitcoin/bitcoin/pull/8772
< GitHub162> [bitcoin] laanwj pushed 55 new commits to 0.13: https://github.com/bitcoin/bitcoin/compare/8d9e8adc05f4...254e990ce5c3
< GitHub85> [bitcoin] laanwj closed pull request #8796: [trivial] fix mempool comment (outdated by BIP125) (master...trivial_comment) https://github.com/bitcoin/bitcoin/pull/8796
< GitHub103> bitcoin/master c14ffd5 jonnynewbs: [trivial] fix mempool comment (outdated by BIP125)
< GitHub103> bitcoin/master 8f1fbf3 Wladimir J. van der Laan: Merge #8796: [trivial] fix mempool comment (outdated by BIP125)...
< GitHub103> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/dd20ed1223b9...8f1fbf36a769
< GitHub160> [bitcoin] laanwj closed pull request #8780: [rpc] Deprecate getinfo (master...Mf1609-getinfoDeprecate) https://github.com/bitcoin/bitcoin/pull/8780
< GitHub105> bitcoin/master dd20ed1 Wladimir J. van der Laan: Merge #8780: [rpc] Deprecate getinfo...
< GitHub105> bitcoin/master fa6e71b MarcoFalke: [qa] Add getinfo smoke tests and rework versionbits test
< GitHub105> bitcoin/master ddddaaf MarcoFalke: [rpc] Deprecate getinfo...
< GitHub105> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/bb843adc8d04...dd20ed1223b9
< GitHub192> [bitcoin] MarcoFalke opened pull request #8814: [wallet, policy] ParameterInteraction: Don't allow 0 fee (master...Mf1607-walletHighFeeWarn) https://github.com/bitcoin/bitcoin/pull/8814
< GitHub154> [bitcoin] laanwj closed pull request #8722: bitcoin-cli: More detailed error reporting (master...2016_09_cli_http_error) https://github.com/bitcoin/bitcoin/pull/8722
< GitHub87> bitcoin/master bb843ad Wladimir J. van der Laan: Merge #8722: bitcoin-cli: More detailed error reporting...
< GitHub87> bitcoin/master 381826d Wladimir J. van der Laan: bitcoin-cli: More detailed error reporting...
< GitHub87> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/ab0b411868e1...bb843adc8d04
< GitHub193> [bitcoin] laanwj closed pull request #7857: Add fee option to fundrawtransaction (master...enhancement/add-fee-to-fundrawtransaction) https://github.com/bitcoin/bitcoin/pull/7857
< MarcoFalke> Hmm, didn't pick it up: https://github.com/bitcoin-core/univalue/pull/3
< MarcoFalke> wumpus: https://travis-ci.org/bitcoin-core/univalue/settings. Is it enabled for pull requests?
< GitHub138> [bitcoin] laanwj closed pull request #8486: [wallet] Add high transaction fee warnings (master...Mf1607-walletHighFeeWarn) https://github.com/bitcoin/bitcoin/pull/8486
< GitHub167> bitcoin/master ab0b411 Wladimir J. van der Laan: Merge #8486: [wallet] Add high transaction fee warnings...
< GitHub167> bitcoin/master faef293 MarcoFalke: [wallet] Add high transaction fee warnings
< GitHub167> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/4e1567acff4b...ab0b411868e1
< GitHub135> [bitcoin] laanwj closed pull request #8249: Enable (and check for) 64-bit ASLR on Windows (master...2016_06_windows64_security) https://github.com/bitcoin/bitcoin/pull/8249
< GitHub61> bitcoin/master 4e1567a Wladimir J. van der Laan: Merge #8249: Enable (and check for) 64-bit ASLR on Windows...