2015-11-04

< GitHub44> [bitcoin] sipa opened pull request #6945: Update LevelDB tree to include #6917 (0.11) (0.11...leveldbfix0.11) https://github.com/bitcoin/bitcoin/pull/6945
< GitHub124> [bitcoin] sipa opened pull request #6944: Update LevelDB tree to include #6917 (master...leveldbfix0.12) https://github.com/bitcoin/bitcoin/pull/6944
< belcher> hmm, so the bitcoin 0.11 you can download from the website doesnt have it? but later commits on the github add it in
< GitHub46> [bitcoin] pstratem opened pull request #6943: [WIP] Wallet: Store hash for encrypted keys (master...ckey_hash) https://github.com/bitcoin/bitcoin/pull/6943
< gmaxwell> But we do not couple soft-forks to bitcoin core major releases.
< davec> I brought it up over in bitcoin-dev, but I'll put it here too for visibility. As of block 583930 on testnet, the CLTV soft fork is active although there is a lot of hash power still mining under the pre-CLTV rules.
< GitHub108> [bitcoin] peterjosling opened pull request #6942: Fix CCoins serialization documentation (master...docfix) https://github.com/bitcoin/bitcoin/pull/6942
< GitHub165> [bitcoin] jonathancross opened pull request #6940: Improving labels for Sent / Received "Bytes" (master...patch-1) https://github.com/bitcoin/bitcoin/pull/6940
< GitHub178> [bitcoin] laanwj closed pull request #6917: leveldb: Win32WritableFile without memory mapping (master...2015_10_leveldb_win_nomap) https://github.com/bitcoin/bitcoin/pull/6917
< GitHub40> [bitcoin] laanwj closed pull request #6938: build: If both Qt4 and Qt5 are installed, use Qt5 (master...2015_11_prefer_qt5) https://github.com/bitcoin/bitcoin/pull/6938
< GitHub145> bitcoin/master 193f7b5 Wladimir J. van der Laan: Merge pull request #6938...
< GitHub145> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/c702521a8564...193f7b553e0a
< GitHub145> bitcoin/master dbacc69 Wladimir J. van der Laan: build: If both Qt4 and Qt5 are installed, use Qt5...
< GitHub17> [bitcoin] MarcoFalke closed pull request #6939: [wallet] [qa] check MAX_STANDARD_TX_SIZE (master...MarcoFalke-2015-largeTx) https://github.com/bitcoin/bitcoin/pull/6939
< GitHub18> [bitcoin] laanwj closed pull request #6864: [qt] Use monospace font (master...MarcoFalke-2015-qtMonospace) https://github.com/bitcoin/bitcoin/pull/6864
< GitHub84> bitcoin/master 268b79e MarcoFalke: [qt] rpcconsole: Scale monospace font to 95%
< GitHub84> bitcoin/master c702521 Wladimir J. van der Laan: Merge pull request #6864...
< GitHub84> bitcoin/master 28313b8 MarcoFalke: [qt] Use fixed pitch font for the rpc console...
< GitHub84> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/8a95a18562b2...c702521a8564
< GitHub61> [bitcoin] MarcoFalke opened pull request #6939: [wallet] [qa] check MAX_STANDARD_TX_SIZE (master...MarcoFalke-2015-largeTx) https://github.com/bitcoin/bitcoin/pull/6939
< GitHub90> [bitcoin] laanwj closed pull request #6669: [wallet] Refactor to use new MIN_CHANGE (master...MarcoFalke-2015-walletCleanup) https://github.com/bitcoin/bitcoin/pull/6669
< GitHub172> bitcoin/master 6342a48 MarcoFalke: Init: Use DEFAULT_TRANSACTION_MINFEE in help message
< GitHub172> bitcoin/master 6b0e622 MarcoFalke: [wallet] Refactor to use new MIN_CHANGE...
< GitHub172> bitcoin/master a9c73a1 MarcoFalke: [wallet] Add comments for doxygen
< GitHub172> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/aa03fb35c4b3...8a95a18562b2
< MarcoFalke> wumpus, anything left to improve https://github.com/bitcoin/bitcoin/pull/6669 ?
< GitHub24> [bitcoin] laanwj closed pull request #6927: Fix ZMQ Notification initialization and shutdown (master...bugfix/zmq-initialization-shutdown) https://github.com/bitcoin/bitcoin/pull/6927
< GitHub11> bitcoin/master aa03fb3 Wladimir J. van der Laan: Merge pull request #6927...
< GitHub11> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/29c3c43e19ca...aa03fb35c4b3
< GitHub11> bitcoin/master de0499d João Barbosa: Fix ZMQ Notification initialization and shutdown...
< GitHub159> [bitcoin] laanwj closed pull request #6933: build: Improve build instructions (master...2015_10_modernize_linux_build_instructions) https://github.com/bitcoin/bitcoin/pull/6933
< GitHub113> bitcoin/master 29c3c43 Wladimir J. van der Laan: Merge pull request #6933...
< GitHub113> bitcoin/master 35bb381 Wladimir J. van der Laan: build: Improve build instructions...
< GitHub113> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/aca0c00ae1fc...29c3c43e19ca
< GitHub120> [bitcoin] laanwj opened pull request #6938: build: If both Qt4 and Qt5 are installed, use Qt5 (master...2015_11_prefer_qt5) https://github.com/bitcoin/bitcoin/pull/6938
< GitHub126> [bitcoin] laanwj closed pull request #6905: Use constants and minor fixes by luke-jr (master...lukejr-constants-no-mergeConf) https://github.com/bitcoin/bitcoin/pull/6905
< GitHub158> bitcoin/master 5f9260f Luke Dashjr: Bugfix: If genproclimit is omitted to RPC setgenerate, don't change it; also show correct default in getmininginfo
< GitHub158> bitcoin/master 420a82f Luke Dashjr: Bugfix: Describe dblogsize option correctly (it refers to the wallet database, not memory pool)
< GitHub158> bitcoin/master caa3d42 Luke Dashjr: Bugfix: RPC: blockchain: Display correct defaults in help for verifychain method
< GitHub158> [bitcoin] laanwj pushed 5 new commits to master: https://github.com/bitcoin/bitcoin/compare/42f339ef780b...aca0c00ae1fc
< jonasschnelli> wumpus: I'm using default values... just double-clicked bitcoin-qt.exe
< jonasschnelli> wumpus: regarding https://github.com/bitcoin/bitcoin/pull/6917#issuecomment-153667846 ... do you thing a sudden VMWare shutdown will emulate the problem realistic enought?
< midnightmagic> You *may* wish to temporarily /mode * +b $j:#bitcoin-global-bans while we track that guy.
< GitHub186> [bitcoin] arowser opened pull request #6937: Fix Boots 1.58.0 build for mips arch (master...mips-options-fix) https://github.com/bitcoin/bitcoin/pull/6937
< GitHub123> [bitcoin] gmaxwell closed pull request #6935: 0.10 (master...0.10) https://github.com/bitcoin/bitcoin/pull/6935

2015-11-03

< GitHub187> [bitcoin] morcos opened pull request #6936: [WIP] Keep pcoinsTip cache warm (master...HotCache) https://github.com/bitcoin/bitcoin/pull/6936
< Luke-Jr> JoyStick: lack of answers in #bitcoin is not a reason to ask support questions here.
< Luke-Jr> JoyStick: rule 1) never ask to ask, just ask; rule 2) if it isn't *development-related*, ask in #bitcoin
< GitHub139> [bitcoin] SegfredB opened pull request #6935: 0.10 (master...0.10) https://github.com/bitcoin/bitcoin/pull/6935
< GitHub128> [bitcoin] gmaxwell opened pull request #6934: Restores mempool only BIP113 enforces (master...undo_undo) https://github.com/bitcoin/bitcoin/pull/6934
< GitHub39> [bitcoin] jtimon closed pull request #6068: Policy: Create CPolicy interface and CStandardPolicy class implementing it (master...policy_new) https://github.com/bitcoin/bitcoin/pull/6068
< wumpus> nice, bitcoin-qt works as-is in QtWayland in Weston w/ no X server. Prepared for the future :)
< GitHub197> [bitcoin] laanwj opened pull request #6933: build: Improve build instructions (master...2015_10_modernize_linux_build_instructions) https://github.com/bitcoin/bitcoin/pull/6933
< GitHub131> [bitcoin] laanwj closed pull request #6930: build: don't distribute tests_config.py (master...2015_11_testsconfig_nodist) https://github.com/bitcoin/bitcoin/pull/6930
< GitHub83> bitcoin/master 42f339e Wladimir J. van der Laan: Merge pull request #6930...
< GitHub83> bitcoin/master ff2a2af Wladimir J. van der Laan: build: don't distribute tests_config.py...
< GitHub83> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/8fe30fb4d130...42f339ef780b
< GitHub1> [bitcoin] morcos opened pull request #6932: ModifyNewCoins saves database lookups (master...newCoinsThinAir) https://github.com/bitcoin/bitcoin/pull/6932
< GitHub30> [bitcoin] morcos opened pull request #6931: Skip BIP 30 verification where not necessary (master...skipBIP30) https://github.com/bitcoin/bitcoin/pull/6931

2015-11-02

< GitHub153> [bitcoin] laanwj reopened pull request #6248: Fix Qt build on arch by setting -fPIC (master...archbuild) https://github.com/bitcoin/bitcoin/pull/6248
< GitHub48> [bitcoin] laanwj closed pull request #6928: Revert MedianTimePast / mempool-only BIP113 (master...undo_113) https://github.com/bitcoin/bitcoin/pull/6928
< GitHub99> bitcoin/master 8fe30fb Wladimir J. van der Laan: Merge pull request #6928...
< GitHub99> bitcoin/master 40cd32e Gregory Maxwell: Revert "Add rules--presently disabled--for using GetMedianTimePast as endpoint for lock-time calculations"...
< GitHub99> bitcoin/master 8537ecd Gregory Maxwell: Revert "Enable policy enforcing GetMedianTimePast as the end point of lock-time constraints"...
< GitHub99> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/a6d0d623fc95...8fe30fb4d130
< GitHub87> [bitcoin] laanwj closed pull request #6902: policy: Add new constant MAX_STANDARD_MULTISIG_KEYS (master...multisig_keys) https://github.com/bitcoin/bitcoin/pull/6902
< GitHub198> [bitcoin] laanwj closed pull request #6839: clang-format: Recently added files (master...MarcoFalke-2015-clangFormatRecent) https://github.com/bitcoin/bitcoin/pull/6839
< GitHub122> [bitcoin] laanwj closed pull request #6926: tests: Initialize networking on windows (master...2015_10_tests_networking_init) https://github.com/bitcoin/bitcoin/pull/6926
< GitHub27> bitcoin/master a6d0d62 Wladimir J. van der Laan: Merge pull request #6926...
< GitHub27> bitcoin/master 7497e80 Wladimir J. van der Laan: tests: Initialize networking on windows
< GitHub27> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/02a95be97759...a6d0d623fc95
< GitHub164> [bitcoin] laanwj closed pull request #6925: Added zmq flag for owned transactions (master...master) https://github.com/bitcoin/bitcoin/pull/6925
< GitHub36> [bitcoin] laanwj opened pull request #6930: build: don't distribute tests_config.py (master...2015_11_testsconfig_nodist) https://github.com/bitcoin/bitcoin/pull/6930

2015-11-01

< Eliel_> http://pastebin.com/Phkhh128 (yes, the Invalid Bitcoin Address thing is intentional)
< Eliel_> bitcoin-cli help sendmany
< Eliel_> It looks like the curl example in the help text you get with "bitcoin-cli help" does not work as is. It has the address list doubly JSON encoded while bitcoind doesn't expect two layers of JSON as input.
< GitHub136> [bitcoin] gmaxwell opened pull request #6928: Revert MedianTimePast / mempool-only BIP113 (master...undo_113) https://github.com/bitcoin/bitcoin/pull/6928
< GitHub127> [bitcoin] promag opened pull request #6927: Fix ZMQ Notification initialization and shutdown (master...bugfix/zmq-initialization-shutdown) https://github.com/bitcoin/bitcoin/pull/6927
< GitHub110> bitcoin/0.10 72a0adf Wladimir J. van der Laan: qt: Final translations update on 0.10 branch...
< GitHub110> [bitcoin] laanwj pushed 1 new commit to 0.10: https://github.com/bitcoin/bitcoin/commit/72a0adfb3c364173d28bab03bf46349ab696c107
< GitHub96> bitcoin/master 02a95be Wladimir J. van der Laan: qt: translation update prior to opening 0.12 translations...
< GitHub96> [bitcoin] laanwj pushed 1 new commit to master: https://github.com/bitcoin/bitcoin/commit/02a95be97759e70d9e7537775c95eec219c8333e
< GitHub76> [bitcoin] laanwj opened pull request #6926: tests: Initialize networking on windows (master...2015_10_tests_networking_init) https://github.com/bitcoin/bitcoin/pull/6926

2015-10-31

< GitHub162> [bitcoin] BTCDDev opened pull request #6925: Added zmq flag for owned transactions (master...master) https://github.com/bitcoin/bitcoin/pull/6925
< GitHub0> [bitcoin] Diapolo closed pull request #5466: [Qt] log cert errors for payment requests and harden test-cases (master...pr_harden_certchecks) https://github.com/bitcoin/bitcoin/pull/5466
< GitHub180> [bitcoin] Diapolo closed pull request #6309: [Qt] simplify code for third party TX urls handling (master...txurls) https://github.com/bitcoin/bitcoin/pull/6309
< GitHub100> [bitcoin] Diapolo closed pull request #6371: banlist updates (master...banlist) https://github.com/bitcoin/bitcoin/pull/6371
< GitHub49> [bitcoin] Diapolo closed pull request #6710: [Qt] polish new ban ui from #6315 (master...polish_ban_ui) https://github.com/bitcoin/bitcoin/pull/6710
< GitHub161> [bitcoin] Diapolo closed pull request #6922: minor updates/changes in txmempool.h/.cpp (master...txmempool) https://github.com/bitcoin/bitcoin/pull/6922
< GitHub197> [bitcoin] laanwj closed pull request #6920: remove unneded include from rpcserver.cpp and rpcclient.cpp (master...univalue) https://github.com/bitcoin/bitcoin/pull/6920
< GitHub125> [bitcoin] laanwj closed pull request #6921: [Trivial] remove unneeded spaces from init.cpp (master...trivial_typo_space) https://github.com/bitcoin/bitcoin/pull/6921
< GitHub104> [bitcoin] laanwj closed pull request #6883: Add BIP65 CHECKLOCKTIMEVERIFY to release notes (master...cltv-release-notes-v0.12.0) https://github.com/bitcoin/bitcoin/pull/6883
< GitHub88> bitcoin/master a6e80e4 Wladimir J. van der Laan: Merge pull request #6883...
< GitHub88> bitcoin/master c939792 Peter Todd: Add BIP65 CHECKLOCKTIMEVERIFY to release notes
< GitHub88> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/48b5b84ee511...a6e80e401716
< wumpus> I don't think so (you may be better off asking in #bitcoin though, apart from the counting methodology this is not essentially dev related)
< wumpus> "strip: unable to rename 'bitcoin-qt.exe'; reason: File exists"
< GitHub10> [bitcoin] Diapolo opened pull request #6922: minor updates/changes in txmempool.h/.cpp (master...txmempool) https://github.com/bitcoin/bitcoin/pull/6922
< wumpus> (and the Sync requires an ugly construct to unwrap the file handle then, as we do here https://github.com/bitcoin/bitcoin/blob/master/src/util.cpp#L608 )
< GitHub69> [bitcoin] Diapolo opened pull request #6921: [Trivial] remove unneeded spaces from init.cpp (master...trivial_typo_space) https://github.com/bitcoin/bitcoin/pull/6921
< GitHub48> [bitcoin] Diapolo opened pull request #6920: remove unneded include from rpcserver.cpp and rpcclient.cpp (master...univalue) https://github.com/bitcoin/bitcoin/pull/6920
< GitHub195> [bitcoin] laanwj opened pull request #6919: Backport chainstate obfuscation to 0.11 (0.11...2015_10_backport_chainstate_obfuscation) https://github.com/bitcoin/bitcoin/pull/6919
< belcher> oh damn this channel, didnt realise, ask in #bitcoin
< diegoviola> will bitcoin core implement SPV like electrum and seeds?

2015-10-30

< GitHub146> [bitcoin] laanwj closed pull request #6906: Reject invalid pubkeys when reading ckey items from the wallet. (master...ckey_pubkey_check) https://github.com/bitcoin/bitcoin/pull/6906
< GitHub164> bitcoin/master 48b5b84 Wladimir J. van der Laan: Merge pull request #6906...
< GitHub164> bitcoin/master 30d9662 Gregory Maxwell: Reject invalid pubkeys when reading ckey items from the wallet....
< GitHub164> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/d482c0a7b246...48b5b84ee511
< GitHub192> [bitcoin] laanwj closed pull request #6896: Make -checkmempool=1 not fail through int32 overflow (master...fixchainsize) https://github.com/bitcoin/bitcoin/pull/6896
< GitHub43> bitcoin/master d482c0a Wladimir J. van der Laan: Merge pull request #6896...
< GitHub43> bitcoin/master e9e6163 Pieter Wuille: Make -checkmempool=1 not fail through int32 overflow
< GitHub43> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/725539ea0376...d482c0a7b246
< GitHub83> [bitcoin] laanwj closed pull request #6893: forward-declare univalue in rpcclient + remove include in rpcserver.cpp (master...univalue) https://github.com/bitcoin/bitcoin/pull/6893
< GitHub91> [bitcoin] sipa opened pull request #6918: Make sigcache faster, more efficient, larger (master...smallsigcache) https://github.com/bitcoin/bitcoin/pull/6918
< gmaxwell> I expect most of the work related to my NBD suggestion will be discovering that all filesystems are broken, and being unable to get results against bitcoin until the file systems are fixed. :(
< GitHub21> [bitcoin] laanwj opened pull request #6917: leveldb: Win32WritableFile without memory mapping (master...2015_10_leveldb_win_nomap) https://github.com/bitcoin/bitcoin/pull/6917
< wumpus> replaced the leveldb win32 writable file, let's see how it goes https://github.com/laanwj/bitcoin/commit/86050deaabe7ca0a19db1c5a65e3174c9cb14511
< GitHub118> [bitcoin] sipa closed pull request #6916: Remove BIP30 enforcement, as it is impossible to trigger since BIP34 (master...nobip30) https://github.com/bitcoin/bitcoin/pull/6916
< GitHub92> [bitcoin] sipa opened pull request #6916: Remove BIP30 enforcement, as it is impossible to trigger since BIP34 (master...nobip30) https://github.com/bitcoin/bitcoin/pull/6916
< wumpus> https://github.com/bitcoin/bitcoin/blob/master/src/leveldb/util/env_win.cc#L587 shouldn't the FlushFuleBuffers and FlushViewOfFile be the other way around?
< wumpus> and not sure bitcoin core should be resilient to motherboard-bends-while-application-is-running crashes :-)
< wumpus> cfields: what would be the best way to include the windows determinism postprocessing in the gitian build (in https://github.com/bitcoin/bitcoin/pull/6900) - call it from the descriptor or 'make'?
< GitHub158> [bitcoin] laanwj closed pull request #6913: 0.11 (master...0.11) https://github.com/bitcoin/bitcoin/pull/6913
< dcousens> sipa: conceptually, how does https://github.com/bitcoin/bitcoin/pull/6914 save memory, except that in some cases it is allocated on the stack?
< GitHub7> [bitcoin] sdaftuar opened pull request #6915: [Mempool] Improve removal of invalid transactions after reorgs (master...fix-reorg-handling) https://github.com/bitcoin/bitcoin/pull/6915
< GitHub139> [bitcoin] sipa opened pull request #6914: [WIP] Add pre-allocated vector type and use it for CScript (master...prevector) https://github.com/bitcoin/bitcoin/pull/6914
< GitHub45> [bitcoin] nathanimpact opened pull request #6913: 0.11 (master...0.11) https://github.com/bitcoin/bitcoin/pull/6913
< GitHub198> [bitcoin] gmaxwell closed pull request #6912: 0.11 (master...0.11) https://github.com/bitcoin/bitcoin/pull/6912
< GitHub93> [bitcoin] gmaxwell closed pull request #6911: 0.10 (master...0.10) https://github.com/bitcoin/bitcoin/pull/6911
< GitHub69> [bitcoin] nathanimpact opened pull request #6912: 0.11 (master...0.11) https://github.com/bitcoin/bitcoin/pull/6912
< GitHub112> [bitcoin] gmaxwell closed pull request #6910: 0.9 (master...0.9) https://github.com/bitcoin/bitcoin/pull/6910
< GitHub40> [bitcoin] nathanimpact opened pull request #6911: 0.10 (master...0.10) https://github.com/bitcoin/bitcoin/pull/6911
< GitHub190> [bitcoin] nathanimpact opened pull request #6910: 0.9 (master...0.9) https://github.com/bitcoin/bitcoin/pull/6910
< GitHub198> [bitcoin] nathanimpact opened pull request #6909: 0.8 (master...0.8) https://github.com/bitcoin/bitcoin/pull/6909
< GitHub180> [bitcoin] gmaxwell closed pull request #6909: 0.8 (master...0.8) https://github.com/bitcoin/bitcoin/pull/6909
< gmaxwell> Seperate subject; https://github.com/bitcoin/bitcoin/pull/6902 I feel that changes like this make the software more opaque. Now to understand whats being done there you must read two places instead of one. If you only look at policy.h you will likely erroniously believe it applies to p2sh somehow.
< gmaxwell> warren: it's all ugly, bitcoin core is currently not intended for system level operation.
< gmaxwell> warren: what happens if a second datadir is specified to bitcoin-cli?
< warren> Would it be bad if the system-level systemd service were named something like bitcoinsys.service, with a selinux context bitcoinsys_t, and a bitcoin-cli wrapper named something like /usr/bin/bitcoinsys_cli that only adds the --datadir /path/to/system/datadir/
< gmaxwell> warren: as far as selinux policy, I think it's unfortunate that you cannot confine bitcoin core to have no access outside of its own directory.
< gmaxwell> warren: this channel should be for bitcoin core exclusive discussion.

2015-10-29

< gmaxwell> Can someone build me windows binaries for https://github.com/bitcoin/bitcoin/pull/6906 ? (oh I miss the testers that produced binaries as a side effect...)
< GitHub7> [bitcoin] MarcoFalke reopened pull request #6905: Use constants and minor fixes by luke-jr (master...lukejr-constants-no-mergeConf) https://github.com/bitcoin/bitcoin/pull/6905
< GitHub107> [bitcoin] jtimon opened 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
< GitHub71> [bitcoin] jtimon opened pull request #6907: Chainparams: Use a regular factory for creating chainparams (master...chainparams-factory-0.12.99) https://github.com/bitcoin/bitcoin/pull/6907
< wumpus> right it's the minimum *per KB*, all fees in bitcoin core are per kB
< sipa> but it was somehow part of the wider bitcoin irc meeting, which took place in #bitcoin-dev an hour ago
< wumpus> this channel has a very narrow scope: code changes to the bitcoin core project
< MarcoFalke> Do those channels have different scopes? I remember bitcoin-dev is deprecated?
< jgarzik> already covered in #bitcoin-dev meeting
< GitHub171> [bitcoin] MarcoFalke closed pull request #6905: Use constants and minor fixes by luke-jr (master...lukejr-constants-no-mergeConf) https://github.com/bitcoin/bitcoin/pull/6905
< GitHub81> [bitcoin] gmaxwell opened pull request #6906: Reject invalid pubkeys when reading ckey items from the wallet. (master...ckey_pubkey_check) https://github.com/bitcoin/bitcoin/pull/6906
< GitHub129> [bitcoin] MarcoFalke opened pull request #6905: Use constants and minor fixes by luke-jr (master...lukejr-constants-no-mergeConf) https://github.com/bitcoin/bitcoin/pull/6905
< GitHub5> [bitcoin] LordOfTheCoin closed pull request #6904: LevelDB Fixes and Enhancements (master...2015_sqlite) https://github.com/bitcoin/bitcoin/pull/6904
< GitHub8> [bitcoin] LordOfTheCoin reopened pull request #6904: LevelDB Fixes and Enhancements (master...2015_sqlite) https://github.com/bitcoin/bitcoin/pull/6904
< GitHub101> [bitcoin] LordOfTheCoin closed pull request #6904: LevelDB Fixes and Enhancements (master...2015_sqlite) https://github.com/bitcoin/bitcoin/pull/6904
< GitHub195> [bitcoin] LordOfTheCoin opened pull request #6904: LevelDB Fixes and Enhancements (master...2015_sqlite) https://github.com/bitcoin/bitcoin/pull/6904
< mcelrath> Hmm I can't assign this to myself: https://github.com/bitcoin/bitcoin/issues/6903
< GitHub33> [bitcoin] dajohi opened pull request #6902: policy: Add new constant MAX_STANDARD_MULTISIG_KEYS (master...multisig_keys) https://github.com/bitcoin/bitcoin/pull/6902
< wumpus> majority plays no role in bitcoin - if *your* node forks from the block chain, that's a risk to you
< mcelrath> Is bitcoin anymore consensus-dependent on leveldb? If we swap it out with sqlite or lmdb, will it still be consensus critical? (and can it be made non-consensus critical?)
< dcousens> I can put bitcoin-qt on a old gaming rig if necessary, have to dust it off a little but, if its worth it
< GitHub146> [bitcoin] laanwj closed pull request #6863: [Test Suite] Fix test for null tx input (master...null-txin-test) https://github.com/bitcoin/bitcoin/pull/6863
< GitHub189> bitcoin/master 725539e Wladimir J. van der Laan: Merge pull request #6863...
< GitHub189> bitcoin/master 0be387a Daniel Kraft: unittest: fix test for null tx input...
< GitHub189> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b28c229324d0...725539ea0376
< GitHub144> [bitcoin] laanwj closed pull request #6899: Add explicit shared_ptr constructor due to C++11 error (master...cpp11) https://github.com/bitcoin/bitcoin/pull/6899
< GitHub57> bitcoin/master b28c229 Wladimir J. van der Laan: Merge pull request #6899...
< GitHub57> bitcoin/master a83f3c2 Bob McElrath: Add explicit shared_ptr constructor due to C++11 error
< GitHub57> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b2ce2c1f0fb1...b28c229324d0
< GitHub152> [bitcoin] laanwj closed pull request #6870: [trivial] Misc cleanup and translations (master...MarcoFalke-2015-trivial3) https://github.com/bitcoin/bitcoin/pull/6870
< GitHub178> bitcoin/master 6782f58 MarcoFalke: [trivial] Latest config.guess...
< GitHub178> bitcoin/master bf68191 MarcoFalke: [trivial] rpcnet: fix typo
< GitHub178> bitcoin/master 95f4291 MarcoFalke: [trivial] Rewrite help text for feature enabled by default...
< GitHub178> [bitcoin] laanwj pushed 5 new commits to master: https://github.com/bitcoin/bitcoin/compare/26752767df45...b2ce2c1f0fb1
< GitHub96> [bitcoin] laanwj closed pull request #6894: [Tests] Fix BIP65 p2p test (master...fix-bip65-p2p-test) https://github.com/bitcoin/bitcoin/pull/6894
< GitHub111> bitcoin/master 2675276 Wladimir J. van der Laan: Merge pull request #6894...
< GitHub111> bitcoin/master 3e187f2 Suhas Daftuar: Fix BIP65 p2p test...
< GitHub111> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/8daffe227bc6...26752767df45
< GitHub195> [bitcoin] laanwj closed pull request #6781: [QT] pretty print (indent) multiline html output (master...2015/10/qt_rpcconsole_pp) https://github.com/bitcoin/bitcoin/pull/6781
< GitHub14> [bitcoin] laanwj opened pull request #6900: gitian: build on ubuntu 14.04 (master...2015_10_gitian_trusty) https://github.com/bitcoin/bitcoin/pull/6900
< CodeShark> I don't really fricking care, personally - I don't really run a bunch of bitcoin core instances on windows - I'm just repeating what I've heard from others
< wumpus> I don't want to hear "leveldb is broken on windows" every day, this is not tea time with bitcoin-dev, we should be constructive here
< Luke-Jr> dcousens: #bitcoin maybe?
< bsm117532> Quick git question...I did a push -f to overwrite my previous commit, because I like having clean histories. I could have also made a second commit on this PR. Does anyone care? Would github have squashed them anyway? (Re: https://github.com/bitcoin/bitcoin/pull/6899 )
< bsm117532> That's a very hard problem that I don't think bitcoin can solve by choice of db.
< sipa> bsm117532: jeff has a branch with bitcoin core running on sqlite3
< sipa> if we need to change something to our leveldb tree, the correct way is submit it as a PR to the bitcoin/leveldb repo, and then bitcoin core can update to a new version
< bsm117532> AFAICT, when I clone bitcoin, I don't get the bitcoin/leveldb repo. I get bitcoin/src/leveldb which is entirely separate. Am I wrong about that?
< bsm117532> FWIW I'd have put it in a separate repo under bitcoin, rather than import the code directly.
< gmaxwell> aj: rethinking this resulted in coming up with the seggregated witness approach thats in elements alpha, and which may be possible to soft-fork into bitcoin.
< gmaxwell> aj: well they're one in the same in Bitcoin Core; these changes are accomplished via validation flags. To make them non-standard we add the new restrictions to the set of flags used to verify transactions going into the mempool.

2015-10-28

< GitHub107> [bitcoin] mcelrath opened pull request #6899: Warnings clean with C++11 (master...cpp11) https://github.com/bitcoin/bitcoin/pull/6899
< GitHub181> [bitcoin] morcos opened pull request #6898: [WIP] Rewrite CreateNewBlock (master...fasterCNB) https://github.com/bitcoin/bitcoin/pull/6898
< gmaxwell> Most of the delays in mining right now appear to be from outside of bitcoin core, actually.
< GitHub4> [bitcoin] sipa opened pull request #6896: Make -checkmempool=1 not fail through int32 overflow (master...fixchainsize) https://github.com/bitcoin/bitcoin/pull/6896
< GitHub1> bitcoin/master 8daffe2 Pieter Wuille: Merge pull request #6891...
< GitHub130> [bitcoin] sipa closed pull request #6891: constify missing catch cases (master...const-ex) https://github.com/bitcoin/bitcoin/pull/6891
< GitHub1> bitcoin/master ad5aae1 Philip Kaufmann: constify missing catch cases...
< GitHub1> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/4764f5db9d2c...8daffe227bc6
< GitHub110> [bitcoin] sipa closed pull request #6892: [Trivial] ensure minimal header conventions (master...headers-new) https://github.com/bitcoin/bitcoin/pull/6892
< GitHub67> bitcoin/master 4764f5d Pieter Wuille: Merge pull request #6892...
< GitHub67> bitcoin/master 214de7e Philip Kaufmann: [Trivial] ensure minimal header conventions...
< GitHub67> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/e06c14fb59ee...4764f5db9d2c
< GitHub73> [bitcoin] sipa closed pull request #6776: Support -checkmempool=N, which runs checks once every N transactions (master...fraccheck) https://github.com/bitcoin/bitcoin/pull/6776
< GitHub93> bitcoin/master e06c14f Pieter Wuille: Merge pull request #6776...
< GitHub93> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/8756c986420c...e06c14fb59ee
< GitHub93> bitcoin/master ab1f560 Pieter Wuille: Support -checkmempool=N, which runs checks on average once every N transactions
< GitHub179> [bitcoin] sipa closed pull request #6895: Update to my new key (master...newkey) https://github.com/bitcoin/bitcoin/pull/6895
< GitHub157> bitcoin/master 8756c98 Pieter Wuille: Merge pull request #6895...
< GitHub157> bitcoin/master 4252cd0 Pieter Wuille: Update to my new key
< GitHub157> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/93521a4f56ce...8756c986420c
< GitHub113> [bitcoin] sipa closed pull request #6875: Fix pre-push-hook regexes (master...verify-commits-fixes) https://github.com/bitcoin/bitcoin/pull/6875
< GitHub43> bitcoin/master 6e800c2 Matt Corallo: Add Pieter's new PGP key to verify-commits/trusted-keys
< GitHub43> bitcoin/master 1d94b72 Matt Corallo: Whitelist commits signed with Pieter's now-revoked key
< GitHub43> bitcoin/master 27252b7 Matt Corallo: Fix pre-push-hook regexes
< GitHub43> [bitcoin] sipa pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/d0badb916e51...93521a4f56ce
< GitHub22> [bitcoin] gmaxwell closed pull request #6865: Fix chainstate serialized_size computation (master...fixchainsize) https://github.com/bitcoin/bitcoin/pull/6865
< GitHub19> bitcoin/master d0badb9 Gregory Maxwell: Merge pull request #6865...
< GitHub19> bitcoin/master 298e040 Pieter Wuille: Fix chainstate serialized_size computation
< GitHub19> [bitcoin] gmaxwell pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/8f3b3cdee497...d0badb916e51
< GitHub30> [bitcoin] sipa opened pull request #6895: Update to my new key (master...newkey) https://github.com/bitcoin/bitcoin/pull/6895

2015-10-27

< GitHub51> [bitcoin] sdaftuar opened pull request #6894: [Tests] Fix BIP65 p2p test (master...fix-bip65-p2p-test) https://github.com/bitcoin/bitcoin/pull/6894
< GitHub163> [bitcoin] giacecco closed pull request #6885: Instructions on how to make the Homebrew OpenSSL headers visible... (master...master) https://github.com/bitcoin/bitcoin/pull/6885
< btcdrak> Trolling for review: https://github.com/bitcoin/bitcoin/pull/6312
< GitHub176> [bitcoin] Diapolo opened pull request #6893: forward-declare univalue in rpcclient + remove include in rpcserver.cpp (master...univalue) https://github.com/bitcoin/bitcoin/pull/6893
< GitHub81> [bitcoin] Diapolo opened pull request #6892: [Trivial] ensure minimal header conventions (master...headers-new) https://github.com/bitcoin/bitcoin/pull/6892
< GitHub34> [bitcoin] Diapolo opened pull request #6891: constify missing catch cases (master...const-ex) https://github.com/bitcoin/bitcoin/pull/6891
< wumpus> for more abstract things, nitty gritty implementation details for bitcoin core belong here
< bsm1175321> Can discuss c++11 here if you prefer. But then what's #bitcoin-dev for?
< wumpus> bitcoin script, hah, even better
< sipa> we should use bitcoin script with the op_x86 extension
< GitHub144> [bitcoin] laanwj closed pull request #6464: Always clean up manual transaction prioritization (master...manual-tx-prioritization-cleanup) https://github.com/bitcoin/bitcoin/pull/6464
< GitHub78> bitcoin/master 8f3b3cd Wladimir J. van der Laan: Merge pull request #6464...
< GitHub78> bitcoin/master 2d8c49d Casey Rodarmor: Clean up tx prioritization when conflict mined
< GitHub78> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/38369dda325d...8f3b3cdee497
< GitHub182> [bitcoin] paveljanik closed pull request #5525: Add removeaddress RPC call (remove watch-only address) (master...rpc_removeaddress) https://github.com/bitcoin/bitcoin/pull/5525
< GitHub109> [bitcoin] laanwj closed pull request #6889: fix locking issue with new mempool limiting (master...2015/10/fix_mempool_lock) https://github.com/bitcoin/bitcoin/pull/6889
< GitHub175> bitcoin/master 38369dd Wladimir J. van der Laan: Merge pull request #6889...
< GitHub175> bitcoin/master 0d699fc Jonas Schnelli: fix locking issue with new mempool limiting...
< GitHub175> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/2b625510d374...38369dda325d
< wumpus> 2GB should certainly be enough. See e.g. https://github.com/bitcoin/bitcoin/issues/6658 , worst contender main.cpp uses 1.2MB while compiling (on gcc 4.8). Adding swap may help though, linux' memory management works better if swap is enabled even if you don't need the extra memory

2015-10-26

< GitHub108> [bitcoin] laanwj closed pull request #6881: Debug: Add option for microsecond precision in debug.log (master...add-microsecond-timestamps) https://github.com/bitcoin/bitcoin/pull/6881
< GitHub138> bitcoin/master 7bbc7c3 Suhas Daftuar: Add option for microsecond precision in debug.log
< GitHub138> bitcoin/master 2b62551 Wladimir J. van der Laan: Merge pull request #6881...
< GitHub138> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/7939164d8985...2b625510d374
< GitHub11> bitcoin/master 7939164 Wladimir J. van der Laan: Merge pull request #6622...
< GitHub11> bitcoin/master 872fee3 Jonas Schnelli: Introduce -maxuploadtarget...
< GitHub11> bitcoin/master 17a073a Suhas Daftuar: Add RPC test for -maxuploadtarget
< GitHub69> [bitcoin] laanwj closed pull request #6622: Introduce -maxuploadtarget (master...2015/09/maxuploadtarget) https://github.com/bitcoin/bitcoin/pull/6622
< GitHub11> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/dbc5ee821ecd...7939164d8985
< GitHub118> [bitcoin] laanwj closed pull request #6621: added org.bitcoin.bitcoind.plist for launchd (OS X) (master...master) https://github.com/bitcoin/bitcoin/pull/6621
< GitHub16> bitcoin/master e04b0b6 Kevin Cooper: added OS X documentation to doc/init.md
< GitHub16> bitcoin/master dbc5ee8 Wladimir J. van der Laan: Merge pull request #6621...
< GitHub16> bitcoin/master d4aa54c Kevin Cooper: added org.bitcoin.bitcoind.plist for launchd (OS X)
< GitHub16> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/c8322ff7f754...dbc5ee821ecd
< GitHub96> [bitcoin] laanwj closed pull request #6888: Clear strMiscWarning before running PartitionAlert (master...alert_tests) https://github.com/bitcoin/bitcoin/pull/6888
< GitHub52> bitcoin/master c8322ff Wladimir J. van der Laan: Merge pull request #6888...
< GitHub52> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/ff057f41aa14...c8322ff7f754
< GitHub52> bitcoin/master 143d173 Eric Lombrozo: Use BOOST_CHECK_MESSAGE() rather than BOOST_CHECK() in alerts_tests.cpp and initialize strMiscWarning before calling PartitionCheck()."
< gmaxwell> (a failure to make this optimization is part of what contributes to miners mining on other miners work without validating-- they go and 'optimize' at a layer higher than Bitcoin, and as a resut complex details like validation go out the window. :) )
< GitHub175> [bitcoin] jonasschnelli opened pull request #6889: fix locking issue with new mempool limiting (master...2015/10/fix_mempool_lock) https://github.com/bitcoin/bitcoin/pull/6889
< wumpus> this is curious, anyone else having problems with address parsing on windows 10? https://github.com/bitcoin/bitcoin/issues/6886
< GitHub161> [bitcoin] laanwj closed pull request #6566: BIP-113: Mempool-only median time-past as endpoint for lock-time calculations (master...medianpasttimelock) https://github.com/bitcoin/bitcoin/pull/6566
< GitHub168> bitcoin/master ff057f4 Wladimir J. van der Laan: Merge pull request #6566...
< GitHub168> bitcoin/master dea8d21 Mark Friedenbach: Enable policy enforcing GetMedianTimePast as the end point of lock-time constraints...
< GitHub168> bitcoin/master 9d55050 Mark Friedenbach: Add rules--presently disabled--for using GetMedianTimePast as endpoint for lock-time calculations...
< GitHub168> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/26f5b34e8838...ff057f41aa14
< jonasschnelli> wumpus: https://github.com/bitcoin/bitcoin/commit/a09297010e171af28a7a3fcad65a4e0aefea53ba seams to break things.. how can this even be possible...
< jonasschnelli> wumpus: i can successfully build up to https://github.com/bitcoin/bitcoin/commit/579b863cd7586b98974484ad55e19be2a54d241d
< btcdrak> dcousens: FYI, my .bitcoin folder is 67GB with addrindex turned on. What is it without?
< GitHub170> [bitcoin] laanwj closed pull request #6877: rpc: Add maxmempool and effective min fee to getmempoolinfo (master...2015_10_mempool_effective_fee) https://github.com/bitcoin/bitcoin/pull/6877
< GitHub136> bitcoin/master 26f5b34 Wladimir J. van der Laan: Merge pull request #6877...
< GitHub136> bitcoin/master 10e2eae Wladimir J. van der Laan: rpc: Add maxmempool and effective min fee to getmempoolinfo
< GitHub136> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/5242bb32c723...26f5b34e8838
< GitHub173> [bitcoin] laanwj closed pull request #6813: Support gathering code coverage data for RPC tests with lcov (master...btc-test-lcov-rpc) https://github.com/bitcoin/bitcoin/pull/6813
< GitHub137> bitcoin/master 8e3a27b dexX7: Require Python for RPC tests, when using lcov...
< GitHub137> bitcoin/master d425877 dexX7: Remove coverage and test related files, when cleaning up...
< GitHub137> bitcoin/master 4d2a926 dexX7: Ignore coverage data related and temporary test files
< GitHub137> [bitcoin] laanwj pushed 7 new commits to master: https://github.com/bitcoin/bitcoin/compare/867d6c90b850...5242bb32c723
< GitHub17> [bitcoin] laanwj closed pull request #6878: doc: Add developer notes about gitignore (master...2015_10_ignore_files) https://github.com/bitcoin/bitcoin/pull/6878
< GitHub163> bitcoin/master 867d6c9 Wladimir J. van der Laan: Merge pull request #6878...
< GitHub163> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/450893769f7a...867d6c90b850
< GitHub163> bitcoin/master dca7bd3 Wladimir J. van der Laan: doc: Add developer notes about gitignore...
< GitHub137> [bitcoin] laanwj closed pull request #6879: doc: mention BIP65 softfork in bips.md (master...2015_10_bip65) https://github.com/bitcoin/bitcoin/pull/6879
< GitHub133> bitcoin/master 4508937 Wladimir J. van der Laan: Merge pull request #6879...
< GitHub133> bitcoin/master ceb2a9c Wladimir J. van der Laan: doc: mention BIP65 softfork in bips.md
< GitHub133> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/46f74379b86b...450893769f7a

2015-10-25

< GitHub9> [bitcoin] CodeShark opened pull request #6888: Clear strMiscWarning before running PartitionAlert (master...alert_tests) https://github.com/bitcoin/bitcoin/pull/6888
< CodeShark> hmmm - this won't work for versionbits at all: https://github.com/bitcoin/bitcoin/blob/master/src/main.cpp#L2034