12016-07-27T00:00:21  *** BashCo has quit IRC
  22016-07-27T00:01:14  <luke-jr> https://github.com/bitcoin/libblkmaker/pull/6 review requested
  32016-07-27T00:04:53  *** BashCo has joined #bitcoin-core-dev
  42016-07-27T00:14:11  *** belcher has quit IRC
  52016-07-27T00:19:29  *** Guest19449 has joined #bitcoin-core-dev
  62016-07-27T00:21:01  *** Guest19449 is now known as roidster
  72016-07-27T00:37:04  <phantomcircuit> wumpus: what else more do i need to do on https://github.com/bitcoin/bitcoin/pull/8152 ?
  82016-07-27T00:37:18  <phantomcircuit> jonasschnelli: same question ^
  92016-07-27T00:43:32  *** roidster has quit IRC
 102016-07-27T01:05:49  *** Ylbam has quit IRC
 112016-07-27T01:27:45  *** fengling has joined #bitcoin-core-dev
 122016-07-27T01:29:38  *** molly has quit IRC
 132016-07-27T01:30:05  *** moli has joined #bitcoin-core-dev
 142016-07-27T01:33:26  *** molz has joined #bitcoin-core-dev
 152016-07-27T01:36:05  *** moli has quit IRC
 162016-07-27T01:49:03  *** laurentmt has quit IRC
 172016-07-27T02:00:36  *** justanotheruser has quit IRC
 182016-07-27T02:05:35  *** justanotheruser has joined #bitcoin-core-dev
 192016-07-27T02:17:46  *** fengling has quit IRC
 202016-07-27T02:21:25  *** fengling has joined #bitcoin-core-dev
 212016-07-27T02:29:56  *** zooko has joined #bitcoin-core-dev
 222016-07-27T03:23:28  *** jtimon has quit IRC
 232016-07-27T03:49:43  *** pmienk has quit IRC
 242016-07-27T04:05:29  *** pmienk has joined #bitcoin-core-dev
 252016-07-27T04:17:37  *** Alopex has quit IRC
 262016-07-27T04:18:42  *** Alopex has joined #bitcoin-core-dev
 272016-07-27T04:28:21  *** Alopex has quit IRC
 282016-07-27T04:29:26  *** Alopex has joined #bitcoin-core-dev
 292016-07-27T04:39:02  *** anu0 has joined #bitcoin-core-dev
 302016-07-27T04:46:02  *** Alopex has quit IRC
 312016-07-27T04:47:07  *** Alopex has joined #bitcoin-core-dev
 322016-07-27T04:50:25  *** d_t has joined #bitcoin-core-dev
 332016-07-27T04:54:55  *** d_t has quit IRC
 342016-07-27T05:07:40  *** morcos has quit IRC
 352016-07-27T05:08:20  *** zxzzt has quit IRC
 362016-07-27T05:09:01  *** d_t has joined #bitcoin-core-dev
 372016-07-27T05:10:03  *** zxzzt has joined #bitcoin-core-dev
 382016-07-27T05:12:22  *** zooko has quit IRC
 392016-07-27T05:23:30  *** morcos has joined #bitcoin-core-dev
 402016-07-27T05:32:21  *** Alopex has quit IRC
 412016-07-27T05:33:27  *** Alopex has joined #bitcoin-core-dev
 422016-07-27T06:06:20  *** Yv7trNY has joined #bitcoin-core-dev
 432016-07-27T06:10:19  *** adiabat has joined #bitcoin-core-dev
 442016-07-27T06:25:10  *** Squidicc has joined #bitcoin-core-dev
 452016-07-27T06:28:40  *** squidicuz has quit IRC
 462016-07-27T06:28:41  *** Yv7trNY has quit IRC
 472016-07-27T06:35:12  *** Yv7trNY has joined #bitcoin-core-dev
 482016-07-27T06:35:41  *** BashCo has quit IRC
 492016-07-27T06:42:17  *** d_t has quit IRC
 502016-07-27T06:52:31  *** jl2012 has quit IRC
 512016-07-27T06:52:58  *** jl2012 has joined #bitcoin-core-dev
 522016-07-27T07:03:41  *** BashCo has joined #bitcoin-core-dev
 532016-07-27T07:05:16  *** Alopex has quit IRC
 542016-07-27T07:06:22  *** Alopex has joined #bitcoin-core-dev
 552016-07-27T07:17:40  *** Guyver2 has joined #bitcoin-core-dev
 562016-07-27T07:31:53  *** Ylbam has joined #bitcoin-core-dev
 572016-07-27T07:35:03  *** dgenr8 has quit IRC
 582016-07-27T07:35:07  *** AtashiCon has quit IRC
 592016-07-27T07:35:21  *** AtashiCon has joined #bitcoin-core-dev
 602016-07-27T07:39:24  *** dgenr8 has joined #bitcoin-core-dev
 612016-07-27T07:39:25  *** aalex_ has quit IRC
 622016-07-27T07:39:25  *** kanzure has quit IRC
 632016-07-27T07:39:26  *** kanzure has joined #bitcoin-core-dev
 642016-07-27T07:40:33  *** aalex_ has joined #bitcoin-core-dev
 652016-07-27T07:52:54  *** G1lius has quit IRC
 662016-07-27T08:11:12  *** kanzure has quit IRC
 672016-07-27T08:11:15  *** kanzure has joined #bitcoin-core-dev
 682016-07-27T08:13:07  *** mturquette has quit IRC
 692016-07-27T08:13:47  *** mturquette has joined #bitcoin-core-dev
 702016-07-27T08:31:22  *** Guyver2 has quit IRC
 712016-07-27T08:31:54  *** aalex_ has quit IRC
 722016-07-27T08:34:20  <jonasschnelli> phantomcircuit: Looks good. I really want to run some tests. I'll do that soon. But 8152 is definitively useful and will be merged soon.
 732016-07-27T08:35:33  *** aalex_ has joined #bitcoin-core-dev
 742016-07-27T08:38:26  *** kadoban has quit IRC
 752016-07-27T08:40:16  *** Yv7trNY has quit IRC
 762016-07-27T08:40:58  *** aalex_ has quit IRC
 772016-07-27T08:41:21  *** aalex_ has joined #bitcoin-core-dev
 782016-07-27T08:53:08  <jonasschnelli> wumpus: I think https://github.com/bitcoin/bitcoin/pull/8152 is ready for merge (into master)
 792016-07-27T08:55:13  <wumpus> thanks, taking a look
 802016-07-27T09:00:59  <wumpus> I'm still not entirely convinced that it won't reduce performance by causing more flushes to disk
 812016-07-27T09:01:28  *** aalex_ has quit IRC
 822016-07-27T09:01:50  <wumpus> e.g. the reason for passing those CWalletDB in the first place (it wasn't always that way) was to prevent extraneous flushes, as sipa says in that pull
 832016-07-27T09:03:18  <wumpus> but if anyone can convince me that that problem has been handled in another way I'm ok with merging it
 842016-07-27T09:03:30  <wumpus> (or even better, benchmarks)
 852016-07-27T09:04:25  <wumpus> jonasschnelli: re: 8407, I think the second commit makes the code unncesarily complex; I think we can already handle all upgrade scenarios without it
 862016-07-27T09:04:44  *** aalex_ has joined #bitcoin-core-dev
 872016-07-27T09:04:57  <jonasschnelli> wumpus: IMO 8152 is only a refactoring to avoid having CWalletDB access in rpc code.
 882016-07-27T09:05:12  <jonasschnelli> wumpus: re 8407, I agree. I will remove the second commit asap
 892016-07-27T09:06:30  <wumpus> yes avoiding CWalletDB access in the RPC code is great; but that could also have been accomplished by adding a wrapper method in CWallet
 902016-07-27T09:06:59  <wumpus> that passes through the wtx to AddToWallet and opens the walletdb
 912016-07-27T09:07:12  <wumpus> what I'm worried about is the internal uses of AddToWallet that do pass their own walletdb
 922016-07-27T09:08:16  <wumpus> eh - do those exist at all?
 932016-07-27T09:08:25  <jonasschnelli> wumpus: I have checked the internal AddToWallet uses. And it seems like they all open CWalletDB shortly before resulting in the ~same behavior.
 942016-07-27T09:09:03  <jonasschnelli> I think the change form (external calls) pwalletMain->AddToWallet(wtx, false, &walletdb); to  pwalletMain->AddToWallet(wtx); is good.
 952016-07-27T09:09:20  <jonasschnelli> It hides CWalletDB behind CWallet
 962016-07-27T09:09:24  *** davidlj95 has joined #bitcoin-core-dev
 972016-07-27T09:09:35  <wumpus> hmm I agree
 982016-07-27T09:09:37  <jonasschnelli> (for callers using AddToWallet)
 992016-07-27T09:10:01  <wumpus> LOL @ line 910
1002016-07-27T09:10:03  <jonasschnelli> But as I said. I think its a refactoring PR
1012016-07-27T09:10:34  <jonasschnelli> wumpus: you mean the return false at L910
1022016-07-27T09:10:45  <wumpus> of wallet.cpp - that line could be deleted, no one is using that walletdb :-)
1032016-07-27T09:10:58  <wumpus> CWalletDB walletdb(strWalletFile, "r+", false);    return AddToWallet(wtx);
1042016-07-27T09:11:27  <wumpus> I'm looking at the merged version, line numbers may be off
1052016-07-27T09:11:42  <jonasschnelli> Ah. Yes. That should be removed I guess.
1062016-07-27T09:12:28  <wumpus> so what changes here is the parameters under which walletdb is opened
1072016-07-27T09:12:55  <wumpus> for that AddWallet it used to be the ("r+", false), but now it wil open the walletdb inside AddToWallet with default parameters
1082016-07-27T09:13:21  *** mkarrer has joined #bitcoin-core-dev
1092016-07-27T09:14:11  <wumpus> ok, commented on the pull
1102016-07-27T09:14:26  *** laurentmt has joined #bitcoin-core-dev
1112016-07-27T09:14:48  *** laurentmt has quit IRC
1122016-07-27T09:16:56  *** mkarrer_ has quit IRC
1132016-07-27T09:23:36  *** TomMc has joined #bitcoin-core-dev
1142016-07-27T09:32:54  *** TomMc has quit IRC
1152016-07-27T09:33:02  *** Giszmo has joined #bitcoin-core-dev
1162016-07-27T09:39:36  *** cdecker has joined #bitcoin-core-dev
1172016-07-27T09:42:40  *** mkarrer has quit IRC
1182016-07-27T09:43:17  *** mkarrer has joined #bitcoin-core-dev
1192016-07-27T10:32:32  *** Ginnarr has joined #bitcoin-core-dev
1202016-07-27T10:38:42  *** spudowiar has joined #bitcoin-core-dev
1212016-07-27T11:03:57  *** BashCo_ has joined #bitcoin-core-dev
1222016-07-27T11:07:15  *** BashCo has quit IRC
1232016-07-27T11:22:12  *** cryptapus has joined #bitcoin-core-dev
1242016-07-27T11:26:34  <jonasschnelli> I think this one is ready for merge https://github.com/bitcoin/bitcoin/pull/8206
1252016-07-27T11:26:45  <jonasschnelli> Its a required 0.13 backport and relatively risk free
1262016-07-27T11:34:40  *** davidlj95 has quit IRC
1272016-07-27T11:44:39  *** spudowiar has quit IRC
1282016-07-27T11:45:03  *** spudowiar1 has joined #bitcoin-core-dev
1292016-07-27T11:47:28  *** spudowiar1 is now known as spudowiar
1302016-07-27T11:53:13  *** spudowiar has quit IRC
1312016-07-27T11:57:30  *** laurentmt has joined #bitcoin-core-dev
1322016-07-27T12:13:01  *** Giszmo has quit IRC
1332016-07-27T12:20:21  *** zooko has joined #bitcoin-core-dev
1342016-07-27T12:24:33  *** anu1 has joined #bitcoin-core-dev
1352016-07-27T12:25:22  *** Ginnarr has quit IRC
1362016-07-27T12:27:52  *** anu0 has quit IRC
1372016-07-27T12:37:20  *** Chris_Stewart_5 has joined #bitcoin-core-dev
1382016-07-27T12:41:02  *** Chris_Stewart_5 has quit IRC
1392016-07-27T12:41:14  <wumpus> jonasschnelli: yes I was planning to test it, will do so
1402016-07-27T12:41:24  <jonasschnelli> Thanks
1412016-07-27T12:45:06  *** fengling has quit IRC
1422016-07-27T12:59:59  *** Chris_Stewart_5 has joined #bitcoin-core-dev
1432016-07-27T13:02:12  <wumpus> jonasschnelli: a small nit: cVjNjjiDXCsTZKVP3VzjH3PPdNdeS7t51RXJ6rt99GgnzaTFtrxz 2016-07-27T13:00:33Z label= # addr=mfZRKaG9dfW5xEqUWiJFFFohe8XXoW297N hdkeypath: m/0'/0'/0'
1442016-07-27T13:02:28  <wumpus> let's format it hdkeypath= instead of hdkeypath:
1452016-07-27T13:02:33  <wumpus> that's more consistent
1462016-07-27T13:02:47  <jonasschnelli> ah. right. Let me change that directly
1472016-07-27T13:03:23  <wumpus> thanks
1482016-07-27T13:03:43  <wumpus> apart from that it works, tested with a non-HD and a HD wallet
1492016-07-27T13:03:59  <jonasschnelli> wumpus: what do you think about changing "oldhdmaster" to "inactivehdmaster"?
1502016-07-27T13:04:15  <jonasschnelli> oldhdmaster sound not ideal
1512016-07-27T13:04:28  <wumpus> yes seems more on-point
1522016-07-27T13:04:39  <jonasschnelli> okay. Will change that as well
1532016-07-27T13:04:51  *** Guyver2 has joined #bitcoin-core-dev
1542016-07-27T13:05:08  <wumpus> it not so much matters whether old or new, just that it's not used at the moment
1552016-07-27T13:05:49  <wumpus> or maybe a more heretical idea: number the master keys, and store what master key was used with the key metadata
1562016-07-27T13:06:09  <wumpus> maybe for 0.14
1572016-07-27T13:07:08  <jonasschnelli> wumpus: yes. Though about that. But would add nother "long" string to each line
1582016-07-27T13:07:20  <jonasschnelli> or we could add an index to each master key (for simplification)
1592016-07-27T13:08:05  <wumpus> yes indeed my idea was an index/handle, not the full master key id
1602016-07-27T13:08:54  <wumpus> that would indeed be ugly, and would take up significant extra space in the metadata
1612016-07-27T13:09:01  <jonasschnelli> Yes. This would make sense for 0.14 I guess.
1622016-07-27T13:09:36  <jonasschnelli> https://github.com/bitcoin/bitcoin/pull/8206 is featureish and I think we only backport it to 0.13 because of possible complains because of lack of exporting the xpriv
1632016-07-27T13:10:16  <wumpus> yes I agree we don't want to do this for 0.13
1642016-07-27T13:10:19  <wumpus> it was just a wild idea
1652016-07-27T13:10:39  <wumpus> for 8206 let's just fix the output format and merge it
1662016-07-27T13:12:02  <jonasschnelli> wumpus: I think 8206 is ready now
1672016-07-27T13:13:09  <wumpus> re-testing
1682016-07-27T13:18:38  <wumpus> jonasschnelli: cRv7Mpp7w5vtf4y8joSPs6kwGZAuKfPa76GzhGS7Vecq3ss4t5Fh 2016-07-27T13:00:33Z reserve=1 # addr=mpEFHw5CXTi69tHcZm62x9DJgbQewP731j hdkeypath= m/0'/0'/1'
1692016-07-27T13:18:46  <wumpus> I think that's a space too much after = :)
1702016-07-27T13:19:09  <jonasschnelli> Damit... :) force pushed.
1712016-07-27T13:19:32  <jonasschnelli> ah. Forgot "git add".
1722016-07-27T13:19:40  <jonasschnelli> wumpus: now pushed
1732016-07-27T13:26:13  <GitHub44> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/beadffae6d65...4d4970fe530a
1742016-07-27T13:26:13  <GitHub44> bitcoin/master 77c912d Jonas Schnelli: [Wallet] add HD xpriv to dumpwallet
1752016-07-27T13:26:14  <GitHub44> bitcoin/master 4d4970f Wladimir J. van der Laan: Merge #8206: [Wallet] Add HD xpriv to dumpwallet...
1762016-07-27T13:26:18  <GitHub179> [bitcoin] laanwj closed pull request #8206: [Wallet] Add HD xpriv to dumpwallet (master...2016/06/hd_info) https://github.com/bitcoin/bitcoin/pull/8206
1772016-07-27T13:27:16  <jonasschnelli> wumpus: does 77c912d apply cleanly to 0.13 (should)? Just tell me if you want me to open a PR against 0.13
1782016-07-27T13:27:35  <wumpus> going to try
1792016-07-27T13:29:25  <wumpus> seems to just apply
1802016-07-27T13:41:25  <GitHub42> [bitcoin] laanwj pushed 1 new commit to 0.13: https://github.com/bitcoin/bitcoin/commit/18b8ee1cd1b2c95faac53e49b9023200679f2bb1
1812016-07-27T13:41:25  <GitHub42> bitcoin/0.13 18b8ee1 Jonas Schnelli: [Wallet] add HD xpriv to dumpwallet...
1822016-07-27T13:41:31  *** fengling has joined #bitcoin-core-dev
1832016-07-27T13:42:58  *** YOU-JI has joined #bitcoin-core-dev
1842016-07-27T13:43:22  *** jtimon has joined #bitcoin-core-dev
1852016-07-27T13:46:26  *** fengling has quit IRC
1862016-07-27T13:48:56  *** Squidicc is now known as squidicuz
1872016-07-27T13:52:53  <wumpus> 8389 needs rebase
1882016-07-27T14:03:25  <jonasschnelli> rebased
1892016-07-27T14:07:25  *** Chris_Stewart_5 has quit IRC
1902016-07-27T14:09:10  *** YOU-JI has quit IRC
1912016-07-27T14:21:23  *** Chris_Stewart_5 has joined #bitcoin-core-dev
1922016-07-27T14:26:23  <wumpus> jonasschnelli: did you see my remark here: https://github.com/bitcoin/bitcoin/pull/8407/files#r72257114  ... I'm not sure what settings.value(strSettingsVersionKey) returns in the case !settings.contains(strSettingsVersionKey), but if it's garbage or raises an exception this may be problematic
1932016-07-27T14:31:12  <jonasschnelli> wumpus: yes saw it. But I thought the second expression in the if gets not executed if the first one (.contains()) fails. It's an OR.
1942016-07-27T14:31:51  <wumpus> yes that is true, but look inside the {} there is another settings.value(strSettingsVersionKey)
1952016-07-27T14:32:02  <wumpus> if (settings.value(strSettingsVersionKey) < 130000 &&  ...
1962016-07-27T14:32:22  <jonasschnelli> Ah. Right!
1972016-07-27T14:32:45  <jonasschnelli> That needs to be fixed. Will do soon (afk/phone typing).
1982016-07-27T14:32:56  <wumpus> yes, no hurry
1992016-07-27T14:43:04  *** fengling has joined #bitcoin-core-dev
2002016-07-27T14:44:48  *** laurentmt has quit IRC
2012016-07-27T14:47:46  *** fengling has quit IRC
2022016-07-27T14:59:09  *** zooko has quit IRC
2032016-07-27T15:00:04  *** laurentmt has joined #bitcoin-core-dev
2042016-07-27T15:00:05  *** laurentmt has quit IRC
2052016-07-27T15:17:29  *** Chris_Stewart_5 has quit IRC
2062016-07-27T15:28:48  *** TomMc has joined #bitcoin-core-dev
2072016-07-27T15:33:46  *** murch has joined #bitcoin-core-dev
2082016-07-27T15:39:09  *** Chris_Stewart_5 has joined #bitcoin-core-dev
2092016-07-27T15:45:06  *** fengling has joined #bitcoin-core-dev
2102016-07-27T15:47:04  *** Chris_Stewart_5 has quit IRC
2112016-07-27T15:47:51  *** Chris_Stewart_5 has joined #bitcoin-core-dev
2122016-07-27T15:50:06  *** fengling has quit IRC
2132016-07-27T16:05:16  *** Cory has quit IRC
2142016-07-27T16:08:12  *** murch has quit IRC
2152016-07-27T16:17:12  *** Chris_Stewart_5 has quit IRC
2162016-07-27T16:31:31  *** gijensen is now known as gijensen3
2172016-07-27T16:33:17  *** Chris_Stewart_5 has joined #bitcoin-core-dev
2182016-07-27T16:33:44  *** gijensen3 is now known as gijensen2
2192016-07-27T16:33:53  <GitHub191> [bitcoin] laanwj pushed 1 new commit to 0.13: https://github.com/bitcoin/bitcoin/commit/0179a39f9da1fa417a592e7bf3ebbb1390a292b9
2202016-07-27T16:33:53  <GitHub191> bitcoin/0.13 0179a39 Wladimir J. van der Laan: qt: periodic translations update
2212016-07-27T16:34:30  *** Cory has joined #bitcoin-core-dev
2222016-07-27T16:35:34  *** gijensen2 is now known as gijensen3
2232016-07-27T16:40:20  *** netzin has joined #bitcoin-core-dev
2242016-07-27T16:43:25  *** slackircbridge1 has quit IRC
2252016-07-27T16:43:31  *** slackircbridge has joined #bitcoin-core-dev
2262016-07-27T16:46:32  *** fengling has joined #bitcoin-core-dev
2272016-07-27T16:51:06  *** fengling has quit IRC
2282016-07-27T16:52:10  *** laurentmt has joined #bitcoin-core-dev
2292016-07-27T16:52:52  *** laurentmt has quit IRC
2302016-07-27T16:56:55  *** Cory has quit IRC
2312016-07-27T17:17:02  *** Cory has joined #bitcoin-core-dev
2322016-07-27T17:33:32  *** aalex__ has joined #bitcoin-core-dev
2332016-07-27T17:34:08  *** d_t has joined #bitcoin-core-dev
2342016-07-27T17:34:52  *** aalex_ has quit IRC
2352016-07-27T17:36:26  *** d_t has joined #bitcoin-core-dev
2362016-07-27T17:44:24  *** jtimon has quit IRC
2372016-07-27T17:47:31  *** fengling has joined #bitcoin-core-dev
2382016-07-27T17:48:14  *** netzin has quit IRC
2392016-07-27T17:52:26  *** fengling has quit IRC
2402016-07-27T17:53:22  *** eenoch has quit IRC
2412016-07-27T17:53:33  *** eenoch has joined #bitcoin-core-dev
2422016-07-27T17:54:10  *** kadoban has joined #bitcoin-core-dev
2432016-07-27T18:00:32  *** menix01 has quit IRC
2442016-07-27T18:01:50  *** netzin has joined #bitcoin-core-dev
2452016-07-27T18:02:45  *** goatpig has joined #bitcoin-core-dev
2462016-07-27T18:03:36  <goatpig> hi
2472016-07-27T18:04:09  <goatpig> you don't need the mask and flag when creating a tx with only segwit outputs, do you?
2482016-07-27T18:04:27  <goatpig> as in, no segwit inputs redeeming, only creating segwit outputs
2492016-07-27T18:04:32  <goatpig> non nested
2502016-07-27T18:07:36  *** jtimon has joined #bitcoin-core-dev
2512016-07-27T18:13:24  <luke-jr> goatpig: that is my understanding, correct
2522016-07-27T18:14:01  <luke-jr> furthermore, I think you can even include such outputs in non-segwit blocks. albeit, they would be vulnerable to easy theft until segwit activates.
2532016-07-27T18:14:18  <luke-jr> (but it's possible to mine non-segwit blocks even after segwit activates, so not entirely useless)
2542016-07-27T18:15:10  <goatpig> in this case they would just behave as anyone can spend wouldn't they? at least until miners start enforcing segwit rules
2552016-07-27T18:17:14  <luke-jr> right
2562016-07-27T18:17:40  <goatpig> so the proper guideline would be to add mask and flag when creating non nested segwit outputs, regardless of inputs?
2572016-07-27T18:18:08  <luke-jr> even after segwit activates, libblkmaker will produce non-segwit blocks if there are no witness-as-input transactions being mined
2582016-07-27T18:18:16  <gmaxwell> Well they're non-standard currently.
2592016-07-27T18:18:35  <luke-jr> goatpig: I don't understand your question there
2602016-07-27T18:18:46  <goatpig> im thinking for after segwit is activated, I don't intent to allow ppl to create SW tx with Armory until a few weeks after that happens on the mainnet
2612016-07-27T18:18:59  <luke-jr> IIRC, it's invalid to have dummy/flag if there's no witness data for inputs
2622016-07-27T18:19:08  <goatpig> luke-jr: oic
2632016-07-27T18:19:19  <luke-jr> (I could be wrong on that, but IIRC)
2642016-07-27T18:19:42  <goatpig> wouldn't that mean that you can't mix regular and sw outputs in a same tx?
2652016-07-27T18:19:58  <luke-jr> goatpig: no, you can have dummy/flag so long as at least one input needs witness data
2662016-07-27T18:20:06  <goatpig> ok
2672016-07-27T18:20:33  <goatpig> aight that sums it for me
2682016-07-27T18:20:35  <goatpig> thanks for the help
2692016-07-27T18:31:41  *** jannes has quit IRC
2702016-07-27T18:45:37  *** laurentmt has joined #bitcoin-core-dev
2712016-07-27T18:45:50  *** laurentmt has quit IRC
2722016-07-27T18:49:03  *** fengling has joined #bitcoin-core-dev
2732016-07-27T18:53:46  *** fengling has quit IRC
2742016-07-27T19:08:44  *** OxADADA has quit IRC
2752016-07-27T19:09:12  *** OxADADA has joined #bitcoin-core-dev
2762016-07-27T19:46:10  <jtimon> mhmm, there's a bitcoinconsensus_SCRIPT_FLAGS_VERIFY_WITNESS but not a bitcoinconsensus_SCRIPT_VERIFY_CHECKSEQUENCEVERIFY in script/bitcoinconsensus.h...
2772016-07-27T19:49:38  <sipa> jtimon: good point, we should fix that
2782016-07-27T19:50:18  <jtimon> I'll PR it, just wanted to confirm first
2792016-07-27T19:50:23  <jtimon> sipa: thanks
2802016-07-27T19:50:40  *** fengling has joined #bitcoin-core-dev
2812016-07-27T19:51:18  <jtimon> actually found out while adapting my consensus branch to ScriptFlagsFromConsensus()as discussed (WIP)
2822016-07-27T19:55:06  *** fengling has quit IRC
2832016-07-27T20:03:48  *** cryptapus has quit IRC
2842016-07-27T20:03:50  *** goatpig has quit IRC
2852016-07-27T20:13:06  *** d_t has quit IRC
2862016-07-27T20:24:21  *** anu0 has joined #bitcoin-core-dev
2872016-07-27T20:27:12  *** anu1 has quit IRC
2882016-07-27T20:28:33  *** d_t has joined #bitcoin-core-dev
2892016-07-27T20:36:34  *** Chris_Stewart_5 has quit IRC
2902016-07-27T20:51:35  *** fengling has joined #bitcoin-core-dev
2912016-07-27T20:53:12  *** owowo has quit IRC
2922016-07-27T20:54:21  *** owowo has joined #bitcoin-core-dev
2932016-07-27T20:56:26  *** fengling has quit IRC
2942016-07-27T21:02:12  *** owowo has quit IRC
2952016-07-27T21:08:57  *** BashCo_ has quit IRC
2962016-07-27T21:13:39  <GitHub48> [bitcoin] jtimon opened pull request #8412: libconsensus: Expose a flag for BIP112 (master...0.13-consensus-bip112-flag) https://github.com/bitcoin/bitcoin/pull/8412
2972016-07-27T21:14:09  <jtimon> does this little thing break rc1?
2982016-07-27T21:23:57  *** droark has quit IRC
2992016-07-27T21:25:21  *** BashCo has joined #bitcoin-core-dev
3002016-07-27T21:31:22  *** owowo has joined #bitcoin-core-dev
3012016-07-27T21:35:58  *** belcher has joined #bitcoin-core-dev
3022016-07-27T21:49:36  <GitHub132> [bitcoin] jtimon opened pull request #8413: Trivial: pass Consensus::Params& instead of CChainParams& in ContextualCheckBlock (master...0.13-consensus-last-params) https://github.com/bitcoin/bitcoin/pull/8413
3032016-07-27T21:53:05  *** fengling has joined #bitcoin-core-dev
3042016-07-27T21:57:46  *** fengling has quit IRC
3052016-07-27T21:59:13  *** spudowiar has joined #bitcoin-core-dev
3062016-07-27T22:13:30  *** Guyver2 has quit IRC
3072016-07-27T22:18:01  <jtimon> btcdrak: quick question https://github.com/bitcoin/bitcoin/pull/8412#issuecomment-235738474
3082016-07-27T22:19:21  *** TomMc has quit IRC
3092016-07-27T22:26:57  <jtimon> never mind, updated without bip112 at the end
3102016-07-27T22:32:45  *** TomMc has joined #bitcoin-core-dev
3112016-07-27T22:37:00  *** Chris_Stewart_5 has joined #bitcoin-core-dev
3122016-07-27T22:52:48  *** d_t has quit IRC
3132016-07-27T22:54:05  *** fengling has joined #bitcoin-core-dev
3142016-07-27T22:58:46  *** fengling has quit IRC
3152016-07-27T23:32:16  *** cdecker has quit IRC
3162016-07-27T23:32:27  <GitHub43> [bitcoin] kazcw opened pull request #8414: prepend license statement to indirectmap.h (master...indirectmap-license) https://github.com/bitcoin/bitcoin/pull/8414
3172016-07-27T23:35:27  *** fengling has joined #bitcoin-core-dev
3182016-07-27T23:55:02  *** netzin has quit IRC