12018-10-20T00:04:16  *** justanotheruser has quit IRC
  22018-10-20T00:05:05  *** Murch has joined #bitcoin-core-dev
  32018-10-20T00:05:58  *** fanquake has joined #bitcoin-core-dev
  42018-10-20T00:15:17  *** Aaronvan_ has quit IRC
  52018-10-20T00:15:54  *** AaronvanW has joined #bitcoin-core-dev
  62018-10-20T00:26:07  *** bralyclow3 has joined #bitcoin-core-dev
  72018-10-20T00:27:17  *** bralyclow3 has quit IRC
  82018-10-20T00:28:57  *** Murch has quit IRC
  92018-10-20T00:30:43  *** IGHOR has quit IRC
 102018-10-20T00:34:12  *** intcat has quit IRC
 112018-10-20T00:35:03  *** Chris_Stewart_5 has quit IRC
 122018-10-20T00:36:12  *** intcat has joined #bitcoin-core-dev
 132018-10-20T00:39:49  *** IGHOR has joined #bitcoin-core-dev
 142018-10-20T00:41:30  *** Murch has joined #bitcoin-core-dev
 152018-10-20T00:44:11  *** dviola has joined #bitcoin-core-dev
 162018-10-20T00:55:06  *** spinza has quit IRC
 172018-10-20T00:56:53  *** jcorgan has joined #bitcoin-core-dev
 182018-10-20T00:59:08  <gwillen> achow101: can you explain the comment "Get all of the previous transactions" in rawtransaction.cpp (in finalizepsbt)?
 192018-10-20T00:59:28  <gwillen> it appears to be just using SignPSBTInput to check if inputs are signed -- is it also intentionally mutating the transaction in some way or something?
 202018-10-20T00:59:45  <gwillen> or is it safe to replace with "is PSBT input signed"?
 212018-10-20T01:00:08  <achow101> that's very likely to be the result of copy-paste from some other rpc
 222018-10-20T01:00:15  * sipa guesses: the comment was copied from rpcwallet.cpp
 232018-10-20T01:04:26  *** dviola has quit IRC
 242018-10-20T01:05:28  *** Murch has quit IRC
 252018-10-20T01:11:19  *** AaronvanW has quit IRC
 262018-10-20T01:12:34  *** j|l has joined #bitcoin-core-dev
 272018-10-20T01:13:17  *** kexkey has joined #bitcoin-core-dev
 282018-10-20T01:13:39  *** proletesseract has joined #bitcoin-core-dev
 292018-10-20T01:13:49  *** jpe_ has joined #bitcoin-core-dev
 302018-10-20T01:16:51  *** jpe__ has quit IRC
 312018-10-20T01:19:49  *** spinza has joined #bitcoin-core-dev
 322018-10-20T01:23:59  *** proletesseract has quit IRC
 332018-10-20T01:26:25  *** j|l has quit IRC
 342018-10-20T01:40:34  *** Krellan_ has quit IRC
 352018-10-20T01:45:16  *** bitcoin-git has joined #bitcoin-core-dev
 362018-10-20T01:45:17  <bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/d387507aeca6...8010ded6da56
 372018-10-20T01:45:17  <bitcoin-git> bitcoin/master 7e18673 Kristaps Kaupe: Fix typo
 382018-10-20T01:45:18  <bitcoin-git> bitcoin/master 8010ded Pieter Wuille: Merge #14524: Trivial: fix typo...
 392018-10-20T01:45:18  *** bitcoin-git has left #bitcoin-core-dev
 402018-10-20T01:46:31  *** bitcoin-git has joined #bitcoin-core-dev
 412018-10-20T01:46:32  <bitcoin-git> [bitcoin] sipa closed pull request #14524: Trivial: fix typo (master...typos) https://github.com/bitcoin/bitcoin/pull/14524
 422018-10-20T01:46:32  *** bitcoin-git has left #bitcoin-core-dev
 432018-10-20T01:48:59  *** Chris_Stewart_5 has joined #bitcoin-core-dev
 442018-10-20T01:53:06  *** bitcoin-git has joined #bitcoin-core-dev
 452018-10-20T01:53:07  <bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/8010ded6da56...b14db5abab40
 462018-10-20T01:53:08  <bitcoin-git> bitcoin/master bc60c61 practicalswift: Avoid 1 << 31 (UB) in calculation of SEQUENCE_LOCKTIME_DISABLE_FLAG
 472018-10-20T01:53:08  <bitcoin-git> bitcoin/master b14db5a Pieter Wuille: Merge #14513: Avoid 1 << 31 (UB) in calculation of SEQUENCE_LOCKTIME_DISABLE_FLAG...
 482018-10-20T01:53:09  *** bitcoin-git has left #bitcoin-core-dev
 492018-10-20T01:54:19  *** bitcoin-git has joined #bitcoin-core-dev
 502018-10-20T01:54:20  <bitcoin-git> [bitcoin] sipa closed pull request #14513: Avoid 1 << 31 (UB) in calculation of SEQUENCE_LOCKTIME_DISABLE_FLAG (master...1<<31-again) https://github.com/bitcoin/bitcoin/pull/14513
 512018-10-20T01:54:20  *** bitcoin-git has left #bitcoin-core-dev
 522018-10-20T01:57:07  *** dviola has joined #bitcoin-core-dev
 532018-10-20T02:00:14  *** bitcoin-git has joined #bitcoin-core-dev
 542018-10-20T02:00:15  <bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b14db5abab40...b2863c0685a5
 552018-10-20T02:00:15  <bitcoin-git> bitcoin/master 369244f Chun Kuan Lee: utils: Fix broken Windows filelock
 562018-10-20T02:00:16  <bitcoin-git> bitcoin/master b2863c0 Pieter Wuille: Merge #14426: utils: Fix broken Windows filelock...
 572018-10-20T02:00:17  *** bitcoin-git has left #bitcoin-core-dev
 582018-10-20T02:01:14  *** bitcoin-git has joined #bitcoin-core-dev
 592018-10-20T02:01:14  <bitcoin-git> [bitcoin] sipa closed pull request #14426: utils: Fix broken Windows filelock (master...filelock-test) https://github.com/bitcoin/bitcoin/pull/14426
 602018-10-20T02:01:14  *** bitcoin-git has left #bitcoin-core-dev
 612018-10-20T02:02:18  *** Chris_Stewart_5 has quit IRC
 622018-10-20T02:03:06  *** bitcoin-git has joined #bitcoin-core-dev
 632018-10-20T02:03:07  <bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b2863c0685a5...dc1e54206d76
 642018-10-20T02:03:07  <bitcoin-git> bitcoin/master 1f01fe0 Antoine Le Calvez: bitcoin-tx: Use constant for n pubkeys check...
 652018-10-20T02:03:08  <bitcoin-git> bitcoin/master dc1e542 Pieter Wuille: Merge #14474: bitcoin-tx: Use constant for n pubkeys check...
 662018-10-20T02:03:08  *** bitcoin-git has left #bitcoin-core-dev
 672018-10-20T02:04:09  *** bitcoin-git has joined #bitcoin-core-dev
 682018-10-20T02:04:10  <bitcoin-git> [bitcoin] sipa closed pull request #14474: bitcoin-tx: Use constant for n pubkeys check (master...bitcoin_tx_use_constant) https://github.com/bitcoin/bitcoin/pull/14474
 692018-10-20T02:04:10  *** bitcoin-git has left #bitcoin-core-dev
 702018-10-20T02:05:04  *** bitcoin-git has joined #bitcoin-core-dev
 712018-10-20T02:05:05  <bitcoin-git> [bitcoin] sipa pushed 2 new commits to 0.17: https://github.com/bitcoin/bitcoin/compare/5b47b8efd48d...1b5af2c177ee
 722018-10-20T02:05:05  <bitcoin-git> bitcoin/0.17 f7dbcaa Sjors Provoost: [doc] getblocktemplate: use SegWit in example
 732018-10-20T02:05:06  <bitcoin-git> bitcoin/0.17 1b5af2c Pieter Wuille: Merge #14509: [0.17] doc: use SegWit in getblocktemplate example...
 742018-10-20T02:05:06  *** bitcoin-git has left #bitcoin-core-dev
 752018-10-20T02:08:59  <fanquake> sipa It'd be great if you could also merged #14011
 762018-10-20T02:09:01  <gribble> https://github.com/bitcoin/bitcoin/issues/14011 | Disable wallet and address book Qt tests on macOS minimal platform by ryanofsky · Pull Request #14011 · bitcoin/bitcoin · GitHub
 772018-10-20T02:12:19  <sipa> fanquake: sgtm
 782018-10-20T02:13:18  *** ula has quit IRC
 792018-10-20T02:14:33  *** bitcoin-git has joined #bitcoin-core-dev
 802018-10-20T02:14:33  <bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/dc1e54206d76...e754c6e33194
 812018-10-20T02:14:34  <bitcoin-git> bitcoin/master a3197c5 Russell Yanofsky: Disable wallet and address book Qt tests on macOS minimal platform...
 822018-10-20T02:14:34  <bitcoin-git> bitcoin/master e754c6e Pieter Wuille: Merge #14011: Disable wallet and address book Qt tests on macOS minimal platform...
 832018-10-20T02:14:35  *** bitcoin-git has left #bitcoin-core-dev
 842018-10-20T02:15:03  <sipa> anything else?
 852018-10-20T02:15:15  *** bitcoin-git has joined #bitcoin-core-dev
 862018-10-20T02:15:15  <bitcoin-git> [bitcoin] sipa closed pull request #14011: Disable wallet and address book Qt tests on macOS minimal platform (master...pr/fuqtmac) https://github.com/bitcoin/bitcoin/pull/14011
 872018-10-20T02:15:15  *** bitcoin-git has left #bitcoin-core-dev
 882018-10-20T02:15:23  *** ghost43 has quit IRC
 892018-10-20T02:15:23  *** ExtraCrispy has quit IRC
 902018-10-20T02:15:23  *** morcos has quit IRC
 912018-10-20T02:15:23  *** sipa has quit IRC
 922018-10-20T02:15:24  *** intcat has quit IRC
 932018-10-20T02:15:40  *** ghost43 has joined #bitcoin-core-dev
 942018-10-20T02:15:43  *** sipa has joined #bitcoin-core-dev
 952018-10-20T02:15:43  *** morcos has joined #bitcoin-core-dev
 962018-10-20T02:15:55  <fanquake> sipa cheers. #14512 is also a trivial merge.
 972018-10-20T02:15:56  <gribble> https://github.com/bitcoin/bitcoin/issues/14512 | docs: Textual improvements in README.md by merland · Pull Request #14512 · bitcoin/bitcoin · GitHub
 982018-10-20T02:20:11  *** intcat has joined #bitcoin-core-dev
 992018-10-20T02:24:09  *** bitcoin-git has joined #bitcoin-core-dev
1002018-10-20T02:24:10  <bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/e754c6e33194...91482e5bf22d
1012018-10-20T02:24:10  <bitcoin-git> bitcoin/master 29ed2d6 Hennadii Stepanov: Improve CAmount tests...
1022018-10-20T02:24:11  <bitcoin-git> bitcoin/master 91482e5 Pieter Wuille: Merge #14460: tests: Improve 'CAmount' tests...
1032018-10-20T02:24:11  *** bitcoin-git has left #bitcoin-core-dev
1042018-10-20T02:25:14  *** bitcoin-git has joined #bitcoin-core-dev
1052018-10-20T02:25:15  <bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/91482e5bf22d...544f3234384b
1062018-10-20T02:25:15  <bitcoin-git> bitcoin/master b6b9915 Martin Erlandsson: Textual improvements
1072018-10-20T02:25:16  <bitcoin-git> bitcoin/master 544f323 Pieter Wuille: Merge #14512: docs: Textual improvements in README.md...
1082018-10-20T02:25:16  *** bitcoin-git has left #bitcoin-core-dev
1092018-10-20T02:33:02  *** rh0nj has quit IRC
1102018-10-20T02:34:08  *** rh0nj has joined #bitcoin-core-dev
1112018-10-20T02:46:29  *** proletesseract has joined #bitcoin-core-dev
1122018-10-20T02:48:49  *** dviola has quit IRC
1132018-10-20T02:56:13  *** justanotheruser has joined #bitcoin-core-dev
1142018-10-20T03:03:04  *** proletesseract has quit IRC
1152018-10-20T03:23:05  *** polydin has joined #bitcoin-core-dev
1162018-10-20T03:26:05  *** polydin has quit IRC
1172018-10-20T03:33:24  *** michaelsdunn1 has quit IRC
1182018-10-20T03:45:44  *** bitconner has quit IRC
1192018-10-20T03:55:33  *** schnerch_ has joined #bitcoin-core-dev
1202018-10-20T03:58:42  *** schnerchi has quit IRC
1212018-10-20T03:59:26  *** copumpkin has joined #bitcoin-core-dev
1222018-10-20T04:28:17  *** luke-jr has quit IRC
1232018-10-20T04:28:30  *** luke-jr has joined #bitcoin-core-dev
1242018-10-20T04:37:12  *** luke-jr has quit IRC
1252018-10-20T04:42:24  *** Tralfaz has quit IRC
1262018-10-20T04:42:28  *** copumpkin has quit IRC
1272018-10-20T04:43:20  *** Tralfaz has joined #bitcoin-core-dev
1282018-10-20T04:43:28  *** copumpkin has joined #bitcoin-core-dev
1292018-10-20T04:44:47  *** luke-jr has joined #bitcoin-core-dev
1302018-10-20T05:11:11  *** bitcoin-git has joined #bitcoin-core-dev
1312018-10-20T05:11:11  <bitcoin-git> [bitcoin] fanquake opened pull request #14526: docs: Document lint tests (master...revive-document-lint-tests) https://github.com/bitcoin/bitcoin/pull/14526
1322018-10-20T05:11:11  *** bitcoin-git has left #bitcoin-core-dev
1332018-10-20T05:11:26  *** bitcoin-git has joined #bitcoin-core-dev
1342018-10-20T05:11:27  <bitcoin-git> [bitcoin] fanquake closed pull request #13708: docs: Document lint tests (master...document-lint-tests) https://github.com/bitcoin/bitcoin/pull/13708
1352018-10-20T05:11:27  *** bitcoin-git has left #bitcoin-core-dev
1362018-10-20T05:24:42  *** bitcoin-git has joined #bitcoin-core-dev
1372018-10-20T05:24:42  <bitcoin-git> [bitcoin] fanquake closed pull request #13542: Adding Docker/docker-compose files (master...docker) https://github.com/bitcoin/bitcoin/pull/13542
1382018-10-20T05:24:42  *** bitcoin-git has left #bitcoin-core-dev
1392018-10-20T05:30:12  *** proletesseract has joined #bitcoin-core-dev
1402018-10-20T05:31:06  *** bitcoin-git has joined #bitcoin-core-dev
1412018-10-20T05:31:06  <bitcoin-git> [bitcoin] fanquake closed pull request #14014: Scripts and tools: fix gitian-build.py --verify option (master...gitian-verify) https://github.com/bitcoin/bitcoin/pull/14014
1422018-10-20T05:31:06  *** bitcoin-git has left #bitcoin-core-dev
1432018-10-20T05:37:38  *** Tralfaz has quit IRC
1442018-10-20T05:37:57  *** Tralfaz has joined #bitcoin-core-dev
1452018-10-20T05:44:10  *** Tralfaz has joined #bitcoin-core-dev
1462018-10-20T05:44:51  *** Squidicuz has quit IRC
1472018-10-20T05:45:12  *** Squidicuz has joined #bitcoin-core-dev
1482018-10-20T05:46:30  *** keymone has quit IRC
1492018-10-20T05:47:13  *** Tralfaz has quit IRC
1502018-10-20T05:47:37  *** Tralfaz has joined #bitcoin-core-dev
1512018-10-20T05:48:23  *** Tralfaz has quit IRC
1522018-10-20T05:49:38  *** keymone has joined #bitcoin-core-dev
1532018-10-20T05:56:54  <wumpus> midnightmagic: well... currently only run OpenBSD in VM; and yes, bitcoin builds fine on OpenBSD, there's special instructions for that, which fanquake updated for 6.4 and I'l like to test out but couldn't yet
1542018-10-20T05:56:58  *** jpe has joined #bitcoin-core-dev
1552018-10-20T05:59:38  *** jpe_ has quit IRC
1562018-10-20T05:59:41  <fanquake> wumpus I've looked at the BSD configure issue a bit more. Looks like a bug fix in 1.16 might have broken something we do, https://github.com/bitcoin/bitcoin/issues/14404#issuecomment-431542526
1572018-10-20T05:59:58  <fanquake> Still need to get an actual fix.
1582018-10-20T06:00:13  *** Tuna_Golem has joined #bitcoin-core-dev
1592018-10-20T06:01:33  <fanquake> *automake 1.16
1602018-10-20T06:01:58  <midnightmagic> nice
1612018-10-20T06:02:28  <midnightmagic> if I could get netbsd into my talos somehow, I'd be testing in there.
1622018-10-20T06:02:33  *** hebasto has joined #bitcoin-core-dev
1632018-10-20T06:03:27  *** Krellan has joined #bitcoin-core-dev
1642018-10-20T06:05:37  <wumpus> fanquake: thanks for investigating; glad you could narrow it down to a specific automake change
1652018-10-20T06:06:31  *** Krellan has joined #bitcoin-core-dev
1662018-10-20T06:07:59  <wumpus> midnightmagic: yeah would love to run *BSD on my SiFive board, but that's a pipe dream for now, even a linux kernel that works is hard to find :)
1672018-10-20T06:08:33  <midnightmagic> you mean the hifive unleashed?
1682018-10-20T06:08:41  <midnightmagic> did you get the expansion board too?
1692018-10-20T06:09:01  <wumpus> it's the first time I bothered with fedora as that seems to be the best supported distro for it
1702018-10-20T06:09:06  <wumpus> yes that one
1712018-10-20T06:09:44  <midnightmagic> i wonder if that page-fault-computing usenix method can be used on the sifive stuff
1722018-10-20T06:09:48  <wumpus> FWIW it runs a bitcoin + lightning node + tor fine :D
1732018-10-20T06:09:55  <midnightmagic> wow, nice.
1742018-10-20T06:11:26  *** luke-jr has quit IRC
1752018-10-20T06:11:51  *** luke-jr has joined #bitcoin-core-dev
1762018-10-20T06:12:00  <wumpus> yes it's kind of cool; I do use nbd for storage, though—no expansion board was too late for that
1772018-10-20T06:12:35  <midnightmagic> still unavailable, too, apparently.
1782018-10-20T06:12:49  <wumpus> I got one of the last unleashed boards, literally a week later the option to order them was closed :)
1792018-10-20T06:12:57  <midnightmagic> This kind of hardware, I don't mind learning new assembly for.
1802018-10-20T06:13:08  <sipa> that's risc-v?
1812018-10-20T06:13:16  <midnightmagic> yeah
1822018-10-20T06:13:20  <sipa> nice
1832018-10-20T06:13:22  <wumpus> midnightmagic: yeah it's not much work either, the instruction set is super straightforward
1842018-10-20T06:14:31  <midnightmagic> runs bitcoin okay? how long did it take to sync with nbd..?
1852018-10-20T06:14:39  <midnightmagic> (Also, why not nfs?)
1862018-10-20T06:14:43  <wumpus> midnightmagic: this lists all of them: https://github.com/michaeljclark/riscv-meta/blob/master/opcode-fullnames
1872018-10-20T06:14:52  <wumpus> (including extensions)
1882018-10-20T06:15:05  <midnightmagic> whoah, short.
1892018-10-20T06:15:46  <midnightmagic> did you ever see the power9 instruction set?
1902018-10-20T06:16:06  *** luke-jr has quit IRC
1912018-10-20T06:16:15  *** luke-jr has joined #bitcoin-core-dev
1922018-10-20T06:16:21  *** bitcoin-git has joined #bitcoin-core-dev
1932018-10-20T06:16:22  <bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/544f3234384b...d433239a8d54
1942018-10-20T06:16:22  <bitcoin-git> bitcoin/master 94e21c1 James O'Beirne: test: forward timeouts properly in send_blocks_and_test
1952018-10-20T06:16:23  <bitcoin-git> bitcoin/master d433239 MarcoFalke: Merge #14456: test: forward timeouts properly in send_blocks_and_test...
1962018-10-20T06:16:23  *** bitcoin-git has left #bitcoin-core-dev
1972018-10-20T06:16:50  <wumpus> the only thing I really had to get used to, is that loading longer immediates is split over multiple instructions instead of in-line (such as auipc and jlr for jump)
1982018-10-20T06:17:04  <wumpus> midnightmagic: I don't think so!
1992018-10-20T06:17:20  <midnightmagic> you have to do weird things like load in addresses in four pieces, carefully shifting them and arranging them..
2002018-10-20T06:17:21  *** bitcoin-git has joined #bitcoin-core-dev
2012018-10-20T06:17:22  <bitcoin-git> [bitcoin] MarcoFalke closed pull request #14456: test: forward timeouts properly in send_blocks_and_test (master...2018-10-test-forward-timeouts) https://github.com/bitcoin/bitcoin/pull/14456
2022018-10-20T06:17:22  *** bitcoin-git has left #bitcoin-core-dev
2032018-10-20T06:18:00  <wumpus> at least I hope risc-v is the last assembly language I ever have to learn and it will take over everything xD
2042018-10-20T06:19:14  <wumpus> yes, bitcoin runs ok, syncing doesn't go super fast but it was a bit faster than most ARM boards from a few years ago I've tried on
2052018-10-20T06:20:21  <midnightmagic> wumpus: it's *huge*. The PowerISA reference manual is 1240 pages.
2062018-10-20T06:20:48  <wumpus> the SD card driver was kind of broken so that's why I'm using nbd, nbd in contrast to nfs is very easy to set up, no need to bother with username mappings, rpc servers, and horrible stuff like that
2072018-10-20T06:22:01  <midnightmagic> "Appendix F. Power ISA Instruction Set Sorted by Mnemonic" => 17 pages is tiny little writing.
2082018-10-20T06:22:15  <wumpus> that's kind of much for a *reduced* instruction set platform
2092018-10-20T06:22:41  <wumpus> though it's still less than Intel's multi-volume bibles
2102018-10-20T06:22:41  <midnightmagic> 56 opcodes per page.
2112018-10-20T06:23:48  <midnightmagic> What the hell does "Transaction Abort Doubleword Conditional Immediate & record" even mean lol and yet, after discovering the weird futureAlienTech IBM put into it. Dude.
2122018-10-20T06:25:57  <wumpus> heh those pages are either scribbled with occult markings or ▓███▒▓█-censored ... nah, sounds like some combinatorial complexity blowup
2132018-10-20T06:26:34  <wumpus> 'let's make 2^n variants of instruction and all document them separately, that's a way to fill a book'
2142018-10-20T06:27:28  *** SopaXorzTaker has joined #bitcoin-core-dev
2152018-10-20T06:27:43  <midnightmagic> If a pcie card tries to twiddle ram without a dma properly setup, the power chips intercept it and helpfully fire off an OS handler do you can decide what to do about it yourself. I guess on normal hardware it just lets it get away with it.
2162018-10-20T06:28:19  <wumpus> anyhow, risc-v is nice and clean, though if it really catches on with many vendors i'd expect ehere might be a cambrian explosion of (even proprietary) extensions at some point though
2172018-10-20T06:28:24  <midnightmagic> I have a supermicro sas card here that tried to munge main memory *on boot* and had to be disabled by the pre-OS checks. wtf.
2182018-10-20T06:28:36  <sipa> wumpus: i just learned from wikipedia that the r in risc does not refer to the size of the instruction set, but to how complex operations per instruction can be (in particular, how many memory accesses)
2192018-10-20T06:28:38  <midnightmagic> well I sure hope so.
2202018-10-20T06:28:52  <wumpus> sipa: oh! TIL too
2212018-10-20T06:29:40  <sipa> and there have been CISC platforms with just 8 instructions
2222018-10-20T06:29:50  <wumpus> midnightmagic: I agree, it'd be a form of open competition which is good
2232018-10-20T06:30:11  <ossifrage> I thought early RISC was along the lines of what can be done in an cycle (with most instructions being register to register)
2242018-10-20T06:31:43  <midnightmagic> still burns me atmel dropped avr for arm. wumpus: well if it does anything interesting with the bitcoin node, I know I at least would love to hear about it.
2252018-10-20T06:33:23  <ossifrage> I always found it cleaner to decouple memory to register instructions from register to alu to register instructions
2262018-10-20T06:33:30  <wumpus> ossifrage: apparently yes that's what it means
2272018-10-20T06:35:37  <wumpus> definitely, having, all the addressing modes for every instruction is a lot of complexity, and then you end up with things like 'MOV is turing-complete'
2282018-10-20T06:36:03  <ossifrage> If you don't have a pipeline writing a RISC CPU in verilog is surprisingly simple.
2292018-10-20T06:36:36  <wumpus> yes handling timing and pipelines is where it gets complex
2302018-10-20T06:37:13  <fanquake> MarcoFalke If you want to merge #14497 I think that's ready to go.
2312018-10-20T06:37:15  <gribble> https://github.com/bitcoin/bitcoin/issues/14497 | docs: Add doc/bitcoin-conf.md by hebasto · Pull Request #14497 · bitcoin/bitcoin · GitHub
2322018-10-20T06:37:28  <wumpus> e.g. this is a whole rv32 CPU https://github.com/cliffordwolf/picorv32/blob/master/picorv32.v
2332018-10-20T06:37:44  <ossifrage> wumpus, the bugs in the chip I worked on where in the hazard logic and the pipeline bypass. Stuff that managed to get past verification
2342018-10-20T06:38:12  <ossifrage> I can't imagine how painful it is to verify a CISC processor
2352018-10-20T06:38:39  <wumpus> it can fit in the small FPGAs supported by the open source yosys toolchain
2362018-10-20T06:40:34  <wumpus> fanquake: will merge
2372018-10-20T06:42:04  <wumpus> but yes it's kind of sad i missed the expansion board, would have been neat to connect a GPU and keyboard and have an actual RISC-V PC
2382018-10-20T06:44:22  *** bitcoin-git has joined #bitcoin-core-dev
2392018-10-20T06:44:23  <bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/d433239a8d54...6746a8951923
2402018-10-20T06:44:23  <bitcoin-git> bitcoin/master 1fb3c16 Hennadii Stepanov: Add `doc/bitcoin-conf.md`
2412018-10-20T06:44:23  <bitcoin-git> bitcoin/master 6746a89 Wladimir J. van der Laan: Merge #14497: docs: Add doc/bitcoin-conf.md...
2422018-10-20T06:44:24  *** bitcoin-git has left #bitcoin-core-dev
2432018-10-20T06:45:22  *** bitcoin-git has joined #bitcoin-core-dev
2442018-10-20T06:45:22  <bitcoin-git> [bitcoin] laanwj closed pull request #14497: docs: Add doc/bitcoin-conf.md (master...20181016-bitcoin-conf-md) https://github.com/bitcoin/bitcoin/pull/14497
2452018-10-20T06:45:22  *** bitcoin-git has left #bitcoin-core-dev
2462018-10-20T06:47:25  <wumpus> maybe they'll re-open production at some point! at least the unleashed board, despite the outrageous price, were popular enough for a new batch
2472018-10-20T06:53:32  <wumpus> midnightmagic: it's reassuring that the IOMMU on that board is consistent and doesn't make an exception for boot-time memory twiddling
2482018-10-20T06:55:04  <wumpus> midnightmagic: btw re: POWER cna you weigh in on #14066 please
2492018-10-20T06:55:06  <gribble> https://github.com/bitcoin/bitcoin/issues/14066 | gitian-linux: Build binaries for 64-bit POWER by luke-jr · Pull Request #14066 · bitcoin/bitcoin · GitHub
2502018-10-20T06:55:06  *** ken2812221 has joined #bitcoin-core-dev
2512018-10-20T06:55:33  <fanquake> wumpus Would we be able to fork emil-e/rapidcheck to github.com/bitcoin-core/?
2522018-10-20T06:55:34  <midnightmagic> sure
2532018-10-20T06:55:37  <fanquake> Marco has a fork that we currently use for depends, however it'd be good to move to a newer version for some bug fixes and a potentially simpler build.
2542018-10-20T06:55:43  <fanquake> We need the fork so we can create our own archive, because emil-e/rapidcheck isn’t tagging releases yet.
2552018-10-20T06:56:06  <wumpus> fanquake: yep; do we need our own fork of that?
2562018-10-20T06:56:12  <wumpus> yes, sure
2572018-10-20T07:01:05  <fanquake> wumpus thanks. More context in my comment here: https://github.com/bitcoin/bitcoin/pull/14430#issuecomment-431555605
2582018-10-20T07:05:22  <wumpus> repo created, see https://github.com/bitcoin-core/rapidcheck, who needs commit access?
2592018-10-20T07:05:28  <wumpus> besides MarcoFalke
2602018-10-20T07:08:01  <fanquake> maybe Christewart, he seems to be spearheading most of this work
2612018-10-20T07:21:23  <wumpus> ok!
2622018-10-20T07:22:23  <wumpus> any idea if they're on IRC?
2632018-10-20T07:25:08  *** ExtraCrispy has joined #bitcoin-core-dev
2642018-10-20T07:25:53  <wumpus> 09:25 -!- Christewart There was no such nickname at least not under that name
2652018-10-20T07:26:31  <gwillen> are you looking for Chris_Stewart_5 ? or someone else?
2662018-10-20T07:26:48  <fanquake> We spoke on IRC in tokyo
2672018-10-20T07:26:54  <fanquake> Yea I think that's it ^
2682018-10-20T07:27:11  <gwillen> he doesn't have a bouncer I guess, he seems to be on and off
2692018-10-20T07:27:11  <wumpus> of course!
2702018-10-20T07:27:25  <wumpus> (I vaguely remembered that but wasn't sure)
2712018-10-20T07:33:16  <wumpus> invited to bitcoin and bitcoin-core at least
2722018-10-20T07:35:07  <fanquake> wumpus cheers
2732018-10-20T07:36:35  *** proletesseract has quit IRC
2742018-10-20T07:37:08  *** proletesseract has joined #bitcoin-core-dev
2752018-10-20T07:41:25  *** proletesseract has quit IRC
2762018-10-20T07:53:43  *** luke-jr has quit IRC
2772018-10-20T07:53:56  *** luke-jr has joined #bitcoin-core-dev
2782018-10-20T07:57:09  *** luke-jr has quit IRC
2792018-10-20T07:57:23  *** luke-jr has joined #bitcoin-core-dev
2802018-10-20T07:59:30  *** luke-jr has quit IRC
2812018-10-20T07:59:40  *** luke-jr has joined #bitcoin-core-dev
2822018-10-20T08:04:42  *** luke-jr has quit IRC
2832018-10-20T08:04:51  *** luke-jr has joined #bitcoin-core-dev
2842018-10-20T08:05:52  *** luke-jr has quit IRC
2852018-10-20T08:06:07  *** luke-jr has joined #bitcoin-core-dev
2862018-10-20T08:09:14  *** luke-jr has quit IRC
2872018-10-20T08:09:22  *** luke-jr has joined #bitcoin-core-dev
2882018-10-20T08:13:40  *** Krellan has quit IRC
2892018-10-20T08:14:00  *** bitconner has joined #bitcoin-core-dev
2902018-10-20T08:15:24  *** Krellan has joined #bitcoin-core-dev
2912018-10-20T08:18:00  *** bitconner has quit IRC
2922018-10-20T08:22:06  *** marcoagner has quit IRC
2932018-10-20T08:24:44  *** luke-jr has quit IRC
2942018-10-20T08:25:25  *** luke-jr has joined #bitcoin-core-dev
2952018-10-20T08:28:06  *** ghost43 has quit IRC
2962018-10-20T08:28:20  *** ghost43 has joined #bitcoin-core-dev
2972018-10-20T08:29:02  *** Guyver2 has joined #bitcoin-core-dev
2982018-10-20T08:29:07  *** murrayn[m] has joined #bitcoin-core-dev
2992018-10-20T08:31:12  *** Victorsueca has quit IRC
3002018-10-20T08:32:29  *** Victorsueca has joined #bitcoin-core-dev
3012018-10-20T08:33:05  *** luke-jr has quit IRC
3022018-10-20T08:33:41  *** bitcoin-git has joined #bitcoin-core-dev
3032018-10-20T08:33:42  <bitcoin-git> [bitcoin] MarcoFalke opened pull request #14527: Revert "Make qt wallet test compatible with qt4" (master...Mf1810-qtRemoveQT4file) https://github.com/bitcoin/bitcoin/pull/14527
3042018-10-20T08:33:42  *** bitcoin-git has left #bitcoin-core-dev
3052018-10-20T08:37:16  *** luke-jr has joined #bitcoin-core-dev
3062018-10-20T08:40:22  *** Emcy has quit IRC
3072018-10-20T08:42:10  *** luke-jr has quit IRC
3082018-10-20T08:42:43  *** luke-jr has joined #bitcoin-core-dev
3092018-10-20T09:44:32  *** AaronvanW has joined #bitcoin-core-dev
3102018-10-20T09:57:52  <MarcoFalke> Sorry for the downtime of DrahtBot recently. It should be catching up in the next hour or so.
3112018-10-20T09:58:33  <fanquake> The flood of emails has begun :p
3122018-10-20T10:02:31  *** kexkey has quit IRC
3132018-10-20T10:02:45  *** kexkey has joined #bitcoin-core-dev
3142018-10-20T10:05:12  <promag> +1
3152018-10-20T10:11:29  *** spinza has quit IRC
3162018-10-20T10:16:04  *** spinza has joined #bitcoin-core-dev
3172018-10-20T10:22:51  *** bitcoin-git has joined #bitcoin-core-dev
3182018-10-20T10:22:51  <bitcoin-git> [bitcoin] MarcoFalke opened pull request #14528: travis: Compile once on xenial (master...Mf1810-travisXenial) https://github.com/bitcoin/bitcoin/pull/14528
3192018-10-20T10:22:51  *** bitcoin-git has left #bitcoin-core-dev
3202018-10-20T10:39:52  *** AaronvanW has quit IRC
3212018-10-20T10:40:30  *** AaronvanW has joined #bitcoin-core-dev
3222018-10-20T10:46:59  * luke-jr wonders if we really need to ban regexs due to reviewer regex-illiteracy :x
3232018-10-20T10:59:05  *** cdecker has quit IRC
3242018-10-20T11:07:33  <wumpus> banning regexps above a certain complexity would make sense, I think, also because it's usually symptomic of a deeper issue
3252018-10-20T11:08:48  <wumpus> e.g. *why* is the underlying data so complex, does it need a real lexer/parser (because it's a programming language or markup language), or is the data mangled in an earlier step (which might be avoided in the first place)
3262018-10-20T11:09:58  <wumpus> and yes long regexps are extremely hard to read, though most regexp engines provide means of splitting them into lines and commenting them, but still
3272018-10-20T11:10:01  <promag> MarcoFalke: should provide fs::relative implementation to our source? :/
3282018-10-20T11:20:22  <luke-jr> wumpus: I'm not seeing any non-forking-the-code way to use readelf as a library; the regex is fairly straightforward
3292018-10-20T11:29:00  *** newbie-- has joined #bitcoin-core-dev
3302018-10-20T11:31:54  *** windsok has quit IRC
3312018-10-20T11:34:12  *** windsok has joined #bitcoin-core-dev
3322018-10-20T11:34:12  *** windsok has joined #bitcoin-core-dev
3332018-10-20T11:37:26  *** Chris_Stewart_5 has joined #bitcoin-core-dev
3342018-10-20T11:45:45  <fanquake> Chris_Stewart_5 See the channel history, we've forked rapidcheck to bitcoin-core, could be a chance to use the new make install method etc. You should have gotten invite to /bitcoin and /bitcoin-core as well.
3352018-10-20T11:50:06  <luke-jr> [06:55:43] <fanquake> We need the fork so we can create our own archive, because emil-e/rapidcheck isn’t tagging releases yet. <-- uh, that's no reason for a fork - am I missing something?
3362018-10-20T11:51:00  *** Chris_Stewart_5 has quit IRC
3372018-10-20T11:51:04  <luke-jr> if you want an archive, just use the ones github generates..
3382018-10-20T11:51:21  <wumpus> luke-jr: there are some python modules that can do elf parsing, but agree it's suboptimal to introduce such a dependency; nm might be a good suggestion though depending
3392018-10-20T11:51:50  <wumpus> ... on whether its output is better
3402018-10-20T11:51:57  <luke-jr> its output seems far worse
3412018-10-20T11:52:35  <luke-jr> same column-based output, but with single letter stuff you need the manpage to understand
3422018-10-20T11:59:56  *** copumpkin has quit IRC
3432018-10-20T12:00:13  *** Zenton has quit IRC
3442018-10-20T12:00:48  <wumpus> what information do you need?
3452018-10-20T12:00:49  *** romanz_ has quit IRC
3462018-10-20T12:01:19  *** Victorsueca has quit IRC
3472018-10-20T12:01:22  *** Zenton has joined #bitcoin-core-dev
3482018-10-20T12:02:29  *** Victorsueca has joined #bitcoin-core-dev
3492018-10-20T12:02:51  *** bitcoin-git has joined #bitcoin-core-dev
3502018-10-20T12:02:52  <bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/6746a8951923...96c409c919b9
3512018-10-20T12:02:52  <bitcoin-git> bitcoin/master fadffae MarcoFalke: Revert "Make qt wallet test compatible with qt4"...
3522018-10-20T12:02:53  <bitcoin-git> bitcoin/master 96c409c Wladimir J. van der Laan: Merge #14527: qa: Revert "Make qt wallet test compatible with qt4"...
3532018-10-20T12:02:53  *** bitcoin-git has left #bitcoin-core-dev
3542018-10-20T12:03:18  <luke-jr> wumpus: target architecture, imported symbol versions, and exported symbols
3552018-10-20T12:04:01  *** bitcoin-git has joined #bitcoin-core-dev
3562018-10-20T12:04:02  <bitcoin-git> [bitcoin] laanwj closed pull request #14527: qa: Revert "Make qt wallet test compatible with qt4" (master...Mf1810-qtRemoveQT4file) https://github.com/bitcoin/bitcoin/pull/14527
3572018-10-20T12:04:02  *** bitcoin-git has left #bitcoin-core-dev
3582018-10-20T12:04:09  <wumpus> luke-jr: but as far as I'm aware we were already looking for those; how did this change for POWER, that necessitated the large regexp
3592018-10-20T12:04:58  *** romanz has joined #bitcoin-core-dev
3602018-10-20T12:05:00  <luke-jr> wumpus: the parser does not correctly handle readelf output
3612018-10-20T12:05:19  <luke-jr> simply splitting on spaces fails when the data contains spaces IIRC
3622018-10-20T12:05:30  <luke-jr> eg     3: 0000000000000000     0 FUNC    GLOBAL DEFAULT [<localentry>: 8]   UND mbrtowc@GLIBC_2.17 (2)
3632018-10-20T12:05:39  <luke-jr> this [<localentry>: 8] stuff
3642018-10-20T12:05:40  *** Tuna_Golem has quit IRC
3652018-10-20T12:06:00  <wumpus> oh some new information in columns was introduced, interesting
3662018-10-20T12:07:40  <wumpus> it is apparently a powerPC ABI specific thing where a function can have two entry points
3672018-10-20T12:08:06  <luke-jr> hmm
3682018-10-20T12:08:15  <wumpus> we don't *need* that information but indeed it messes with the current parsing
3692018-10-20T12:09:21  <luke-jr> sorry, guess the motivation behind the change was missing >_<
3702018-10-20T12:13:14  <wumpus> indeed; so it seems fair enough
3712018-10-20T12:13:34  <wumpus> I still wonder if it could be done with a simpler regexp, but yes, documenting why you do something is important
3722018-10-20T12:14:20  *** Tuna_Golem has joined #bitcoin-core-dev
3732018-10-20T12:14:22  <luke-jr> I don't really see how it can get any simpler :x
3742018-10-20T12:14:23  *** bitconner has joined #bitcoin-core-dev
3752018-10-20T12:14:34  <luke-jr> wumpus: do you have a link for the PPC-specific details?
3762018-10-20T12:16:36  <wumpus> luke-jr: http://openpowerfoundation.org/wp-content/uploads/resources/leabi/content/dbdoclet.50655241_95185.html
3772018-10-20T12:16:44  <luke-jr> ty
3782018-10-20T12:18:25  <wumpus> luke-jr: ok, I think it's not *that* complex understanding what it does
3792018-10-20T12:19:22  *** bitconner has quit IRC
3802018-10-20T12:24:37  *** laurentmt has joined #bitcoin-core-dev
3812018-10-20T12:25:27  *** laurentmt has quit IRC
3822018-10-20T12:25:34  *** josephnicholas has joined #bitcoin-core-dev
3832018-10-20T12:26:52  *** Emcy has joined #bitcoin-core-dev
3842018-10-20T12:28:41  *** ott0disk has joined #bitcoin-core-dev
3852018-10-20T12:30:03  *** Krellan has quit IRC
3862018-10-20T12:31:12  *** Krellan has joined #bitcoin-core-dev
3872018-10-20T12:33:53  *** mistergold has joined #bitcoin-core-dev
3882018-10-20T12:34:48  <wumpus> regarding fs::relative, is this a new API that did get introduced in a new c++ standard? does anyone know why doesn't it exist on xenial?
3892018-10-20T12:35:29  <wumpus> oh wait—it's a *boost* thing, fs is boost::fs
3902018-10-20T12:36:46  <promag> right, 1.58 doesn't have it
3912018-10-20T12:37:53  <wumpus> we need to support 1.47 according to dependencies.md
3922018-10-20T12:37:59  <wumpus> so using that is really out of the question
3932018-10-20T12:38:05  <wumpus> at least without fallback
3942018-10-20T12:38:43  *** AaronvanW has quit IRC
3952018-10-20T12:39:10  <luke-jr> I thought we were trying to avoid boost anyway?
3962018-10-20T12:39:14  *** AaronvanW has joined #bitcoin-core-dev
3972018-10-20T12:40:48  <promag> 1.47?
3982018-10-20T12:41:12  <promag> ah right, min version
3992018-10-20T12:41:24  <promag> err,
4002018-10-20T12:41:52  <promag> I'll submit a fix
4012018-10-20T12:44:23  <wumpus> luke-jr: really can't in this case until there's a replacement for boost::filesystem
4022018-10-20T12:44:36  <wumpus> (which would require a newer c++ than 11)
4032018-10-20T12:45:17  <wumpus> it's fairly straightforward to redefine the fs namespace then
4042018-10-20T12:47:44  <luke-jr> oh well
4052018-10-20T12:52:41  *** AaronvanW has quit IRC
4062018-10-20T12:54:34  *** AaronvanW has joined #bitcoin-core-dev
4072018-10-20T12:55:43  <wumpus> promag: thanks!
4082018-10-20T12:59:21  *** AaronvanW has quit IRC
4092018-10-20T13:00:29  <wumpus> luke-jr: I know it gets tiring but moving away from boost is really a long-term project
4102018-10-20T13:01:45  <wumpus> it needs to be done without interfering with anything else and there's so many different concernt to juggle
4112018-10-20T13:02:47  *** cdecker has joined #bitcoin-core-dev
4122018-10-20T13:03:53  <luke-jr> wumpus: well, if it serves a purpose, I'm fine with it :p
4132018-10-20T13:03:57  <luke-jr> better than re-inventing
4142018-10-20T13:04:49  *** AaronvanW has joined #bitcoin-core-dev
4152018-10-20T13:05:10  <wumpus> yes, exactly,
4162018-10-20T13:09:27  *** AaronvanW has quit IRC
4172018-10-20T13:09:45  *** josephni_ has joined #bitcoin-core-dev
4182018-10-20T13:10:00  *** josephnicholas has quit IRC
4192018-10-20T13:14:48  *** bitcoin-git has joined #bitcoin-core-dev
4202018-10-20T13:14:48  <bitcoin-git> [bitcoin] MarcoFalke opened pull request #14530: scripted-diff: Use RPCHelpMan on RPCs with no args (master...Mf1810-rpcHelpMan) https://github.com/bitcoin/bitcoin/pull/14530
4212018-10-20T13:14:48  *** bitcoin-git has left #bitcoin-core-dev
4222018-10-20T13:16:34  <Sentineo> hm does anybody else have a different behavior with multiwallets in testnet vs mainnet?
4232018-10-20T13:18:26  <Sentineo> the testnet version ignores multiple wallets, even though they are specified with testnet.wallet directive. Loading a wallet from rpc works, then wallet.dat is marked as [default]. When I try to make "loadwalelt wallet.dat" again it crashes - core dumped. On mainnet bitcoind correctly tels me the wallet is already loaded + does not ignore the wallet directives in the config file.
4242018-10-20T13:19:25  <Sentineo> v0.17.0 btw
4252018-10-20T13:19:50  <promag> wumpus: MarcoFalke: #12842
4262018-10-20T13:19:52  <gribble> https://github.com/bitcoin/bitcoin/issues/12842 | Prevent concurrent savemempool by promag · Pull Request #12842 · bitcoin/bitcoin · GitHub
4272018-10-20T13:22:13  *** promag_ has joined #bitcoin-core-dev
4282018-10-20T13:22:13  *** promag has quit IRC
4292018-10-20T13:25:10  *** promag_ is now known as promag
4302018-10-20T13:26:05  *** AaronvanW has joined #bitcoin-core-dev
4312018-10-20T13:29:56  *** Aaronvan_ has joined #bitcoin-core-dev
4322018-10-20T13:30:41  *** AaronvanW has quit IRC
4332018-10-20T13:33:46  *** bitcoin-git has joined #bitcoin-core-dev
4342018-10-20T13:33:47  <bitcoin-git> [bitcoin] promag opened pull request #14531: Replace fs::relative call with custom GetRelativePath (master...2018-10-getrelativepath) https://github.com/bitcoin/bitcoin/pull/14531
4352018-10-20T13:33:47  *** bitcoin-git has left #bitcoin-core-dev
4362018-10-20T13:34:06  <promag> wumpus: MarcoFalke ^
4372018-10-20T13:34:12  *** Aaronvan_ has quit IRC
4382018-10-20T13:35:30  *** AaronvanW has joined #bitcoin-core-dev
4392018-10-20T13:36:11  *** josephni_ has quit IRC
4402018-10-20T13:36:46  *** josephnicholas has joined #bitcoin-core-dev
4412018-10-20T13:40:04  *** Chris_Stewart_5 has joined #bitcoin-core-dev
4422018-10-20T13:53:33  *** josephni_ has joined #bitcoin-core-dev
4432018-10-20T13:53:34  *** josephnicholas has quit IRC
4442018-10-20T13:55:12  *** josephni_ has quit IRC
4452018-10-20T13:59:21  *** AaronvanW has quit IRC
4462018-10-20T13:59:59  *** AaronvanW has joined #bitcoin-core-dev
4472018-10-20T14:01:25  *** josephnicholas has joined #bitcoin-core-dev
4482018-10-20T14:02:52  *** iphelix has joined #bitcoin-core-dev
4492018-10-20T14:03:32  *** iphelix has left #bitcoin-core-dev
4502018-10-20T14:11:34  *** fanquake has quit IRC
4512018-10-20T14:20:12  *** waxwing has quit IRC
4522018-10-20T14:26:56  *** ott0disk has quit IRC
4532018-10-20T15:01:16  *** bitcoin-git has joined #bitcoin-core-dev
4542018-10-20T15:01:17  <bitcoin-git> [bitcoin] luke-jr opened pull request #14532: Never bind INADDR_ANY by default, and warn when doing so explicitly (master...rpcbind_explicit) https://github.com/bitcoin/bitcoin/pull/14532
4552018-10-20T15:01:17  *** bitcoin-git has left #bitcoin-core-dev
4562018-10-20T15:05:08  *** harrymm has quit IRC
4572018-10-20T15:05:09  *** laurentmt has joined #bitcoin-core-dev
4582018-10-20T15:06:13  *** harrymm has joined #bitcoin-core-dev
4592018-10-20T15:13:18  *** waxwing has joined #bitcoin-core-dev
4602018-10-20T15:13:52  *** ula has joined #bitcoin-core-dev
4612018-10-20T15:23:39  *** ula has quit IRC
4622018-10-20T15:26:03  *** ula has joined #bitcoin-core-dev
4632018-10-20T15:39:06  *** laurentmt has quit IRC
4642018-10-20T15:42:15  *** bitconner has joined #bitcoin-core-dev
4652018-10-20T15:46:28  *** bitconner has quit IRC
4662018-10-20T15:48:32  *** mistergold has quit IRC
4672018-10-20T15:52:01  *** rh0nj has quit IRC
4682018-10-20T15:53:07  *** rh0nj has joined #bitcoin-core-dev
4692018-10-20T16:20:01  *** irc_viewer_test has joined #bitcoin-core-dev
4702018-10-20T16:21:36  *** Chris_Stewart_5 has quit IRC
4712018-10-20T16:23:29  *** AaronvanW has quit IRC
4722018-10-20T16:25:39  *** lnostdal has quit IRC
4732018-10-20T16:25:53  *** lnostdal has joined #bitcoin-core-dev
4742018-10-20T16:37:12  <kakobrekla> 
4752018-10-20T16:41:40  *** grubles_ has joined #bitcoin-core-dev
4762018-10-20T16:41:43  *** laurentmt has joined #bitcoin-core-dev
4772018-10-20T16:42:39  *** AaronvanW has joined #bitcoin-core-dev
4782018-10-20T16:42:58  *** grubles has quit IRC
4792018-10-20T16:47:15  *** AaronvanW has quit IRC
4802018-10-20T16:52:00  *** Krellan has quit IRC
4812018-10-20T16:53:36  *** Krellan has joined #bitcoin-core-dev
4822018-10-20T16:56:18  *** laurentmt has quit IRC
4832018-10-20T16:59:18  <karelb> MarcoFalke thx for feedback on the RPC stuff, will reply tomorrow after I sleep on it
4842018-10-20T17:35:55  *** Krellan has quit IRC
4852018-10-20T17:36:49  *** AaronvanW has joined #bitcoin-core-dev
4862018-10-20T17:41:09  *** AaronvanW has quit IRC
4872018-10-20T17:53:45  *** spinza has quit IRC
4882018-10-20T18:00:54  *** booyah has quit IRC
4892018-10-20T18:02:12  *** booyah has joined #bitcoin-core-dev
4902018-10-20T18:03:24  *** spinza has joined #bitcoin-core-dev
4912018-10-20T18:05:58  *** owowo has quit IRC
4922018-10-20T18:06:58  *** owowo has joined #bitcoin-core-dev
4932018-10-20T18:16:18  *** irc_viewer_test has quit IRC
4942018-10-20T18:19:06  *** esotericnonsense has quit IRC
4952018-10-20T18:20:06  *** sipa has quit IRC
4962018-10-20T18:20:33  *** esotericnonsense has joined #bitcoin-core-dev
4972018-10-20T18:20:44  *** sipa has joined #bitcoin-core-dev
4982018-10-20T18:29:05  *** mistergold has joined #bitcoin-core-dev
4992018-10-20T18:50:25  *** SopaXorzTaker has quit IRC
5002018-10-20T18:52:56  *** Chris_Stewart_5 has joined #bitcoin-core-dev
5012018-10-20T18:56:39  *** gribble has quit IRC
5022018-10-20T18:58:38  *** Lexman has joined #bitcoin-core-dev
5032018-10-20T19:00:20  *** irc_viewer_test has joined #bitcoin-core-dev
5042018-10-20T19:08:26  *** bitcoin-git has joined #bitcoin-core-dev
5052018-10-20T19:08:27  <bitcoin-git> [bitcoin] practicalswift reopened pull request #13909: validation: Pass chainparams in AcceptToMemoryPoolWorker(...) (master...remove-chainparams-argument-to-AcceptToMemoryPoolWorker) https://github.com/bitcoin/bitcoin/pull/13909
5062018-10-20T19:08:27  *** bitcoin-git has left #bitcoin-core-dev
5072018-10-20T19:11:08  *** gribble has joined #bitcoin-core-dev
5082018-10-20T19:11:43  *** shesek has quit IRC
5092018-10-20T19:14:57  *** belcher_ has joined #bitcoin-core-dev
5102018-10-20T19:16:55  *** irc_viewer_test has quit IRC
5112018-10-20T19:17:41  *** hebasto has quit IRC
5122018-10-20T19:18:30  *** reallll has quit IRC
5132018-10-20T19:25:04  *** AaronvanW has joined #bitcoin-core-dev
5142018-10-20T19:29:30  *** AaronvanW has quit IRC
5152018-10-20T19:32:02  *** meshcollider_ has joined #bitcoin-core-dev
5162018-10-20T19:45:17  <phantomcircuit> wumpus, they're unlikely to do another run of the same chips for sifive, im guessing that was recouping the cost of the smallest run they could get
5172018-10-20T19:45:36  <phantomcircuit> wumpus, but that also means the next one might be a more fully baked thing
5182018-10-20T19:47:04  *** mistergold has quit IRC
5192018-10-20T19:54:17  *** mistergold has joined #bitcoin-core-dev
5202018-10-20T19:54:26  *** bitcoin-git has joined #bitcoin-core-dev
5212018-10-20T19:54:26  <bitcoin-git> [bitcoin] mrwhythat opened pull request #14533: wallet: ensure wallet files are not reused across chains (master...wallet-file-reuse-prevention) https://github.com/bitcoin/bitcoin/pull/14533
5222018-10-20T19:54:26  *** bitcoin-git has left #bitcoin-core-dev
5232018-10-20T20:11:42  *** waxwing_ has joined #bitcoin-core-dev
5242018-10-20T20:11:52  *** waxwing has quit IRC
5252018-10-20T20:12:30  *** dstadulis has joined #bitcoin-core-dev
5262018-10-20T20:14:15  <wumpus> phantomcircuit: on one hand I hope they do improve it, on the other, I hope it doesn't mean my board is no longer supported
5272018-10-20T20:14:57  *** waxwing_ has quit IRC
5282018-10-20T20:16:44  *** waxwing has joined #bitcoin-core-dev
5292018-10-20T20:17:13  *** waxwing has quit IRC
5302018-10-20T20:17:13  *** waxwing has joined #bitcoin-core-dev
5312018-10-20T20:18:46  <phantomcircuit> wumpus, i mean what does supported even mean for a dev board like that
5322018-10-20T20:19:00  <phantomcircuit> usually you're lucky to get insanely hacked together kernel source
5332018-10-20T20:19:04  *** AaronvanW has joined #bitcoin-core-dev
5342018-10-20T20:19:49  <wumpus> welll fedora and debian distributions target it now because it's the only RISC-V board that supports Linux
5352018-10-20T20:19:51  <phantomcircuit> how about a 2.6.x branch with random bits from 4.x in it?
5362018-10-20T20:20:08  <phantomcircuit> which is no joke the norm for embeded type dev board things
5372018-10-20T20:20:14  <wumpus> kernel is just a matter of time, they're working on getting everything upstreamed
5382018-10-20T20:20:37  <phantomcircuit> yeah that's my point
5392018-10-20T20:20:43  <phantomcircuit> it seems documented enough to actually do that
5402018-10-20T20:20:46  <wumpus> they more or less did, got it to boot but had some problem with the network interface
5412018-10-20T20:20:56  <wumpus> (with mainline kernel I mean)
5422018-10-20T20:21:09  <wumpus> it's *certainly* not as bad as some ARM boards, as you say with ancient kernels
5432018-10-20T20:23:45  *** AaronvanW has quit IRC
5442018-10-20T20:24:16  <phantomcircuit> wumpus, also it seems most of the poll() issues have to do with unconnected sockets and using it as a sleep
5452018-10-20T20:24:17  <wumpus> but you're right, it's not that big a concern in this case, I'm just too use to ARM SoCs where the next thing means the previous one is dropped
5462018-10-20T20:24:26  <phantomcircuit> which im avoiding completely now
5472018-10-20T20:24:41  <phantomcircuit> so im thinking we can use poll() on windows as long as it's not used on unconnected sockets
5482018-10-20T20:24:45  <wumpus> using unconnected sockets as a sleep wut?!?
5492018-10-20T20:26:09  <wumpus> well unconnected sockets shouldn't be emitting any events I guess
5502018-10-20T20:26:50  <wumpus> but it should be unnecessary to pass them to poll/select, you can use either function as a sleep by not passing anything...
5512018-10-20T20:27:00  *** grubles_ is now known as grubles
5522018-10-20T20:28:58  *** shesek has joined #bitcoin-core-dev
5532018-10-20T20:30:13  <wumpus> as for using poll() on windows, be careful, I don't think that's very common, either because it's known as flaky or because it's only been introduced in more recent versions
5542018-10-20T20:32:34  <gmaxwell> It's also not terribly important on widows. Select uses a linked list there and doesn't have an FD range limit.
5552018-10-20T20:33:02  <wumpus> exactly...
5562018-10-20T20:33:55  *** Lexman has quit IRC
5572018-10-20T20:34:34  <phantomcircuit> wumpus, no im saying the main select/poll loop doesn't have any sockets which haven't connected
5582018-10-20T20:34:39  <phantomcircuit> we have another thing that does that
5592018-10-20T20:34:47  <phantomcircuit> or maybe im misunderstanding the way this is all setup
5602018-10-20T20:35:09  <phantomcircuit> the windows bug is about sockets where you call connect non-blocking and then select it
5612018-10-20T20:35:21  *** laurentmt has joined #bitcoin-core-dev
5622018-10-20T20:35:56  <wumpus> but things like this is why I preferred using libevent, it chooses the best polling abstraction for the OS, and is pretty well tested because it's used by Tor
5632018-10-20T20:36:40  <wumpus> introducing this into our own code means having to handle all who-knows-what OS dependent edge cases ourselves
5642018-10-20T20:38:06  <phantomcircuit> wumpus, the first step to using libevent is to move to a callback style
5652018-10-20T20:38:13  <phantomcircuit> which we cna do while still using the loop
5662018-10-20T20:38:19  <phantomcircuit> (which is what libevent is doing internally)
5672018-10-20T20:39:13  <wumpus> yes, cfields was working on this at some point
5682018-10-20T20:39:30  <wumpus> to be clear I think it's fine to move to poll but I *expected* it to be much less of a hassle
5692018-10-20T20:39:33  *** laurentmt has quit IRC
5702018-10-20T20:39:43  <wumpus> I think it makes sense to focus on UNIXish OS and forget windows just leave that at select
5712018-10-20T20:40:24  <wumpus> it seems like you're trying to extend the scope of this too much
5722018-10-20T20:41:37  <phantomcircuit> wumpus, im just thinking about what to do next, need to cleanup the current pr a bunch before it's ready
5732018-10-20T20:41:49  <phantomcircuit> so i was thinking if i was going to do anything that would make that harder on accident
5742018-10-20T20:47:10  *** Chris_Stewart_5 has quit IRC
5752018-10-20T21:03:21  *** lnostdal has quit IRC
5762018-10-20T21:13:09  *** AaronvanW has joined #bitcoin-core-dev
5772018-10-20T21:13:46  *** dstadulis has quit IRC
5782018-10-20T21:15:41  *** lnostdal has joined #bitcoin-core-dev
5792018-10-20T21:17:39  *** AaronvanW has quit IRC
5802018-10-20T21:20:15  *** bralyclow has joined #bitcoin-core-dev
5812018-10-20T21:22:49  *** bralycl__ has quit IRC
5822018-10-20T21:26:35  *** AaronvanW has joined #bitcoin-core-dev
5832018-10-20T21:31:03  *** murrayn[m] has quit IRC
5842018-10-20T21:32:39  *** Chris_Stewart_5 has joined #bitcoin-core-dev
5852018-10-20T21:33:15  *** lnostdal has quit IRC
5862018-10-20T21:33:39  *** murrayn[m] has joined #bitcoin-core-dev
5872018-10-20T21:48:29  *** lnostdal has joined #bitcoin-core-dev
5882018-10-20T21:57:00  *** bralyclo_ has joined #bitcoin-core-dev
5892018-10-20T21:59:25  *** dviola has joined #bitcoin-core-dev
5902018-10-20T21:59:27  *** bralyclow has quit IRC
5912018-10-20T22:05:09  *** murrayn[m] has quit IRC
5922018-10-20T22:06:24  *** robo7 has joined #bitcoin-core-dev
5932018-10-20T22:14:22  *** murrayn[m] has joined #bitcoin-core-dev
5942018-10-20T22:17:07  *** bitcoin-git has joined #bitcoin-core-dev
5952018-10-20T22:17:07  <bitcoin-git> [bitcoin] jbampton opened pull request #14534: Enable flake8 rule E225 which checks for missing whitespace around op… (master...flake8-fix-E225) https://github.com/bitcoin/bitcoin/pull/14534
5962018-10-20T22:17:07  *** bitcoin-git has left #bitcoin-core-dev
5972018-10-20T22:17:46  *** bralyclo_ has quit IRC
5982018-10-20T22:21:37  *** bralyclow has joined #bitcoin-core-dev
5992018-10-20T22:24:24  *** Zenton has quit IRC
6002018-10-20T22:28:41  *** spinza has quit IRC
6012018-10-20T22:33:25  *** spinza has joined #bitcoin-core-dev
6022018-10-20T22:37:53  *** Guest20006 has quit IRC
6032018-10-20T22:41:53  *** spinza has quit IRC
6042018-10-20T22:46:27  *** brianhoffman has joined #bitcoin-core-dev
6052018-10-20T22:46:47  *** Chris_Stewart_5 has quit IRC
6062018-10-20T22:48:48  *** spinza has joined #bitcoin-core-dev
6072018-10-20T22:53:18  *** dviola has quit IRC
6082018-10-20T22:55:07  *** ula has quit IRC
6092018-10-20T22:58:51  *** dviola has joined #bitcoin-core-dev
6102018-10-20T23:01:48  *** meshcollider_ has quit IRC
6112018-10-20T23:07:49  *** Guyver2 has quit IRC
6122018-10-20T23:10:26  *** Zenton has joined #bitcoin-core-dev
6132018-10-20T23:15:29  <booyah> btw, why not asio, instead of all this problems with pool/libevent and all
6142018-10-20T23:16:38  <phantomcircuit> booyah, asio and libevent are both callback based
6152018-10-20T23:16:46  <phantomcircuit> while the current logic is polling based
6162018-10-20T23:17:23  <phantomcircuit> improving what's there is less error prone than switching the entire paradigm
6172018-10-20T23:21:07  <sipa> one step at a time
6182018-10-20T23:25:50  <wumpus> asio used to be used for the RPC server, that was pretty awful, also we don't really want to add any boost dependencies
6192018-10-20T23:27:00  <booyah> wumpus: what was afful there, in that rpc?
6202018-10-20T23:29:28  *** pope has joined #bitcoin-core-dev
6212018-10-20T23:45:53  *** pope has quit IRC
6222018-10-20T23:48:06  *** mistergold has quit IRC
6232018-10-20T23:55:20  *** AaronvanW has quit IRC