2016-09-26

< GitHub61> bitcoin/master 9a75d29 Wladimir J. van der Laan: devtools: Check for high-entropy ASLR in 64-bit PE executables...
< GitHub61> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/37871f216e0d...4e1567acff4b
< GitHub50> [bitcoin] laanwj closed pull request #8451: Get rid of the const field in CTransaction (master...noconsttx) https://github.com/bitcoin/bitcoin/pull/8451
< GitHub11> [bitcoin] laanwj closed pull request #8278: Forking daemon (master...forking-daemon) https://github.com/bitcoin/bitcoin/pull/8278
< GitHub26> [bitcoin] laanwj opened pull request #8813: bitcoind: Daemonise using daemon(3) (master...2016_09_daemonize) https://github.com/bitcoin/bitcoin/pull/8813
< paveljanik> jonasschnelli, I have started testnet Qt with the current master from scratch, only bitcoin.conf remained. There is no overlay window showing it is synchronizing. Should it be displayed?
< wumpus> this would make https://github.com/bitcoin/bitcoin/pull/8278 trivial
< wumpus> so I think we can just rely on that any OS that support daemonization in the first place and runs bitcoin core has that call
< sipa> bitcoin core is sending you a getdata

2016-09-25

< GitHub70> [bitcoin] laanwj opened pull request #8811: rpc: Add support for JSON-RPC named arguments (master...2016_09_rpc_named_arguments) https://github.com/bitcoin/bitcoin/pull/8811
< MarcoFalke> wumpus: Can you enable travis on https://github.com/bitcoin-core/univalue?
< GitHub2> [bitcoin] laanwj opened pull request #8810: tests: Add exception error message for JSONRPCException (master...2016_09_tests_rpc_error) https://github.com/bitcoin/bitcoin/pull/8810
< GitHub186> [bitcoin] MarcoFalke opened pull request #8809: [qt] sync-overlay: Don't show redundant information (master...Mf1609-qtSyncInf) https://github.com/bitcoin/bitcoin/pull/8809
< GitHub25> [bitcoin] paveljanik opened pull request #8808: Do not shadow variables (gcc set) (master...20160925_Wshadow_gcc) https://github.com/bitcoin/bitcoin/pull/8808
< GitHub139> [bitcoin] laanwj closed pull request #8807: [univalue] Pull subtree from upstream (master...Mf1609-univalueSync) https://github.com/bitcoin/bitcoin/pull/8807
< GitHub168> bitcoin/master 37871f2 Wladimir J. van der Laan: Merge #8807: [univalue] Pull subtree from upstream...
< GitHub168> bitcoin/master 9bf41af MarcoFalke: Merge commit '3650668cdbbf369dd8f30c8e8eb5bb883325942d' into HEAD
< GitHub168> bitcoin/master 3650668 MarcoFalke: Squashed 'src/univalue/' changes from f32df99..daf1285...
< GitHub168> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/bae178f3ca78...37871f216e0d
< wumpus> btcdrak: something like https://github.com/bitcoin/bitcoin/projects/6 ?
< GitHub161> [bitcoin] MarcoFalke opened pull request #8807: [univalue] Pull subtree from upstream (master...Mf1609-univalueSync) https://github.com/bitcoin/bitcoin/pull/8807
< wumpus> the annoying thing is that these get URLs like https://github.com/bitcoin/bitcoin/files/491570/bitcoin.zip, as if we've uploaded them
< jonasschnelli> wumpus: wait, ... is this an ELF executable attached to this issue: https://github.com/bitcoin/bitcoin/issues/8806
< GitHub12> [bitcoin] laanwj closed 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
< GitHub152> bitcoin/master b194872 fanquake: Remove old manpages from contrib/debian
< GitHub152> bitcoin/master bae178f Wladimir J. van der Laan: Merge #8743: Remove old manpages from contrib/debian in favour of doc/man...
< GitHub152> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b694b0d5a5b4...bae178f3ca78
< GitHub175> [bitcoin] laanwj closed pull request #8791: [travis] cross-mac: explicitly enable gui (master...Mf1609-travisGui) https://github.com/bitcoin/bitcoin/pull/8791
< GitHub49> bitcoin/master b694b0d Wladimir J. van der Laan: Merge #8791: [travis] cross-mac: explicitly enable gui...
< GitHub49> bitcoin/master fa16991 MarcoFalke: [travis] cross-mac: explicitly enable gui
< GitHub49> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b8d10fd4ef8f...b694b0d5a5b4
< GitHub62> [bitcoin] laanwj closed pull request #8802: Fix future copyright year (master...future_copyright) https://github.com/bitcoin/bitcoin/pull/8802
< GitHub183> bitcoin/master b8d10fd Wladimir J. van der Laan: Merge #8802: Fix future copyright year...
< GitHub183> bitcoin/master edeaf24 Mitchell Cash: Fix future copyright year
< GitHub183> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/bc9e3ab29f9b...b8d10fd4ef8f
< GitHub70> [bitcoin] laanwj closed pull request #8771: CONTRIBUTING: Mention not to open several pulls (master...Mf1609-ContributeDoc) https://github.com/bitcoin/bitcoin/pull/8771
< GitHub25> bitcoin/master bc9e3ab Wladimir J. van der Laan: Merge #8771: CONTRIBUTING: Mention not to open several pulls...
< GitHub25> bitcoin/master faa91bc MarcoFalke: CONTRIBUTING: Mention not to open several pulls
< GitHub25> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/279bbadc8ba9...bc9e3ab29f9b
< wumpus> is libconsensus work still underway in an active enough fashion to warrant a project in https://github.com/bitcoin/bitcoin/projects? I haven't heard of it in quite a while. It may have been because of sidetracking due to segwit ofcourse.
< GitHub105> [bitcoin] laanwj closed pull request #8787: [Doc] Add missing autogen to example builds (master...patch-1) https://github.com/bitcoin/bitcoin/pull/8787
< GitHub66> bitcoin/master 279bbad Wladimir J. van der Laan: Merge #8787: [Doc] Add missing autogen to example builds...
< GitHub66> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/9da7366be19b...279bbadc8ba9
< GitHub66> bitcoin/master 3f58a28 Amir Abrams: [Doc] Add missing autogen to example builds
< GitHub196> [bitcoin] laanwj closed pull request #8786: Mandatory copyright agreement (master...copyright-contributing) https://github.com/bitcoin/bitcoin/pull/8786
< GitHub98> bitcoin/master 9da7366 Wladimir J. van der Laan: Merge #8786: Mandatory copyright agreement...
< GitHub98> bitcoin/master 4b67402 Andrew Chow: Mandatory copyright agreement...
< GitHub98> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/08cc5fd66645...9da7366be19b
< GitHub151> [bitcoin] laanwj closed pull request #8801: [trivial] Switching from Boost for-each macros to C++11 for-each (master...tjps_foreach) https://github.com/bitcoin/bitcoin/pull/8801

2016-09-24

< GitHub104> [bitcoin] Derek701 opened pull request #8805: Trivial: Grammar and capitalization (master...master) https://github.com/bitcoin/bitcoin/pull/8805
< GitHub179> [bitcoin] MarcoFalke closed pull request #8790: [test] Remove redundant debug print in addrman_tests (master...Mf1609-testsDeletePrint) https://github.com/bitcoin/bitcoin/pull/8790
< GitHub153> bitcoin/master 08cc5fd MarcoFalke: Merge #8790: [test] Remove redundant debug print in addrman_tests...
< GitHub153> bitcoin/master 3333bd2 MarcoFalke: [test] Remove redundant print in addrman_tests
< GitHub153> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/e31a43c725eb...08cc5fd66645
< GitHub68> [bitcoin] MarcoFalke closed pull request #8789: [qa] pull-tester: Only print output when failed (master...Mf1609-qaPrintFailedOnly) https://github.com/bitcoin/bitcoin/pull/8789
< GitHub185> bitcoin/master e31a43c MarcoFalke: Merge #8789: [qa] pull-tester: Only print output when failed...
< GitHub185> bitcoin/master fa427ce MarcoFalke: [qa] pull-tester: Only print output when failed
< GitHub185> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/24f72e9f3fd0...e31a43c725eb
< GitHub187> [bitcoin] fanquake closed pull request #8804: bitcoin -> litecoin (0.13...0.13) https://github.com/bitcoin/bitcoin/pull/8804
< GitHub0> [bitcoin] wangxinxi opened pull request #8804: bitcoin -> litecoin (0.13...0.13) https://github.com/bitcoin/bitcoin/pull/8804
< GitHub134> [bitcoin] jl2012 opened pull request #8803: Reduce number of repeated tests in p2p-segwit.py (master...patch-17) https://github.com/bitcoin/bitcoin/pull/8803
< luke-jr> ask in #bitcoin if it isn't about development
< tucker111> got some questions on bitcoin core if anyone could help...

2016-09-23

< GitHub166> [bitcoin] tjps opened pull request #8801: [trivial] Switching from Boost for-each macros to C++11 for-each (master...tjps_foreach) https://github.com/bitcoin/bitcoin/pull/8801
< cfields> morcos: i believe the only hot move is in the checkqueue. https://github.com/bitcoin/bitcoin/blob/master/src/checkqueue.h#L150
< GitHub168> [bitcoin] jonasschnelli closed pull request #8371: [Qt] Add out-of-sync modal info layer (master...2016/07/UI-out-of-sync) https://github.com/bitcoin/bitcoin/pull/8371
< GitHub48> bitcoin/master a001f18 Jonas Schnelli: [Qt] Always pass the numBlocksChanged signal for headers tip changed
< GitHub48> bitcoin/master bd44a04 Jonas Schnelli: [Qt] make Out-Of-Sync warning icon clickable
< GitHub48> bitcoin/master 0904c3c Jonas Schnelli: [Refactor] refactor function that forms human readable text out of a timeoffset
< GitHub48> [bitcoin] jonasschnelli pushed 8 new commits to master: https://github.com/bitcoin/bitcoin/compare/d2e46558ba0e...24f72e9f3fd0
< jonasschnelli> Anyone interested in reviewing the "generic"-statistics PR (currently mempool only)? https://github.com/bitcoin/bitcoin/pull/8501
< GitHub141> [bitcoin] laanwj closed pull request #8661: Do not set an addr time penalty when a peer advertises itself. (master...addrman_nopenalty_self) https://github.com/bitcoin/bitcoin/pull/8661
< GitHub8> bitcoin/master d2e4655 Wladimir J. van der Laan: Merge #8661: Do not set an addr time penalty when a peer advertises itself....
< GitHub8> bitcoin/master 6d0ced1 Gregory Maxwell: Do not set an addr time penalty when a peer advertises itself....
< GitHub8> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/5d0219d983b6...d2e46558ba0e
< GitHub132> bitcoin/master 5d0219d Wladimir J. van der Laan: Merge #8793: Do not shadow in src/qt...
< GitHub128> [bitcoin] laanwj closed pull request #8793: Do not shadow in src/qt (master...20160922_Wshadow_qt) https://github.com/bitcoin/bitcoin/pull/8793
< GitHub132> bitcoin/master f839350 Pavel Janík: Do not shadow in src/qt
< GitHub132> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/2b514aa2eae6...5d0219d983b6
< blaker> hi bitcoin core devs. I am at a conference where a talk later today called 'exploiting trust in deterministic builds' is going to be presented. Since you use reproducible builds maybe it is of interest. paper should be available via springer
< gmaxwell> fwiw, I found more bandwidth was being used by bitcoinj clients than ibding bitcoin core.
< gmaxwell> hm. that allow edits thing defaults to on, so next time someone PRs their whole altcoin to our repo we could go and replace their code with bitcoin? :P "we upgraded your altcoin, enjoy!"
< GitHub44> [bitcoin] gmaxwell opened pull request #8800: Fetch w/o CB if mempool empty, don't use HB mode if blocks only. (master...no-hb-in-bo) https://github.com/bitcoin/bitcoin/pull/8800

2016-09-22

< luke-jr> Bitcoin started off with a wxWidgets Windows-only GUI client which was abandoned a long time ago
< luke-jr> Chris_Stewart_5: this was before Bitcoin Core's time, and also before my time :p
< Chris_Stewart_5> luke-jr: Was that functionality inside of Bitcoin Core? Inputing their ip address?
< luke-jr> always annoying when people hear "I'm running Bitcoin Core" and their immediate reply is "you should switch to Electrum!"
< morcos> cfields: what happened to: https://github.com/theuni/bitcoin/tree/copy-move ?
< CodeShark> Hell, if I wanted to go that route I could probably write a much thinner custom server - or a custom build of bitcoin core, even
< CodeShark> I run multiple bitcoin core instances and reluctantly use bip37
< petertodd> CodeShark: indeed - python-bitcoinlib is likely used 99% of the time with a local copy of Bitcoin Core
< instagibbs> is the bitcoin wiki down for others for the last few days?
< wumpus> this is not bitcoin core specific but everyone-that-embeds-that-pubkey specific
< wumpus> it applies to everyone using the key, not just users of bitcoin core
< gmaxwell> I was thinking of limiting the applicability of the penultimate alert to users of Bitcoin Core, open to suggestions.
< gmaxwell> petertodd: if I can parition your network I can make your client show "Your bitcoin is outdated and vulnerable. You MUST download an update to continue. http://bitcoinscam.eu/"
< gmaxwell> (1) Create a bitcoincore.org or bitcoin.org announcement message.
< BlueMatt> (ie reaching out to non-bitcoin core people)
< wumpus> yes good point btcdrak PSA: I've started using the new feature of github projects for tracking a few longer-running projects: https://github.com/bitcoin/bitcoin/projects
< btcdrak> There is also this nice project https://github.com/bitcoin/bitcoin/projects/5
< btcdrak> sipa: I think there are a couple of niggles on the BIP also https://github.com/bitcoin/bips/pull/423
< wumpus> there seems to be disagreement on the RPC behavior change in Fix issue with conflicted mempool tx in listsinceblock https://github.com/bitcoin/bitcoin/pull/8757
< gmaxwell> reminder on milestone 22, https://github.com/bitcoin/bitcoin/milestone/22 (and if there are 0.13 things in that, which aren't tagged, make sure they get tagged)
< wumpus> Make non-minimal OP_IF/NOTIF argument non-standard for P2WSH https://github.com/bitcoin/bitcoin/pull/8526
< wumpus> Check bad witness and implement several policy limits for segwit scripts https://github.com/bitcoin/bitcoin/pull/8499
< wumpus> Add policy: null signature for failed CHECK(MULTI)SIG https://github.com/bitcoin/bitcoin/pull/8634
< wumpus> <jl2012> I may not join the meeting today but I suggest not to do https://github.com/bitcoin/bitcoin/pull/8654 in 0.13.1
< 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
< GitHub197> [bitcoin] jonnynewbs opened pull request #8796: [trivial] fix mempool comment (outdated by BIP125) (master...trivial_comment) https://github.com/bitcoin/bitcoin/pull/8796
< haakonn> agh, sorry for the noise, thought this was #bitcoin
< wumpus> #bitcoin
< JackH> I am just thinking in terms of opening up Bitcoin, beside fixing malleability, if there is anything on the table?
< JackH> Is there any reason to think that Bitcoin wont eventually be taken over in terms of usage by Ethereum (serious question)
< GitHub162> [bitcoin] laudaa opened pull request #8795: [doc] Mention Gitian building script in documents. (master...master) https://github.com/bitcoin/bitcoin/pull/8795
< GitHub168> [bitcoin] paveljanik opened pull request #8794: Enable -Wshadow by default (master...20160922_Wshadow_enable) https://github.com/bitcoin/bitcoin/pull/8794
< GitHub82> [bitcoin] laanwj closed pull request #8785: Comment on CNode::nLocalServices meaning (master...nlocalservisme) https://github.com/bitcoin/bitcoin/pull/8785
< GitHub41> bitcoin/master 2b514aa Wladimir J. van der Laan: Merge #8785: Comment on CNode::nLocalServices meaning...
< GitHub41> bitcoin/master b5ccded instagibbs: Comment on CConnman::nLocalServices meaning
< GitHub41> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/26b370a93700...2b514aa2eae6
< GitHub97> [bitcoin] paveljanik opened pull request #8793: Do not shadow in src/qt (master...20160922_Wshadow_qt) https://github.com/bitcoin/bitcoin/pull/8793
< GitHub99> [bitcoin] MarcoFalke opened pull request #8791: [travis] cross-mac: explicitly enable gui (master...Mf1609-travisGui) https://github.com/bitcoin/bitcoin/pull/8791
< GitHub35> [bitcoin] MarcoFalke opened pull request #8790: [test] Remove redundant debug print in addrman_tests (master...Mf1609-testsDeletePrint) https://github.com/bitcoin/bitcoin/pull/8790
< GitHub101> [bitcoin] MarcoFalke opened pull request #8789: [qa] pull-tester: Only print output when failed (master...Mf1609-qaPrintFailedOnly) https://github.com/bitcoin/bitcoin/pull/8789
< GitHub156> [bitcoin] laanwj closed pull request #8636: Implement NULLDUMMY softfork (BIP147) (master...nulldummy) https://github.com/bitcoin/bitcoin/pull/8636
< GitHub84> bitcoin/master 26b370a Wladimir J. van der Laan: Merge #8636: Implement NULLDUMMY softfork (BIP147)...
< GitHub84> bitcoin/master 482f852 Johnson Lau: Implement NULLDUMMY softfork
< GitHub84> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/7008e28136b5...26b370a93700
< jonasschnelli> where the bitcoin-cli tools would use --wallet=<walletid> to pass the request to /<walletid>
< jonasschnelli> If we would select the wallet depending on the endpoint, we could just use something like bitcoin-cli --wallet=<walletid> getbalance
< GitHub79> [bitcoin] jonasschnelli opened pull request #8788: [RPC] Give RPC commands more information about the RPC request (master...2016/09/rpc_container) https://github.com/bitcoin/bitcoin/pull/8788
< jonasschnelli> Linking errors are here: https://travis-ci.org/bitcoin/bitcoin/jobs/160477991#L1567
< jonasschnelli> cfields: do you see a/the reason why this fails on gcc but not on clang? https://github.com/bitcoin/bitcoin/pull/8745/files#diff-480477e89f9b6ddafb30c4383dcdd705R407
< GitHub19> [bitcoin] laanwj closed pull request #8779: [contrib] Delete spendfrom (master...Mf1609-deleteAllTheThings) https://github.com/bitcoin/bitcoin/pull/8779
< GitHub101> bitcoin/master 7008e28 Wladimir J. van der Laan: Merge #8779: [contrib] Delete spendfrom...
< GitHub101> bitcoin/master fa81d09 MarcoFalke: [contrib] Delete spendfrom
< GitHub101> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/3166dff48f35...7008e28136b5
< GitHub4> [bitcoin] laanwj closed pull request #8423: [depends] expat 2.2.0, ccache 3.3.1, fontconfig 2.12.1 (master...expat-ccache-jul) https://github.com/bitcoin/bitcoin/pull/8423
< GitHub16> bitcoin/master 86d410d fanquake: [depends] fontconfig 2.12.1
< GitHub16> bitcoin/master 9616ac8 fanquake: [depends] ccache 3.3.1
< GitHub16> bitcoin/master 6b6cbdd fanquake: [depends] expat 2.2.0
< GitHub16> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/64dc6457454a...3166dff48f35
< GitHub28> [bitcoin] laanwj closed pull request #8186: [0.12.2] backport: getblockchaininfo: make bip9_softforks an object, not an array. (0.12...Mf1606-rpcBip9Backport) https://github.com/bitcoin/bitcoin/pull/8186
< GitHub166> [bitcoin] laanwj closed pull request #8783: [share] remove qt/protobuf.pri (master...Mf1609-deleteqtProto) https://github.com/bitcoin/bitcoin/pull/8783
< GitHub146> bitcoin/master 64dc645 Wladimir J. van der Laan: Merge #8783: [share] remove qt/protobuf.pri...
< GitHub146> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/ca69ef4880d1...64dc6457454a
< GitHub146> bitcoin/master fa13c5c MarcoFalke: [share] remove qt/protobuf.pri...
< GitHub172> [bitcoin] laanwj closed pull request #8781: [contrib] delete qt_translations.py (master...Mf1609-deleteQtTrans) https://github.com/bitcoin/bitcoin/pull/8781
< GitHub21> bitcoin/master ca69ef4 Wladimir J. van der Laan: Merge #8781: [contrib] delete qt_translations.py...
< GitHub21> bitcoin/master faf87af MarcoFalke: [contrib] delete qt_translations.py...
< GitHub21> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/cf5ebaa921a9...ca69ef4880d1
< wumpus> cfields: I've created a project for your P2P refactor, please add if I missed anything: https://github.com/bitcoin/bitcoin/projects/4
< wumpus> achow101: seems to work fairly well https://github.com/bitcoin/bitcoin/projects/1
< GitHub131> [bitcoin] AmirAbrams opened pull request #8787: [Doc] Add missing autogen to example builds (master...patch-1) https://github.com/bitcoin/bitcoin/pull/8787
< wumpus> just paste the script from https://github.com/bitcoin/bitcoin/pull/8674 somewhere into the test process in travis.yml
< wumpus> I've approved https://github.com/bitcoin/bitcoin/pull/8783 but it still shows as 0
< GitHub82> [bitcoin] achow101 opened pull request #8786: Mandatory copyright agreement (master...copyright-contributing) https://github.com/bitcoin/bitcoin/pull/8786
< GitHub154> [bitcoin] instagibbs opened pull request #8785: Comment on CNode::nLocalServices meaning (master...nlocalservisme) https://github.com/bitcoin/bitcoin/pull/8785

2016-09-21

< GitHub136> [bitcoin] luke-jr opened pull request #8784: Copyright headers for build scripts (master...license_build) https://github.com/bitcoin/bitcoin/pull/8784
< GitHub177> [bitcoin] MarcoFalke opened pull request #8783: [share] remove qt/protobuf.pri (master...Mf1609-deleteqtProto) https://github.com/bitcoin/bitcoin/pull/8783
< murch1> completely off-topic, but Pidgin sucks for IRC • I used to have a decent client when I still used Linux, but I've only started using IRC for Bitcoin again recently. Any recommendation for a decent IRC client?
< murch1> gmaxwell: Here is a question. For a very long time I thought that spending several outputs from the same address required only one signature. I recently saw an answer on Bitcoin.SE that said that spending transactions from the same address still uses a full signature each. Is the only concern privacy then?
< GitHub125> [bitcoin] MarcoFalke opened pull request #8781: [contrib] delete qt_translations.py (master...Mf1609-deleteQtTrans) https://github.com/bitcoin/bitcoin/pull/8781
< GitHub54> [bitcoin] MarcoFalke opened pull request #8780: [rpc] Deprecate getinfo (master...Mf1609-getinfoDeprecate) https://github.com/bitcoin/bitcoin/pull/8780
< GitHub24> [bitcoin] MarcoFalke opened pull request #8779: [contrib] Delete spendfrom (master...Mf1609-deleteAllTheThings) https://github.com/bitcoin/bitcoin/pull/8779
< instagibbs> this channel is for near-term bitcoin core development, your question is about an implementation of elements sidechain and/or future quantum advances
< instagibbs> xinxi: maybe #bitcoin-wizards ?
< murch> gmaxwell: Did you have something in particular in mind to try? We (sipa, you, instagibbs, me) did discuss this a few months ago already though. And I've also read your Bitcoin Wiki page on Coin Selection. :)
< murch> sipa: E.g. Bitcoin Core uses an initial guess of the fee to restrict solutions in the selection pass through. It will only update the fee guess after the selection attempt if it didn't produce a satisfying result.
< BlueMatt> murch: hmmm, i cant say i recall exactly what the behavior there is...its also very much based on what bitcoin core did at the time (and its non-fee anti-spam measures)
< murch> gmaxwell: I was planning to provide it around Scaling Bitcoin. There was a few more things I wanted to do before publishing it. Trying other strategies is a case of inheritance and overriding a single function though.
< gmaxwell> BlueMatt: you mean making the final utxo set 126 times larger than bitcoin core isn't good?
< sipa> you can disable it by setting listenonion=0 in bitcoin.conf
< skyraider> (for additional context, this pull request supposedly fixes that message, but apparently not https://github.com/bitcoin/bitcoin/pull/7637)
< murch> instagibbs: All bitcoin values are in satoshi.
< sipa> BlueMatt: maybe a #!/bin/sh echo "The Bitcoin Core PPA no longer supports Ubuntu 12.04" would be better than just empty package?
< BlueMatt> sipa: yea, if i get emails then #bitcoin is generally guaranteed to complain
< sipa> BlueMatt: a few people in #bitcoin were confused by bitcoind disappearing
< jonasschnelli> BlueMatt: while your here.. :), do you use the contrib/debian packaging dir for your bitcoin PPA?
< GitHub78> [bitcoin] MarcoFalke closed pull request #8536: [qa] Adjust poll interval for micro-optimization of run time (master...Mf1608-qaOptSync) https://github.com/bitcoin/bitcoin/pull/8536
< GitHub191> [bitcoin] laanwj closed pull request #8656: Trivial: Do not shadow global variable fileout (master...20160902_Wshadow_fileout) https://github.com/bitcoin/bitcoin/pull/8656
< GitHub170> bitcoin/master cf5ebaa Wladimir J. van der Laan: Merge #8656: Trivial: Do not shadow global variable fileout...
< GitHub170> bitcoin/master 7c069a7 Pavel Janík: Do not shadow global variable
< GitHub170> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/fec6af744014...cf5ebaa921a9
< GitHub97> [bitcoin] laanwj closed pull request #8777: [qt] WalletModel: Expose disablewallet (master...Mf1609-initDisableWallet) https://github.com/bitcoin/bitcoin/pull/8777
< GitHub146> bitcoin/master fec6af7 Wladimir J. van der Laan: Merge #8777: [qt] WalletModel: Expose disablewallet...
< GitHub146> bitcoin/master 6666ca6 MarcoFalke: [qt] WalletModel: Expose disablewallet
< GitHub146> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/52b5a8785de7...fec6af744014
< wumpus> I should likely put https://github.com/bitcoin/bitcoin/pull/8249 on ice; having less effective ASLR is great compared to the alternative of crashing with stack protectors :)
< sipa> wumpus: i guess i'm biased because i learned c++ from bitcoin's codebase :)
< GitHub122> [bitcoin] MarcoFalke opened pull request #8777: [qt] WalletModel: Expose disablewallet (master...Mf1609-initDisableWallet) https://github.com/bitcoin/bitcoin/pull/8777
< wumpus> re: wallet, I'd really like to get https://github.com/bitcoin/bitcoin/pull/7551 in
< GitHub143> [bitcoin] MarcoFalke closed pull request #8773: Trivial Bugfix: doc/gitian-building.md: Link to release-process needs to be updated (master...bugfix_link_20160921) https://github.com/bitcoin/bitcoin/pull/8773
< GitHub109> bitcoin/master 52b5a87 MarcoFalke: Merge #8773: Trivial Bugfix: doc/gitian-building.md: Link to release-process needs to be updated...
< GitHub109> bitcoin/master 6f933c6 Luke Dashjr: Trivial Bugfix: doc/gitian-building.md: Link to release-process needs to be updated...
< GitHub109> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/886e8c9b7269...52b5a8785de7
< wumpus> MarcoFalke: certainly trivial things like https://github.com/bitcoin/bitcoin/pull/8773 you can just merge if they check out
< GitHub81> [bitcoin] laanwj closed pull request #8768: init: Get rid of fDisableWallet (master...Mf1609-initDisableWallet) https://github.com/bitcoin/bitcoin/pull/8768
< GitHub94> bitcoin/master 886e8c9 Wladimir J. van der Laan: Merge #8768: init: Get rid of fDisableWallet...
< GitHub94> bitcoin/master fab9107 MarcoFalke: init: Get rid of fDisableWallet
< GitHub94> bitcoin/master fa58edb MarcoFalke: [wallet] Introduce DEFAULT_DISABLE_WALLET
< GitHub94> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/84c9a048559d...886e8c9b7269
< GitHub181> [bitcoin] luke-jr opened pull request #8776: Wallet refactoring leading up to multiwallet (master...multiwallet_prefactor_wallet) https://github.com/bitcoin/bitcoin/pull/8776
< GitHub30> [bitcoin] luke-jr opened pull request #8775: RPC refactoring: Never access wallet directly, only via new CRPCRequestInfo (master...multiwallet_prefactor_rpc) https://github.com/bitcoin/bitcoin/pull/8775
< GitHub124> [bitcoin] luke-jr opened pull request #8774: Qt refactors to better abstract wallet access (master...multiwallet_prefactor_qt) https://github.com/bitcoin/bitcoin/pull/8774
< GitHub137> [bitcoin] luke-jr opened pull request #8773: Trivial Bugfix: doc/gitian-building.md: Link to release-process needs to be updated (master...bugfix_link_20160921) https://github.com/bitcoin/bitcoin/pull/8773
< GitHub10> [bitcoin] luke-jr opened pull request #8772: [0.13] Backports (0.13...backports-0.13) https://github.com/bitcoin/bitcoin/pull/8772
< MarcoFalke> Open a pull request to bitcoin:master in some other repo. Force push to bitcoin:master...
< MarcoFalke> wumpus, can you do "git push -f git@github.com:MarcoFalke/bitcoin.git HEAD:Mf1609-ContributeDoc"?
< GitHub138> [bitcoin] laanwj closed pull request #8769: Trivial: Fix ISO URL, capitalization (master...patch-4) https://github.com/bitcoin/bitcoin/pull/8769
< GitHub93> bitcoin/master 84c9a04 Wladimir J. van der Laan: Merge #8769: Trivial: Fix ISO URL, capitalization...
< GitHub93> bitcoin/master 04d91f4 unsystemizer: Trivial: Fix ISO URL, capitalization...
< GitHub93> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b4f53641a95d...84c9a048559d
< GitHub92> [bitcoin] luke-jr opened pull request #8771: CONTRIBUTING: Mention not to open several pulls (master...Mf1609-ContributeDoc) https://github.com/bitcoin/bitcoin/pull/8771
< wumpus> "git push -f git@github.com:unsystemizer/bitcoin.git HEAD:patch-4" seems to have worked
< wumpus> MarcoFalke: yes I was thining to difficult, I thought I had to push to some special pulls branch on bitcoin/bitcoin
< wumpus> "Add more commits by pushing to the patch-4 branch on unsystemizer/bitcoin." ah
< jonasschnelli> Luke-Jr: Yes. And sorry for "shooting" in the same direction with https://github.com/bitcoin/bitcoin/pull/8764 I like your mutliwallet PR and really like to boost getting this in
< jonasschnelli> Mostly search and replace thought: https://github.com/bitcoin/bitcoin/pull/8764
< jonasschnelli> Luke-Jr: I agree that it would be better to first merge https://github.com/bitcoin/bitcoin/pull/8694 and then improving multiwallet stuff.
< jonasschnelli> https://github.com/bitcoin/bitcoin/pull/8371 needs another ACK (or a re-ack from MarcoFalke) before I merge it.
< GitHub106> [bitcoin] laanwj closed pull request #8765: [Wallet] remove "unused" ThreadFlushWalletDB from removeprunedfunds (master...2016/09/flush_wallet_dump) https://github.com/bitcoin/bitcoin/pull/8765
< GitHub23> bitcoin/master b4f5364 Wladimir J. van der Laan: Merge #8765: [Wallet] remove "unused" ThreadFlushWalletDB from removeprunedfunds...
< GitHub23> bitcoin/master c6f5ca8 Jonas Schnelli: [Wallet] remove "unused" ThreadFlushWalletDB from removeprunedfunds
< GitHub23> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/1c24d5f63720...b4f53641a95d
< GitHub113> [bitcoin] laanwj closed pull request #8687: Bugfix: RPC/Wallet: removeprunedfunds should flush the wallet db (master...bugfix_removeprunedfunds) https://github.com/bitcoin/bitcoin/pull/8687
< luke-jr> wumpus: I guess I should just close https://github.com/bitcoin/bitcoin/pull/8687 and simply remove the call in a new PR?
< wumpus> phantomcircuit: yes, the default mutex used in bitcoin core is a recursive one

2016-09-20

< GitHub150> [bitcoin] MarcoFalke closed pull request #8286: [doc] Docs and copyright headers bump (master...Mf1607-trivialPre13) https://github.com/bitcoin/bitcoin/pull/8286
< GitHub79> [bitcoin] unsystemizer opened pull request #8769: Trivial: Fix ISO URL, capitalization (master...patch-4) https://github.com/bitcoin/bitcoin/pull/8769
< GitHub4> [bitcoin] unsystemizer closed pull request #8766: Fix URL for Debian 8.5.0 ISO (master...patch-3) https://github.com/bitcoin/bitcoin/pull/8766
< gmaxwell> someone in #bitcoin with a gdb backtrace of a node that keeps crashing during sync running 0.13 ppa... crashing at assert(coins) in CheckTxInputs.
< gmaxwell> tucker111: arubi: achow101: etc. please move to #bitcoin for further discussion. :)
< arubi> tucker111, -> #bitcoin
< achow101> so if your wallet.dat is gone (as it likely is) then you have lost your bitcoin
< achow101> you don't have the private key so you can't get any of the Bitcoin that was associated with it
< GitHub147> [bitcoin] MarcoFalke opened pull request #8768: init: Get rid of fDisableWallet (master...Mf1609-initDisableWallet) https://github.com/bitcoin/bitcoin/pull/8768
< GitHub110> [bitcoin] unsystemizer opened pull request #8766: Fix URL for Debian 8.5.0 ISO (master...patch-3) https://github.com/bitcoin/bitcoin/pull/8766
< GitHub94> [bitcoin] laanwj closed pull request #8739: [qa] Fix broken sendcmpct test in p2p-compactblocks.py (master...fix-cb-test) https://github.com/bitcoin/bitcoin/pull/8739
< GitHub189> bitcoin/master 1c24d5f Wladimir J. van der Laan: Merge #8739: [qa] Fix broken sendcmpct test in p2p-compactblocks.py...
< GitHub189> bitcoin/master 157254a Suhas Daftuar: Fix broken sendcmpct test in p2p-compactblocks.py...
< GitHub189> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/aff69278063c...1c24d5f63720
< GitHub50> [bitcoin] jonasschnelli opened pull request #8765: [Wallet] remove "unused" ThreadFlushWalletDB from removeprunedfunds (master...2016/09/flush_wallet_dump) https://github.com/bitcoin/bitcoin/pull/8765
< wumpus> the whole thing is NOP, *apart* from renaming the RPC thread to 'bitcoin-wallet' :')
< GitHub164> [bitcoin] laanwj closed pull request #8762: Trivial: Fix typo (master...patch-1) https://github.com/bitcoin/bitcoin/pull/8762
< GitHub175> bitcoin/master aff6927 Wladimir J. van der Laan: Merge #8762: Trivial: Fix typo...
< GitHub175> bitcoin/master 12a721b Marty Jones: Trivial: Fix typo
< GitHub175> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/6052d5091057...aff69278063c
< GitHub165> [bitcoin] jonasschnelli closed pull request #8672: Qt: Show transaction size in transaction details window (master...qttxsizeindetails) https://github.com/bitcoin/bitcoin/pull/8672
< GitHub161> bitcoin/master c015634 Hampus Sjöberg: qt: Adding transaction size to transaction details window
< GitHub161> bitcoin/master 6052d50 Jonas Schnelli: Merge #8672: Qt: Show transaction size in transaction details window...
< GitHub161> bitcoin/master fdf82fb Hampus Sjöberg: Adding method GetTotalSize() to CTransaction...
< GitHub161> [bitcoin] jonasschnelli pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/4335d5a41bc2...6052d5091057
< GitHub129> [bitcoin] jonasschnelli opened pull request #8764: [Wallet] get rid of pwalletMain, add simple CWallets infrastructure (master...2016/09/wallet_pointer) https://github.com/bitcoin/bitcoin/pull/8764
< GitHub141> [bitcoin] laanwj closed pull request #7783: [Qt] RPC-Console: support nested commands and simple value queries (master...2016/04/qt_console_nested) https://github.com/bitcoin/bitcoin/pull/7783
< GitHub67> bitcoin/master 1586044 Jonas Schnelli: [Qt] RPC-Console: support nested commands and simple value queries...
< GitHub67> bitcoin/master 4335d5a Wladimir J. van der Laan: Merge #7783: [Qt] RPC-Console: support nested commands and simple value queries...
< GitHub67> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/a1f8d3ed9569...4335d5a41bc2
< wumpus> 'wallet applications' that run inside bitcoin core
< GitHub169> bitcoin/0.13 d6ebe13 MarcoFalke: [qa] Refactor RPCTestHandler to prevent TimeoutExpired...
< GitHub169> bitcoin/0.13 ae8c7df MarcoFalke: [qa] create_cache: Delete temp dir when done...
< GitHub169> bitcoin/0.13 63462c2 whythat: [qa] remove root test directory for RPC tests...
< GitHub21> [bitcoin] laanwj closed pull request #8744: [0.13.1] qa Backports (0.13...Mf1609-qaBackports) https://github.com/bitcoin/bitcoin/pull/8744
< GitHub169> [bitcoin] laanwj pushed 4 new commits to 0.13: https://github.com/bitcoin/bitcoin/compare/4731623777ab...8d9e8adc05f4
< GitHub182> [bitcoin] laanwj closed pull request #8696: [Wallet] Remove last external reference to CWalletDB (master...2016-09-09-cwallet-listaccountcreditdebit) https://github.com/bitcoin/bitcoin/pull/8696
< GitHub55> bitcoin/master 02e2a81 Patrick Strateman: Remove pwalletdb parameter from CWallet::AddAccountingEntry
< GitHub55> bitcoin/master d2e678d Patrick Strateman: Add CWallet::ReorderTransactions and use in accounting_tests.cpp
< GitHub55> bitcoin/master 59adc86 Patrick Strateman: Add CWallet::ListAccountCreditDebit...
< GitHub55> [bitcoin] laanwj pushed 5 new commits to master: https://github.com/bitcoin/bitcoin/compare/02ac669730c0...a1f8d3ed9569
< GitHub78> [bitcoin] laanwj closed pull request #8760: [init] Get rid of some ENABLE_WALLET (master...Mf1609-walletInitGuard) https://github.com/bitcoin/bitcoin/pull/8760
< GitHub13> bitcoin/master 02ac669 Wladimir J. van der Laan: Merge #8760: [init] Get rid of some ENABLE_WALLET...
< GitHub13> bitcoin/master faddd62 MarcoFalke: init: Get rid of some ENABLE_WALLET
< GitHub13> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/82eacc786d8d...02ac669730c0
< mang0_> Does anyone know the best place to get a small fast bitcoin loan?
< wumpus> I was surprised openbsd gives 5MiB or so of locked ram by default to processes, it's not usually known for its lenient ulimits ( https://github.com/bitcoin/bitcoin/blob/master/doc/build-openbsd.md#resource-limits :p)
< GitHub147> [bitcoin] laanwj closed pull request #8700: [copyright] add MIT license headers to .sh scripts where missing (master...sh-copyright) https://github.com/bitcoin/bitcoin/pull/8700
< GitHub102> bitcoin/master 82eacc7 Wladimir J. van der Laan: Merge #8700: [copyright] add MIT license headers to .sh scripts where missing...
< GitHub102> bitcoin/master 0766d1c isle2983: [copyright] add MIT license headers to .sh scripts where missing...
< GitHub102> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/d8b4b631c5be...82eacc786d8d
< jonasschnelli> Anyone interested in reviewing/acking https://github.com/bitcoin/bitcoin/pull/7783 (nested commands for Qt), and https://github.com/bitcoin/bitcoin/pull/8371 (out-of-sync info layer)?
< GitHub2> [bitcoin] CryptAxe opened pull request #8763: Optionally sweep funds from private key. (master...2751PullRequest) https://github.com/bitcoin/bitcoin/pull/8763
< GitHub27> [bitcoin] murtyjones opened pull request #8762: Trivial: Fix typo (master...patch-1) https://github.com/bitcoin/bitcoin/pull/8762

2016-09-19

< GitHub173> [bitcoin] laanwj closed pull request #8676: Add missing copyright headers (master...missing-copyright) https://github.com/bitcoin/bitcoin/pull/8676
< GitHub23> bitcoin/master d8b4b63 Wladimir J. van der Laan: Merge #8676: Add missing copyright headers...
< GitHub23> bitcoin/master 783e930 isle2983: [copyright] Add missing copyright headers
< GitHub23> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/047ded0b12f3...d8b4b631c5be