2016-09-19

< GitHub191> [bitcoin] laanwj closed pull request #8716: [qa] wallet: Check legacy wallet as well (master...Mf1609-qaWalletLegacy) https://github.com/bitcoin/bitcoin/pull/8716
< GitHub197> bitcoin/master fa644d0 MarcoFalke: [qa] wallet: Check legacy wallet as well
< GitHub197> bitcoin/master 9ac0130 Wladimir J. van der Laan: Merge #8716: [qa] wallet: Check legacy wallet as well...
< GitHub197> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/2ebb8099d9a2...9ac01301f5c4
< GitHub155> [bitcoin] laanwj closed pull request #8742: Specify Protobuf version 2 in paymentrequest.proto (master...proto2-vs-proto3) https://github.com/bitcoin/bitcoin/pull/8742
< GitHub181> bitcoin/master 2ebb809 Wladimir J. van der Laan: Merge #8742: Specify Protobuf version 2 in paymentrequest.proto...
< GitHub181> bitcoin/master c4084c2 fanquake: Specify Protobuf version 2 in paymentrequest.proto
< GitHub181> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/0171a1e559aa...2ebb8099d9a2
< GitHub123> [bitcoin] laanwj closed pull request #8720: Minor change in section name (master...patch-1) https://github.com/bitcoin/bitcoin/pull/8720
< GitHub185> bitcoin/master 0171a1e Wladimir J. van der Laan: Merge #8720: Minor change in section name...
< GitHub185> bitcoin/master dad932c Anders Øyvind Urke-Sætre: Minor change in section name...
< GitHub185> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/254ea97e940f...0171a1e559aa
< GitHub35> [bitcoin] laanwj closed pull request #8754: [Doc] Target protobuf 2.6 in OS X build notes. (master...osx-protobuf-doc) https://github.com/bitcoin/bitcoin/pull/8754
< GitHub53> bitcoin/master 254ea97 Wladimir J. van der Laan: Merge #8754: [Doc] Target protobuf 2.6 in OS X build notes....
< GitHub53> bitcoin/master b16a7f6 Michael Ford: [Doc] Target protobuf 2.6 in OS X build notes....
< GitHub53> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/a5b20edd3dc6...254ea97e940f
< GitHub73> [bitcoin] jonasschnelli opened pull request #8757: Fix issue with conflicted mempool tx in listsinceblock (master...2016/09/listsinceblock) https://github.com/bitcoin/bitcoin/pull/8757
< GitHub56> [bitcoin] laanwj closed pull request #8494: [init, wallet] ParameterInteraction() iff wallet enabled (master...Mf1608-initWalletParamInt) https://github.com/bitcoin/bitcoin/pull/8494
< GitHub157> bitcoin/master a5b20ed Wladimir J. van der Laan: Merge #8494: [init, wallet] ParameterInteraction() iff wallet enabled...
< GitHub157> bitcoin/master fa5d276 MarcoFalke: [init] ParameterInteraction() iff wallet enabled
< GitHub157> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/483e8e4f4875...a5b20edd3dc6
< GitHub31> [bitcoin] MarcoFalke closed pull request #8724: [qa] walletbackup: Sync blocks inside the loop (master...Mf1609-qaWalletBackupFix) https://github.com/bitcoin/bitcoin/pull/8724
< GitHub140> bitcoin/master 483e8e4 MarcoFalke: Merge #8724: [qa] walletbackup: Sync blocks inside the loop...
< GitHub140> bitcoin/master fad41f3 MarcoFalke: [qa] walletbackup: Sync blocks inside the loop
< GitHub140> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/239e8ce95896...483e8e4f4875
< GitHub139> bitcoin/master 239e8ce Wladimir J. van der Laan: Merge #8740: net: No longer send local address in addrMe...
< GitHub126> [bitcoin] laanwj closed pull request #8740: net: No longer send local address in addrMe (master...2016_09_addrfrom_version) https://github.com/bitcoin/bitcoin/pull/8740
< GitHub139> bitcoin/master d9c99c3 Wladimir J. van der Laan: net: No longer send local address in addrMe...
< GitHub139> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/1e5799c52535...239e8ce95896
< GitHub82> [bitcoin] jl2012 opened pull request #8756: Implement excessive sighashing protection policy with conservative counting (master...sighashpolicylite) https://github.com/bitcoin/bitcoin/pull/8756

2016-09-18

< GitHub154> [bitcoin] jl2012 opened pull request #8755: Implement excessive sighashing protection policy (master...sighashpolicy) https://github.com/bitcoin/bitcoin/pull/8755
< GitHub25> [bitcoin] fanquake opened pull request #8754: [Doc] Target protobuf 2.6 in OS X build notes. (master...osx-protobuf-doc) https://github.com/bitcoin/bitcoin/pull/8754
< GitHub94> [bitcoin] laanwj opened pull request #8753: Locked memory manager (master...2016_09_lockedpool) https://github.com/bitcoin/bitcoin/pull/8753

2016-09-17

< GitHub101> [bitcoin] afk11 opened pull request #8751: RPC: Add parameter to addmultisigaddress / createmultisig to sort public keys (master...sort-multisigs) https://github.com/bitcoin/bitcoin/pull/8751
< GitHub88> [bitcoin] MarcoFalke opened pull request #8750: [qa] Refactor RPCTestHandler to prevent TimeoutExpired (master...Mf1609-qaRefactorTimeout) https://github.com/bitcoin/bitcoin/pull/8750
< luke-jr> rebroad: I do consulting for various Bitcoin stuff, but I try to minimise travel even then :p
< gmaxwell> there are many things which people would like to, I'd suggest reading through the bitcoin core zurich meeting notes for things people are working on/planning on working on.
< GitHub128> [bitcoin] pstratem closed pull request #8585: [Wallet] Move IncOrderPosNext to CWallet::AddAccountingEntry (master...2016-08-24-cwallet-incorderposnext) https://github.com/bitcoin/bitcoin/pull/8585

2016-09-16

< GitHub107> [bitcoin] jonnynewbs opened pull request #8747: [rpc] Fix transaction size comments and RPC help text. (master...rpc_comments) https://github.com/bitcoin/bitcoin/pull/8747
< GitHub64> [bitcoin] achow101 opened pull request #8746: [Qt][RPC] Hide passphrases in debug console history (master...hide-walletpassphrase) https://github.com/bitcoin/bitcoin/pull/8746
< GitHub44> [bitcoin] jonasschnelli opened pull request #8745: [PoC] Add wallet inspection and modification tool "bitcoin-wallet-tool" (master...2016/09/wallet-tool) https://github.com/bitcoin/bitcoin/pull/8745
< GitHub149> [bitcoin] MarcoFalke opened pull request #8744: [0.13.1] qa Backports (0.13...Mf1609-qaBackports) https://github.com/bitcoin/bitcoin/pull/8744
< GitHub24> [bitcoin] MarcoFalke closed pull request #8713: [qa] create_cache: Delete temp dir when done (master...Mf1609-qaCacheTempdir) https://github.com/bitcoin/bitcoin/pull/8713
< GitHub99> bitcoin/master 1e5799c MarcoFalke: Merge #8713: [qa] create_cache: Delete temp dir when done...
< GitHub99> bitcoin/master fa27d99 MarcoFalke: [qa] create_cache: Delete temp dir when done
< GitHub99> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/a82e5d8220bb...1e5799c52535
< jonasschnelli> dgenr8: and the magic point where qt gets not compiled for ARM is here: https://github.com/bitcoin/bitcoin/blob/master/depends/packages/packages.mk#L7
< jonasschnelli> You might want to check the bitcoin-qt.m4 macro file
< GitHub3> [bitcoin] fanquake opened pull request #8743: Remove old manpages from contrib/debian in favour of doc/man (master...remove-old-manpages) https://github.com/bitcoin/bitcoin/pull/8743
< GitHub14> [bitcoin] fanquake opened pull request #8742: Specify Protobuf version 2 in paymentrequest.proto (master...proto2-vs-proto3) https://github.com/bitcoin/bitcoin/pull/8742

2016-09-15

< BlueMatt> argh ffs...can we all agree to not use github's fancy new review mode thing? it seems to be breaking github's emails as they are no longer threaded together for a single issue...hopefully they can fix in a day or two but they really broke my bitcoin-email workflow :(
< gmaxwell> would it be unreasonable for us to keep a bitcoin-core keyring file that has the pgp keys of all the regulars around here in it?
< GitHub48> [bitcoin] laanwj opened pull request #8740: net: No longer send local address in addrMe (master...2016_09_addrfrom_version) https://github.com/bitcoin/bitcoin/pull/8740
< wumpus> cfields_: yes did you see https://github.com/bitcoin/bitcoin/pull/8730?
< jonasschnelli> jl2012: what do you think about https://github.com/bitcoin/bitcoin/pull/8654?
< btcdrak> Here are the remaining PRs and issues for 0.13.1 https://github.com/bitcoin/bitcoin/milestone/22
< wumpus> I've posted a proposal for the 0.14 release schedule here: https://github.com/bitcoin/bitcoin/issues/8719
< wumpus> yes the bitcoin.org page needs to be updated
< gmaxwell> #bitcoin-core-dev Meeting: wumpus sipa gmaxwell jonasschnelli morcos luke-jr btcdrak sdaftuar jtimon cfields petertodd kanzure bluematt instagibbs phantomcircuit codeshark michagogo marcofalke paveljanik NicolasDorier
< paveljanik> wumpus, jonasschnelli: here is a rough QT -Wshadow commit (https://github.com/paveljanik/bitcoin/commit/f9d00d7e624a6760a099ca61caca25d7982844d1). I'd like to hear what you like or dislike about such changes or what to do in some other way.
< GitHub112> [bitcoin] sdaftuar opened pull request #8739: [qa] Fix broken sendcmpct test in p2p-compactblocks.py (master...fix-cb-test) https://github.com/bitcoin/bitcoin/pull/8739
< GitHub111> [bitcoin] rebroad closed pull request #8729: More granular debug (master...MoreGranularDebug6) https://github.com/bitcoin/bitcoin/pull/8729
< GitHub86> [bitcoin] paveljanik opened pull request #8737: Trivial: UndoReadFromDisk works on undo files (rev), not on block files. (master...20160915_Undo_error_message_fix) https://github.com/bitcoin/bitcoin/pull/8737
< GitHub24> [bitcoin] wjx opened pull request #8736: base58: Improve DecodeBase58 performance. (master...speedup-decodebase58) https://github.com/bitcoin/bitcoin/pull/8736
< GitHub156> [bitcoin] jonasschnelli opened pull request #8735: [Wallet] add option for a custom extended master privat key (xpriv) (master...2016/09/hd_set_seed) https://github.com/bitcoin/bitcoin/pull/8735
< cfields_> wumpus: lots of work remains. But, mingw depends build fully, and bitcoin build breaks with the same threading problems that have been reported, so it's enough to help with testing/debugging.
< dcousens> achow101: RE privileges to approve, you don't need any see https://github.com/bitcoin/bitcoin/pull/8723#pullrequestreview-82853
< GitHub54> [bitcoin] rebroad opened pull request #8734: Send NOTFOUND when we don't have the block data. (master...NotfoundIfPruned) https://github.com/bitcoin/bitcoin/pull/8734

2016-09-14

< GitHub198> [bitcoin] rebroad closed pull request #8596: [WIP] Feeler code and debugging. (master...FeelerFixes) https://github.com/bitcoin/bitcoin/pull/8596
< GitHub147> [bitcoin] MarcoFalke closed pull request #8731: Debug headers received ("block" for new block announcement, "block2" … (master...DebugHeadersReceived) https://github.com/bitcoin/bitcoin/pull/8731
< GitHub131> [bitcoin] rebroad opened pull request #8733: More granular debug [WIP] (master...MoreGranularDebug.wip) https://github.com/bitcoin/bitcoin/pull/8733
< GitHub25> [bitcoin] rebroad closed pull request #8727: Move logic for TX INVs together (master...MoreGranularDebug4) https://github.com/bitcoin/bitcoin/pull/8727
< GitHub62> [bitcoin] rebroad closed pull request #8726: Move a bunch of fairly verbose debug messages from mempool to mempool2 (master...MoreGranularDebug3) https://github.com/bitcoin/bitcoin/pull/8726
< GitHub28> [bitcoin] rebroad closed pull request #8725: Split debug for estimatefee into {estimatefee,2} (master...MoreGranularDebug2) https://github.com/bitcoin/bitcoin/pull/8725
< GitHub167> [bitcoin] rebroad closed pull request #8728: More granular debug5 (master...MoreGranularDebug5) https://github.com/bitcoin/bitcoin/pull/8728
< rebroad> if it's granular enough, then no one should need to add adhoc debug message as you are suggesting... there'd be a debug message already there that they can activate just by editing bitcoin.conf
< wumpus> if I became really obsessed about this, and wanted this a default part of bitcoin, and became angry if other developers disagreed, that would be an ego commit
< GitHub9> [bitcoin] rebroad closed pull request #8484: More granular debug (master...MoreGranularDebug) https://github.com/bitcoin/bitcoin/pull/8484
< rebroad> wumpus, i just spend 3 days having to rebase all these debug commits due to changes that were made recently.. I am getting very very fed up with the amount of time wasted rebasing when I could be spending that time adding functionality to bitcoin. this is why this needs to get merged!
< GitHub154> [bitcoin] rebroad opened pull request #8731: Debug headers received ("block" for new block announcement, "block2" … (master...DebugHeadersReceived) https://github.com/bitcoin/bitcoin/pull/8731
< GitHub64> [bitcoin] laanwj opened pull request #8730: depends: Add libevent compatibility patch for windows (master...2016_09_libevent_windows_gcc_531) https://github.com/bitcoin/bitcoin/pull/8730
< GitHub29> [bitcoin] rebroad opened pull request #8729: More granular debug6 (master...MoreGranularDebug6) https://github.com/bitcoin/bitcoin/pull/8729
< GitHub85> [bitcoin] rebroad opened pull request #8728: More granular debug5 (master...MoreGranularDebug5) https://github.com/bitcoin/bitcoin/pull/8728
< GitHub66> [bitcoin] rebroad opened pull request #8727: Move logic for TX INVs together (master...MoreGranularDebug4) https://github.com/bitcoin/bitcoin/pull/8727
< GitHub174> [bitcoin] rebroad opened pull request #8726: Move a bunch of fairly verbose debug messages from mempool to mempool2 (master...MoreGranularDebug3) https://github.com/bitcoin/bitcoin/pull/8726
< GitHub12> [bitcoin] rebroad opened pull request #8725: Split debug for estimatefee into {estimatefee,2} (master...MoreGranularDebug2) https://github.com/bitcoin/bitcoin/pull/8725
< achow101> This is the discussion if it's of any help: https://botbot.me/freenode/bitcoin-core-dev/2016-08-08/?msg=70975840&page=1
< wumpus> it only seemed like bitcoin-qt was working: I had disabled wallet support, apparently it works then
< GitHub197> [bitcoin] MarcoFalke opened pull request #8724: [qa] walletbackup: Sync blocks inside the loop (master...Mf1609-qaWalletBackupFix) https://github.com/bitcoin/bitcoin/pull/8724
< MarcoFalke> Hmm. Github allows us to write to branches that have a pull request open against bitcoin.
< GitHub7> [bitcoin] MarcoFalke closed pull request #8714: [qa] gitignore: Remove unused lines (master...Mf1609-qaUnused) https://github.com/bitcoin/bitcoin/pull/8714
< GitHub93> bitcoin/master a82e5d8 MarcoFalke: Merge #8714: [qa] gitignore: Remove unused lines...
< GitHub93> bitcoin/master 1111ddb MarcoFalke: gitignore: Remove unused lines
< GitHub93> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/881d7eaf29f7...a82e5d8220bb
< GitHub10> [bitcoin] jonasschnelli opened pull request #8723: [Wallet] Add support for flexible BIP32/HD keypath-scheme (master...2016/09/hd_flex) https://github.com/bitcoin/bitcoin/pull/8723
< GitHub88> [bitcoin] laanwj opened pull request #8722: bitcoin-cli: More detailed error reporting (master...2016_09_cli_http_error) https://github.com/bitcoin/bitcoin/pull/8722
< GitHub17> [bitcoin] andersoyvind opened pull request #8720: Minor change in section name (master...patch-1) https://github.com/bitcoin/bitcoin/pull/8720
< wumpus> bitcoin core win64 cross-build is in a state of crap with ubuntu 16.04
< GitHub0> [bitcoin] laanwj closed pull request #8715: net: only delete CConnman if it's been created (master...fix-connman-shutdown) https://github.com/bitcoin/bitcoin/pull/8715
< GitHub179> bitcoin/master 881d7ea Wladimir J. van der Laan: Merge #8715: net: only delete CConnman if it's been created...
< GitHub179> bitcoin/master 36fa01f Cory Fields: net: only delete CConnman if it's been created...
< GitHub179> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/57b34599b2de...881d7eaf29f7
< wumpus> jonasschnelli: hehe, maybe if bitcoin was designed 10 years earlier
< * jonasschnelli> thinks we should fork to use CORBA as our bitcoin scripting language and SOAP as http API :}
< wumpus> many people just use the RPC through curl, or bitcoin-cli, which involves executing an external process per connection per command, and they don't complain either
< dcousens> but, my point was, even my own implementation has nothing to do bitcoin as such, its just an RPC implementation
< wumpus> this presentation talks about people using MQTT for bitcoin wallet control, among other things: https://media.defcon.org/DEF%20CON%2024/DEF%20CON%2024%20presentations/DEFCON-24-Lucas-Lundgren-Light-Weight%20Protocol-Critical-Implications.pdf of course, without authentication or encryption and on the public internet :)
< dcousens> I've found it much easier to just always use a well-tested rpc library than use something "bitcoin tailored"
< gmaxwell> the standard python bitcoin json library that people get linked to is not at all reliable (gets disconnected and then throws exceptions, and has to be wrapped to be usable) and I don't see anyone even complaining about it.
< dcousens> bitcoin-cli*, that is
< wumpus> no interpreters in bitcoin please
< wumpus> I mean with bitcoin-cli you have full access to bash' scripting/variable munging/etc capabilities, in the Qt console you don't, so this adds a bit of shell power to it
< jonasschnelli> I think its better to start with it client-side QT only, and if it turns to be useful, add it to bitcoin-cli.
< jonasschnelli> A better approach for the RPC layer would be to add it into bitcoin-cli
< jonasschnelli> Would be nice if anyone could review/ACK/NACK https://github.com/bitcoin/bitcoin/pull/7783 (nested RPC commands).
< GitHub42> [bitcoin] jonasschnelli closed pull request #8712: Remove maxuploadtargets recommended minimum (master...2016/09/rem_maxupt_min) https://github.com/bitcoin/bitcoin/pull/8712
< GitHub186> bitcoin/master 57b3459 Jonas Schnelli: Merge #8712: Remove maxuploadtargets recommended minimum...
< GitHub186> bitcoin/master 1b6bcdd Jonas Schnelli: Remove maxuploadtargets recommended minimum
< GitHub186> [bitcoin] jonasschnelli pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/fa7caf6d9116...57b34599b2de

2016-09-13

< GitHub169> [bitcoin] SCDeveloper closed pull request #8703: Addition of ImmatureCreditCached to MarkDirty() (master...patch-5) https://github.com/bitcoin/bitcoin/pull/8703
< GitHub132> [bitcoin] spencerlievens opened pull request #8717: [WALLET] Addition of ImmatureCreditCached to MarkDirty() (master...patch-2) https://github.com/bitcoin/bitcoin/pull/8717
< GitHub83> [bitcoin] MarcoFalke opened pull request #8716: [qa] wallet: Check legacy wallet as well (master...Mf1609-qaWalletLegacy) https://github.com/bitcoin/bitcoin/pull/8716
< GitHub32> [bitcoin] theuni opened pull request #8715: net: only delete CConnman if it's been created (master...fix-connman-shutdown) https://github.com/bitcoin/bitcoin/pull/8715
< GitHub155> [bitcoin] MarcoFalke opened pull request #8714: [qa] gitignore, travis: Remove unused lines (master...Mf1609-qaUnused) https://github.com/bitcoin/bitcoin/pull/8714
< GitHub170> [bitcoin] MarcoFalke opened pull request #8713: [qa] create_cache: Delete temp dir when done (master...Mf1609-qaCacheTempdir) https://github.com/bitcoin/bitcoin/pull/8713
< GitHub20> [bitcoin] jonasschnelli closed pull request #8559: Change maxuploadtarget recommended minimum calculation (master...2016/08/max_ut) https://github.com/bitcoin/bitcoin/pull/8559
< GitHub151> [bitcoin] jonasschnelli opened pull request #8712: Remove maxuploadtargets recommended minimum (master...2016/09/rem_maxupt_min) https://github.com/bitcoin/bitcoin/pull/8712
< GitHub58> [bitcoin] MarcoFalke closed pull request #7728: Fees: Tests: Check CFeeRate internal precision in mempool_tests.cpp (master...0.12.99-feerate-precision-test) https://github.com/bitcoin/bitcoin/pull/7728
< GitHub110> [bitcoin] laanwj closed pull request #8457: Add block height support in rpc call getblock (master...feature/add-get-block-by-number) https://github.com/bitcoin/bitcoin/pull/8457
< cfields> btw, https://github.com/bitcoin/bitcoin/pull/8708/commits/89c57428e335fefe87c6f5b4fd173c9ba9bdb486 may be worth backporting. I'm not sure what the implications are, but i assume it's possible for that to cause a stall in initial sync. Not quite sure how to test.
< jonasschnelli> Re: https://github.com/bitcoin/bitcoin/pull/8559 ... should we just remove the maxuploadtargets "recommended" minimum?
< GitHub93> [bitcoin] laanwj closed pull request #8710: [0.13.1] qt Backports (0.13...Mf1609-qtBackports) https://github.com/bitcoin/bitcoin/pull/8710
< GitHub11> bitcoin/0.13 a37cec5 Andrew Chow: Persist the datadir after option reset...
< GitHub11> bitcoin/0.13 d9f0d4e adlawren: Fix minimize and close bugs...
< GitHub11> bitcoin/0.13 41fd852 rodasmith: fix op order to append first alert...
< GitHub11> [bitcoin] laanwj pushed 4 new commits to 0.13: https://github.com/bitcoin/bitcoin/compare/a9429ca26dd8...4731623777ab
< GitHub8> [bitcoin] laanwj closed pull request #8693: add witness address to address book (master...addwitbook) https://github.com/bitcoin/bitcoin/pull/8693
< GitHub4> bitcoin/master fa7caf6 Wladimir J. van der Laan: Merge #8693: add witness address to address book...
< GitHub4> bitcoin/master 62ffbbd instagibbs: add witness address to address book
< GitHub4> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/c9914c209468...fa7caf6d9116
< GitHub163> [bitcoin] laanwj closed pull request #8699: Remove createwitnessaddress RPC command (master...nocreatewitadd) https://github.com/bitcoin/bitcoin/pull/8699
< GitHub83> bitcoin/master 86c3f8d Johnson Lau: Remove createwitnessaddress...
< GitHub83> bitcoin/master c9914c2 Wladimir J. van der Laan: Merge #8699: Remove createwitnessaddress RPC command...
< GitHub83> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/7fe6c5c99370...c9914c209468
< wumpus> I still don't get the situation around the createwitnessaddress command https://github.com/bitcoin/bitcoin/pull/8699
< GitHub50> [bitcoin] jl2012 closed pull request #8685: Discourage P2WSH with too big script or stack (master...bigp2wsh) https://github.com/bitcoin/bitcoin/pull/8685
< GitHub174> [bitcoin] MarcoFalke closed pull request #8652: [qa]: remove root test directory for RPC tests (master...cleanup) https://github.com/bitcoin/bitcoin/pull/8652
< GitHub114> bitcoin/master c62cc4e whythat: fix path for bak file
< GitHub114> bitcoin/master 7fe6c5c MarcoFalke: Merge #8652: [qa]: remove root test directory for RPC tests...
< GitHub114> bitcoin/master 438e94d whythat: remove root test directory for RPC tests
< GitHub114> [bitcoin] MarcoFalke pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/37ac67816afb...7fe6c5c99370
< GitHub97> [bitcoin] MarcoFalke opened pull request #8710: [0.13.1] qt Backports (0.13...Mf1609-qtBackports) https://github.com/bitcoin/bitcoin/pull/8710
< GitHub171> bitcoin/master 86726d8 Wladimir J. van der Laan: Rename `-optintofullrbf` option to `-walletrbf`...
< GitHub29> [bitcoin] laanwj closed pull request #8601: Add option to opt into full-RBF when sending funds (rebase, original by petertodd) (master...2016_08_full_rbf_option) https://github.com/bitcoin/bitcoin/pull/8601
< GitHub171> bitcoin/master 05fa823 Wladimir J. van der Laan: wallet: Add BIP125 comment for MAXINT-1/-2 behavior
< GitHub171> bitcoin/master 152f45b Peter Todd: Add option to opt into full-RBF when sending funds
< GitHub171> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/39ac1ec64264...37ac67816afb
< wumpus> so is https://github.com/bitcoin/bitcoin/pull/8699 an intermediate measure for 0.13.1 or does the `createwitnessaddress` command need to disappear forever?
< GitHub145> [bitcoin] laanwj closed pull request #8697: fix op order to append first alert (master...fix-op-order-1st-alert) https://github.com/bitcoin/bitcoin/pull/8697
< GitHub142> bitcoin/master 39ac1ec Wladimir J. van der Laan: Merge #8697: fix op order to append first alert...
< GitHub142> bitcoin/master 1d635ae rodasmith: fix op order to append first alert
< GitHub142> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/256215244105...39ac1ec64264
< GitHub118> [bitcoin] laanwj closed pull request #8662: Update btcdrak signing key (master...updatekey) https://github.com/bitcoin/bitcoin/pull/8662
< GitHub197> bitcoin/master 2562152 Wladimir J. van der Laan: Merge #8662: Update btcdrak signing key...
< GitHub197> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/4ced5de71d4d...256215244105
< GitHub197> bitcoin/master 46606af BtcDrak: Update btcdrak signing key
< GitHub57> [bitcoin] laanwj closed pull request #8528: Update p2p-segwit.py to reflect correct behavior (master...rejectsw) https://github.com/bitcoin/bitcoin/pull/8528
< GitHub125> bitcoin/master 4ced5de Wladimir J. van der Laan: Merge #8528: Update p2p-segwit.py to reflect correct behavior...
< GitHub125> bitcoin/master 5547aeb instagibbs: p2psegwit.py transaction is rejected due to premature witness not size
< GitHub125> bitcoin/master bc1d1f2 instagibbs: Update p2p-segwit.py to reflect correct AskFor behavior
< GitHub125> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/7e9ab9555cab...4ced5de71d4d
< GitHub42> [bitcoin] laanwj closed pull request #8608: Install manpages via make install, also add some autogenerated manpages (master...man_automake2) https://github.com/bitcoin/bitcoin/pull/8608
< GitHub32> bitcoin/master 6edf2fd nomnombtc: add gen-manpages.sh description to README.md
< GitHub32> bitcoin/master eb5643b nomnombtc: add autogenerated manpages by help2man
< GitHub32> bitcoin/master d2cd9c0 nomnombtc: add script to generate manpages with help2man
< GitHub32> [bitcoin] laanwj pushed 10 new commits to master: https://github.com/bitcoin/bitcoin/compare/2a0836f6d5e7...7e9ab9555cab
< wumpus> I understand lazy developers, but being lazy with security issues areound bitcoin is a deadly sin
< jonasschnelli> https://github.com/bitcoin/bitcoin/pull/8653 is indeed silly...
< GitHub140> [bitcoin] rebroad opened pull request #8709: Allow filterclear messages for enabling TX relay only. (master...AllowFilterclear) https://github.com/bitcoin/bitcoin/pull/8709
< GitHub101> [bitcoin] theuni opened pull request #8708: net: have CConnman handle message sending (master...connman-send) https://github.com/bitcoin/bitcoin/pull/8708
< GitHub12> [bitcoin] theuni opened pull request #8707: net: fix maxuploadtarget setting (master...fix-maxupload) https://github.com/bitcoin/bitcoin/pull/8707

2016-09-12

< GitHub63> [bitcoin] achow101 opened pull request #8704: [RPC] Transaction details in getblock (master...getblock-extraverbose) https://github.com/bitcoin/bitcoin/pull/8704
< GitHub105> [bitcoin] SCDeveloper opened pull request #8703: Addition of ImmatureCreditCached to MarkDirty() (master...patch-5) https://github.com/bitcoin/bitcoin/pull/8703

2016-09-11

< GitHub2> [bitcoin] isle2983 opened pull request #8702: [copyright] add MIT License copyright header to remaining Python files (master...misc-copyright) https://github.com/bitcoin/bitcoin/pull/8702
< GitHub127> [bitcoin] isle2983 opened pull request #8701: [copyright] add MIT License copyright header to zmq_sub.py (master...zmq_sub-copyright) https://github.com/bitcoin/bitcoin/pull/8701
< GitHub94> [bitcoin] isle2983 opened pull request #8700: [copyright] add MIT license headers to .sh scripts where missing (master...sh-copyright) https://github.com/bitcoin/bitcoin/pull/8700
< GitHub193> [bitcoin] jl2012 opened pull request #8699: Disable createwitnessaddress in mainnet (master...nocreatewitadd) https://github.com/bitcoin/bitcoin/pull/8699
< luke-jr> jasonv75: #bitcoin-dev ?

2016-09-10

< GitHub112> [bitcoin] rodasmith opened pull request #8697: fix op order to append first alert (master...fix-op-order-1st-alert) https://github.com/bitcoin/bitcoin/pull/8697
< TheJoeB> Please send me a quick message if anyone is interested in a short conversation regarding your personal experiences with Bitcoin and/or Ethereum (any blockchain technology). It shouldn't take longer than 10 minutes and is for my graduate work on these technologies: http://www.thejoeblankenship.com/research-proposal Your privacy, confidentially, and anonymity will be protected.
< GitHub77> [bitcoin] TheBlueMatt closed pull request #8235: Compact Block Tweaks (master...cmpctblock) https://github.com/bitcoin/bitcoin/pull/8235
< GitHub13> [bitcoin] luke-jr closed pull request #8132: wallet: Add key generation type (master...keygentype) https://github.com/bitcoin/bitcoin/pull/8132
< GitHub126> [bitcoin] pstratem opened pull request #8696: [Wallet] Remove last external reference to CWalletDB (master...2016-09-09-cwallet-listaccountcreditdebit) https://github.com/bitcoin/bitcoin/pull/8696
< luke-jr> "If checked, users with write access to bitcoin/bitcoin can add new commits to your multiwallet branch. You can always change this setting later." … on by default O.o
< GitHub117> [bitcoin] pstratem opened pull request #8695: Retry feeler connection if OpenNetworkConnection fails in under 1ms. (master...2016-09-09-net-feeler-timeout) https://github.com/bitcoin/bitcoin/pull/8695

2016-09-09

< GitHub25> [bitcoin] MarcoFalke closed pull request #8667: Fix SIGHASH_SINGLE bug in test_framework SignatureHash (master...patch-16) https://github.com/bitcoin/bitcoin/pull/8667
< GitHub64> bitcoin/master 2a0836f MarcoFalke: Merge #8667: Fix SIGHASH_SINGLE bug in test_framework SignatureHash...
< GitHub64> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/702e6e059b3d...2a0836f6d5e7
< GitHub64> bitcoin/master 2f2548d Johnson Lau: Fix SIGHASH_SINGLE bug in test_framework SignatureHash...
< GitHub57> [bitcoin] luke-jr opened pull request #8694: Basic multiwallet support (master...multiwallet) https://github.com/bitcoin/bitcoin/pull/8694
< BlueMatt> indeed, overcommit is required to run bitcoin core in consensus
< GitHub77> [bitcoin] instagibbs opened pull request #8693: add witness address to address book (master...addwitbook) https://github.com/bitcoin/bitcoin/pull/8693
< BlueMatt> sipa: ok, take a look at https://github.com/TheBlueMatt/bitcoin/tree/segwitcb - removed the boolean and I think simplified some of the logic
< GitHub17> [bitcoin] jonasschnelli closed pull request #8678: [Qt][CoinControl] fix UI bug that could result in paying unexpected fee (master...2016/09/qt_cc_ui_radrio_fix) https://github.com/bitcoin/bitcoin/pull/8678
< GitHub155> bitcoin/master 702e6e0 Jonas Schnelli: Merge #8678: [Qt][CoinControl] fix UI bug that could result in paying unexpected fee...
< GitHub155> bitcoin/master 0480293 Jonas Schnelli: [Qt][CoinControl] fix UI bug that could result in paying unexpected fee
< GitHub155> [bitcoin] jonasschnelli pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/689821340981...702e6e059b3d
< GitHub103> [bitcoin] sipa opened pull request #8690: Use a heap to not fully sort all nodes for addr relay (master...heapaddrsort) https://github.com/bitcoin/bitcoin/pull/8690
< GitHub34> [bitcoin] sipa closed pull request #8681: Performance Regression Fix: Pre-Allocate txChanged vector (master...fix-perf-regressed-txChanged) https://github.com/bitcoin/bitcoin/pull/8681
< GitHub163> bitcoin/master 6898213 Pieter Wuille: Merge #8681: Performance Regression Fix: Pre-Allocate txChanged vector...
< GitHub163> bitcoin/master ec81881 Jeremy Rubin: Performance Regression Fix: Pre-Allocate txChanged vector
< GitHub163> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/2abfe5956eef...689821340981
< GitHub126> [bitcoin] sipa opened pull request #8688: Move static global randomizer seeds into CConnman (master...detrandconnman) https://github.com/bitcoin/bitcoin/pull/8688
< GitHub43> [bitcoin] laanwj closed pull request #8664: Fix segwit-related wallet bug (master...segwit-wallet-bug) https://github.com/bitcoin/bitcoin/pull/8664
< GitHub33> bitcoin/master 2abfe59 Wladimir J. van der Laan: Merge #8664: Fix segwit-related wallet bug...
< GitHub33> bitcoin/master c40b034 Suhas Daftuar: Clear witness with vin/vout in CWallet::CreateTransaction()
< GitHub33> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/6423116741de...2abfe5956eef
< GitHub149> [bitcoin] laanwj closed pull request #8646: [0.13 backport] Reduce default number of blocks to check at startup (0.13...reduceblocks_backport) https://github.com/bitcoin/bitcoin/pull/8646
< GitHub90> bitcoin/0.13 a9429ca Pieter Wuille: Reduce default number of blocks to check at startup...
< GitHub90> [bitcoin] laanwj pushed 1 new commit to 0.13: https://github.com/bitcoin/bitcoin/commit/a9429ca26dd8f4555def2dc8bd8ea7fc4e32fce6
< GitHub48> bitcoin/0.13 2611ad7 Ethan Heilman: Added feeler connections increasing good addrs in the tried table....
< GitHub48> bitcoin/0.13 1db3352 Wladimir J. van der Laan: qt: Fix random segfault when closing "Choose data directory" dialog...
< GitHub48> bitcoin/0.13 75f2065 Wladimir J. van der Laan: build: Remove check for `openssl/ec.h`...
< GitHub48> [bitcoin] laanwj pushed 13 new commits to 0.13: https://github.com/bitcoin/bitcoin/compare/122fdfdae915...32269449185d
< GitHub198> [bitcoin] laanwj closed pull request #8679: [0.13] Various backports (0.13...backports_0.13) https://github.com/bitcoin/bitcoin/pull/8679
< GitHub100> [bitcoin] laanwj closed pull request #8085: p2p: Begin encapsulation (master...net-refactor13) https://github.com/bitcoin/bitcoin/pull/8085
< GitHub123> bitcoin/master cd16f48 Cory Fields: net: Create CConnman to encapsulate p2p connections
< GitHub123> bitcoin/master 531214f Cory Fields: gui: add NodeID to the peer table
< GitHub123> bitcoin/master d93b14d Cory Fields: net: move CBanDB and CAddrDB out of net.h/cpp...
< GitHub123> [bitcoin] laanwj pushed 34 new commits to master: https://github.com/bitcoin/bitcoin/compare/4daf02a03f9e...6423116741de
< GitHub91> [bitcoin] luke-jr opened pull request #8687: Bugfix: RPC/Wallet: removeprunedfunds should flush the wallet db (master...bugfix_removeprunedfunds) https://github.com/bitcoin/bitcoin/pull/8687
< gmaxwell> and it also means you couldn't use bitcoin anymore on a connection of less than about 50kbit/sec which, otherwise, could keep up with the chain.
< GitHub48> [bitcoin] laanwj closed pull request #8677: Do not shadow upper local variable 'send', prevent -Wshadow compiler warning. (master...20160907_Wshadow_8606) https://github.com/bitcoin/bitcoin/pull/8677
< GitHub33> bitcoin/master 125b946 Pavel Janík: Do not shadow upper local variable 'send', prevent -Wshadow compiler warning.
< GitHub33> bitcoin/master 4daf02a Wladimir J. van der Laan: Merge #8677: Do not shadow upper local variable 'send', prevent -Wshadow compiler warning....
< GitHub33> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/7f8b677aeb79...4daf02a03f9e
< gmaxwell> I was explaining this in #bitcoin two days ago in fact.
< GitHub157> [bitcoin] laanwj closed pull request #8563: Add configure check for -latomic (master...autoconf-latomic) https://github.com/bitcoin/bitcoin/pull/8563
< GitHub91> bitcoin/master 7f8b677 Wladimir J. van der Laan: Merge #8563: Add configure check for -latomic...
< GitHub91> bitcoin/master 878faac Anthony Towns: Add configure check for -latomic
< GitHub91> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/666eaf03cf25...7f8b677aeb79
< GitHub147> [bitcoin] laanwj closed pull request #8680: Address Travis spurious failures (master...rpc-waitforblock) https://github.com/bitcoin/bitcoin/pull/8680
< GitHub118> bitcoin/master 666eaf0 Wladimir J. van der Laan: Merge #8680: Address Travis spurious failures...
< GitHub118> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/80a4f21d377a...666eaf03cf25
< GitHub118> bitcoin/master d6a5dc4 Cory Fields: add waitfornewblock/waitforblock/waitforblockheight rpcs and use them for tests...
< GitHub50> [bitcoin] laanwj closed pull request #8686: translate to np (master...master) https://github.com/bitcoin/bitcoin/pull/8686
< GitHub190> [bitcoin] bitcoinsSG opened pull request #8686: translate to np (master...master) https://github.com/bitcoin/bitcoin/pull/8686
< GitHub131> [bitcoin] laanwj closed pull request #8525: Do not store witness txn in rejection cache (master...nowitnessreject) https://github.com/bitcoin/bitcoin/pull/8525
< GitHub170> bitcoin/master ca10a03 instagibbs: Add basic test for IsStandard witness transaction blinding
< GitHub170> bitcoin/master 80a4f21 Wladimir J. van der Laan: Merge #8525: Do not store witness txn in rejection cache...
< GitHub170> bitcoin/master 34521e4 Pieter Wuille: Do not store witness txn in rejection cache
< GitHub170> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/17347d6a5915...80a4f21d377a
< GitHub14> [bitcoin] laanwj closed pull request #8683: fix incorrect file name bitcoin.qrc (master...patch-1) https://github.com/bitcoin/bitcoin/pull/8683
< GitHub90> bitcoin/master df2d2e7 Gaurav Rana: update name of file bitcoin.qrc
< GitHub90> bitcoin/master 17347d6 Wladimir J. van der Laan: Merge #8683: fix incorrect file name bitcoin.qrc...
< GitHub90> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/ddc308068d69...17347d6a5915
< CodeShark> in my ideal world, Bitcoin Core would just do peer discovery, validation, and relay
< CodeShark> I have a process connect to bitcoin core via p2p which receives inv messages and sends getdata
< midnightmagic> it should then be in an external library. putting reliable message delivery into bitcoin core would also make the binary liable in the event something (inevitably) goes wrong.

2016-09-08

< BlueMatt> sipa: https://github.com/TheBlueMatt/bitcoin/commit/96cef326f9184aefd1c562f64a21298a15f0adde simplifies the total diff from master to be more readable, to me...it matches the bip, but another way to look at it is that you have two booleans: which version of compact blocks they want (which you use to send to them) and whether they will send us segwit cmpctblocks...you simply introduce the the do-they-support-v2 bool as an additional
< cfields> kanzure: and #8680 introduces blocking calls. So you could "./bitcoin-cli waitfornewblock && ./dosomething.sh", which would essentially be blocknotify. Obviously the same could be done for wallet stuff
< kanzure> so- for example- imagine you are operating a lot of bitcoin infrastructure. you do a bunch of per-block processing. when you are running functional/integration tests, you can't run your test checks immediately after issuing a command because things take a while. so when do you expect to run? well you setup a hook and you ask your infrastructure to ping your tester as soon as the data's available, or you poll for some internal state.
< BlueMatt> (comment text from above link https://github.com/bitcoin/bips/pull/423#issuecomment-245629813 )
< wumpus> #link re: queue of things for 0.13.1, link is https://github.com/bitcoin/bitcoin/milestones/0.13.1
< sipa> BlueMatt: i haven't reviewed the changes for the bip you suggested - does it require any code changes to the bitcoin core pr?
< 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
< GitHub36> [bitcoin] jl2012 opened pull request #8685: Discourage P2WSH with too big script or stack (master...bigp2wsh) https://github.com/bitcoin/bitcoin/pull/8685
< morcos> for instance you could imagine someone using bitcoin core has come to depend on that behavior
< morcos> cfields: i was just trying to review #8660, so this is also a regression caused by https://github.com/bitcoin/bitcoin/pull/7946?
< GitHub24> [bitcoin] bitcoinsSG opened pull request #8683: fix incorrect file name bitcoin.qrc (master...patch-1) https://github.com/bitcoin/bitcoin/pull/8683
< translatoree3> I've seen transfix, so translations there get credit on the github bitcoin repo?
< GitHub67> [bitcoin] MarcoFalke closed pull request #8671: Minimal fix to slow prevector tests as stopgap measure (master...simple_faster_tests) https://github.com/bitcoin/bitcoin/pull/8671
< GitHub87> bitcoin/master ddc3080 MarcoFalke: Merge #8671: Minimal fix to slow prevector tests as stopgap measure...
< GitHub87> bitcoin/master f71d4a3 Jeremy Rubin: Minimal fix to slow prevector tests as stopgap measure
< GitHub87> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/ec139a5621a9...ddc308068d69
< wumpus> /home/user/projects/bitcoin/bitcoin/src/test/addrman_tests.cpp:336:471: warning: stack frame size of 328360 bytes in function 'addrman_tests::addrman_delete::test_method' [-Wframe-larger-than=] whoa :) luckily only in the tests
< wumpus> let's try to make cfields network refactor (https://github.com/bitcoin/bitcoin/pull/8085/files) happen , it's been open so long now, line-wise there's a lot of changes so he keeps having to rebase it