1 2018-10-20T00:04:16  *** justanotheruser has quit IRC
  2 2018-10-20T00:05:05  *** Murch has joined #bitcoin-core-dev
  3 2018-10-20T00:05:58  *** fanquake has joined #bitcoin-core-dev
  4 2018-10-20T00:15:17  *** Aaronvan_ has quit IRC
  5 2018-10-20T00:15:54  *** AaronvanW has joined #bitcoin-core-dev
  6 2018-10-20T00:26:07  *** bralyclow3 has joined #bitcoin-core-dev
  7 2018-10-20T00:27:17  *** bralyclow3 has quit IRC
  8 2018-10-20T00:28:57  *** Murch has quit IRC
  9 2018-10-20T00:30:43  *** IGHOR has quit IRC
 10 2018-10-20T00:34:12  *** intcat has quit IRC
 11 2018-10-20T00:35:03  *** Chris_Stewart_5 has quit IRC
 12 2018-10-20T00:36:12  *** intcat has joined #bitcoin-core-dev
 13 2018-10-20T00:39:49  *** IGHOR has joined #bitcoin-core-dev
 14 2018-10-20T00:41:30  *** Murch has joined #bitcoin-core-dev
 15 2018-10-20T00:44:11  *** dviola has joined #bitcoin-core-dev
 16 2018-10-20T00:55:06  *** spinza has quit IRC
 17 2018-10-20T00:56:53  *** jcorgan has joined #bitcoin-core-dev
 18 2018-10-20T00:59:08  <gwillen> achow101: can you explain the comment "Get all of the previous transactions" in rawtransaction.cpp (in finalizepsbt)?
 19 2018-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?
 20 2018-10-20T00:59:45  <gwillen> or is it safe to replace with "is PSBT input signed"?
 21 2018-10-20T01:00:08  <achow101> that's very likely to be the result of copy-paste from some other rpc
 22 2018-10-20T01:00:15  * sipa guesses: the comment was copied from rpcwallet.cpp
 23 2018-10-20T01:04:26  *** dviola has quit IRC
 24 2018-10-20T01:05:28  *** Murch has quit IRC
 25 2018-10-20T01:11:19  *** AaronvanW has quit IRC
 26 2018-10-20T01:12:34  *** j|l has joined #bitcoin-core-dev
 27 2018-10-20T01:13:17  *** kexkey has joined #bitcoin-core-dev
 28 2018-10-20T01:13:39  *** proletesseract has joined #bitcoin-core-dev
 29 2018-10-20T01:13:49  *** jpe_ has joined #bitcoin-core-dev
 30 2018-10-20T01:16:51  *** jpe__ has quit IRC
 31 2018-10-20T01:19:49  *** spinza has joined #bitcoin-core-dev
 32 2018-10-20T01:23:59  *** proletesseract has quit IRC
 33 2018-10-20T01:26:25  *** j|l has quit IRC
 34 2018-10-20T01:40:34  *** Krellan_ has quit IRC
 35 2018-10-20T01:45:16  *** bitcoin-git has joined #bitcoin-core-dev
 36 2018-10-20T01:45:17  <bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/d387507aeca6...8010ded6da56
 37 2018-10-20T01:45:17  <bitcoin-git> bitcoin/master 7e18673 Kristaps Kaupe: Fix typo
 38 2018-10-20T01:45:18  <bitcoin-git> bitcoin/master 8010ded Pieter Wuille: Merge #14524: Trivial: fix typo...
 39 2018-10-20T01:45:18  *** bitcoin-git has left #bitcoin-core-dev
 40 2018-10-20T01:46:31  *** bitcoin-git has joined #bitcoin-core-dev
 41 2018-10-20T01:46:32  <bitcoin-git> [bitcoin] sipa closed pull request #14524: Trivial: fix typo (master...typos) https://github.com/bitcoin/bitcoin/pull/14524
 42 2018-10-20T01:46:32  *** bitcoin-git has left #bitcoin-core-dev
 43 2018-10-20T01:48:59  *** Chris_Stewart_5 has joined #bitcoin-core-dev
 44 2018-10-20T01:53:06  *** bitcoin-git has joined #bitcoin-core-dev
 45 2018-10-20T01:53:07  <bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/8010ded6da56...b14db5abab40
 46 2018-10-20T01:53:08  <bitcoin-git> bitcoin/master bc60c61 practicalswift: Avoid 1 << 31 (UB) in calculation of SEQUENCE_LOCKTIME_DISABLE_FLAG
 47 2018-10-20T01:53:08  <bitcoin-git> bitcoin/master b14db5a Pieter Wuille: Merge #14513: Avoid 1 << 31 (UB) in calculation of SEQUENCE_LOCKTIME_DISABLE_FLAG...
 48 2018-10-20T01:53:09  *** bitcoin-git has left #bitcoin-core-dev
 49 2018-10-20T01:54:19  *** bitcoin-git has joined #bitcoin-core-dev
 50 2018-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
 51 2018-10-20T01:54:20  *** bitcoin-git has left #bitcoin-core-dev
 52 2018-10-20T01:57:07  *** dviola has joined #bitcoin-core-dev
 53 2018-10-20T02:00:14  *** bitcoin-git has joined #bitcoin-core-dev
 54 2018-10-20T02:00:15  <bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b14db5abab40...b2863c0685a5
 55 2018-10-20T02:00:15  <bitcoin-git> bitcoin/master 369244f Chun Kuan Lee: utils: Fix broken Windows filelock
 56 2018-10-20T02:00:16  <bitcoin-git> bitcoin/master b2863c0 Pieter Wuille: Merge #14426: utils: Fix broken Windows filelock...
 57 2018-10-20T02:00:17  *** bitcoin-git has left #bitcoin-core-dev
 58 2018-10-20T02:01:14  *** bitcoin-git has joined #bitcoin-core-dev
 59 2018-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
 60 2018-10-20T02:01:14  *** bitcoin-git has left #bitcoin-core-dev
 61 2018-10-20T02:02:18  *** Chris_Stewart_5 has quit IRC
 62 2018-10-20T02:03:06  *** bitcoin-git has joined #bitcoin-core-dev
 63 2018-10-20T02:03:07  <bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b2863c0685a5...dc1e54206d76
 64 2018-10-20T02:03:07  <bitcoin-git> bitcoin/master 1f01fe0 Antoine Le Calvez: bitcoin-tx: Use constant for n pubkeys check...
 65 2018-10-20T02:03:08  <bitcoin-git> bitcoin/master dc1e542 Pieter Wuille: Merge #14474: bitcoin-tx: Use constant for n pubkeys check...
 66 2018-10-20T02:03:08  *** bitcoin-git has left #bitcoin-core-dev
 67 2018-10-20T02:04:09  *** bitcoin-git has joined #bitcoin-core-dev
 68 2018-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
 69 2018-10-20T02:04:10  *** bitcoin-git has left #bitcoin-core-dev
 70 2018-10-20T02:05:04  *** bitcoin-git has joined #bitcoin-core-dev
 71 2018-10-20T02:05:05  <bitcoin-git> [bitcoin] sipa pushed 2 new commits to 0.17: https://github.com/bitcoin/bitcoin/compare/5b47b8efd48d...1b5af2c177ee
 72 2018-10-20T02:05:05  <bitcoin-git> bitcoin/0.17 f7dbcaa Sjors Provoost: [doc] getblocktemplate: use SegWit in example
 73 2018-10-20T02:05:06  <bitcoin-git> bitcoin/0.17 1b5af2c Pieter Wuille: Merge #14509: [0.17] doc: use SegWit in getblocktemplate example...
 74 2018-10-20T02:05:06  *** bitcoin-git has left #bitcoin-core-dev
 75 2018-10-20T02:08:59  <fanquake> sipa It'd be great if you could also merged #14011
 76 2018-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
 77 2018-10-20T02:12:19  <sipa> fanquake: sgtm
 78 2018-10-20T02:13:18  *** ula has quit IRC
 79 2018-10-20T02:14:33  *** bitcoin-git has joined #bitcoin-core-dev
 80 2018-10-20T02:14:33  <bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/dc1e54206d76...e754c6e33194
 81 2018-10-20T02:14:34  <bitcoin-git> bitcoin/master a3197c5 Russell Yanofsky: Disable wallet and address book Qt tests on macOS minimal platform...
 82 2018-10-20T02:14:34  <bitcoin-git> bitcoin/master e754c6e Pieter Wuille: Merge #14011: Disable wallet and address book Qt tests on macOS minimal platform...
 83 2018-10-20T02:14:35  *** bitcoin-git has left #bitcoin-core-dev
 84 2018-10-20T02:15:03  <sipa> anything else?
 85 2018-10-20T02:15:15  *** bitcoin-git has joined #bitcoin-core-dev
 86 2018-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
 87 2018-10-20T02:15:15  *** bitcoin-git has left #bitcoin-core-dev
 88 2018-10-20T02:15:23  *** ghost43 has quit IRC
 89 2018-10-20T02:15:23  *** ExtraCrispy has quit IRC
 90 2018-10-20T02:15:23  *** morcos has quit IRC
 91 2018-10-20T02:15:23  *** sipa has quit IRC
 92 2018-10-20T02:15:24  *** intcat has quit IRC
 93 2018-10-20T02:15:40  *** ghost43 has joined #bitcoin-core-dev
 94 2018-10-20T02:15:43  *** sipa has joined #bitcoin-core-dev
 95 2018-10-20T02:15:43  *** morcos has joined #bitcoin-core-dev
 96 2018-10-20T02:15:55  <fanquake> sipa cheers. #14512 is also a trivial merge.
 97 2018-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
 98 2018-10-20T02:20:11  *** intcat has joined #bitcoin-core-dev
 99 2018-10-20T02:24:09  *** bitcoin-git has joined #bitcoin-core-dev
100 2018-10-20T02:24:10  <bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/e754c6e33194...91482e5bf22d
101 2018-10-20T02:24:10  <bitcoin-git> bitcoin/master 29ed2d6 Hennadii Stepanov: Improve CAmount tests...
102 2018-10-20T02:24:11  <bitcoin-git> bitcoin/master 91482e5 Pieter Wuille: Merge #14460: tests: Improve 'CAmount' tests...
103 2018-10-20T02:24:11  *** bitcoin-git has left #bitcoin-core-dev
104 2018-10-20T02:25:14  *** bitcoin-git has joined #bitcoin-core-dev
105 2018-10-20T02:25:15  <bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/91482e5bf22d...544f3234384b
106 2018-10-20T02:25:15  <bitcoin-git> bitcoin/master b6b9915 Martin Erlandsson: Textual improvements
107 2018-10-20T02:25:16  <bitcoin-git> bitcoin/master 544f323 Pieter Wuille: Merge #14512: docs: Textual improvements in README.md...
108 2018-10-20T02:25:16  *** bitcoin-git has left #bitcoin-core-dev
109 2018-10-20T02:33:02  *** rh0nj has quit IRC
110 2018-10-20T02:34:08  *** rh0nj has joined #bitcoin-core-dev
111 2018-10-20T02:46:29  *** proletesseract has joined #bitcoin-core-dev
112 2018-10-20T02:48:49  *** dviola has quit IRC
113 2018-10-20T02:56:13  *** justanotheruser has joined #bitcoin-core-dev
114 2018-10-20T03:03:04  *** proletesseract has quit IRC
115 2018-10-20T03:23:05  *** polydin has joined #bitcoin-core-dev
116 2018-10-20T03:26:05  *** polydin has quit IRC
117 2018-10-20T03:33:24  *** michaelsdunn1 has quit IRC
118 2018-10-20T03:45:44  *** bitconner has quit IRC
119 2018-10-20T03:55:33  *** schnerch_ has joined #bitcoin-core-dev
120 2018-10-20T03:58:42  *** schnerchi has quit IRC
121 2018-10-20T03:59:26  *** copumpkin has joined #bitcoin-core-dev
122 2018-10-20T04:28:17  *** luke-jr has quit IRC
123 2018-10-20T04:28:30  *** luke-jr has joined #bitcoin-core-dev
124 2018-10-20T04:37:12  *** luke-jr has quit IRC
125 2018-10-20T04:42:24  *** Tralfaz has quit IRC
126 2018-10-20T04:42:28  *** copumpkin has quit IRC
127 2018-10-20T04:43:20  *** Tralfaz has joined #bitcoin-core-dev
128 2018-10-20T04:43:28  *** copumpkin has joined #bitcoin-core-dev
129 2018-10-20T04:44:47  *** luke-jr has joined #bitcoin-core-dev
130 2018-10-20T05:11:11  *** bitcoin-git has joined #bitcoin-core-dev
131 2018-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
132 2018-10-20T05:11:11  *** bitcoin-git has left #bitcoin-core-dev
133 2018-10-20T05:11:26  *** bitcoin-git has joined #bitcoin-core-dev
134 2018-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
135 2018-10-20T05:11:27  *** bitcoin-git has left #bitcoin-core-dev
136 2018-10-20T05:24:42  *** bitcoin-git has joined #bitcoin-core-dev
137 2018-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
138 2018-10-20T05:24:42  *** bitcoin-git has left #bitcoin-core-dev
139 2018-10-20T05:30:12  *** proletesseract has joined #bitcoin-core-dev
140 2018-10-20T05:31:06  *** bitcoin-git has joined #bitcoin-core-dev
141 2018-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
142 2018-10-20T05:31:06  *** bitcoin-git has left #bitcoin-core-dev
143 2018-10-20T05:37:38  *** Tralfaz has quit IRC
144 2018-10-20T05:37:57  *** Tralfaz has joined #bitcoin-core-dev
145 2018-10-20T05:44:10  *** Tralfaz has joined #bitcoin-core-dev
146 2018-10-20T05:44:51  *** Squidicuz has quit IRC
147 2018-10-20T05:45:12  *** Squidicuz has joined #bitcoin-core-dev
148 2018-10-20T05:46:30  *** keymone has quit IRC
149 2018-10-20T05:47:13  *** Tralfaz has quit IRC
150 2018-10-20T05:47:37  *** Tralfaz has joined #bitcoin-core-dev
151 2018-10-20T05:48:23  *** Tralfaz has quit IRC
152 2018-10-20T05:49:38  *** keymone has joined #bitcoin-core-dev
153 2018-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
154 2018-10-20T05:56:58  *** jpe has joined #bitcoin-core-dev
155 2018-10-20T05:59:38  *** jpe_ has quit IRC
156 2018-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
157 2018-10-20T05:59:58  <fanquake> Still need to get an actual fix.
158 2018-10-20T06:00:13  *** Tuna_Golem has joined #bitcoin-core-dev
159 2018-10-20T06:01:33  <fanquake> *automake 1.16
160 2018-10-20T06:01:58  <midnightmagic> nice
161 2018-10-20T06:02:28  <midnightmagic> if I could get netbsd into my talos somehow, I'd be testing in there.
162 2018-10-20T06:02:33  *** hebasto has joined #bitcoin-core-dev
163 2018-10-20T06:03:27  *** Krellan has joined #bitcoin-core-dev
164 2018-10-20T06:05:37  <wumpus> fanquake: thanks for investigating; glad you could narrow it down to a specific automake change
165 2018-10-20T06:06:31  *** Krellan has joined #bitcoin-core-dev
166 2018-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 :)
167 2018-10-20T06:08:33  <midnightmagic> you mean the hifive unleashed?
168 2018-10-20T06:08:41  <midnightmagic> did you get the expansion board too?
169 2018-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
170 2018-10-20T06:09:06  <wumpus> yes that one
171 2018-10-20T06:09:44  <midnightmagic> i wonder if that page-fault-computing usenix method can be used on the sifive stuff
172 2018-10-20T06:09:48  <wumpus> FWIW it runs a bitcoin + lightning node + tor fine :D
173 2018-10-20T06:09:55  <midnightmagic> wow, nice.
174 2018-10-20T06:11:26  *** luke-jr has quit IRC
175 2018-10-20T06:11:51  *** luke-jr has joined #bitcoin-core-dev
176 2018-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
177 2018-10-20T06:12:35  <midnightmagic> still unavailable, too, apparently.
178 2018-10-20T06:12:49  <wumpus> I got one of the last unleashed boards, literally a week later the option to order them was closed :)
179 2018-10-20T06:12:57  <midnightmagic> This kind of hardware, I don't mind learning new assembly for.
180 2018-10-20T06:13:08  <sipa> that's risc-v?
181 2018-10-20T06:13:16  <midnightmagic> yeah
182 2018-10-20T06:13:20  <sipa> nice
183 2018-10-20T06:13:22  <wumpus> midnightmagic: yeah it's not much work either, the instruction set is super straightforward
184 2018-10-20T06:14:31  <midnightmagic> runs bitcoin okay? how long did it take to sync with nbd..?
185 2018-10-20T06:14:39  <midnightmagic> (Also, why not nfs?)
186 2018-10-20T06:14:43  <wumpus> midnightmagic: this lists all of them: https://github.com/michaeljclark/riscv-meta/blob/master/opcode-fullnames
187 2018-10-20T06:14:52  <wumpus> (including extensions)
188 2018-10-20T06:15:05  <midnightmagic> whoah, short.
189 2018-10-20T06:15:46  <midnightmagic> did you ever see the power9 instruction set?
190 2018-10-20T06:16:06  *** luke-jr has quit IRC
191 2018-10-20T06:16:15  *** luke-jr has joined #bitcoin-core-dev
192 2018-10-20T06:16:21  *** bitcoin-git has joined #bitcoin-core-dev
193 2018-10-20T06:16:22  <bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/544f3234384b...d433239a8d54
194 2018-10-20T06:16:22  <bitcoin-git> bitcoin/master 94e21c1 James O'Beirne: test: forward timeouts properly in send_blocks_and_test
195 2018-10-20T06:16:23  <bitcoin-git> bitcoin/master d433239 MarcoFalke: Merge #14456: test: forward timeouts properly in send_blocks_and_test...
196 2018-10-20T06:16:23  *** bitcoin-git has left #bitcoin-core-dev
197 2018-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)
198 2018-10-20T06:17:04  <wumpus> midnightmagic: I don't think so!
199 2018-10-20T06:17:20  <midnightmagic> you have to do weird things like load in addresses in four pieces, carefully shifting them and arranging them..
200 2018-10-20T06:17:21  *** bitcoin-git has joined #bitcoin-core-dev
201 2018-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
202 2018-10-20T06:17:22  *** bitcoin-git has left #bitcoin-core-dev
203 2018-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
204 2018-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
205 2018-10-20T06:20:21  <midnightmagic> wumpus: it's *huge*. The PowerISA reference manual is 1240 pages.
206 2018-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
207 2018-10-20T06:22:01  <midnightmagic> "Appendix F. Power ISA Instruction Set Sorted by Mnemonic" => 17 pages is tiny little writing.
208 2018-10-20T06:22:15  <wumpus> that's kind of much for a *reduced* instruction set platform
209 2018-10-20T06:22:41  <wumpus> though it's still less than Intel's multi-volume bibles
210 2018-10-20T06:22:41  <midnightmagic> 56 opcodes per page.
211 2018-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.
212 2018-10-20T06:25:57  <wumpus> heh those pages are either scribbled with occult markings or ▓███▒▓█-censored ... nah, sounds like some combinatorial complexity blowup
213 2018-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'
214 2018-10-20T06:27:28  *** SopaXorzTaker has joined #bitcoin-core-dev
215 2018-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.
216 2018-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
217 2018-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.
218 2018-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)
219 2018-10-20T06:28:38  <midnightmagic> well I sure hope so.
220 2018-10-20T06:28:52  <wumpus> sipa: oh! TIL too
221 2018-10-20T06:29:40  <sipa> and there have been CISC platforms with just 8 instructions
222 2018-10-20T06:29:50  <wumpus> midnightmagic: I agree, it'd be a form of open competition which is good
223 2018-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)
224 2018-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.
225 2018-10-20T06:33:23  <ossifrage> I always found it cleaner to decouple memory to register instructions from register to alu to register instructions
226 2018-10-20T06:33:30  <wumpus> ossifrage: apparently yes that's what it means
227 2018-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'
228 2018-10-20T06:36:03  <ossifrage> If you don't have a pipeline writing a RISC CPU in verilog is surprisingly simple.
229 2018-10-20T06:36:36  <wumpus> yes handling timing and pipelines is where it gets complex
230 2018-10-20T06:37:13  <fanquake> MarcoFalke If you want to merge #14497 I think that's ready to go.
231 2018-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
232 2018-10-20T06:37:28  <wumpus> e.g. this is a whole rv32 CPU https://github.com/cliffordwolf/picorv32/blob/master/picorv32.v
233 2018-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
234 2018-10-20T06:38:12  <ossifrage> I can't imagine how painful it is to verify a CISC processor
235 2018-10-20T06:38:39  <wumpus> it can fit in the small FPGAs supported by the open source yosys toolchain
236 2018-10-20T06:40:34  <wumpus> fanquake: will merge
237 2018-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
238 2018-10-20T06:44:22  *** bitcoin-git has joined #bitcoin-core-dev
239 2018-10-20T06:44:23  <bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/d433239a8d54...6746a8951923
240 2018-10-20T06:44:23  <bitcoin-git> bitcoin/master 1fb3c16 Hennadii Stepanov: Add `doc/bitcoin-conf.md`
241 2018-10-20T06:44:23  <bitcoin-git> bitcoin/master 6746a89 Wladimir J. van der Laan: Merge #14497: docs: Add doc/bitcoin-conf.md...
242 2018-10-20T06:44:24  *** bitcoin-git has left #bitcoin-core-dev
243 2018-10-20T06:45:22  *** bitcoin-git has joined #bitcoin-core-dev
244 2018-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
245 2018-10-20T06:45:22  *** bitcoin-git has left #bitcoin-core-dev
246 2018-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
247 2018-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
248 2018-10-20T06:55:04  <wumpus> midnightmagic: btw re: POWER cna you weigh in on #14066 please
249 2018-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
250 2018-10-20T06:55:06  *** ken2812221 has joined #bitcoin-core-dev
251 2018-10-20T06:55:33  <fanquake> wumpus Would we be able to fork emil-e/rapidcheck to github.com/bitcoin-core/?
252 2018-10-20T06:55:34  <midnightmagic> sure
253 2018-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.
254 2018-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.
255 2018-10-20T06:56:06  <wumpus> fanquake: yep; do we need our own fork of that?
256 2018-10-20T06:56:12  <wumpus> yes, sure
257 2018-10-20T07:01:05  <fanquake> wumpus thanks. More context in my comment here: https://github.com/bitcoin/bitcoin/pull/14430#issuecomment-431555605
258 2018-10-20T07:05:22  <wumpus> repo created, see https://github.com/bitcoin-core/rapidcheck, who needs commit access?
259 2018-10-20T07:05:28  <wumpus> besides MarcoFalke
260 2018-10-20T07:08:01  <fanquake> maybe Christewart, he seems to be spearheading most of this work
261 2018-10-20T07:21:23  <wumpus> ok!
262 2018-10-20T07:22:23  <wumpus> any idea if they're on IRC?
263 2018-10-20T07:25:08  *** ExtraCrispy has joined #bitcoin-core-dev
264 2018-10-20T07:25:53  <wumpus> 09:25 -!- Christewart There was no such nickname at least not under that name
265 2018-10-20T07:26:31  <gwillen> are you looking for Chris_Stewart_5 ? or someone else?
266 2018-10-20T07:26:48  <fanquake> We spoke on IRC in tokyo
267 2018-10-20T07:26:54  <fanquake> Yea I think that's it ^
268 2018-10-20T07:27:11  <gwillen> he doesn't have a bouncer I guess, he seems to be on and off
269 2018-10-20T07:27:11  <wumpus> of course!
270 2018-10-20T07:27:25  <wumpus> (I vaguely remembered that but wasn't sure)
271 2018-10-20T07:33:16  <wumpus> invited to bitcoin and bitcoin-core at least
272 2018-10-20T07:35:07  <fanquake> wumpus cheers
273 2018-10-20T07:36:35  *** proletesseract has quit IRC
274 2018-10-20T07:37:08  *** proletesseract has joined #bitcoin-core-dev
275 2018-10-20T07:41:25  *** proletesseract has quit IRC
276 2018-10-20T07:53:43  *** luke-jr has quit IRC
277 2018-10-20T07:53:56  *** luke-jr has joined #bitcoin-core-dev
278 2018-10-20T07:57:09  *** luke-jr has quit IRC
279 2018-10-20T07:57:23  *** luke-jr has joined #bitcoin-core-dev
280 2018-10-20T07:59:30  *** luke-jr has quit IRC
281 2018-10-20T07:59:40  *** luke-jr has joined #bitcoin-core-dev
282 2018-10-20T08:04:42  *** luke-jr has quit IRC
283 2018-10-20T08:04:51  *** luke-jr has joined #bitcoin-core-dev
284 2018-10-20T08:05:52  *** luke-jr has quit IRC
285 2018-10-20T08:06:07  *** luke-jr has joined #bitcoin-core-dev
286 2018-10-20T08:09:14  *** luke-jr has quit IRC
287 2018-10-20T08:09:22  *** luke-jr has joined #bitcoin-core-dev
288 2018-10-20T08:13:40  *** Krellan has quit IRC
289 2018-10-20T08:14:00  *** bitconner has joined #bitcoin-core-dev
290 2018-10-20T08:15:24  *** Krellan has joined #bitcoin-core-dev
291 2018-10-20T08:18:00  *** bitconner has quit IRC
292 2018-10-20T08:22:06  *** marcoagner has quit IRC
293 2018-10-20T08:24:44  *** luke-jr has quit IRC
294 2018-10-20T08:25:25  *** luke-jr has joined #bitcoin-core-dev
295 2018-10-20T08:28:06  *** ghost43 has quit IRC
296 2018-10-20T08:28:20  *** ghost43 has joined #bitcoin-core-dev
297 2018-10-20T08:29:02  *** Guyver2 has joined #bitcoin-core-dev
298 2018-10-20T08:29:07  *** murrayn[m] has joined #bitcoin-core-dev
299 2018-10-20T08:31:12  *** Victorsueca has quit IRC
300 2018-10-20T08:32:29  *** Victorsueca has joined #bitcoin-core-dev
301 2018-10-20T08:33:05  *** luke-jr has quit IRC
302 2018-10-20T08:33:41  *** bitcoin-git has joined #bitcoin-core-dev
303 2018-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
304 2018-10-20T08:33:42  *** bitcoin-git has left #bitcoin-core-dev
305 2018-10-20T08:37:16  *** luke-jr has joined #bitcoin-core-dev
306 2018-10-20T08:40:22  *** Emcy has quit IRC
307 2018-10-20T08:42:10  *** luke-jr has quit IRC
308 2018-10-20T08:42:43  *** luke-jr has joined #bitcoin-core-dev
309 2018-10-20T09:44:32  *** AaronvanW has joined #bitcoin-core-dev
310 2018-10-20T09:57:52  <MarcoFalke> Sorry for the downtime of DrahtBot recently. It should be catching up in the next hour or so.
311 2018-10-20T09:58:33  <fanquake> The flood of emails has begun :p
312 2018-10-20T10:02:31  *** kexkey has quit IRC
313 2018-10-20T10:02:45  *** kexkey has joined #bitcoin-core-dev
314 2018-10-20T10:05:12  <promag> +1
315 2018-10-20T10:11:29  *** spinza has quit IRC
316 2018-10-20T10:16:04  *** spinza has joined #bitcoin-core-dev
317 2018-10-20T10:22:51  *** bitcoin-git has joined #bitcoin-core-dev
318 2018-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
319 2018-10-20T10:22:51  *** bitcoin-git has left #bitcoin-core-dev
320 2018-10-20T10:39:52  *** AaronvanW has quit IRC
321 2018-10-20T10:40:30  *** AaronvanW has joined #bitcoin-core-dev
322 2018-10-20T10:46:59  * luke-jr wonders if we really need to ban regexs due to reviewer regex-illiteracy :x
323 2018-10-20T10:59:05  *** cdecker has quit IRC
324 2018-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
325 2018-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)
326 2018-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
327 2018-10-20T11:10:01  <promag> MarcoFalke: should provide fs::relative implementation to our source? :/
328 2018-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
329 2018-10-20T11:29:00  *** newbie-- has joined #bitcoin-core-dev
330 2018-10-20T11:31:54  *** windsok has quit IRC
331 2018-10-20T11:34:12  *** windsok has joined #bitcoin-core-dev
332 2018-10-20T11:34:12  *** windsok has joined #bitcoin-core-dev
333 2018-10-20T11:37:26  *** Chris_Stewart_5 has joined #bitcoin-core-dev
334 2018-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.
335 2018-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?
336 2018-10-20T11:51:00  *** Chris_Stewart_5 has quit IRC
337 2018-10-20T11:51:04  <luke-jr> if you want an archive, just use the ones github generates..
338 2018-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
339 2018-10-20T11:51:50  <wumpus> ... on whether its output is better
340 2018-10-20T11:51:57  <luke-jr> its output seems far worse
341 2018-10-20T11:52:35  <luke-jr> same column-based output, but with single letter stuff you need the manpage to understand
342 2018-10-20T11:59:56  *** copumpkin has quit IRC
343 2018-10-20T12:00:13  *** Zenton has quit IRC
344 2018-10-20T12:00:48  <wumpus> what information do you need?
345 2018-10-20T12:00:49  *** romanz_ has quit IRC
346 2018-10-20T12:01:19  *** Victorsueca has quit IRC
347 2018-10-20T12:01:22  *** Zenton has joined #bitcoin-core-dev
348 2018-10-20T12:02:29  *** Victorsueca has joined #bitcoin-core-dev
349 2018-10-20T12:02:51  *** bitcoin-git has joined #bitcoin-core-dev
350 2018-10-20T12:02:52  <bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/6746a8951923...96c409c919b9
351 2018-10-20T12:02:52  <bitcoin-git> bitcoin/master fadffae MarcoFalke: Revert "Make qt wallet test compatible with qt4"...
352 2018-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"...
353 2018-10-20T12:02:53  *** bitcoin-git has left #bitcoin-core-dev
354 2018-10-20T12:03:18  <luke-jr> wumpus: target architecture, imported symbol versions, and exported symbols
355 2018-10-20T12:04:01  *** bitcoin-git has joined #bitcoin-core-dev
356 2018-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
357 2018-10-20T12:04:02  *** bitcoin-git has left #bitcoin-core-dev
358 2018-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
359 2018-10-20T12:04:58  *** romanz has joined #bitcoin-core-dev
360 2018-10-20T12:05:00  <luke-jr> wumpus: the parser does not correctly handle readelf output
361 2018-10-20T12:05:19  <luke-jr> simply splitting on spaces fails when the data contains spaces IIRC
362 2018-10-20T12:05:30  <luke-jr> eg     3: 0000000000000000     0 FUNC    GLOBAL DEFAULT [<localentry>: 8]   UND mbrtowc@GLIBC_2.17 (2)
363 2018-10-20T12:05:39  <luke-jr> this [<localentry>: 8] stuff
364 2018-10-20T12:05:40  *** Tuna_Golem has quit IRC
365 2018-10-20T12:06:00  <wumpus> oh some new information in columns was introduced, interesting
366 2018-10-20T12:07:40  <wumpus> it is apparently a powerPC ABI specific thing where a function can have two entry points
367 2018-10-20T12:08:06  <luke-jr> hmm
368 2018-10-20T12:08:15  <wumpus> we don't *need* that information but indeed it messes with the current parsing
369 2018-10-20T12:09:21  <luke-jr> sorry, guess the motivation behind the change was missing >_<
370 2018-10-20T12:13:14  <wumpus> indeed; so it seems fair enough
371 2018-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
372 2018-10-20T12:14:20  *** Tuna_Golem has joined #bitcoin-core-dev
373 2018-10-20T12:14:22  <luke-jr> I don't really see how it can get any simpler :x
374 2018-10-20T12:14:23  *** bitconner has joined #bitcoin-core-dev
375 2018-10-20T12:14:34  <luke-jr> wumpus: do you have a link for the PPC-specific details?
376 2018-10-20T12:16:36  <wumpus> luke-jr: http://openpowerfoundation.org/wp-content/uploads/resources/leabi/content/dbdoclet.50655241_95185.html
377 2018-10-20T12:16:44  <luke-jr> ty
378 2018-10-20T12:18:25  <wumpus> luke-jr: ok, I think it's not *that* complex understanding what it does
379 2018-10-20T12:19:22  *** bitconner has quit IRC
380 2018-10-20T12:24:37  *** laurentmt has joined #bitcoin-core-dev
381 2018-10-20T12:25:27  *** laurentmt has quit IRC
382 2018-10-20T12:25:34  *** josephnicholas has joined #bitcoin-core-dev
383 2018-10-20T12:26:52  *** Emcy has joined #bitcoin-core-dev
384 2018-10-20T12:28:41  *** ott0disk has joined #bitcoin-core-dev
385 2018-10-20T12:30:03  *** Krellan has quit IRC
386 2018-10-20T12:31:12  *** Krellan has joined #bitcoin-core-dev
387 2018-10-20T12:33:53  *** mistergold has joined #bitcoin-core-dev
388 2018-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?
389 2018-10-20T12:35:29  <wumpus> oh wait—it's a *boost* thing, fs is boost::fs
390 2018-10-20T12:36:46  <promag> right, 1.58 doesn't have it
391 2018-10-20T12:37:53  <wumpus> we need to support 1.47 according to dependencies.md
392 2018-10-20T12:37:59  <wumpus> so using that is really out of the question
393 2018-10-20T12:38:05  <wumpus> at least without fallback
394 2018-10-20T12:38:43  *** AaronvanW has quit IRC
395 2018-10-20T12:39:10  <luke-jr> I thought we were trying to avoid boost anyway?
396 2018-10-20T12:39:14  *** AaronvanW has joined #bitcoin-core-dev
397 2018-10-20T12:40:48  <promag> 1.47?
398 2018-10-20T12:41:12  <promag> ah right, min version
399 2018-10-20T12:41:24  <promag> err,
400 2018-10-20T12:41:52  <promag> I'll submit a fix
401 2018-10-20T12:44:23  <wumpus> luke-jr: really can't in this case until there's a replacement for boost::filesystem
402 2018-10-20T12:44:36  <wumpus> (which would require a newer c++ than 11)
403 2018-10-20T12:45:17  <wumpus> it's fairly straightforward to redefine the fs namespace then
404 2018-10-20T12:47:44  <luke-jr> oh well
405 2018-10-20T12:52:41  *** AaronvanW has quit IRC
406 2018-10-20T12:54:34  *** AaronvanW has joined #bitcoin-core-dev
407 2018-10-20T12:55:43  <wumpus> promag: thanks!
408 2018-10-20T12:59:21  *** AaronvanW has quit IRC
409 2018-10-20T13:00:29  <wumpus> luke-jr: I know it gets tiring but moving away from boost is really a long-term project
410 2018-10-20T13:01:45  <wumpus> it needs to be done without interfering with anything else and there's so many different concernt to juggle
411 2018-10-20T13:02:47  *** cdecker has joined #bitcoin-core-dev
412 2018-10-20T13:03:53  <luke-jr> wumpus: well, if it serves a purpose, I'm fine with it :p
413 2018-10-20T13:03:57  <luke-jr> better than re-inventing
414 2018-10-20T13:04:49  *** AaronvanW has joined #bitcoin-core-dev
415 2018-10-20T13:05:10  <wumpus> yes, exactly,
416 2018-10-20T13:09:27  *** AaronvanW has quit IRC
417 2018-10-20T13:09:45  *** josephni_ has joined #bitcoin-core-dev
418 2018-10-20T13:10:00  *** josephnicholas has quit IRC
419 2018-10-20T13:14:48  *** bitcoin-git has joined #bitcoin-core-dev
420 2018-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
421 2018-10-20T13:14:48  *** bitcoin-git has left #bitcoin-core-dev
422 2018-10-20T13:16:34  <Sentineo> hm does anybody else have a different behavior with multiwallets in testnet vs mainnet?
423 2018-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.
424 2018-10-20T13:19:25  <Sentineo> v0.17.0 btw
425 2018-10-20T13:19:50  <promag> wumpus: MarcoFalke: #12842
426 2018-10-20T13:19:52  <gribble> https://github.com/bitcoin/bitcoin/issues/12842 | Prevent concurrent savemempool by promag · Pull Request #12842 · bitcoin/bitcoin · GitHub
427 2018-10-20T13:22:13  *** promag_ has joined #bitcoin-core-dev
428 2018-10-20T13:22:13  *** promag has quit IRC
429 2018-10-20T13:25:10  *** promag_ is now known as promag
430 2018-10-20T13:26:05  *** AaronvanW has joined #bitcoin-core-dev
431 2018-10-20T13:29:56  *** Aaronvan_ has joined #bitcoin-core-dev
432 2018-10-20T13:30:41  *** AaronvanW has quit IRC
433 2018-10-20T13:33:46  *** bitcoin-git has joined #bitcoin-core-dev
434 2018-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
435 2018-10-20T13:33:47  *** bitcoin-git has left #bitcoin-core-dev
436 2018-10-20T13:34:06  <promag> wumpus: MarcoFalke ^
437 2018-10-20T13:34:12  *** Aaronvan_ has quit IRC
438 2018-10-20T13:35:30  *** AaronvanW has joined #bitcoin-core-dev
439 2018-10-20T13:36:11  *** josephni_ has quit IRC
440 2018-10-20T13:36:46  *** josephnicholas has joined #bitcoin-core-dev
441 2018-10-20T13:40:04  *** Chris_Stewart_5 has joined #bitcoin-core-dev
442 2018-10-20T13:53:33  *** josephni_ has joined #bitcoin-core-dev
443 2018-10-20T13:53:34  *** josephnicholas has quit IRC
444 2018-10-20T13:55:12  *** josephni_ has quit IRC
445 2018-10-20T13:59:21  *** AaronvanW has quit IRC
446 2018-10-20T13:59:59  *** AaronvanW has joined #bitcoin-core-dev
447 2018-10-20T14:01:25  *** josephnicholas has joined #bitcoin-core-dev
448 2018-10-20T14:02:52  *** iphelix has joined #bitcoin-core-dev
449 2018-10-20T14:03:32  *** iphelix has left #bitcoin-core-dev
450 2018-10-20T14:11:34  *** fanquake has quit IRC
451 2018-10-20T14:20:12  *** waxwing has quit IRC
452 2018-10-20T14:26:56  *** ott0disk has quit IRC
453 2018-10-20T15:01:16  *** bitcoin-git has joined #bitcoin-core-dev
454 2018-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
455 2018-10-20T15:01:17  *** bitcoin-git has left #bitcoin-core-dev
456 2018-10-20T15:05:08  *** harrymm has quit IRC
457 2018-10-20T15:05:09  *** laurentmt has joined #bitcoin-core-dev
458 2018-10-20T15:06:13  *** harrymm has joined #bitcoin-core-dev
459 2018-10-20T15:13:18  *** waxwing has joined #bitcoin-core-dev
460 2018-10-20T15:13:52  *** ula has joined #bitcoin-core-dev
461 2018-10-20T15:23:39  *** ula has quit IRC
462 2018-10-20T15:26:03  *** ula has joined #bitcoin-core-dev
463 2018-10-20T15:39:06  *** laurentmt has quit IRC
464 2018-10-20T15:42:15  *** bitconner has joined #bitcoin-core-dev
465 2018-10-20T15:46:28  *** bitconner has quit IRC
466 2018-10-20T15:48:32  *** mistergold has quit IRC
467 2018-10-20T15:52:01  *** rh0nj has quit IRC
468 2018-10-20T15:53:07  *** rh0nj has joined #bitcoin-core-dev
469 2018-10-20T16:20:01  *** irc_viewer_test has joined #bitcoin-core-dev
470 2018-10-20T16:21:36  *** Chris_Stewart_5 has quit IRC
471 2018-10-20T16:23:29  *** AaronvanW has quit IRC
472 2018-10-20T16:25:39  *** lnostdal has quit IRC
473 2018-10-20T16:25:53  *** lnostdal has joined #bitcoin-core-dev
474 2018-10-20T16:37:12  <kakobrekla> 
475 2018-10-20T16:41:40  *** grubles_ has joined #bitcoin-core-dev
476 2018-10-20T16:41:43  *** laurentmt has joined #bitcoin-core-dev
477 2018-10-20T16:42:39  *** AaronvanW has joined #bitcoin-core-dev
478 2018-10-20T16:42:58  *** grubles has quit IRC
479 2018-10-20T16:47:15  *** AaronvanW has quit IRC
480 2018-10-20T16:52:00  *** Krellan has quit IRC
481 2018-10-20T16:53:36  *** Krellan has joined #bitcoin-core-dev
482 2018-10-20T16:56:18  *** laurentmt has quit IRC
483 2018-10-20T16:59:18  <karelb> MarcoFalke thx for feedback on the RPC stuff, will reply tomorrow after I sleep on it
484 2018-10-20T17:35:55  *** Krellan has quit IRC
485 2018-10-20T17:36:49  *** AaronvanW has joined #bitcoin-core-dev
486 2018-10-20T17:41:09  *** AaronvanW has quit IRC
487 2018-10-20T17:53:45  *** spinza has quit IRC
488 2018-10-20T18:00:54  *** booyah has quit IRC
489 2018-10-20T18:02:12  *** booyah has joined #bitcoin-core-dev
490 2018-10-20T18:03:24  *** spinza has joined #bitcoin-core-dev
491 2018-10-20T18:05:58  *** owowo has quit IRC
492 2018-10-20T18:06:58  *** owowo has joined #bitcoin-core-dev
493 2018-10-20T18:16:18  *** irc_viewer_test has quit IRC
494 2018-10-20T18:19:06  *** esotericnonsense has quit IRC
495 2018-10-20T18:20:06  *** sipa has quit IRC
496 2018-10-20T18:20:33  *** esotericnonsense has joined #bitcoin-core-dev
497 2018-10-20T18:20:44  *** sipa has joined #bitcoin-core-dev
498 2018-10-20T18:29:05  *** mistergold has joined #bitcoin-core-dev
499 2018-10-20T18:50:25  *** SopaXorzTaker has quit IRC
500 2018-10-20T18:52:56  *** Chris_Stewart_5 has joined #bitcoin-core-dev
501 2018-10-20T18:56:39  *** gribble has quit IRC
502 2018-10-20T18:58:38  *** Lexman has joined #bitcoin-core-dev
503 2018-10-20T19:00:20  *** irc_viewer_test has joined #bitcoin-core-dev
504 2018-10-20T19:08:26  *** bitcoin-git has joined #bitcoin-core-dev
505 2018-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
506 2018-10-20T19:08:27  *** bitcoin-git has left #bitcoin-core-dev
507 2018-10-20T19:11:08  *** gribble has joined #bitcoin-core-dev
508 2018-10-20T19:11:43  *** shesek has quit IRC
509 2018-10-20T19:14:57  *** belcher_ has joined #bitcoin-core-dev
510 2018-10-20T19:16:55  *** irc_viewer_test has quit IRC
511 2018-10-20T19:17:41  *** hebasto has quit IRC
512 2018-10-20T19:18:30  *** reallll has quit IRC
513 2018-10-20T19:25:04  *** AaronvanW has joined #bitcoin-core-dev
514 2018-10-20T19:29:30  *** AaronvanW has quit IRC
515 2018-10-20T19:32:02  *** meshcollider_ has joined #bitcoin-core-dev
516 2018-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
517 2018-10-20T19:45:36  <phantomcircuit> wumpus, but that also means the next one might be a more fully baked thing
518 2018-10-20T19:47:04  *** mistergold has quit IRC
519 2018-10-20T19:54:17  *** mistergold has joined #bitcoin-core-dev
520 2018-10-20T19:54:26  *** bitcoin-git has joined #bitcoin-core-dev
521 2018-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
522 2018-10-20T19:54:26  *** bitcoin-git has left #bitcoin-core-dev
523 2018-10-20T20:11:42  *** waxwing_ has joined #bitcoin-core-dev
524 2018-10-20T20:11:52  *** waxwing has quit IRC
525 2018-10-20T20:12:30  *** dstadulis has joined #bitcoin-core-dev
526 2018-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
527 2018-10-20T20:14:57  *** waxwing_ has quit IRC
528 2018-10-20T20:16:44  *** waxwing has joined #bitcoin-core-dev
529 2018-10-20T20:17:13  *** waxwing has quit IRC
530 2018-10-20T20:17:13  *** waxwing has joined #bitcoin-core-dev
531 2018-10-20T20:18:46  <phantomcircuit> wumpus, i mean what does supported even mean for a dev board like that
532 2018-10-20T20:19:00  <phantomcircuit> usually you're lucky to get insanely hacked together kernel source
533 2018-10-20T20:19:04  *** AaronvanW has joined #bitcoin-core-dev
534 2018-10-20T20:19:49  <wumpus> welll fedora and debian distributions target it now because it's the only RISC-V board that supports Linux
535 2018-10-20T20:19:51  <phantomcircuit> how about a 2.6.x branch with random bits from 4.x in it?
536 2018-10-20T20:20:08  <phantomcircuit> which is no joke the norm for embeded type dev board things
537 2018-10-20T20:20:14  <wumpus> kernel is just a matter of time, they're working on getting everything upstreamed
538 2018-10-20T20:20:37  <phantomcircuit> yeah that's my point
539 2018-10-20T20:20:43  <phantomcircuit> it seems documented enough to actually do that
540 2018-10-20T20:20:46  <wumpus> they more or less did, got it to boot but had some problem with the network interface
541 2018-10-20T20:20:56  <wumpus> (with mainline kernel I mean)
542 2018-10-20T20:21:09  <wumpus> it's *certainly* not as bad as some ARM boards, as you say with ancient kernels
543 2018-10-20T20:23:45  *** AaronvanW has quit IRC
544 2018-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
545 2018-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
546 2018-10-20T20:24:26  <phantomcircuit> which im avoiding completely now
547 2018-10-20T20:24:41  <phantomcircuit> so im thinking we can use poll() on windows as long as it's not used on unconnected sockets
548 2018-10-20T20:24:45  <wumpus> using unconnected sockets as a sleep wut?!?
549 2018-10-20T20:26:09  <wumpus> well unconnected sockets shouldn't be emitting any events I guess
550 2018-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...
551 2018-10-20T20:27:00  *** grubles_ is now known as grubles
552 2018-10-20T20:28:58  *** shesek has joined #bitcoin-core-dev
553 2018-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
554 2018-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.
555 2018-10-20T20:33:02  <wumpus> exactly...
556 2018-10-20T20:33:55  *** Lexman has quit IRC
557 2018-10-20T20:34:34  <phantomcircuit> wumpus, no im saying the main select/poll loop doesn't have any sockets which haven't connected
558 2018-10-20T20:34:39  <phantomcircuit> we have another thing that does that
559 2018-10-20T20:34:47  <phantomcircuit> or maybe im misunderstanding the way this is all setup
560 2018-10-20T20:35:09  <phantomcircuit> the windows bug is about sockets where you call connect non-blocking and then select it
561 2018-10-20T20:35:21  *** laurentmt has joined #bitcoin-core-dev
562 2018-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
563 2018-10-20T20:36:40  <wumpus> introducing this into our own code means having to handle all who-knows-what OS dependent edge cases ourselves
564 2018-10-20T20:38:06  <phantomcircuit> wumpus, the first step to using libevent is to move to a callback style
565 2018-10-20T20:38:13  <phantomcircuit> which we cna do while still using the loop
566 2018-10-20T20:38:19  <phantomcircuit> (which is what libevent is doing internally)
567 2018-10-20T20:39:13  <wumpus> yes, cfields was working on this at some point
568 2018-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
569 2018-10-20T20:39:33  *** laurentmt has quit IRC
570 2018-10-20T20:39:43  <wumpus> I think it makes sense to focus on UNIXish OS and forget windows just leave that at select
571 2018-10-20T20:40:24  <wumpus> it seems like you're trying to extend the scope of this too much
572 2018-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
573 2018-10-20T20:41:49  <phantomcircuit> so i was thinking if i was going to do anything that would make that harder on accident
574 2018-10-20T20:47:10  *** Chris_Stewart_5 has quit IRC
575 2018-10-20T21:03:21  *** lnostdal has quit IRC
576 2018-10-20T21:13:09  *** AaronvanW has joined #bitcoin-core-dev
577 2018-10-20T21:13:46  *** dstadulis has quit IRC
578 2018-10-20T21:15:41  *** lnostdal has joined #bitcoin-core-dev
579 2018-10-20T21:17:39  *** AaronvanW has quit IRC
580 2018-10-20T21:20:15  *** bralyclow has joined #bitcoin-core-dev
581 2018-10-20T21:22:49  *** bralycl__ has quit IRC
582 2018-10-20T21:26:35  *** AaronvanW has joined #bitcoin-core-dev
583 2018-10-20T21:31:03  *** murrayn[m] has quit IRC
584 2018-10-20T21:32:39  *** Chris_Stewart_5 has joined #bitcoin-core-dev
585 2018-10-20T21:33:15  *** lnostdal has quit IRC
586 2018-10-20T21:33:39  *** murrayn[m] has joined #bitcoin-core-dev
587 2018-10-20T21:48:29  *** lnostdal has joined #bitcoin-core-dev
588 2018-10-20T21:57:00  *** bralyclo_ has joined #bitcoin-core-dev
589 2018-10-20T21:59:25  *** dviola has joined #bitcoin-core-dev
590 2018-10-20T21:59:27  *** bralyclow has quit IRC
591 2018-10-20T22:05:09  *** murrayn[m] has quit IRC
592 2018-10-20T22:06:24  *** robo7 has joined #bitcoin-core-dev
593 2018-10-20T22:14:22  *** murrayn[m] has joined #bitcoin-core-dev
594 2018-10-20T22:17:07  *** bitcoin-git has joined #bitcoin-core-dev
595 2018-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
596 2018-10-20T22:17:07  *** bitcoin-git has left #bitcoin-core-dev
597 2018-10-20T22:17:46  *** bralyclo_ has quit IRC
598 2018-10-20T22:21:37  *** bralyclow has joined #bitcoin-core-dev
599 2018-10-20T22:24:24  *** Zenton has quit IRC
600 2018-10-20T22:28:41  *** spinza has quit IRC
601 2018-10-20T22:33:25  *** spinza has joined #bitcoin-core-dev
602 2018-10-20T22:37:53  *** Guest20006 has quit IRC
603 2018-10-20T22:41:53  *** spinza has quit IRC
604 2018-10-20T22:46:27  *** brianhoffman has joined #bitcoin-core-dev
605 2018-10-20T22:46:47  *** Chris_Stewart_5 has quit IRC
606 2018-10-20T22:48:48  *** spinza has joined #bitcoin-core-dev
607 2018-10-20T22:53:18  *** dviola has quit IRC
608 2018-10-20T22:55:07  *** ula has quit IRC
609 2018-10-20T22:58:51  *** dviola has joined #bitcoin-core-dev
610 2018-10-20T23:01:48  *** meshcollider_ has quit IRC
611 2018-10-20T23:07:49  *** Guyver2 has quit IRC
612 2018-10-20T23:10:26  *** Zenton has joined #bitcoin-core-dev
613 2018-10-20T23:15:29  <booyah> btw, why not asio, instead of all this problems with pool/libevent and all
614 2018-10-20T23:16:38  <phantomcircuit> booyah, asio and libevent are both callback based
615 2018-10-20T23:16:46  <phantomcircuit> while the current logic is polling based
616 2018-10-20T23:17:23  <phantomcircuit> improving what's there is less error prone than switching the entire paradigm
617 2018-10-20T23:21:07  <sipa> one step at a time
618 2018-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
619 2018-10-20T23:27:00  <booyah> wumpus: what was afful there, in that rpc?
620 2018-10-20T23:29:28  *** pope has joined #bitcoin-core-dev
621 2018-10-20T23:45:53  *** pope has quit IRC
622 2018-10-20T23:48:06  *** mistergold has quit IRC
623 2018-10-20T23:55:20  *** AaronvanW has quit IRC