12017-06-07T00:00:46 <bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/75e898c094ee...be3e042c20e2
22017-06-07T00:00:46 <bitcoin-git> bitcoin/master 656dbd8 practicalswift: Perform member initialization in initialization lists where possible
32017-06-07T00:00:47 <bitcoin-git> bitcoin/master be3e042 Pieter Wuille: Merge #10523: Perform member initialization in initialization lists where possible...
42017-06-07T00:00:53 *** jheathco has left #bitcoin-core-dev
52017-06-07T00:01:22 <bitcoin-git> [bitcoin] sipa closed pull request #10523: Perform member initialization in initialization lists where possible (master...initialization-list) https://github.com/bitcoin/bitcoin/pull/10523
62017-06-07T00:24:52 *** Chris_Stewart_5 has joined #bitcoin-core-dev
72017-06-07T00:33:00 *** AaronvanW has quit IRC
82017-06-07T00:41:00 *** talmai has joined #bitcoin-core-dev
92017-06-07T00:41:33 *** Ylbam has quit IRC
102017-06-07T00:44:02 *** btcdrak has joined #bitcoin-core-dev
112017-06-07T00:49:27 *** jtimon has quit IRC
122017-06-07T01:13:09 *** echonaut has quit IRC
132017-06-07T01:13:13 *** echonaut1 has joined #bitcoin-core-dev
142017-06-07T01:14:16 *** beatrootfarmer has joined #bitcoin-core-dev
152017-06-07T01:17:23 *** goatturneer has quit IRC
162017-06-07T01:26:20 *** jl2012 has joined #bitcoin-core-dev
172017-06-07T01:26:59 *** d_t has joined #bitcoin-core-dev
182017-06-07T01:30:03 *** dermoth has quit IRC
192017-06-07T01:30:59 *** dermoth has joined #bitcoin-core-dev
202017-06-07T01:44:11 *** dabura667 has joined #bitcoin-core-dev
212017-06-07T01:48:26 *** dabura667 has quit IRC
222017-06-07T01:48:37 *** dabura667_ has joined #bitcoin-core-dev
232017-06-07T01:59:43 *** Giszmo has quit IRC
242017-06-07T02:11:43 *** goatturneer has joined #bitcoin-core-dev
252017-06-07T02:15:16 *** beatrootfarmer has quit IRC
262017-06-07T02:15:31 *** beatrootfarmer has joined #bitcoin-core-dev
272017-06-07T02:19:07 *** goatturneer has quit IRC
282017-06-07T02:33:10 *** goatturneer has joined #bitcoin-core-dev
292017-06-07T02:35:23 *** goatturner has joined #bitcoin-core-dev
302017-06-07T02:37:05 *** beatrootfarmer has quit IRC
312017-06-07T02:38:29 *** goatturneer has quit IRC
322017-06-07T02:40:41 <cfields> gitian builders: 0.14.2rc2 sigs are pushed
332017-06-07T02:42:38 *** goatturneer has joined #bitcoin-core-dev
342017-06-07T02:46:11 *** goatturner has quit IRC
352017-06-07T02:46:17 *** Chris_Stewart_5 has quit IRC
362017-06-07T02:48:45 *** goatturner has joined #bitcoin-core-dev
372017-06-07T02:50:21 *** beatrootfarmer has joined #bitcoin-core-dev
382017-06-07T02:52:15 *** goatturneer has quit IRC
392017-06-07T02:54:14 *** goatturner has quit IRC
402017-06-07T02:59:47 *** Dyaheon has quit IRC
412017-06-07T03:02:16 *** Dyaheon has joined #bitcoin-core-dev
422017-06-07T03:07:38 *** RubenSomsen has joined #bitcoin-core-dev
432017-06-07T04:04:28 *** talmai has quit IRC
442017-06-07T04:05:45 *** jamesob has joined #bitcoin-core-dev
452017-06-07T04:21:08 *** goatturneer has joined #bitcoin-core-dev
462017-06-07T04:24:32 *** beatrootfarmer has quit IRC
472017-06-07T04:31:06 *** beatrootfarmer has joined #bitcoin-core-dev
482017-06-07T04:34:55 *** goatturneer has quit IRC
492017-06-07T04:41:00 *** goatturneer has joined #bitcoin-core-dev
502017-06-07T04:44:29 *** beatrootfarmer has quit IRC
512017-06-07T04:47:17 *** RubenSomsen has quit IRC
522017-06-07T04:53:33 *** talmai has joined #bitcoin-core-dev
532017-06-07T04:56:01 *** beatrootfarmer has joined #bitcoin-core-dev
542017-06-07T04:59:30 *** goatturneer has quit IRC
552017-06-07T05:02:57 *** wangchun has quit IRC
562017-06-07T05:03:40 *** wangchun has joined #bitcoin-core-dev
572017-06-07T05:16:01 *** d9b4bef9 has quit IRC
582017-06-07T05:16:15 *** goatturneer has joined #bitcoin-core-dev
592017-06-07T05:17:07 *** d9b4bef9 has joined #bitcoin-core-dev
602017-06-07T05:17:35 *** goatturner has joined #bitcoin-core-dev
612017-06-07T05:20:11 *** beatrootfarmer has quit IRC
622017-06-07T05:20:22 *** beatrootfarmer has joined #bitcoin-core-dev
632017-06-07T05:20:53 *** goatturneer has quit IRC
642017-06-07T05:22:17 *** goatturner has quit IRC
652017-06-07T05:54:49 *** jamesob has quit IRC
662017-06-07T06:16:30 *** goatturneer has joined #bitcoin-core-dev
672017-06-07T06:18:20 *** goatturner has joined #bitcoin-core-dev
682017-06-07T06:20:23 *** beatrootfarmer has quit IRC
692017-06-07T06:21:26 *** goatturneer has quit IRC
702017-06-07T06:23:15 *** talmai has quit IRC
712017-06-07T06:24:03 *** RubenSomsen has joined #bitcoin-core-dev
722017-06-07T06:44:05 *** JackH has joined #bitcoin-core-dev
732017-06-07T06:53:38 *** JackH has quit IRC
742017-06-07T06:55:35 *** jamesob has joined #bitcoin-core-dev
752017-06-07T06:56:37 *** SopaXorzTaker has joined #bitcoin-core-dev
762017-06-07T06:57:28 *** BashCo has quit IRC
772017-06-07T06:59:56 *** jamesob has quit IRC
782017-06-07T07:02:42 *** AaronvanW has joined #bitcoin-core-dev
792017-06-07T07:04:07 *** Aaronvan_ has joined #bitcoin-core-dev
802017-06-07T07:07:38 *** AaronvanW has quit IRC
812017-06-07T07:13:25 *** BashCo has joined #bitcoin-core-dev
822017-06-07T07:26:10 *** JackH has joined #bitcoin-core-dev
832017-06-07T07:57:03 *** jamesob has joined #bitcoin-core-dev
842017-06-07T07:59:08 *** timothy has joined #bitcoin-core-dev
852017-06-07T07:59:25 *** goatturneer has joined #bitcoin-core-dev
862017-06-07T08:01:46 *** jamesob has quit IRC
872017-06-07T08:02:56 *** goatturner has quit IRC
882017-06-07T08:03:53 *** v3d has joined #bitcoin-core-dev
892017-06-07T08:05:57 *** hephaestus has quit IRC
902017-06-07T08:12:38 *** riemann has joined #bitcoin-core-dev
912017-06-07T08:24:30 *** riemann has quit IRC
922017-06-07T08:24:32 *** riemann_ has joined #bitcoin-core-dev
932017-06-07T08:39:50 *** RubenSomsen has quit IRC
942017-06-07T08:56:58 *** john1 has joined #bitcoin-core-dev
952017-06-07T08:58:21 *** jannes has joined #bitcoin-core-dev
962017-06-07T09:01:39 *** RubenSomsen has joined #bitcoin-core-dev
972017-06-07T09:03:48 *** john1 has quit IRC
982017-06-07T09:15:35 *** RubenSomsen has quit IRC
992017-06-07T09:20:33 *** hephaestus has joined #bitcoin-core-dev
1002017-06-07T09:23:05 *** v3d has quit IRC
1012017-06-07T09:24:01 *** sjums has joined #bitcoin-core-dev
1022017-06-07T09:29:27 *** jtimon has joined #bitcoin-core-dev
1032017-06-07T09:45:37 <bitcoin-git> [bitcoin] practicalswift opened pull request #10546: Remove unused Boost includes (master...remove-unused-boost-includes) https://github.com/bitcoin/bitcoin/pull/10546
1042017-06-07T09:51:07 *** tunafizz has quit IRC
1052017-06-07T09:51:15 *** tunafizz has joined #bitcoin-core-dev
1062017-06-07T09:58:42 *** jamesob has joined #bitcoin-core-dev
1072017-06-07T10:03:08 *** jamesob has quit IRC
1082017-06-07T10:22:23 *** chjj has quit IRC
1092017-06-07T10:27:51 <wumpus> 0.14.2rc2 executables up https://bitcoin.org/bin/bitcoin-core-0.14.2/test.rc2/
1102017-06-07T10:31:16 *** laurentmt has joined #bitcoin-core-dev
1112017-06-07T10:32:11 *** gielbier has quit IRC
1122017-06-07T10:35:30 *** chjj has joined #bitcoin-core-dev
1132017-06-07T10:41:58 *** laurentmt has quit IRC
1142017-06-07T10:47:42 <timothy> hi, do you think -bip148 option will be included in 0.14.2 final?
1152017-06-07T10:52:30 <SopaXorzTaker> https://bitcointalk.org/index.php?topic=1955073.0
1162017-06-07T10:52:36 <wumpus> timothy: no
1172017-06-07T10:53:06 <timothy> wumpus: bad :P
1182017-06-07T10:53:09 <wumpus> there's no agreement to add bip148 support to core at all, and adding a major feature like that between rcs doesn't happen
1192017-06-07T10:53:18 <wumpus> yeah I know...
1202017-06-07T10:54:41 *** beatrootfarmer has joined #bitcoin-core-dev
1212017-06-07T10:54:49 <timothy> it's disabled by default and optional :P
1222017-06-07T10:56:46 <wumpus> I'm sure there will be a bip148 variant of -final
1232017-06-07T10:57:18 <wumpus> and you can always just cherry-pick the UASF commit on top while testing
1242017-06-07T10:57:56 *** goatturneer has quit IRC
1252017-06-07T10:58:08 <timothy> the UASF commits, since luke-jr is working on DoS stuff
1262017-06-07T11:07:41 <bitcoin-git> [bitcoin] laanwj pushed 1 new commit to 0.14: https://github.com/bitcoin/bitcoin/commit/7a643511b474d53f952d3cd403af51aabd104044
1272017-06-07T11:07:41 <bitcoin-git> bitcoin/0.14 7a64351 Wladimir J. van der Laan: doc: Fill in details about miniupnp CVE-2017-8798
1282017-06-07T11:08:04 <wumpus> anything else that needs special notice in the 0.14.2 changelog?
1292017-06-07T11:13:17 <wumpus> ref: https://github.com/bitcoin/bitcoin/blob/0.14/doc/release-notes.md
1302017-06-07T11:14:04 *** BashCo has quit IRC
1312017-06-07T11:16:00 *** BashCo has joined #bitcoin-core-dev
1322017-06-07T11:22:10 *** john has joined #bitcoin-core-dev
1332017-06-07T11:22:34 *** john is now known as Guest83978
1342017-06-07T11:28:03 *** goatturneer has joined #bitcoin-core-dev
1352017-06-07T11:29:56 *** nemgun has joined #bitcoin-core-dev
1362017-06-07T11:31:32 *** beatrootfarmer has quit IRC
1372017-06-07T11:31:58 *** goatturner has joined #bitcoin-core-dev
1382017-06-07T11:33:00 *** str4d has joined #bitcoin-core-dev
1392017-06-07T11:34:30 *** goatturneer has quit IRC
1402017-06-07T11:39:53 *** vicenteH has joined #bitcoin-core-dev
1412017-06-07T11:50:05 *** str4d has quit IRC
1422017-06-07T11:56:42 *** dabura667_ has quit IRC
1432017-06-07T12:00:11 *** jamesob has joined #bitcoin-core-dev
1442017-06-07T12:03:04 *** Ylbam has joined #bitcoin-core-dev
1452017-06-07T12:04:26 *** jamesob has quit IRC
1462017-06-07T12:17:33 *** dabura667_ has joined #bitcoin-core-dev
1472017-06-07T12:18:48 *** Aaronvan_ is now known as AaronvanW
1482017-06-07T12:19:02 <bitcoin-git> [bitcoin] practicalswift opened pull request #10547: Use std::random::mt19937/uniform_int_distribution (C++11) instead of boost::random equivalents (master...remove-boost-random-dependency) https://github.com/bitcoin/bitcoin/pull/10547
1492017-06-07T12:27:18 <gmaxwell> what why do we have any of that? grr scheduler.
1502017-06-07T12:27:34 <wumpus> yea that should use our own fast random context
1512017-06-07T12:30:43 <wumpus> though this allows for direct conversion of the code, I don't think there's much of a point as we use no fancy statistical distribution stuff at all
1522017-06-07T12:32:27 *** JackH has quit IRC
1532017-06-07T12:32:45 *** fanquake has joined #bitcoin-core-dev
1542017-06-07T12:44:56 *** Chris_Stewart_5 has joined #bitcoin-core-dev
1552017-06-07T13:10:49 <bitcoin-git> [bitcoin] practicalswift opened pull request #10548: Use std::unordered_{map,set} (C++11) instead of boost::unordered_{map,set} (master...unordered_map) https://github.com/bitcoin/bitcoin/pull/10548
1562017-06-07T13:31:46 <bitcoin-git> [bitcoin] jnewbery closed pull request #10540: [WIP] Salvage wallet should not set the aggressive flag on Db::verify() (master...fixsalvage) https://github.com/bitcoin/bitcoin/pull/10540
1572017-06-07T13:32:10 <bitcoin-git> [bitcoin] laanwj pushed 9 new commits to master: https://github.com/bitcoin/bitcoin/compare/be3e042c20e2...46311e792f4e
1582017-06-07T13:32:11 <bitcoin-git> bitcoin/master 578ec80 Luke Dashjr: RPC: rawtransaction: Add RBF support for createrawtransaction
1592017-06-07T13:32:11 <bitcoin-git> bitcoin/master 891c5ee Luke Dashjr: Wallet: Refactor FundTransaction to accept parameters via CCoinControl
1602017-06-07T13:32:12 <bitcoin-git> bitcoin/master 36bcab2 Luke Dashjr: RPC/Wallet: Add RBF support for fundrawtransaction
1612017-06-07T13:32:31 <bitcoin-git> [bitcoin] laanwj closed pull request #9672: Opt-into-RBF for RPC & bitcoin-tx (master...rpc_rbf) https://github.com/bitcoin/bitcoin/pull/9672
1622017-06-07T13:34:20 *** jnewbery has joined #bitcoin-core-dev
1632017-06-07T13:53:43 *** btcdrak has quit IRC
1642017-06-07T14:01:30 *** jamesob has joined #bitcoin-core-dev
1652017-06-07T14:06:07 *** jamesob has quit IRC
1662017-06-07T14:07:33 *** AaronvanW has quit IRC
1672017-06-07T14:09:25 *** Guyver2 has joined #bitcoin-core-dev
1682017-06-07T14:11:55 *** AaronvanW has joined #bitcoin-core-dev
1692017-06-07T14:14:04 <wumpus> any opinion on reverting https://github.com/bitcoin/bitcoin/pull/9672 ? seems to have been a bit of a mistake
1702017-06-07T14:15:24 *** fanquake has quit IRC
1712017-06-07T14:19:51 *** Guest83978 has quit IRC
1722017-06-07T14:24:29 <jnewbery> no need to revert. I was just voicing surprise that it got merged without response to review comments or test cases for the new functionality in bitcoin-tx
1732017-06-07T14:26:14 <wumpus> yes I agree
1742017-06-07T14:27:12 <wumpus> anyhow can be added later I guess, and your comments should still be addressed
1752017-06-07T14:38:14 <wumpus> let's wait for luke-jr's response
1762017-06-07T14:41:11 *** talmai has joined #bitcoin-core-dev
1772017-06-07T14:45:44 *** talmai has quit IRC
1782017-06-07T14:51:34 *** nemgun1 has joined #bitcoin-core-dev
1792017-06-07T14:54:28 *** nemgun has quit IRC
1802017-06-07T15:01:05 *** riemann_ has quit IRC
1812017-06-07T15:01:28 *** gribble has quit IRC
1822017-06-07T15:08:29 *** gribble has joined #bitcoin-core-dev
1832017-06-07T15:11:05 <jonasschnelli> wumpus, jnewbery: Yes. I think 9672 is okay. No need to revert... if there is something, we can fix it up later. Some of the commits also have reviews/utACKs in other PRs.
1842017-06-07T15:16:14 *** BashCo has quit IRC
1852017-06-07T15:16:23 <wumpus> thanks for weighing in, maybe that's what confused me too
1862017-06-07T15:18:00 *** Guest83978 has joined #bitcoin-core-dev
1872017-06-07T15:33:37 *** RubenSomsen has joined #bitcoin-core-dev
1882017-06-07T15:37:00 *** BashCo has joined #bitcoin-core-dev
1892017-06-07T15:37:24 *** arowser has quit IRC
1902017-06-07T15:40:19 *** Chris_Stewart_5 has quit IRC
1912017-06-07T15:43:24 *** arowser has joined #bitcoin-core-dev
1922017-06-07T15:44:30 <wumpus> checking for QT... no
1932017-06-07T15:44:30 <wumpus> checking for QT... no
1942017-06-07T15:44:37 <wumpus> strange, that appears double on freebsd at least
1952017-06-07T15:47:32 *** abpa has joined #bitcoin-core-dev
1962017-06-07T15:56:15 *** Chris_Stewart_5 has joined #bitcoin-core-dev
1972017-06-07T16:00:19 *** btcdrak has joined #bitcoin-core-dev
1982017-06-07T16:00:48 *** Dyaheon has quit IRC
1992017-06-07T16:02:09 *** Dyaheon has joined #bitcoin-core-dev
2002017-06-07T16:04:02 *** laurentmt has joined #bitcoin-core-dev
2012017-06-07T16:05:32 *** ula has quit IRC
2022017-06-07T16:07:08 *** laurentmt has quit IRC
2032017-06-07T16:07:45 *** Giszmo has joined #bitcoin-core-dev
2042017-06-07T16:15:41 *** Chris_Stewart_5 has quit IRC
2052017-06-07T16:29:57 *** AaronvanW has quit IRC
2062017-06-07T16:38:45 *** Chris_Stewart_5 has joined #bitcoin-core-dev
2072017-06-07T16:44:29 <timothy> wumpus: me too (fedora 26)
2082017-06-07T16:45:00 <wumpus> timothy: that's also a case in which QT is not detected?
2092017-06-07T16:45:51 <timothy> qt4 and qt5
2102017-06-07T16:45:56 <timothy> PKG_CHECK_MODULES([QT], [$qt5_modules], [QT_INCLUDES="$QT_CFLAGS"; have_qt=yes],[have_qt=no])
2112017-06-07T16:45:58 <timothy> PKG_CHECK_MODULES([QT], [$qt4_modules], [QT_INCLUDES="$QT_CFLAGS"; have_qt=yes], [have_qt=no])
2122017-06-07T16:47:25 <wumpus> oh, it makes sense then, if only the message was clear about that :)
2132017-06-07T16:47:41 <timothy> yes, I'll propose a pull request to change QT with QT5 and QT4
2142017-06-07T16:51:47 *** chjj has quit IRC
2152017-06-07T16:54:48 <bitcoin-git> [bitcoin] drizzt opened pull request #10549: Avoid printing generic and duplicated "checking for QT" during ./configure (master...check_qt) https://github.com/bitcoin/bitcoin/pull/10549
2162017-06-07T16:55:13 <timothy> wumpus: ^^
2172017-06-07T16:55:59 <wumpus> timothy: thanks!
2182017-06-07T16:56:29 <timothy> (un)likely I work often with autotools :P
2192017-06-07T16:59:38 <morcos> sipa: i'm still reviewing #10195, but i think i have one issue. if you agree, i can fix it after i finish review if you want, or you can.
2202017-06-07T16:59:44 <gribble> https://github.com/bitcoin/bitcoin/issues/10195 | Switch chainstate db and cache to per-txout model by sipa · Pull Request #10195 · bitcoin/bitcoin · GitHub
2212017-06-07T17:00:19 <morcos> in ATMP, it seems like it will be too slow to do HaveCoins for every outpoint of a new tx, in the common case, that's hitting disk for every outpoint of every new tx
2222017-06-07T17:01:04 *** timothy has quit IRC
2232017-06-07T17:01:31 <morcos> we can probably ditch that whole block of code altogether, and rely on AlreadyHave which is called just before ATMP in net_processing, or if we need to because of other places ATMP is called, we can duplicate the idea of just checking 1 or 2 outputs that AlreadyHave uses
2242017-06-07T17:02:27 <sipa> or change it to HaveCoinInCache?
2252017-06-07T17:02:43 <sipa> ah no, nvm
2262017-06-07T17:05:33 *** chjj has joined #bitcoin-core-dev
2272017-06-07T17:05:38 *** talmai has joined #bitcoin-core-dev
2282017-06-07T17:17:23 *** PaulCapestany has quit IRC
2292017-06-07T17:17:41 *** ula has joined #bitcoin-core-dev
2302017-06-07T17:18:08 *** PaulCapestany has joined #bitcoin-core-dev
2312017-06-07T17:20:59 <bitcoin-git> [bitcoin] pavlosantoniou closed pull request #10530: Fix possibly unsafe accesses of array in class base_uint<BITS> (master...master) https://github.com/bitcoin/bitcoin/pull/10530
2322017-06-07T17:23:47 <bitcoin-git> [bitcoin] pavlosantoniou reopened pull request #10530: Fix possibly unsafe accesses of array in class base_uint<BITS> (master...master) https://github.com/bitcoin/bitcoin/pull/10530
2332017-06-07T17:26:43 *** SopaXorzTaker has quit IRC
2342017-06-07T17:28:34 *** goatturneer has joined #bitcoin-core-dev
2352017-06-07T17:31:50 *** goatturner has quit IRC
2362017-06-07T17:33:05 *** vicenteH has quit IRC
2372017-06-07T17:45:46 *** Chris_Stewart_5 has quit IRC
2382017-06-07T17:58:55 *** talmai has quit IRC
2392017-06-07T18:04:14 *** Dyaheon has quit IRC
2402017-06-07T18:04:57 <luke-jr> wumpus: BIP148 is a softfork. We typically consider those bugfixes and only added on minor bumps..
2412017-06-07T18:05:34 <luke-jr> [10:58:08] <timothy> the UASF commits, since luke-jr is working on DoS stuff <-- for now, it may be safer to omit some of the DoS stuff; that's additional on top of BIP148 (not necessary), and has less review so far
2422017-06-07T18:05:58 *** Chris_Stewart_5 has joined #bitcoin-core-dev
2432017-06-07T18:06:52 *** Dyaheon has joined #bitcoin-core-dev
2442017-06-07T18:07:48 <luke-jr> wumpus: I don't see the point in reverting unless there's an actual problem.
2452017-06-07T18:08:10 *** beatrootfarmer has joined #bitcoin-core-dev
2462017-06-07T18:11:48 *** goatturneer has quit IRC
2472017-06-07T18:15:56 <jtimon> if you are included in https://en.bitcoin.it/wiki/Segwit_support please check that your position is not misrepresented (mine was by mistake)
2482017-06-07T18:18:36 <sipa> luke-jr: i don't "want" BIP148. I want segwit, as I think it's necessary for Bitcoin's future. BIP148 is a overly risky means of obtaining that. That does not mean I oppose it if there were tremendous support, but on itself I think it's a bad idea
2492017-06-07T18:19:45 <sipa> i'm going to delete my line from that table - i don't think it can represent nuanced opinions
2502017-06-07T18:20:24 <jcorgan> i think the "wanting" terminology is confusing
2512017-06-07T18:20:36 <jcorgan> as is the "deficient"
2522017-06-07T18:21:05 <jcorgan> sipa: "wanting" means you like it but think community support is lacking
2532017-06-07T18:21:15 <sipa> i don't like it
2542017-06-07T18:21:23 <jcorgan> that is not how I would take it at first read, and clearly not how you feel
2552017-06-07T18:21:27 <sipa> i would be happy if it worked, but i don't believe it can
2562017-06-07T18:22:15 <jcorgan> i suggest you keep yourself there but just keep the prefer 141 and the rest empty
2572017-06-07T18:23:02 *** SopaXorzTaker has joined #bitcoin-core-dev
2582017-06-07T18:23:48 <BlueMatt> luke-jr: "We typically consider [soft forks] bugfixes and only added on minor bumps.." <-- NO, WE DO NOT
2592017-06-07T18:24:09 <luke-jr> BlueMatt: lolwut? even Segwit was added on a minor bump
2602017-06-07T18:24:18 <BlueMatt> we consider them CONSENSUS CHANGES, and release them in minor version so as to avoid forcing anyone to upgrade to a soft fork for other major-version features they want
2612017-06-07T18:24:37 <BlueMatt> just because it was a minor version number does not mean its a "minor bump"
2622017-06-07T18:27:02 <BlueMatt> if anything, the fact that we release them in minor versions indicates how seriously we take them, not the inverse - they are the only thing in that release (+/-) and get an entire release cycle dedicated just to them
2632017-06-07T18:28:37 <jtimon> who is proposing to change that (ie enabling activation of consensus changes only on minor versions)?
2642017-06-07T18:29:06 <BlueMatt> jtimon: I dont think anyone, I just found luke's comments laughably insulting to the effort that goes into consensus changes
2652017-06-07T18:29:21 <sipa> luke-jr: i think the point is that yes softforks can be merged in minor release, but no that does not mean that any softfork will be merged
2662017-06-07T18:29:45 <wumpus> never between rcs though
2672017-06-07T18:30:11 <BlueMatt> lol @ merge consensus changes between rcs
2682017-06-07T18:30:14 <BlueMatt> actually lol
2692017-06-07T18:30:16 <jtimon> oh, I see, luke wants bip148 for 14.2 ?
2702017-06-07T18:30:51 <luke-jr> wumpus: point taken
2712017-06-07T18:31:15 <luke-jr> jtimon: that would be ideal, but wumpus makes a good point that it's too late for 0.14.2
2722017-06-07T18:32:41 <jtimon> yeah I assumed that was clear. I'm unconvinced a variation of bip149 cannot be merged before mov15 though, so maybe it could make it to 0.14.3
2732017-06-07T18:34:10 <luke-jr> [18:29:06] <BlueMatt> jtimon: I dont think anyone, I just found luke's comments laughably insulting to the effort that goes into consensus changes <-- effort goes into traditional bugfixes as well. "bugfix" does not make any implication of effort.
2742017-06-07T18:34:31 <jtimon> yeah I read that
2752017-06-07T18:34:45 <jtimon> oh, sorry
2762017-06-07T18:35:20 <BlueMatt> luke-jr: well then let me rephrase, consensus changes are *not* bugfixes....they could be if there was an actual bug, but so far we havent seen one (and, no, "segwit hasnt happened" is not a bug...)
2772017-06-07T18:35:55 <BlueMatt> more recent consensus changes have been 6+ month review and tweak cycles, even for simple things
2782017-06-07T18:36:17 <luke-jr> BlueMatt: being vulnerable to miners producing a long chain of invalid blocks, is the bug fixed by (any) softforks
2792017-06-07T18:36:56 * BlueMatt proposes redlisting soft fork again, clearly its a bugfix so must be merged *now*
2802017-06-07T18:36:57 <luke-jr> but call it what you like, it's beside the point
2812017-06-07T18:37:15 <BlueMatt> no, you're missing the point, this is consensus, not a fucking joke
2822017-06-07T18:37:24 <luke-jr> bugs aren't a joke
2832017-06-07T18:38:17 <BlueMatt> you appear to be dead-set on taking it as one
2842017-06-07T18:38:40 <sipa> luke-jr: i don't think you can consider not implementing an new and backward-compatible consensus change a bug
2852017-06-07T18:38:56 <sipa> luke-jr: i wouldn't call 0.13.0 buggy when segwit actives; at best, it is outdated
2862017-06-07T18:39:21 <sipa> the whole point of softforks is that nobody is required to adopt the rule
2872017-06-07T18:39:45 <luke-jr> to retain full node security, you need to adopt the rule
2882017-06-07T18:40:03 <BlueMatt> the idea that its a "bug" (or even an issue) for a user to *not* "upgrade" to enforce a soft fork is laughable...if that were the case, Bitcoin would pretty clearly have no long-term value
2892017-06-07T18:40:08 <sipa> luke-jr: what? no
2902017-06-07T18:40:14 <BlueMatt> "oops, miners are enforcing redlisting, you need to upgrade"
2912017-06-07T18:40:51 <luke-jr> BlueMatt: false equivalency is not helping
2922017-06-07T18:41:10 <sipa> how is that a false equivalency?
2932017-06-07T18:41:16 <BlueMatt> luke-jr: I absolutely do not believe it is false equivalency, actually. We're talking about what precedent is being set for Bitcoin's future and how changes are made in Bitcoin
2942017-06-07T18:41:43 <morcos> Can we talk about when we're upgrading to C++17 instead?
2952017-06-07T18:41:48 <BlueMatt> ACK
2962017-06-07T18:41:55 <BlueMatt> is filesystem in 17?
2972017-06-07T18:41:58 <BlueMatt> or is that 15?
2982017-06-07T18:42:06 <BlueMatt> 14
2992017-06-07T18:42:10 * sipa wants std::this_thread::sleep(10ms);
3002017-06-07T18:42:12 <luke-jr> hopefully compilers support it sooner than C++11
3012017-06-07T18:42:19 <BlueMatt> sipa: wait, thats not 11?
3022017-06-07T18:42:25 <sipa> BlueMatt: no, c++14
3032017-06-07T18:42:25 *** SopaXorzTaker has quit IRC
3042017-06-07T18:42:29 <BlueMatt> ohoh, the ms postfix, yea
3052017-06-07T18:43:07 <sipa> still no concepts :(
3062017-06-07T18:43:12 <sipa> not even in 17
3072017-06-07T18:46:35 *** beatrootfarmer has quit IRC
3082017-06-07T18:47:11 <wumpus> we supported c++11 in.. 2016, so extrapolating that, c++17 will be usable in 2022
3092017-06-07T18:47:33 <luke-jr> :/
3102017-06-07T18:47:51 <wumpus> don't shoot me, I'm just the messenger :)
3112017-06-07T18:50:03 <sipa> c++11 took much longer to be actually available in compilers
3122017-06-07T18:50:11 <sipa> c++14 is already default in gcc 6
3132017-06-07T18:50:34 <sipa> the changes in 14 and 17 are also much smaller than the c++03-c++11 changes
3142017-06-07T18:50:45 <wumpus> std::optional is pretty nice
3152017-06-07T18:51:06 <luke-jr> how long until GCC 6 is on major distros?
3162017-06-07T18:51:20 <luke-jr> Gentoo just got GCC 5 like a month ago
3172017-06-07T18:51:38 <BlueMatt> debian? 2025?
3182017-06-07T18:51:46 <jtimon> sure, ack on moving to c++14, what do we need to do?
3192017-06-07T18:52:01 <sipa> i think GCC 5 fully supports c++14
3202017-06-07T18:52:16 <BlueMatt> oh, thats a lie, wow, next debian is gcc 6
3212017-06-07T18:52:39 <luke-jr> RHEL tends to be the slowest these days
3222017-06-07T18:52:42 <sipa> and c++17... doesn't exist yet
3232017-06-07T18:52:44 <luke-jr> from what I've seen
3242017-06-07T18:52:59 <wumpus> std::string_view is very useful too
3252017-06-07T18:53:00 <luke-jr> lo, was morcos just trolling
3262017-06-07T18:54:07 <BlueMatt> morcos' xkcd 356 is good
3272017-06-07T18:54:24 <wumpus> recent versions of clang already support a lot of c++17 features, even though it doesn't officially exist yet
3282017-06-07T18:54:43 <sipa> Clang finished support for C++14 in 3.4 though under the standard name c++1y.[23] GCC finished support for C++14 in GCC 5, and made C++14 the default C++ standard in GCC 6.[24] Microsoft Visual Studio 2015 has support for some but not all C++14 features.[25]
3292017-06-07T18:55:04 <jtimon> luke-jr: does it matter? the important part is that if we don't move to c++14 coon some people may start moving to ripple, they had c++14 almost from launch </bad joke>
3302017-06-07T18:55:16 <sipa> hahaha
3312017-06-07T18:55:51 <jtimon> s/coon/soon
3322017-06-07T18:55:56 <wumpus> bitcoin core compiles almost entirely without changes in c++17 mode, btw
3332017-06-07T18:56:14 <luke-jr> wumpus: that's a nice start at least
3342017-06-07T18:56:16 <wumpus> there were some minor things last time I tried
3352017-06-07T18:56:33 <jtimon> wumpus: awesome, what are the almost no changes?
3362017-06-07T18:56:41 <wumpus> but those might be clang-weirdnesses as well
3372017-06-07T18:57:03 <wumpus> should be in my cloudabi branch, let me see
3382017-06-07T18:57:46 *** Giszmo has quit IRC
3392017-06-07T18:58:08 <wumpus> jtimon: https://github.com/laanwj/bitcoin/commit/efeec33d19752de7480f1e8d0fbcd79e49548910
3402017-06-07T18:58:13 *** Giszmo has joined #bitcoin-core-dev
3412017-06-07T18:59:26 <BlueMatt> god I keep finding things and being like "wait, didnt I change that???" only to realize its in a queued-next PR based on one still pending :(
3422017-06-07T18:59:39 * BlueMatt really wants to make all CBlockIndexes const :(
3432017-06-07T19:00:14 <sipa> BlueMatt: what do you do with the mutable fields?
3442017-06-07T19:00:25 <sipa> (nChainTx, nStatus, nFile, ...)
3452017-06-07T19:00:27 <BlueMatt> sipa: ehh, sorry, const outside of validation.cpp
3462017-06-07T19:00:37 <sipa> ah!
3472017-06-07T19:00:38 <sipa> yes!
3482017-06-07T19:00:44 <BlueMatt> ie exposed versions in headers are all const, and mapBlockIndex is const too
3492017-06-07T19:01:03 <BlueMatt> sipa: its the next one queued after #10279
3502017-06-07T19:01:07 <gribble> https://github.com/bitcoin/bitcoin/issues/10279 | Add a CChainState class to validation.cpp to take another step towards clarifying internal interfaces by TheBlueMatt · Pull Request #10279 · bitcoin/bitcoin · GitHub
3512017-06-07T19:02:08 *** chjj has quit IRC
3522017-06-07T19:03:43 *** beatrootfarmer has joined #bitcoin-core-dev
3532017-06-07T19:15:30 *** chjj has joined #bitcoin-core-dev
3542017-06-07T19:16:37 <achow101> wumpus: did you change your gitian key?
3552017-06-07T19:16:49 <achow101> or sign with the wrong one?
3562017-06-07T19:18:26 <wumpus> I've added a signing subkey
3572017-06-07T19:18:33 <wumpus> it's the same I use for signing commits
3582017-06-07T19:20:03 <jtimon> wumpus: yeah, that's a very small diff
3592017-06-07T19:20:31 <achow101> wumpus: ah, ok. Apparently I don't have that key
3602017-06-07T19:21:18 <jtimon> BlueMatt: been there
3612017-06-07T19:24:10 <wumpus> achow101: --refresh-keys should get it AFAIK
3622017-06-07T19:24:52 <BlueMatt> jtimon: ehh, its normal 'round here, review is hard
3632017-06-07T19:25:26 <jtimon> yep, and time consuming
3642017-06-07T19:25:38 <BlueMatt> with good reason
3652017-06-07T19:25:49 <jtimon> not saying otherwise
3662017-06-07T19:28:41 *** goatturneer has joined #bitcoin-core-dev
3672017-06-07T19:32:05 *** beatrootfarmer has quit IRC
3682017-06-07T19:36:00 <bitcoin-git> [bitcoin] achow101 opened pull request #10551: [Tests] Wallet encryption functional tests (master...wallet-encrypt-test) https://github.com/bitcoin/bitcoin/pull/10551
3692017-06-07T19:40:42 <luke-jr> btcdrak: are you planning to do a 0.14 addrindex?
3702017-06-07T19:43:36 *** parazyd has quit IRC
3712017-06-07T19:44:25 <jtimon> yeah, I'm interested in knowing that too
3722017-06-07T19:44:38 <sipa> :(
3732017-06-07T19:44:57 <BlueMatt> :) ?
3742017-06-07T19:45:07 <jtimon> for an explorer
3752017-06-07T19:45:40 *** parazyd has joined #bitcoin-core-dev
3762017-06-07T19:45:40 *** parazyd has joined #bitcoin-core-dev
3772017-06-07T19:46:02 <sipa> i wish people wouldn't build solutions that require a fully indexed unpruned blockchain
3782017-06-07T19:47:00 <jtimon> sipa: although honestly like it as I have it with only blockheight, blockhash and txid for searching
3792017-06-07T19:47:24 <luke-jr> sipa: I agree, but a 0.14 addrindex will be less annoying than supporting 0.13 longer than necessary XD
3802017-06-07T19:47:31 *** parazyd has joined #bitcoin-core-dev
3812017-06-07T19:47:31 *** parazyd has joined #bitcoin-core-dev
3822017-06-07T19:50:11 *** parazyd has quit IRC
3832017-06-07T19:51:42 *** parazyd has joined #bitcoin-core-dev
3842017-06-07T19:51:42 *** parazyd has joined #bitcoin-core-dev
3852017-06-07T19:59:51 *** RubenSomsen has quit IRC
3862017-06-07T20:02:14 <BlueMatt> sipa: is deserializing bandb/addrdb any more expensive than any other deserialization?
3872017-06-07T20:02:20 <BlueMatt> its vector operations, mostly, just like the rest, no?
3882017-06-07T20:02:34 <BlueMatt> re: 10248: so I dont see why it matters
3892017-06-07T20:07:26 *** chjj has quit IRC
3902017-06-07T20:10:31 <sipa> BlueMatt: code simplification
3912017-06-07T20:10:47 <BlueMatt> hmm? no, i mean i like 10248
3922017-06-07T20:11:02 <BlueMatt> and dont think the removal of the ability to skip deserialize on hash mismatch is worth worrying about
3932017-06-07T20:12:57 <sipa> oh, i agree
3942017-06-07T20:14:22 <wumpus> yes it doesn't really matter in this case, the hash is not a MAC and it's unlikely the bandb/addrdb will be used as an attack vector in any case
3952017-06-07T20:14:29 <wumpus> so 10248 is fine
3962017-06-07T20:14:42 <BlueMatt> if you can modify my bandb........
3972017-06-07T20:14:52 <BlueMatt> something something wallet in same folder
3982017-06-07T20:14:59 <wumpus> you can also change the hash, sure
3992017-06-07T20:16:48 <wumpus> the checksum is there against accidental corruption, and checking the hash on the fly is just as good for that
4002017-06-07T20:16:56 <sipa> i'd change your bitcoind :p
4012017-06-07T20:18:15 <wumpus> and if there is any way for a corruption to *crash* the deserialization I guess there's worse problems
4022017-06-07T20:21:03 *** chjj has joined #bitcoin-core-dev
4032017-06-07T20:29:49 *** jannes has quit IRC
4042017-06-07T20:31:35 *** QBcrusher has quit IRC
4052017-06-07T20:37:33 *** vicenteH has joined #bitcoin-core-dev
4062017-06-07T20:45:03 *** Guyver2 has quit IRC
4072017-06-07T20:52:20 <ryanofsky> should it be necessary to lock mempool.cs before calling atmp? is this lock needed: https://github.com/bitcoin/bitcoin/blob/master/src/wallet/wallet.cpp#L1546
4082017-06-07T21:01:57 *** Guest83978 has quit IRC
4092017-06-07T21:04:52 <bitcoin-git> [bitcoin] jtimon closed pull request #10427: Consensus: Introduce static GetScriptFlags (mostly MOVEONLY) (master...b15-consensus-script-flags-min) https://github.com/bitcoin/bitcoin/pull/10427
4102017-06-07T21:05:57 *** Guest83978 has joined #bitcoin-core-dev
4112017-06-07T21:15:29 *** [\\\] is now known as tripleslash
4122017-06-07T21:15:35 *** ajd__ has quit IRC
4132017-06-07T21:22:56 *** mkarrer_ has joined #bitcoin-core-dev
4142017-06-07T21:24:23 *** aspect__ has joined #bitcoin-core-dev
4152017-06-07T21:24:26 *** rubensayshi_ has joined #bitcoin-core-dev
4162017-06-07T21:24:32 *** brg444_ has joined #bitcoin-core-dev
4172017-06-07T21:24:53 *** Taek42 has joined #bitcoin-core-dev
4182017-06-07T21:26:56 *** cfields_ has joined #bitcoin-core-dev
4192017-06-07T21:27:18 *** nickler_ has joined #bitcoin-core-dev
4202017-06-07T21:29:23 *** rubensayshi has quit IRC
4212017-06-07T21:29:23 *** aspect_ has quit IRC
4222017-06-07T21:29:24 *** brg444 has quit IRC
4232017-06-07T21:29:24 *** nickler has quit IRC
4242017-06-07T21:29:24 *** _flow_ has quit IRC
4252017-06-07T21:29:24 *** mkarrer has quit IRC
4262017-06-07T21:29:24 *** nsh has quit IRC
4272017-06-07T21:29:24 *** gijensen has quit IRC
4282017-06-07T21:29:24 *** Taek has quit IRC
4292017-06-07T21:29:25 *** cfields has quit IRC
4302017-06-07T21:29:26 *** rubensayshi_ is now known as rubensayshi
4312017-06-07T21:29:27 *** aspect__ is now known as aspect_
4322017-06-07T21:29:30 *** brg444_ is now known as brg444
4332017-06-07T21:29:43 *** nsh has joined #bitcoin-core-dev
4342017-06-07T21:30:10 *** gijensen has joined #bitcoin-core-dev
4352017-06-07T21:31:59 *** _flow_ has joined #bitcoin-core-dev
4362017-06-07T21:35:21 *** Guest47556 has joined #bitcoin-core-dev
4372017-06-07T21:35:46 *** Guest83978 has quit IRC
4382017-06-07T21:41:36 *** Guest47556 has quit IRC
4392017-06-07T21:42:57 *** Chris_Stewart_5 has quit IRC
4402017-06-07T21:47:58 *** Guest47556 has joined #bitcoin-core-dev
4412017-06-07T21:59:43 <BlueMatt> ryanofsky: probably not?
4422017-06-07T22:01:46 *** _flow_ has quit IRC
4432017-06-07T22:04:00 <ryanofsky> ok, thx for sanity check
4442017-06-07T22:04:30 *** nakaluna_ has joined #bitcoin-core-dev
4452017-06-07T22:04:42 *** _flow_ has joined #bitcoin-core-dev
4462017-06-07T22:11:46 *** Guest47556 has quit IRC
4472017-06-07T22:12:44 <bitcoin-git> [bitcoin] sipa pushed 11 new commits to master: https://github.com/bitcoin/bitcoin/compare/46311e792f4e...e801084decf4
4482017-06-07T22:12:45 <bitcoin-git> bitcoin/master 37e864e Pieter Wuille: Add FastRandomContext::rand256() and ::randbytes()...
4492017-06-07T22:12:45 <bitcoin-git> bitcoin/master 90620d6 Pieter Wuille: scripted-diff: Rename cuckoo tests' local rand context...
4502017-06-07T22:12:46 <bitcoin-git> bitcoin/master 124d13a Pieter Wuille: Merge test_random.h into test_bitcoin.h
4512017-06-07T22:13:04 <bitcoin-git> [bitcoin] sipa closed pull request #10321: Use FastRandomContext for all tests (master...fast_rand_tests) https://github.com/bitcoin/bitcoin/pull/10321
4522017-06-07T22:14:40 *** Cheeseo has joined #bitcoin-core-dev
4532017-06-07T22:16:56 *** chjj has quit IRC
4542017-06-07T22:19:16 *** Cheeseo has quit IRC
4552017-06-07T22:20:58 *** nemgun1 has quit IRC
4562017-06-07T22:25:13 *** nakaluna_ is now known as nakaluna
4572017-06-07T22:25:31 *** _flow_ has quit IRC
4582017-06-07T22:26:41 *** nakaluna has joined #bitcoin-core-dev
4592017-06-07T22:31:09 *** chjj has joined #bitcoin-core-dev
4602017-06-07T22:31:42 <jtimon> btw, I got #9717 out of #10339 as requested
4612017-06-07T22:31:43 <gribble> https://github.com/bitcoin/bitcoin/issues/9717 | Pow: Remove fCheckPOW from CheckBlockHeader by jtimon · Pull Request #9717 · bitcoin/bitcoin · GitHub
4622017-06-07T22:31:45 <gribble> https://github.com/bitcoin/bitcoin/issues/10339 | Optimization: Calculate block hash less times by jtimon · Pull Request #10339 · bitcoin/bitcoin · GitHub
4632017-06-07T22:34:12 *** achow101 has left #bitcoin-core-dev
4642017-06-07T22:34:22 *** achow101 has joined #bitcoin-core-dev
4652017-06-07T22:40:57 *** Dyaheon has quit IRC
4662017-06-07T22:42:34 *** Dyaheon has joined #bitcoin-core-dev
4672017-06-07T22:47:56 *** _flow_ has joined #bitcoin-core-dev
4682017-06-07T23:00:43 *** chartractegg has joined #bitcoin-core-dev
4692017-06-07T23:03:43 *** chartractegg has quit IRC
4702017-06-07T23:10:59 *** Guest47556 has joined #bitcoin-core-dev
4712017-06-07T23:16:19 *** PaulCapestany has quit IRC
4722017-06-07T23:22:16 *** chjj has quit IRC
4732017-06-07T23:26:58 *** Guest47556 has quit IRC
4742017-06-07T23:31:39 *** Chris_Stewart_5 has joined #bitcoin-core-dev
4752017-06-07T23:35:35 *** chjj has joined #bitcoin-core-dev
4762017-06-07T23:39:11 *** nakaluna has quit IRC
4772017-06-07T23:48:46 *** Taek42 is now known as Taek
4782017-06-07T23:49:08 *** cryptapus has joined #bitcoin-core-dev
4792017-06-07T23:49:13 *** cryptapus is now known as cryptapus_afk
4802017-06-07T23:59:10 *** abpa has quit IRC