12017-06-03T00:05:49  <luke-jr> spudI think you'll get arguments for using pb at all
  22017-06-03T00:15:06  *** elkalamar has quit IRC
  32017-06-03T00:22:10  *** str4d has joined #bitcoin-core-dev
  42017-06-03T00:33:03  *** Giszmo has quit IRC
  52017-06-03T00:45:45  *** nemgun1 has quit IRC
  62017-06-03T01:03:56  *** Ylbam has quit IRC
  72017-06-03T01:11:06  *** elkalamar has joined #bitcoin-core-dev
  82017-06-03T01:23:24  *** Chris_Stewart_5 has joined #bitcoin-core-dev
  92017-06-03T01:39:10  *** laurentmt has quit IRC
 102017-06-03T02:24:15  <cfields> wumpus: considering there's already an 0.14.1rc1, I'd prefer not to sign the binaries with incorrect version
 112017-06-03T02:24:57  <cfields> i'd say just go ahead with release without signature
 122017-06-03T02:26:58  *** Chris_Stewart_5 has quit IRC
 132017-06-03T02:30:44  *** RubenSomsen has joined #bitcoin-core-dev
 142017-06-03T02:43:10  *** PaulCapestany has quit IRC
 152017-06-03T02:47:23  *** juscamarena has joined #bitcoin-core-dev
 162017-06-03T02:47:23  *** juscamarena_ has joined #bitcoin-core-dev
 172017-06-03T02:47:37  *** juscamarena has quit IRC
 182017-06-03T03:09:01  *** d9b4bef9 has quit IRC
 192017-06-03T03:10:08  *** d9b4bef9 has joined #bitcoin-core-dev
 202017-06-03T03:14:37  *** jtimon has quit IRC
 212017-06-03T03:51:14  *** cryptapus_afk has quit IRC
 222017-06-03T04:16:26  *** kadoban has quit IRC
 232017-06-03T04:24:56  *** PaulCapestany has joined #bitcoin-core-dev
 242017-06-03T04:27:26  *** Squidicc has joined #bitcoin-core-dev
 252017-06-03T04:29:21  *** Squidicuz has quit IRC
 262017-06-03T04:40:08  *** Squidicuz has joined #bitcoin-core-dev
 272017-06-03T04:41:51  *** Squidicc has quit IRC
 282017-06-03T04:50:01  <bitcoin-git> [bitcoin] jimmysong opened pull request #10515: [test] Add test for getchaintxstats (master...test_getchaintxstats) https://github.com/bitcoin/bitcoin/pull/10515
 292017-06-03T04:51:00  *** goatturner has quit IRC
 302017-06-03T04:52:12  <achow101> does anyone know if the waitforblock/waitfornewblock RPCs are used or will be used sometime in the future?
 312017-06-03T04:52:47  <achow101> The PR that adds them says that they are for testing and travis, but I don't see anything which actually uses those commands.
 322017-06-03T04:52:55  <achow101> cfields: ^^^ (you're the one who added them)
 332017-06-03T04:55:03  *** cryptapus has joined #bitcoin-core-dev
 342017-06-03T04:55:07  *** cryptapus is now known as cryptapus_afk
 352017-06-03T05:04:40  *** Guest80280 has joined #bitcoin-core-dev
 362017-06-03T05:06:14  <bitcoin-git> [bitcoin] luke-jr opened pull request #10516: Backports for 0.14.2 (0.14...0.14.2_fixes) https://github.com/bitcoin/bitcoin/pull/10516
 372017-06-03T05:09:05  *** cryptapus_afk has quit IRC
 382017-06-03T05:16:58  *** cryptapus_afk has joined #bitcoin-core-dev
 392017-06-03T05:16:58  *** cryptapus_afk has joined #bitcoin-core-dev
 402017-06-03T05:36:08  *** Yogaqueef has joined #bitcoin-core-dev
 412017-06-03T05:37:28  *** goatturner has joined #bitcoin-core-dev
 422017-06-03T05:47:09  *** RubenSomsen has quit IRC
 432017-06-03T05:51:56  *** sahl has joined #bitcoin-core-dev
 442017-06-03T05:57:01  *** RubenSomsen has joined #bitcoin-core-dev
 452017-06-03T06:00:18  *** goatturneer has joined #bitcoin-core-dev
 462017-06-03T06:01:37  *** Guest80280 has quit IRC
 472017-06-03T06:03:59  *** goatturner has quit IRC
 482017-06-03T06:07:21  *** kanzure has quit IRC
 492017-06-03T06:09:14  *** kanzure has joined #bitcoin-core-dev
 502017-06-03T06:10:57  *** grubles has quit IRC
 512017-06-03T06:12:51  *** zeusa1mighty has joined #bitcoin-core-dev
 522017-06-03T06:13:13  *** zeusalmighty has quit IRC
 532017-06-03T06:15:40  *** grubles has joined #bitcoin-core-dev
 542017-06-03T06:15:40  *** grubles has joined #bitcoin-core-dev
 552017-06-03T06:22:57  *** RubenSomsen has quit IRC
 562017-06-03T06:28:16  *** paveljanik has quit IRC
 572017-06-03T06:30:10  *** beatrootfarmer has joined #bitcoin-core-dev
 582017-06-03T06:33:43  *** goatturner has joined #bitcoin-core-dev
 592017-06-03T06:33:51  *** goatturneer has quit IRC
 602017-06-03T06:37:02  *** beatrootfarmer has quit IRC
 612017-06-03T07:18:18  *** cryptapus_afk has quit IRC
 622017-06-03T07:18:44  *** cryptapus_afk has joined #bitcoin-core-dev
 632017-06-03T07:48:48  <bitcoin-git> [bitcoin] jonasschnelli opened pull request #10517: Factor out CCoinsView based AreInputsStandard/IsWitnessStandard (master...2017/06/policy_compile) https://github.com/bitcoin/bitcoin/pull/10517
 642017-06-03T07:50:17  *** cryptapus_afk has quit IRC
 652017-06-03T07:53:44  *** [Author] has quit IRC
 662017-06-03T07:54:47  *** cryptapus_afk has joined #bitcoin-core-dev
 672017-06-03T07:58:46  *** cryptapus_afk has quit IRC
 682017-06-03T08:01:15  *** [Author] has joined #bitcoin-core-dev
 692017-06-03T08:03:09  *** vicenteH has joined #bitcoin-core-dev
 702017-06-03T08:09:12  *** btcdrak has quit IRC
 712017-06-03T08:12:59  *** goatturneer has joined #bitcoin-core-dev
 722017-06-03T08:16:56  *** goatturner has quit IRC
 732017-06-03T08:18:12  *** Ylbam has joined #bitcoin-core-dev
 742017-06-03T08:28:35  *** beatrootfarmer has joined #bitcoin-core-dev
 752017-06-03T08:30:15  *** goatturner has joined #bitcoin-core-dev
 762017-06-03T08:32:23  *** goatturneer has quit IRC
 772017-06-03T08:34:02  *** beatrootfarmer has quit IRC
 782017-06-03T08:48:51  *** cryptapus_afk has joined #bitcoin-core-dev
 792017-06-03T09:01:54  *** cryptapus_afk has quit IRC
 802017-06-03T09:05:14  *** btcdrak has joined #bitcoin-core-dev
 812017-06-03T09:16:05  *** spudowiar has joined #bitcoin-core-dev
 822017-06-03T09:45:13  *** Guest80280 has joined #bitcoin-core-dev
 832017-06-03T09:51:08  *** tunafizz has quit IRC
 842017-06-03T09:51:34  *** tunafizz has joined #bitcoin-core-dev
 852017-06-03T09:56:34  *** tiagotrs has joined #bitcoin-core-dev
 862017-06-03T10:10:02  *** d9b4bef9 has quit IRC
 872017-06-03T10:11:08  *** d9b4bef9 has joined #bitcoin-core-dev
 882017-06-03T10:33:09  *** rockhouse has joined #bitcoin-core-dev
 892017-06-03T10:41:26  <spudowiar> luke-jr: I meant changing the use of Protocol Buffers in Payment Requests
 902017-06-03T10:43:08  <phantomcircuit> spudowiar, the payments protocol stuff is basically just broken
 912017-06-03T10:43:37  <spudowiar> I was just suggesting switching from Google Protocol Buffers to nanopb, because nanopb generates nicer code
 922017-06-03T10:43:53  <spudowiar> (It's a protoc plugin)
 932017-06-03T10:46:51  *** rockhouse has quit IRC
 942017-06-03T10:48:06  *** rockhouse has joined #bitcoin-core-dev
 952017-06-03T10:48:43  *** tiagotrs has quit IRC
 962017-06-03T10:49:16  *** rockhouse has quit IRC
 972017-06-03T10:49:16  *** rockhouse has joined #bitcoin-core-dev
 982017-06-03T10:56:22  *** chjj has quit IRC
 992017-06-03T11:07:47  <jonasschnelli> spudowiar: is nanopb a dropin replacement for protoc?
1002017-06-03T11:08:29  *** SopaXorzTaker has joined #bitcoin-core-dev
1012017-06-03T11:08:33  *** Guyver2 has joined #bitcoin-core-dev
1022017-06-03T11:10:16  *** chjj has joined #bitcoin-core-dev
1032017-06-03T11:10:51  *** gielbier has quit IRC
1042017-06-03T11:12:51  <spudowiar> jonasschnelli: It's a plugin for protoc. It generates different code but nicer code. It's primarily designed for C and microcontrollers though so it might not be that good a fit. Something to take a look at though.
1052017-06-03T11:13:16  <jonasschnelli> Thanks... will take a closer look soon.
1062017-06-03T11:32:35  *** Yogaqueef has quit IRC
1072017-06-03T11:45:09  <jonasschnelli> Just curios,.. whats the reason for including ctaes.c in aes.c and adding CT_AES to EXTRA_DIST? instead of compiling/linking it regularly?
1082017-06-03T11:48:45  <spudowiar> Has anyone tested secp256k1 with ctgrind?
1092017-06-03T11:49:18  <spudowiar> (Valgrind constant time checker by Adam Langley)
1102017-06-03T11:50:01  <spudowiar> I mean libsecp256k1
1112017-06-03T11:56:25  <spudowiar> Actually, looking at the Modern Crypto mailing list, looks gmaxwell or someone did something similar
1122017-06-03T12:20:54  *** elkalamar has quit IRC
1132017-06-03T12:29:37  *** jtimon has joined #bitcoin-core-dev
1142017-06-03T12:32:50  *** spudowiar has quit IRC
1152017-06-03T12:38:42  *** spudowiar has joined #bitcoin-core-dev
1162017-06-03T12:40:11  <jtimon> jnewbery: perhaps if we can't agree on changing the current make check we can at least start by adding new targets that could replace it, check-functional, check-all or something
1172017-06-03T12:40:40  <jtimon> I would love to run the functional tests from the makefile
1182017-06-03T13:01:56  *** nemgun has joined #bitcoin-core-dev
1192017-06-03T13:21:19  *** Chris_Stewart_5 has joined #bitcoin-core-dev
1202017-06-03T13:25:40  *** RubenSomsen has joined #bitcoin-core-dev
1212017-06-03T14:02:13  *** Guest80280 has quit IRC
1222017-06-03T14:16:11  *** Guest80280 has joined #bitcoin-core-dev
1232017-06-03T14:19:22  *** laurentmt has joined #bitcoin-core-dev
1242017-06-03T14:29:04  <bitcoin-git> [bitcoin] biniek opened pull request #10518: Fix translation error (master...patch-1) https://github.com/bitcoin/bitcoin/pull/10518
1252017-06-03T14:44:35  <cfields> achow101: they were needed at one point to fix a race with the wallet. They're probably unnecessary now.
1262017-06-03T14:46:13  <cfields> jonasschnelli: it's a .c file, but we want to make sure it gets compiled as c++, to avoid accidentally mixing in c flags
1272017-06-03T14:47:05  <cfields> jonasschnelli: because we add everything to cxxflags. We might forget to (for ex) add fPIC/fPIE to CFLAGS
1282017-06-03T15:08:22  *** str4d has quit IRC
1292017-06-03T15:16:22  <jonasschnelli> cfields: thanks. I see.
1302017-06-03T15:16:27  *** chjj has quit IRC
1312017-06-03T15:30:49  *** chjj has joined #bitcoin-core-dev
1322017-06-03T15:51:53  <spudowiar> jonasschnelli: So, if Core RPC doesn't have a method for getting the xpub, should I just create one for my HWW support?
1332017-06-03T15:52:13  <spudowiar> Basically, I'm going to do a HWW plugin that's either a command or a server and supports JSON-RPC
1342017-06-03T15:52:46  <spudowiar> e.g. You could do `bitcoin-qt -hww-cmd=bitcoin-hww-trezor`
1352017-06-03T15:53:08  <spudowiar> Or even `bitcoin-qt -hww-server=http://user:password@othermachine:8332` and connect to another bitcoind
1362017-06-03T15:53:33  <spudowiar> So, I guess I should just create another method or smth
1372017-06-03T15:58:57  *** Dyaheon has quit IRC
1382017-06-03T15:59:41  *** Dyaheon has joined #bitcoin-core-dev
1392017-06-03T16:22:05  *** chjj has quit IRC
1402017-06-03T16:36:32  *** chjj has joined #bitcoin-core-dev
1412017-06-03T16:36:46  <bitcoin-git> [bitcoin] achow101 opened pull request #10519: [RPC] Remove waitforblock and waitfornewblock (master...rm-waitforblock-rpcs) https://github.com/bitcoin/bitcoin/pull/10519
1422017-06-03T16:38:07  <jonasschnelli> spudowiar: for HWW support, Core needs flexible keypath (BIP44) with support for pub-key-derivation (currently only hardened derivation is supported), there is a PR from NicolasDorier (check it out).
1432017-06-03T16:38:28  <jonasschnelli> You don't want to mix xpriv with support for single child key private key exports
1442017-06-03T16:39:05  <jonasschnelli> mixing non hardened derivation with private child key export is kinda careless
1452017-06-03T16:52:55  <spudowiar> jonasschnelli: I have all the necessary patches from NicolasDorier applied to my tree
1462017-06-03T16:53:15  <spudowiar> jonasschnelli: Anyway, the HWW plugin has an RPC method gethwwinfo
1472017-06-03T16:53:23  <spudowiar> That contains some information about the hardware wallet
1482017-06-03T16:53:28  <spudowiar> Like the xpub
1492017-06-03T16:55:17  *** laurentmt has quit IRC
1502017-06-03T16:56:05  <jonasschnelli> spudowiar: So you are adding those methods to Core? Or do you have a "middleware" between Core and the HWW?
1512017-06-03T16:56:12  <jonasschnelli> Do you intend to PR that stuff?
1522017-06-03T16:56:36  <spudowiar> So, I have a HWW plugin (a command called bitcoin-hww-trezor)
1532017-06-03T16:56:49  <spudowiar> And CWallet starts that process
1542017-06-03T16:57:21  <spudowiar> Then I can send RPC requests over stdio to the process and it can send RPC requests back to Core
1552017-06-03T16:57:37  <spudowiar> So, when something is signed, it sends signrawtransaction to the HWW plugin
1562017-06-03T16:57:52  <spudowiar> And the HWW plugin will do stuff like send validateaddress (to get the keypath) and decoderawtransaction, etc.
1572017-06-03T16:58:13  <spudowiar> Also, there's a gethwwinfo which is sent by Core right at the start to get the xpub, etc.
1582017-06-03T17:01:24  <jonasschnelli> cool stuff
1592017-06-03T17:01:32  <spudowiar> Although this is quite a bit more complicated than my previous method so I'm not sure how good an idea it is
1602017-06-03T17:01:53  <spudowiar> Previous method was just start the process when signing needs to happen and pipe a JSON representation of the transaction, with some metadata
1612017-06-03T17:02:02  <jonasschnelli> IMO working with watchonlys on the Core side and send out the unsigned transaction to a plugin could make more sense?
1622017-06-03T17:02:24  <spudowiar> That's what it does
1632017-06-03T17:02:30  <jonasschnelli> ah! cool
1642017-06-03T17:02:55  <spudowiar> So, if it's a hardware wallet, CWallet::IsMine will treat watch only as normal keys
1652017-06-03T17:03:20  <spudowiar> e.g. ISMINE_WATCH_UNSOLVABLE => ISMINE_NO and ISMINE_WATCH_SOLVABLE => ISMINE_SPENDABLE
1662017-06-03T17:29:39  *** d_t has joined #bitcoin-core-dev
1672017-06-03T17:31:23  *** Chris_Stewart_5 has quit IRC
1682017-06-03T17:38:39  *** Chris_Stewart_5 has joined #bitcoin-core-dev
1692017-06-03T18:03:52  *** Chris_Stewart_5 has quit IRC
1702017-06-03T18:03:53  *** Dyaheon has quit IRC
1712017-06-03T18:04:55  *** Dyaheon has joined #bitcoin-core-dev
1722017-06-03T18:15:24  <bitcoin-git> [bitcoin] sipa closed pull request #10518: Fix translation error (master...patch-1) https://github.com/bitcoin/bitcoin/pull/10518
1732017-06-03T18:29:21  *** Guest52697 has joined #bitcoin-core-dev
1742017-06-03T18:38:01  *** RubenSomsen has quit IRC
1752017-06-03T19:04:13  *** elkalamar has joined #bitcoin-core-dev
1762017-06-03T19:05:50  *** davec has quit IRC
1772017-06-03T19:10:44  *** laurentmt has joined #bitcoin-core-dev
1782017-06-03T19:11:14  *** laurentmt has quit IRC
1792017-06-03T19:17:25  *** davec has joined #bitcoin-core-dev
1802017-06-03T19:17:46  <spudowiar> jonasschnelli: What do you think about using a file descriptor for the child? Because having bi directional JSON-RPC gets a bit messy
1812017-06-03T19:18:08  <spudowiar> But I could do stdin, stdout and a file descriptor for a socket (to the Bitcoin daemon RPC server)
1822017-06-03T19:18:17  <spudowiar> Hmm, maybe not actually
1832017-06-03T19:18:31  <spudowiar> To be honest, only when you want to sign something should it handle the messages
1842017-06-03T19:18:33  <spudowiar> nvm
1852017-06-03T19:18:35  <spudowiar> :)
1862017-06-03T19:23:14  <luke-jr> spudowiar: why do you need bi-directional?
1872017-06-03T19:27:34  <spudowiar> luke-jr: Because then you can do "Bitcoin Core asks HWW signrawtransaction", "HWW asks Bitcoin Core gettransaction", "HWW responds to signrawtransaction"
1882017-06-03T19:27:56  <luke-jr> ah
1892017-06-03T19:29:25  <gmaxwell> spudowiar: I think it should be safe to assume the wallet needs the inputs and just pass them.
1902017-06-03T19:29:45  <gmaxwell> it can just drop them if it doesn't need them.
1912017-06-03T19:30:05  <spudowiar> gmaxwell: I was thinking of mimicking an API like signrawtransaction though
1922017-06-03T19:30:42  <gmaxwell> apparently not, since that api doesn't have two way communication. :P
1932017-06-03T19:31:02  <spudowiar> Well, it's going to be over stdio :)
1942017-06-03T19:31:30  <spudowiar> It's basically mimicking the API, because you could just as easily have the HWW plugin connect to the Bitcoin Core over HTTP RPC
1952017-06-03T19:31:42  <spudowiar> Except it's doing it over stdio
1962017-06-03T19:31:52  <spudowiar> Which happens to be where the other call came from :)
1972017-06-03T19:31:54  <sipa> thinking further, it seems strange that the HW/wrapper would need to know what it's missing
1982017-06-03T19:32:15  <spudowiar> What do you mean?
1992017-06-03T19:32:16  <sipa> especially if we ever get schnorr multisig or even aggregation
2002017-06-03T19:32:42  <sipa> the signrawtransaction API is "pass it in partially signed transactions, and do with it what you can"
2012017-06-03T19:33:05  <sipa> i think that's a good approach, also for HW wallets
2022017-06-03T19:33:20  <spudowiar> Yeah, but it needs to have the input transactions
2032017-06-03T19:33:25  <sipa> so pass them
2042017-06-03T19:33:32  <sipa> the wallet knows whether the HW wallet needs them
2052017-06-03T19:33:44  <spudowiar> Right, so don't bother with the JSON-RPC emulation then?
2062017-06-03T19:33:56  <sipa> i think that's unnecessarily complicating the API
2072017-06-03T19:34:19  <spudowiar> Well, I kept my old branch behind (without this stuff), so that's fine :)
2082017-06-03T19:34:23  <spudowiar> This was just an experiment :)
2092017-06-03T19:36:55  *** chjj has quit IRC
2102017-06-03T19:43:51  *** spudowiar has quit IRC
2112017-06-03T19:50:25  *** chjj has joined #bitcoin-core-dev
2122017-06-03T20:29:14  *** Chris_Stewart_5 has joined #bitcoin-core-dev
2132017-06-03T20:35:10  *** roidster has joined #bitcoin-core-dev
2142017-06-03T20:35:10  *** roidster is now known as Guest64521
2152017-06-03T20:36:15  *** d_t has quit IRC
2162017-06-03T20:41:46  *** chjj has quit IRC
2172017-06-03T20:55:01  *** chjj has joined #bitcoin-core-dev
2182017-06-03T21:01:28  *** Chris_Stewart_5 has quit IRC
2192017-06-03T21:27:27  *** Guest64521 is now known as roidster
2202017-06-03T21:37:09  *** Chris_Stewart_5 has joined #bitcoin-core-dev
2212017-06-03T21:42:08  <bitcoin-git> [bitcoin] practicalswift opened pull request #10521: Limit variable scope (master...tighten-scope) https://github.com/bitcoin/bitcoin/pull/10521
2222017-06-03T21:47:08  *** chjj has quit IRC
2232017-06-03T21:55:28  <bitcoin-git> [bitcoin] practicalswift opened pull request #10522: Remove unused variables (master...unused-strings) https://github.com/bitcoin/bitcoin/pull/10522
2242017-06-03T21:57:52  *** Guest80280 has quit IRC
2252017-06-03T21:59:48  *** roidster has quit IRC
2262017-06-03T22:00:53  *** chjj has joined #bitcoin-core-dev
2272017-06-03T22:34:58  *** Guest80280 has joined #bitcoin-core-dev
2282017-06-03T22:35:39  *** Chris_Stewart_5 has quit IRC
2292017-06-03T22:55:54  *** AaronvanW has quit IRC
2302017-06-03T22:56:32  *** AaronvanW has joined #bitcoin-core-dev
2312017-06-03T23:19:40  *** Guyver2 has quit IRC
2322017-06-03T23:20:17  *** Chris_Stewart_5 has joined #bitcoin-core-dev
2332017-06-03T23:20:18  <bitcoin-git> [bitcoin] practicalswift opened pull request #10523: Perform member initialization in initialization lists where possible (master...initialization-list) https://github.com/bitcoin/bitcoin/pull/10523
2342017-06-03T23:31:36  <bitcoin-git> [bitcoin] practicalswift opened pull request #10524: Fix incorrect format strings (master...u-for-unsigned-int) https://github.com/bitcoin/bitcoin/pull/10524
2352017-06-03T23:33:31  *** justanotheruser has quit IRC
2362017-06-03T23:34:56  *** belcher has joined #bitcoin-core-dev
2372017-06-03T23:37:55  *** justanotheruser has joined #bitcoin-core-dev
2382017-06-03T23:51:32  *** Yogaqueef has joined #bitcoin-core-dev
2392017-06-03T23:53:12  *** Guest19621 has joined #bitcoin-core-dev
2402017-06-03T23:56:27  *** Chris_Stewart_5 has quit IRC
2412017-06-03T23:56:32  *** Guest19621 is now known as roidster
2422017-06-03T23:58:31  *** str4d has joined #bitcoin-core-dev