2016-09-28

< GitHub190> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/7b05af634be4...dc641415e75e
< GitHub121> [bitcoin] laanwj closed pull request #8823: doc: Add privacy recommendation when running hidden service (master...2016_09_tor_recommendation) https://github.com/bitcoin/bitcoin/pull/8823
< GitHub61> bitcoin/master 7b05af6 Wladimir J. van der Laan: Merge #8823: doc: Add privacy recommendation when running hidden service...
< GitHub61> bitcoin/master ac01ff2 Wladimir J. van der Laan: doc: Add privacy recommendation when running hidden service
< GitHub61> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/14b7b3fb9f79...7b05af634be4
< jonasschnelli> Maybe I missed something: whats the reason why importmulti requires a scriptPubKey https://github.com/bitcoin/bitcoin/pull/7551/files#diff-522490d83dce5375d423b23886e4125eR979?
< sipa> no objection to moving it to bitcoin-core bte
< wumpus> it isn't part of the bitcoin core release cycle either
< wumpus> I have some other scripts too, a few other code analysis tools, and to build release notes, author lists etc. All can be useful, but it's pretty much a meta-project to bitcoin itself.
< wumpus> should I move/clone https://github.com/laanwj/bitcoin-maintainer-tools into bitcoin-core?
< GitHub50> [bitcoin] laanwj closed pull request #8814: [wallet, policy] ParameterInteraction: Don't allow 0 fee (master...Mf1607-walletHighFeeWarn) https://github.com/bitcoin/bitcoin/pull/8814
< GitHub190> bitcoin/master 14b7b3f Wladimir J. van der Laan: Merge #8814: [wallet, policy] ParameterInteraction: Don't allow 0 fee...
< GitHub190> bitcoin/master fa4bfb4 MarcoFalke: [wallet, policy] ParameterInteraction: Don't allow 0 fee
< GitHub190> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/fc4f4547b7f6...14b7b3fb9f79

2016-09-27

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

2016-09-26

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