1 2016-09-28T00:14:06  *** Alopex has quit IRC
  2 2016-09-28T00:15:12  *** Alopex has joined #bitcoin-core-dev
  3 2016-09-28T00:22:45  *** aureianimus_ has quit IRC
  4 2016-09-28T00:22:54  *** aureianimus has joined #bitcoin-core-dev
  5 2016-09-28T00:45:20  *** Madars has quit IRC
  6 2016-09-28T00:47:00  *** Ylbam has quit IRC
  7 2016-09-28T00:52:51  *** arowser_ has quit IRC
  8 2016-09-28T00:53:18  *** arowser has joined #bitcoin-core-dev
  9 2016-09-28T00:56:32  *** DigiByteDev has joined #bitcoin-core-dev
 10 2016-09-28T01:10:59  *** Giszmo has quit IRC
 11 2016-09-28T01:11:26  *** aureianimus has quit IRC
 12 2016-09-28T01:11:31  *** aureianimus_ has joined #bitcoin-core-dev
 13 2016-09-28T01:26:57  *** aureianimus_ has quit IRC
 14 2016-09-28T01:27:11  *** aureianimus has joined #bitcoin-core-dev
 15 2016-09-28T01:33:11  *** Alopex has quit IRC
 16 2016-09-28T01:34:16  *** Alopex has joined #bitcoin-core-dev
 17 2016-09-28T01:43:14  *** DigiByteDev has quit IRC
 18 2016-09-28T01:50:01  *** aureianimus has quit IRC
 19 2016-09-28T01:50:05  *** aureianimus_ has joined #bitcoin-core-dev
 20 2016-09-28T02:01:03  *** aureianimus_ has quit IRC
 21 2016-09-28T02:01:12  *** aureianimus has joined #bitcoin-core-dev
 22 2016-09-28T02:03:36  *** DigiByteDev has joined #bitcoin-core-dev
 23 2016-09-28T02:17:48  *** jchrome has joined #bitcoin-core-dev
 24 2016-09-28T02:32:06  *** Alopex has quit IRC
 25 2016-09-28T02:32:42  *** aureianimus_ has joined #bitcoin-core-dev
 26 2016-09-28T02:32:53  *** aureianimus has quit IRC
 27 2016-09-28T02:33:11  *** Alopex has joined #bitcoin-core-dev
 28 2016-09-28T02:41:29  *** Madars has joined #bitcoin-core-dev
 29 2016-09-28T02:44:32  *** aureianimus_ has quit IRC
 30 2016-09-28T02:44:39  *** aureianimus has joined #bitcoin-core-dev
 31 2016-09-28T02:57:44  *** aureianimus has quit IRC
 32 2016-09-28T02:57:47  *** aureianimus_ has joined #bitcoin-core-dev
 33 2016-09-28T03:06:16  *** Alopex has quit IRC
 34 2016-09-28T03:07:21  *** Alopex has joined #bitcoin-core-dev
 35 2016-09-28T03:14:30  *** Arnavion has joined #bitcoin-core-dev
 36 2016-09-28T03:28:53  *** aureianimus_ has quit IRC
 37 2016-09-28T03:29:03  *** aureianimus has joined #bitcoin-core-dev
 38 2016-09-28T03:45:07  *** aureianimus has quit IRC
 39 2016-09-28T03:45:11  *** aureianimus_ has joined #bitcoin-core-dev
 40 2016-09-28T03:46:05  *** aureianimus has joined #bitcoin-core-dev
 41 2016-09-28T03:57:01  *** aureianimus_ has joined #bitcoin-core-dev
 42 2016-09-28T03:57:02  *** aureianimus has quit IRC
 43 2016-09-28T04:05:16  *** Alopex has quit IRC
 44 2016-09-28T04:06:22  *** Alopex has joined #bitcoin-core-dev
 45 2016-09-28T04:16:02  *** Alopex has quit IRC
 46 2016-09-28T04:17:07  *** Alopex has joined #bitcoin-core-dev
 47 2016-09-28T04:20:36  *** aureianimus_ has quit IRC
 48 2016-09-28T04:20:47  *** aureianimus has joined #bitcoin-core-dev
 49 2016-09-28T04:28:11  *** Alopex has quit IRC
 50 2016-09-28T04:29:17  *** Alopex has joined #bitcoin-core-dev
 51 2016-09-28T04:33:32  *** aureianimus has quit IRC
 52 2016-09-28T04:33:33  *** aureianimus_ has joined #bitcoin-core-dev
 53 2016-09-28T04:39:01  *** Alopex has quit IRC
 54 2016-09-28T04:40:07  *** Alopex has joined #bitcoin-core-dev
 55 2016-09-28T04:56:11  *** aureianimus has joined #bitcoin-core-dev
 56 2016-09-28T04:56:19  *** aureianimus_ has quit IRC
 57 2016-09-28T04:58:33  *** aj_ is now known as aj
 58 2016-09-28T05:00:18  *** DigiByteDev has quit IRC
 59 2016-09-28T05:00:18  *** dermoth has quit IRC
 60 2016-09-28T05:01:00  *** dermoth has joined #bitcoin-core-dev
 61 2016-09-28T05:01:37  *** DigiByteDev has joined #bitcoin-core-dev
 62 2016-09-28T05:05:42  *** morcos has quit IRC
 63 2016-09-28T05:05:43  *** zxzzt_ has quit IRC
 64 2016-09-28T05:05:43  *** sdaftuar has quit IRC
 65 2016-09-28T05:05:49  *** zxzzt has joined #bitcoin-core-dev
 66 2016-09-28T05:05:56  *** morcos has joined #bitcoin-core-dev
 67 2016-09-28T05:06:07  *** sdaftuar has joined #bitcoin-core-dev
 68 2016-09-28T05:13:51  *** aureianimus has quit IRC
 69 2016-09-28T05:14:02  *** aureianimus has joined #bitcoin-core-dev
 70 2016-09-28T05:24:16  *** fengling has joined #bitcoin-core-dev
 71 2016-09-28T05:29:11  *** Alopex has quit IRC
 72 2016-09-28T05:30:17  *** Alopex has joined #bitcoin-core-dev
 73 2016-09-28T05:33:03  *** baldur has joined #bitcoin-core-dev
 74 2016-09-28T05:34:59  *** aureianimus has quit IRC
 75 2016-09-28T05:35:10  *** aureianimus has joined #bitcoin-core-dev
 76 2016-09-28T05:38:51  *** jchrome has quit IRC
 77 2016-09-28T05:39:56  *** bitcoin-core-dev has joined #bitcoin-core-dev
 78 2016-09-28T05:46:22  *** Alopex has quit IRC
 79 2016-09-28T05:47:27  *** Alopex has joined #bitcoin-core-dev
 80 2016-09-28T05:50:39  *** aureianimus has quit IRC
 81 2016-09-28T05:50:51  *** aureianimus has joined #bitcoin-core-dev
 82 2016-09-28T06:10:09  <GitHub190> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/fc4f4547b7f6...14b7b3fb9f79
 83 2016-09-28T06:10:10  <GitHub190> bitcoin/master fa4bfb4 MarcoFalke: [wallet, policy] ParameterInteraction: Don't allow 0 fee
 84 2016-09-28T06:10:10  *** aureianimus has quit IRC
 85 2016-09-28T06:10:10  <GitHub190> bitcoin/master 14b7b3f Wladimir J. van der Laan: Merge #8814: [wallet, policy] ParameterInteraction: Don't allow 0 fee...
 86 2016-09-28T06:10:16  *** aureianimus has joined #bitcoin-core-dev
 87 2016-09-28T06:10:27  <GitHub50> [bitcoin] laanwj closed pull request #8814: [wallet, policy] ParameterInteraction: Don't allow 0 fee (master...Mf1607-walletHighFeeWarn) https://github.com/bitcoin/bitcoin/pull/8814
 88 2016-09-28T06:16:42  *** bitcoin-core-dev has quit IRC
 89 2016-09-28T06:38:23  *** jannes has joined #bitcoin-core-dev
 90 2016-09-28T06:41:09  *** aureianimus has quit IRC
 91 2016-09-28T06:41:16  *** aureianimus has joined #bitcoin-core-dev
 92 2016-09-28T06:45:08  *** jannes_ has joined #bitcoin-core-dev
 93 2016-09-28T06:47:01  *** jannes has quit IRC
 94 2016-09-28T06:52:18  *** aureianimus has quit IRC
 95 2016-09-28T06:52:26  *** aureianimus has joined #bitcoin-core-dev
 96 2016-09-28T06:53:32  *** aureianimus has joined #bitcoin-core-dev
 97 2016-09-28T07:00:39  *** DigiByteDev has quit IRC
 98 2016-09-28T07:06:00  *** aureianimus has quit IRC
 99 2016-09-28T07:06:11  *** aureianimus has joined #bitcoin-core-dev
100 2016-09-28T07:22:31  *** rubensayshi has joined #bitcoin-core-dev
101 2016-09-28T07:22:54  *** DigiByteDev has joined #bitcoin-core-dev
102 2016-09-28T07:34:03  *** rubensayshi has quit IRC
103 2016-09-28T07:35:41  *** rubensayshi has joined #bitcoin-core-dev
104 2016-09-28T07:47:39  *** aureianimus has quit IRC
105 2016-09-28T07:47:41  *** aureianimus_ has joined #bitcoin-core-dev
106 2016-09-28T08:08:47  *** aureianimus_ has quit IRC
107 2016-09-28T08:08:53  *** aureianimus has joined #bitcoin-core-dev
108 2016-09-28T08:09:26  *** laurentmt has joined #bitcoin-core-dev
109 2016-09-28T08:12:56  *** laurentmt has quit IRC
110 2016-09-28T08:29:55  *** aureianimus_ has joined #bitcoin-core-dev
111 2016-09-28T08:29:55  *** aureianimus has quit IRC
112 2016-09-28T08:38:24  *** phantomcircuit_ is now known as phantomcircuit
113 2016-09-28T08:40:46  *** aureianimus_ has quit IRC
114 2016-09-28T08:40:58  *** aureianimus has joined #bitcoin-core-dev
115 2016-09-28T08:48:49  *** AaronvanW has quit IRC
116 2016-09-28T08:52:07  *** AaronvanW has joined #bitcoin-core-dev
117 2016-09-28T08:55:42  *** aureianimus has quit IRC
118 2016-09-28T08:55:54  *** aureianimus has joined #bitcoin-core-dev
119 2016-09-28T09:22:47  *** aureianimus has quit IRC
120 2016-09-28T09:22:47  *** aureianimus_ has joined #bitcoin-core-dev
121 2016-09-28T09:33:33  *** aureianimus_ has quit IRC
122 2016-09-28T09:33:43  *** aureianimus has joined #bitcoin-core-dev
123 2016-09-28T09:37:01  *** Giszmo has joined #bitcoin-core-dev
124 2016-09-28T09:44:39  *** aureianimus has quit IRC
125 2016-09-28T09:44:44  *** aureianimus_ has joined #bitcoin-core-dev
126 2016-09-28T09:45:30  *** DigiByteDev has quit IRC
127 2016-09-28T09:50:47  *** Ginnarr has joined #bitcoin-core-dev
128 2016-09-28T09:58:26  <jl2012> when I'm working on rpctests, sometimes the node will generate empty blocks despite the mempool is not empty. It usually happens when the transactions in mempool are big. Why would that happen and how could I force it to mine those txs?
129 2016-09-28T09:59:11  <gmaxwell> will it continue to mine empty blocks, not just a one time event?
130 2016-09-28T10:02:43  <sipa> jl2012: perhaps an interaction between max block size an cpfp mining?
131 2016-09-28T10:03:31  <sipa> if you build a chain of dependent transactions that does not actually fit in a block
132 2016-09-28T10:04:09  <gmaxwell> and then there is nothing else to include other than that chain.
133 2016-09-28T10:04:53  <jl2012> all parents are confirmed. And keep mining empty blocks
134 2016-09-28T10:07:31  <jl2012> oh, not really keeping mining empty blocks forever
135 2016-09-28T10:08:44  <jl2012> there is only 1 tx in mempool, but it'd cleanup the mempool only if I generate about 500 blocks
136 2016-09-28T10:09:30  <luke-jr> fee too low, so it's waiting for priority?
137 2016-09-28T10:09:34  <sipa> anything particular about this tx?
138 2016-09-28T10:09:49  <jl2012> about 80kB
139 2016-09-28T10:09:53  *** DigiByteDev has joined #bitcoin-core-dev
140 2016-09-28T10:10:21  <jl2012> luke-jr: so it'd rather mining empty blocks, than mining a tx with insufficient fee?
141 2016-09-28T10:10:55  <luke-jr> jl2012: it shouldn't happen with the current code afaik, but that's what it sounds like if it's mining it after blocks
142 2016-09-28T10:10:59  <luke-jr> I assume there's no locktime?
143 2016-09-28T10:11:12  <jl2012> no locktime
144 2016-09-28T10:12:17  <sipa> so after 500 blocks, it does mine it?
145 2016-09-28T10:12:34  <jl2012> yes, about 400 blocks. I'm try to find the exact number
146 2016-09-28T10:13:33  <gmaxwell> "Interesting"
147 2016-09-28T10:14:18  *** DigiByteDev has quit IRC
148 2016-09-28T10:14:23  <sipa> is the height it activates a nice multiple of 144 after the activation of segwit?
149 2016-09-28T10:16:08  <jl2012> well, I find something more strange
150 2016-09-28T10:16:19  <jl2012> https://www.irccloud.com/pastebin/QiFCc6YN/
151 2016-09-28T10:16:43  <gmaxwell> whats that?
152 2016-09-28T10:17:07  <jl2012> I have a loop. Each round it generates 1 block, and print the results of getrawmempool
153 2016-09-28T10:17:21  *** aureianimus_ has quit IRC
154 2016-09-28T10:17:30  <luke-jr> so it's being pruned and then re-accepted? O.o
155 2016-09-28T10:17:32  *** aureianimus has joined #bitcoin-core-dev
156 2016-09-28T10:17:50  <jl2012> i will push the code, wait
157 2016-09-28T10:19:48  *** murch has joined #bitcoin-core-dev
158 2016-09-28T10:20:21  *** Ginnarr has quit IRC
159 2016-09-28T10:22:35  <jl2012> https://github.com/jl2012/bitcoin/blob/sighashpolicy-test/qa/rpc-tests/sighashlimit.py#L161
160 2016-09-28T10:24:28  <sipa> jl2012: heh
161 2016-09-28T10:26:04  <jl2012> it was it's just generate 1 block, print the mempool
162 2016-09-28T10:26:15  <jl2012> nothing strange
163 2016-09-28T10:27:33  <jl2012> from the output i assume it was mined at the 11th block, but it re-appeared after that
164 2016-09-28T10:27:48  <sipa> that's not possible...
165 2016-09-28T10:27:55  <sipa> or shouldn't be possible
166 2016-09-28T10:28:07  <sipa> its inputs are spent after mining it
167 2016-09-28T10:28:33  <jl2012> by the way the tx is not segwit
168 2016-09-28T10:29:07  <jl2012> maybe i should really see if it is really mined. But that's kind of random
169 2016-09-28T10:29:42  <sipa> you can call gettransaction to see its number of confirmations, if the transaction applies to the local wallet
170 2016-09-28T10:30:29  <jl2012> that's not. But I could read the content of blocks
171 2016-09-28T10:31:12  <gmaxwell> how do these tests mine?
172 2016-09-28T10:31:30  *** aureianimus has quit IRC
173 2016-09-28T10:31:32  <gmaxwell> IIRC getblocktemplate can cache templates.
174 2016-09-28T10:31:42  *** aureianimus has joined #bitcoin-core-dev
175 2016-09-28T10:32:16  <jl2012> gmaxwell, the mempool has only 1 tx, if the mempool is cleaned up after mining one block, the tx is not mined (i suppose)
176 2016-09-28T10:37:35  *** laurentmt has joined #bitcoin-core-dev
177 2016-09-28T10:37:55  <warren> btcdrak: wow
178 2016-09-28T10:38:23  *** laurentmt has quit IRC
179 2016-09-28T10:40:54  *** fengling has quit IRC
180 2016-09-28T10:42:01  <btcdrak> warren: I assume you are referring to this "<cfields_> wumpus: i've added thin toolchain builders to depends that rid us of the ubuntu toolchain dependency. Only needed once for bootstrap, then we're self-hosted"
181 2016-09-28T10:43:33  <warren> cfields_: can the same binary result be reached from another distro's toolchain?
182 2016-09-28T10:43:46  <phantomcircuit> luke-jr: there seems to be a bug in getblocktemplate
183 2016-09-28T10:43:55  <phantomcircuit> running master
184 2016-09-28T10:43:55  <phantomcircuit>   "coinbasevalue": 333038622,
185 2016-09-28T10:44:00  <phantomcircuit> 2016-09-28 10:43:08 ERROR: ConnectBlock(): coinbase pays too much (actual=332777360 vs limit=312500000)
186 2016-09-28T10:44:03  <phantomcircuit> (this is on testnet)
187 2016-09-28T10:44:14  <sipa> phantomcircuit: ugh!
188 2016-09-28T10:44:25  <phantomcircuit> or possibly im just not including transactions?
189 2016-09-28T10:44:27  <luke-jr> phantomcircuit: in or outside the context of jl2012's stuff?
190 2016-09-28T10:44:30  <phantomcircuit> hmm
191 2016-09-28T10:44:41  <phantomcircuit> this seems like im not including transactions actually
192 2016-09-28T10:44:56  <sipa> phantomcircuit: that would explain
193 2016-09-28T10:46:23  <phantomcircuit> im just guessing based on the difference
194 2016-09-28T10:47:09  <jl2012> sorry, seems just a bug in my code
195 2016-09-28T10:47:14  <luke-jr> >_<
196 2016-09-28T10:47:21  <jl2012> false alarm
197 2016-09-28T10:47:26  <sipa> jl2012: goox
198 2016-09-28T10:47:28  <sipa> good
199 2016-09-28T10:47:58  <jl2012> I have 2 nodes. I generated blocks in one, and the other one is not synced
200 2016-09-28T10:48:29  <jl2012> so sometimes it will generate a block with the new tx, but the block is then get orphaned
201 2016-09-28T10:48:35  <jl2012> and the tx is returned to the mempool
202 2016-09-28T10:54:23  <gmaxwell> jl2012: oh obvious. Good.
203 2016-09-28T10:55:44  *** cdecker has joined #bitcoin-core-dev
204 2016-09-28T11:06:03  *** midnightmagic has quit IRC
205 2016-09-28T11:10:11  *** fengling has joined #bitcoin-core-dev
206 2016-09-28T11:12:40  *** aureianimus has quit IRC
207 2016-09-28T11:12:41  *** midnightmagic has joined #bitcoin-core-dev
208 2016-09-28T11:12:47  *** aureianimus has joined #bitcoin-core-dev
209 2016-09-28T11:21:22  *** cryptapus has joined #bitcoin-core-dev
210 2016-09-28T11:21:22  *** cryptapus has joined #bitcoin-core-dev
211 2016-09-28T11:34:26  <wumpus> should I move/clone https://github.com/laanwj/bitcoin-maintainer-tools into bitcoin-core?
212 2016-09-28T11:41:31  <sipa> i wonder if we shouldn't try to integrate a buildinfo-less mode into the source tree itself
213 2016-09-28T11:42:02  <sipa> ./configure --disable-buildinfo
214 2016-09-28T11:44:18  *** aureianimus has quit IRC
215 2016-09-28T11:44:30  *** aureianimus has joined #bitcoin-core-dev
216 2016-09-28T11:52:21  <wumpus> maybe, but this is just an experiment for now
217 2016-09-28T11:53:10  <wumpus> I don't think that's an argument against the tool though?
218 2016-09-28T11:54:53  <sipa> sure, just making an unrelated comment
219 2016-09-28T11:55:27  <sipa> but in general of things are useful for the project jn general, i think they should be in repo
220 2016-09-28T11:55:38  <wumpus> having it outside the source tree has its advantages, it means easier experimentation, quicker iteration etc
221 2016-09-28T11:55:43  <wumpus> not everything having to go through series of ACK
222 2016-09-28T11:55:53  <wumpus> these are just my private tools anyhow
223 2016-09-28T11:56:19  <wumpus> I think I tend to disagree with that in practice, not everything has to be in the repo
224 2016-09-28T11:57:22  <wumpus> sure, if something is needed all the time like the github-merge script it makes sense to have it in the repo
225 2016-09-28T11:57:42  *** aureianimus has quit IRC
226 2016-09-28T11:57:50  *** aureianimus has joined #bitcoin-core-dev
227 2016-09-28T11:58:00  <wumpus> although even of that I have my own version, which I incidentally make improvements to, once there's enough I'll file a PR. But having to do that all the time gives a lot of overhead for everyone involved.
228 2016-09-28T11:58:50  <wumpus> I have some other scripts too, a few other code analysis tools, and to build release notes, author lists etc. All can be useful, but it's pretty much a meta-project to bitcoin itself.
229 2016-09-28T12:00:16  <wumpus> it isn't part of the bitcoin core release cycle either
230 2016-09-28T12:00:44  <wumpus> changes can happen any time, even between rcs, or between rc and final
231 2016-09-28T12:00:52  <sipa> fair enough
232 2016-09-28T12:01:04  <sipa> no objection to moving it to bitcoin-core bte
233 2016-09-28T12:01:06  <sipa> *btw
234 2016-09-28T12:01:53  <wumpus> thanks
235 2016-09-28T12:02:56  <wumpus> I also think we can just be more liberal with changes in a tools repo. If you think something is useful, just do it, no need for long review cycles etc.
236 2016-09-28T12:04:10  <sipa> right, agree
237 2016-09-28T12:04:12  <wumpus> (or maintaining a listof 130 PRs *despairs*)
238 2016-09-28T12:15:23  *** fengling has quit IRC
239 2016-09-28T12:18:53  *** aureianimus has quit IRC
240 2016-09-28T12:19:04  *** aureianimus has joined #bitcoin-core-dev
241 2016-09-28T12:30:10  *** aureianimus has quit IRC
242 2016-09-28T12:30:15  *** aureianimus_ has joined #bitcoin-core-dev
243 2016-09-28T12:38:44  *** Chris_Stewart_5 has joined #bitcoin-core-dev
244 2016-09-28T12:39:00  *** cryptapus_ has joined #bitcoin-core-dev
245 2016-09-28T12:42:41  *** cryptapus has quit IRC
246 2016-09-28T13:02:33  *** aureianimus has joined #bitcoin-core-dev
247 2016-09-28T13:02:49  *** aureianimus_ has quit IRC
248 2016-09-28T13:06:14  *** laurentmt has joined #bitcoin-core-dev
249 2016-09-28T13:10:03  *** laurentmt has quit IRC
250 2016-09-28T13:14:17  *** laurentmt has joined #bitcoin-core-dev
251 2016-09-28T13:16:44  *** laurentmt has quit IRC
252 2016-09-28T13:21:26  <jonasschnelli> BlueMatt: A nice... was looking through the code and couldn't find the Auth/MAC
253 2016-09-28T13:23:43  <BlueMatt> jonasschnelli: in UDPMessageHeader - chk1 and chk2
254 2016-09-28T13:24:23  <BlueMatt> those are just the hash of the message = the magic (yea, yea, not how you're supposed to do a MAC....whatever)
255 2016-09-28T13:24:29  <BlueMatt> s/=/+/
256 2016-09-28T13:25:09  * jonasschnelli is checking the code
257 2016-09-28T13:25:49  <BlueMatt> see FillChecksum
258 2016-09-28T13:26:03  <jonasschnelli> need to checkout first,... github not searchable on forks. :(
259 2016-09-28T13:26:28  <BlueMatt> its all in one file, you can just read that :p
260 2016-09-28T13:27:27  <jonasschnelli> wasn't aware of udpnet.cpp :)
261 2016-09-28T13:27:33  <jonasschnelli> FillChecksum(std::get<3>(msg), std::get<1>(msg), std::get<2>(msg));
262 2016-09-28T13:27:49  <jonasschnelli> std::get<3>(msg) is a key or something?
263 2016-09-28T13:28:02  <sipa> it gets the 3rd element of a tuple
264 2016-09-28T13:28:29  <jonasschnelli> Yes.
265 2016-09-28T13:28:34  <jonasschnelli> its declared as a uint64_t
266 2016-09-28T13:28:43  <jonasschnelli> Wonder how it gets popuplated
267 2016-09-28T13:30:29  <jonasschnelli> Or is FillChecksum(uint64_t magic, UDPMessage& msg, const unsigned int length) similar to our TCP p2p message packeges where magic is the network magic?
268 2016-09-28T13:33:41  <jonasschnelli> Okay. Got it...
269 2016-09-28T13:33:54  <jonasschnelli> UDPConnectionInfo.local_magic gets populated through addudpnode
270 2016-09-28T13:34:06  <jonasschnelli> Yes. Thats a nice simple auth
271 2016-09-28T13:34:41  *** Chris_Stewart_5 has quit IRC
272 2016-09-28T13:41:47  <wumpus> yes
273 2016-09-28T13:45:57  *** aureianimus has quit IRC
274 2016-09-28T13:46:01  *** aureianimus_ has joined #bitcoin-core-dev
275 2016-09-28T13:48:45  *** Chris_Stewart_5 has joined #bitcoin-core-dev
276 2016-09-28T13:58:55  *** Guyver2 has joined #bitcoin-core-dev
277 2016-09-28T14:06:09  <jonasschnelli> Maybe I missed something: whats the reason why importmulti requires a scriptPubKey https://github.com/bitcoin/bitcoin/pull/7551/files#diff-522490d83dce5375d423b23886e4125eR979?
278 2016-09-28T14:06:22  <jonasschnelli> Can't I just import pubkeys?
279 2016-09-28T14:06:47  <GitHub61> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/14b7b3fb9f79...7b05af634be4
280 2016-09-28T14:06:47  <GitHub61> bitcoin/master ac01ff2 Wladimir J. van der Laan: doc: Add privacy recommendation when running hidden service
281 2016-09-28T14:06:48  <GitHub61> bitcoin/master 7b05af6 Wladimir J. van der Laan: Merge #8823: doc: Add privacy recommendation when running hidden service...
282 2016-09-28T14:07:01  <GitHub121> [bitcoin] laanwj closed pull request #8823: doc: Add privacy recommendation when running hidden service (master...2016_09_tor_recommendation) https://github.com/bitcoin/bitcoin/pull/8823
283 2016-09-28T14:07:16  <sipa> jonasschnelli: yes, you can, but it's hard to create an API that always has expected results
284 2016-09-28T14:07:41  <sipa> we have so many different ways through which we can start treating a particular script as ours
285 2016-09-28T14:07:47  <sipa> maybe it's an imported address
286 2016-09-28T14:08:06  <sipa> maybe it is p2sh for a redeemscript that contains something we know
287 2016-09-28T14:08:11  <sipa> maybe we have the private key
288 2016-09-28T14:08:16  <sipa> maybe we have a pubkey
289 2016-09-28T14:08:44  <sipa> i think it's impossible to explain all those conditions
290 2016-09-28T14:08:50  <sipa> so what does 'importing a pubkey' mean
291 2016-09-28T14:09:23  <jonasschnelli> Yes. I see your point.
292 2016-09-28T14:09:24  <sipa> should it make sends to that pubkey spendable? or also p2pkh involving those keys? what about a 1-of-3 multisig that involves those keys?
293 2016-09-28T14:09:37  <sipa> by forcing the user to state exactly what they want, you avoid that
294 2016-09-28T14:09:51  <sipa> and you can check whether you can provide the functionality they're asking for
295 2016-09-28T14:10:20  <jonasschnelli> But explicit requiring the scriptPubKey could be a little bit annoying if you just want to import a P2PKH key?!
296 2016-09-28T14:10:42  <sipa> it's a compromise between convenience and clarity, yes
297 2016-09-28T14:11:25  <jonasschnelli> A wait... there is an "address" option for the scriptPubKey... okay. let me continue the testing..
298 2016-09-28T14:11:33  <jonasschnelli> Makes sense so far.
299 2016-09-28T14:11:36  <sipa> also, if you just want a p2pkh, you should import just that address, and set watchonly
300 2016-09-28T14:11:58  <jonasschnelli> Agree. A plain pubkey import is kind of unspecified.
301 2016-09-28T14:12:39  <sipa> the idea is that 1) you state what you want to see made spendable/solvable/watching, and then 2) all necessary information to accomplish that (which may mean giving redeemscripts, or pubkeys, or private keys)
302 2016-09-28T14:12:55  <sipa> and in the simplest cases, (2) is empty
303 2016-09-28T14:17:50  *** aureianimus_ has quit IRC
304 2016-09-28T14:18:01  *** aureianimus has joined #bitcoin-core-dev
305 2016-09-28T14:35:08  *** instagibbs has quit IRC
306 2016-09-28T14:42:29  *** aureianimus_ has joined #bitcoin-core-dev
307 2016-09-28T14:42:34  *** cryptapus_ has quit IRC
308 2016-09-28T14:42:38  <GitHub190> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/7b05af634be4...dc641415e75e
309 2016-09-28T14:42:38  <GitHub190> bitcoin/master 5776e8a fanquake: [depends] Fix Qt compilation with Xcode 8
310 2016-09-28T14:42:39  <GitHub190> bitcoin/master dc64141 Wladimir J. van der Laan: Merge #8820: [depends] Fix Qt compilation with Xcode 8...
311 2016-09-28T14:42:44  *** aureianimus has quit IRC
312 2016-09-28T14:42:53  <GitHub22> [bitcoin] laanwj closed pull request #8820: [depends] Fix Qt compilation with Xcode 8 (master...depends-qt-xcoderun) https://github.com/bitcoin/bitcoin/pull/8820
313 2016-09-28T14:42:54  *** cryptapus_ has joined #bitcoin-core-dev
314 2016-09-28T14:42:55  *** cryptapus_ has joined #bitcoin-core-dev
315 2016-09-28T14:42:58  <GitHub163> [bitcoin] paveljanik opened pull request #8826: Do not include env_win.cc on non-Windows systems (master...20160928_leveldb_no_win) https://github.com/bitcoin/bitcoin/pull/8826
316 2016-09-28T14:44:01  *** Guyver2 has quit IRC
317 2016-09-28T14:44:33  <jonasschnelli> wumpus: IMO this is ready: https://github.com/bitcoin/bitcoin/pull/7551
318 2016-09-28T14:44:59  <jonasschnelli> Maybe the help text is not perfect...
319 2016-09-28T14:45:03  <wumpus> awesome
320 2016-09-28T14:45:08  <wumpus> well, that can be improved later
321 2016-09-28T14:45:12  <sipa> i'd like to review 7551 as well, but haven't found the time yet
322 2016-09-28T14:46:39  <wumpus> ok
323 2016-09-28T14:46:40  <jonasschnelli> Yes. No hurry.
324 2016-09-28T14:47:01  <jonasschnelli> We should just make sure we merge is some month before 0.14 RC
325 2016-09-28T14:47:20  <sipa> absolutely
326 2016-09-28T14:47:22  <wumpus> yes, it is functionality that certainly needs to be in master for a while before the release
327 2016-09-28T14:47:35  <wumpus> it's complex enough, I'm sure there are problems that will only be found in actual use
328 2016-09-28T14:47:58  <jonasschnelli> Yes. Its relatively complex and could lead to coins lost in the worst case.
329 2016-09-28T14:49:00  *** cryptapus_ is now known as cryptapus
330 2016-09-28T14:49:18  *** jnewbery has joined #bitcoin-core-dev
331 2016-09-28T14:51:01  <wumpus> we reallly need it though. After a while we can then start to deprecate the whole zoo of other import calls.
332 2016-09-28T14:58:29  *** pedrobranco has joined #bitcoin-core-dev
333 2016-09-28T15:05:32  *** jnewbery has quit IRC
334 2016-09-28T15:06:07  *** jnewbery has joined #bitcoin-core-dev
335 2016-09-28T15:10:21  *** jnewbery has quit IRC
336 2016-09-28T15:13:21  *** aureianimus_ has quit IRC
337 2016-09-28T15:13:26  *** aureianimus has joined #bitcoin-core-dev
338 2016-09-28T15:19:46  *** rubensayshi has quit IRC
339 2016-09-28T15:29:19  *** aureianimus has quit IRC
340 2016-09-28T15:29:22  *** aureianimus_ has joined #bitcoin-core-dev
341 2016-09-28T15:31:45  *** mrkent has joined #bitcoin-core-dev
342 2016-09-28T15:42:59  *** pedrobranco has quit IRC
343 2016-09-28T15:43:28  *** pedrobranco has joined #bitcoin-core-dev
344 2016-09-28T15:44:52  *** jnewbery has joined #bitcoin-core-dev
345 2016-09-28T15:47:15  *** pedrobranco has quit IRC
346 2016-09-28T15:47:23  *** pedrobranco has joined #bitcoin-core-dev
347 2016-09-28T15:58:24  <GitHub167> [bitcoin] sdaftuar opened pull request #8827: [qa] Split up slow RPC calls to avoid pruning test timeouts (master...fix-pruning-timeout) https://github.com/bitcoin/bitcoin/pull/8827
348 2016-09-28T16:00:30  *** Chris_Stewart_5 has quit IRC
349 2016-09-28T16:01:00  *** aureianimus has joined #bitcoin-core-dev
350 2016-09-28T16:01:14  *** aureianimus_ has quit IRC
351 2016-09-28T16:02:47  <GitHub163> [bitcoin] pstratem opened pull request #8828: Move CWalletDB::ReorderTransactions to CWallet (master...2016-09-28-cwallet-reordertransactions) https://github.com/bitcoin/bitcoin/pull/8828
352 2016-09-28T16:06:45  *** shesek has quit IRC
353 2016-09-28T16:10:58  *** shesek has joined #bitcoin-core-dev
354 2016-09-28T16:11:35  *** andytosh1 has joined #bitcoin-core-dev
355 2016-09-28T16:17:06  *** Chris_Stewart_5 has joined #bitcoin-core-dev
356 2016-09-28T16:19:00  *** jannes_ has quit IRC
357 2016-09-28T16:25:21  *** Chris_Stewart_5 has quit IRC
358 2016-09-28T16:28:52  *** andytosh1 is now known as andytoshi
359 2016-09-28T16:29:46  *** jnewbery has quit IRC
360 2016-09-28T16:39:52  *** Chris_Stewart_5 has joined #bitcoin-core-dev
361 2016-09-28T16:50:36  *** andytoshi has quit IRC
362 2016-09-28T16:50:36  *** andytoshi has joined #bitcoin-core-dev
363 2016-09-28T17:00:22  *** jnewbery has joined #bitcoin-core-dev
364 2016-09-28T17:04:59  *** aureianimus has quit IRC
365 2016-09-28T17:05:04  *** aureianimus_ has joined #bitcoin-core-dev
366 2016-09-28T17:07:21  *** Chris_Stewart_5 has quit IRC
367 2016-09-28T17:17:21  *** aureianimus_ has quit IRC
368 2016-09-28T17:17:26  *** aureianimus has joined #bitcoin-core-dev
369 2016-09-28T17:22:30  *** mrkent has quit IRC
370 2016-09-28T17:24:04  *** Chris_Stewart_5 has joined #bitcoin-core-dev
371 2016-09-28T17:24:59  *** mrkent has joined #bitcoin-core-dev
372 2016-09-28T17:26:08  *** Ylbam has joined #bitcoin-core-dev
373 2016-09-28T17:28:21  *** aureianimus has quit IRC
374 2016-09-28T17:28:29  *** aureianimus has joined #bitcoin-core-dev
375 2016-09-28T17:29:35  *** aureianimus has joined #bitcoin-core-dev
376 2016-09-28T17:31:07  *** aureianimus has quit IRC
377 2016-09-28T17:31:19  *** aureianimus has joined #bitcoin-core-dev
378 2016-09-28T17:39:43  *** Guyver2 has joined #bitcoin-core-dev
379 2016-09-28T17:40:14  <GitHub184> [bitcoin] jnewbery opened pull request #8829: Add bitcoin-tx JSON tests (master...test-bitcoin-tx-json) https://github.com/bitcoin/bitcoin/pull/8829
380 2016-09-28T17:42:07  *** aureianimus has quit IRC
381 2016-09-28T17:42:19  *** aureianimus has joined #bitcoin-core-dev
382 2016-09-28T17:49:27  *** shesek has quit IRC
383 2016-09-28T17:52:51  <GitHub21> [bitcoin] jnewbery opened pull request #8830: [test] Add option to run bitcoin-util-test.py manually (master...test-bitcoin-util-manually) https://github.com/bitcoin/bitcoin/pull/8830
384 2016-09-28T17:53:35  *** shesek has joined #bitcoin-core-dev
385 2016-09-28T17:54:05  <pedrobranco> Hi, anyone is experiencing high disk space usage when running bitcoin tests (env. is osx)?
386 2016-09-28T17:55:26  *** laurentmt has joined #bitcoin-core-dev
387 2016-09-28T17:55:48  <wumpus> what is 'high disk space usage' these days?
388 2016-09-28T17:57:04  *** laurentmt has quit IRC
389 2016-09-28T17:58:20  <pedrobranco> 62gb in temporary test folder
390 2016-09-28T17:58:33  *** Chris_Stewart_5 has quit IRC
391 2016-09-28T17:59:28  <wumpus> for the RPC tests I guess, not the unit tests?
392 2016-09-28T18:00:16  <wumpus> does it at least clean up everything again? the RPC tests have recently been changed to execute tests in parallel, this also means that more disk space is required for the instances running at the same time
393 2016-09-28T18:00:44  *** aureianimus has quit IRC
394 2016-09-28T18:00:45  <pedrobranco> rpc tests only
395 2016-09-28T18:00:54  *** aureianimus has joined #bitcoin-core-dev
396 2016-09-28T18:01:30  <wumpus> you can reduce the parallelism with some command line option if it's too much
397 2016-09-28T18:01:51  <pedrobranco> i have 80G free space and it uses it all before ending rpc tests.
398 2016-09-28T18:02:13  <gmaxwell> 62GB sounds like a lot, do the rpc tests really do that? seems implausable to me.
399 2016-09-28T18:02:21  <pedrobranco> http://pastebin.com/ZDzxBLNF
400 2016-09-28T18:05:18  *** Chris_Stewart_5 has joined #bitcoin-core-dev
401 2016-09-28T18:05:46  <wumpus> try rpc-tests.py -parallel=1
402 2016-09-28T18:06:01  <pedrobranco> sure
403 2016-09-28T18:06:57  <GitHub87> [bitcoin] pstratem opened pull request #8831: Replace CWalletDB::ReadKeyValue with CWallet::LoadKeyValue (master...2016-09-28-cwallet-loadkeyvalue) https://github.com/bitcoin/bitcoin/pull/8831
404 2016-09-28T18:08:24  <wumpus> I also doubt it uses tht much in /tmp though, I only have 22GB free there and never had issues
405 2016-09-28T18:09:13  <wumpus> not sure how much the travis VMs have but it should be a comparable amount, not  a huge lot
406 2016-09-28T18:10:54  <phantomcircuit> can someone check the travis issue on #8828 ?
407 2016-09-28T18:10:57  <phantomcircuit> tests pass locally
408 2016-09-28T18:11:02  <wumpus> or are these the 'extended tests'?
409 2016-09-28T18:13:38  <phantomcircuit> oh wait they dont
410 2016-09-28T18:13:38  <phantomcircuit> huh
411 2016-09-28T18:13:51  <phantomcircuit> they *did*
412 2016-09-28T18:13:51  *** MarcoFalke has joined #bitcoin-core-dev
413 2016-09-28T18:14:20  <pedrobranco> with rpc-tests.py -parallel=1 i'm having the same issue. p2p-fullblocktest.py is creating the 65G folder
414 2016-09-28T18:14:49  <pedrobranco> and fails on the test #133
415 2016-09-28T18:15:19  <wumpus> which file is so big?
416 2016-09-28T18:15:41  <wumpus> in your pastebin there's 65G test3jvyn0m1/45 and everything below it is *much* smaller
417 2016-09-28T18:15:54  <wumpus> so it must be a file in the top levele dir
418 2016-09-28T18:17:22  <pedrobranco> to continue the tests i had to removed the folder, but i will repeat to show a full disk usage.
419 2016-09-28T18:17:36  <pedrobranco> *remove
420 2016-09-28T18:21:36  <pedrobranco> http://pastebin.com/UAYvF1cY
421 2016-09-28T18:21:50  <pedrobranco> still running tests...
422 2016-09-28T18:25:03  <arubi> pedrobranco, out of curiosity, can you tell which test?  run `top`?  should be part of the command line
423 2016-09-28T18:25:28  <wumpus> p2p-fullblocktest
424 2016-09-28T18:25:37  <pedrobranco> ^
425 2016-09-28T18:25:56  <arubi> oh, I thought it was /still/ running, sorry
426 2016-09-28T18:26:04  <pedrobranco> np
427 2016-09-28T18:26:05  <wumpus> I had the same error on openbsd, but that was due to linking a crappy ndb implementation that only supports small files
428 2016-09-28T18:27:28  <wumpus> test is still running here, disk usage looks fairly stable around ~2GB, at subtest 94 now
429 2016-09-28T18:27:54  *** mrkent has quit IRC
430 2016-09-28T18:28:05  <wumpus> so I still wonder what file is growing so big there - I guess blocks.db?
431 2016-09-28T18:28:13  <wumpus> (as it's above the node directories)
432 2016-09-28T18:29:08  <pedrobranco> http://pastebin.com/1cawnCMJ
433 2016-09-28T18:29:29  <pedrobranco> i will now run p2p-fullblocktest only
434 2016-09-28T18:30:55  <wumpus> final size is 2.1G    /tmp/testyxoclyo1/14241 for p2p-fullblocktest.py -nocleanup
435 2016-09-28T18:32:01  <wumpus> blocks.db ends up at 1.1G, the rest goes to the node0 dir. So it must be something local there that causes some file to balloon, do you have any changes compared to master?
436 2016-09-28T18:32:01  *** aureianimus has quit IRC
437 2016-09-28T18:32:01  <jonasschnelli> I hope libevent does a 2.1.x release before our 0.14 (https://github.com/libevent/libevent/releases/)
438 2016-09-28T18:32:03  *** aureianimus_ has joined #bitcoin-core-dev
439 2016-09-28T18:32:28  <wumpus> jonasschnelli: they have done 2.1.x releases, they're just in the beta phase
440 2016-09-28T18:32:57  <jonasschnelli> Yes. I meant a stable one... so package managers like brew (on OSX) can pick this up
441 2016-09-28T18:33:12  <wumpus> (finally, since last year or so, 2.1 has been in dev/alpha since 2009 or so...)
442 2016-09-28T18:33:32  <pedrobranco> it is /var/folders/kt/t6rd1zxx7px8273wlb56c4340000gn/T/testo0lm0oh1/0/blocks.db that has 65G
443 2016-09-28T18:33:59  <wumpus> pedrobranco: interesting. What OS, what version of python do you have?
444 2016-09-28T18:35:28  <wumpus> what does 'import dbm.ndbm; print(dbm.ndbm.library)' in python3 print?
445 2016-09-28T18:35:54  <pedrobranco> OSX 10.11.6 with xcode 7.3.1 , python -V = 2.7.12, python3 -V = 3.5.2
446 2016-09-28T18:35:58  <pedrobranco> let me see
447 2016-09-28T18:36:16  <pedrobranco> prints 'GNU gdbm'
448 2016-09-28T18:36:20  <wumpus> uh oh
449 2016-09-28T18:36:56  <wumpus> if it prints that on a BSD it probably means it is linked against the system's ndbm, BSDs have a ndbm implementation in their libc
450 2016-09-28T18:37:25  <wumpus> in openbsd that's a very silly one that can handle up to 64kb files or so :) but on OSX probably different
451 2016-09-28T18:37:36  <wumpus> let me find something to try for you
452 2016-09-28T18:38:18  <pedrobranco> thanks for the help wumpus, but i need to leave now. i'll be back in a couple of hours.
453 2016-09-28T18:39:04  <wumpus> pedrobranco:  see https://github.com/bitcoin/bitcoin/issues/8605 , that's my openbsd problem, if you replace dbm.ndbm with dbm.dumb it will probably work fine
454 2016-09-28T18:39:20  *** pedrobranco has quit IRC
455 2016-09-28T18:39:22  <wumpus> (in qa/rpc-tests/test-framework/blockstore.py)
456 2016-09-28T18:39:45  *** pedrobranco has joined #bitcoin-core-dev
457 2016-09-28T18:40:15  <wumpus> jonasschnelli: you're on OSX right? did you ever notice p2p-fullblocktest using exorbitant disk space?
458 2016-09-28T18:40:27  <jonasschnelli> I can't remember when i last ran this test...
459 2016-09-28T18:40:30  <jonasschnelli> let me try
460 2016-09-28T18:40:43  <jonasschnelli> (currently compiling, will start in a min)
461 2016-09-28T18:40:51  <wumpus> well it's part of the default pulltester run
462 2016-09-28T18:41:29  <jonasschnelli> Yes. Never noticed big data dirs...
463 2016-09-28T18:41:36  <jonasschnelli> But I have a 1GB SSD..
464 2016-09-28T18:41:41  <jonasschnelli> TB
465 2016-09-28T18:41:43  <jonasschnelli> :-)
466 2016-09-28T18:42:25  * jonasschnelli started ./qa/rpc-tests/p2p-fullblocktest.py --nocleanup
467 2016-09-28T18:42:26  <wumpus> BTW bitcoin core on FreeBSD nearly works out of the box with ports-provided development packages, not nearly the struggle it is on OpenBSD. Though I don't think I've run the RPC tests yet :)
468 2016-09-28T18:42:44  *** pedrobra_ has joined #bitcoin-core-dev
469 2016-09-28T18:43:12  *** aureianimus_ has quit IRC
470 2016-09-28T18:43:16  <jonasschnelli> oh
471 2016-09-28T18:43:17  <jonasschnelli> Unexpected exception caught during testing: error('cannot add item to database',)
472 2016-09-28T18:43:17  *** aureianimus has joined #bitcoin-core-dev
473 2016-09-28T18:43:35  <wumpus> hah there goes your 1TB SSD? :-)
474 2016-09-28T18:43:36  <jonasschnelli> Got the exception after Test 94: PASS [132]
475 2016-09-28T18:43:45  <jonasschnelli> du -sh /var/folders/hp/kb9p9q8x4k3_z_ccy588hxrc0000gn/T/testhuweo6jb/15776
476 2016-09-28T18:43:45  <jonasschnelli>  25M	/var/folders/hp/kb9p9q8x4k3_z_ccy588hxrc0000gn/T/testhuweo6jb/15776
477 2016-09-28T18:43:54  <jonasschnelli> 25MV
478 2016-09-28T18:43:55  <jonasschnelli> MB
479 2016-09-28T18:44:02  <jonasschnelli> Oh.. wait... 128GB
480 2016-09-28T18:44:17  <jonasschnelli> Yes. Same issue. :)
481 2016-09-28T18:44:22  <wumpus> :)
482 2016-09-28T18:44:42  *** pedrobranco has quit IRC
483 2016-09-28T18:45:03  <wumpus> can you try the solution of https://github.com/bitcoin/bitcoin/issues/8605 ? e.g.  replace dbm.ndbm with dbm.dumb in qa/rpc-tests/test-framework/blockstore.py
484 2016-09-28T18:45:18  * jonasschnelli pulling
485 2016-09-28T18:46:42  *** pedrobra_ has quit IRC
486 2016-09-28T18:48:36  <jonasschnelli> running with dbm.dump now
487 2016-09-28T18:49:29  <jonasschnelli> again 128GB
488 2016-09-28T18:49:44  <jonasschnelli> and the execp.
489 2016-09-28T18:51:28  <wumpus> that's very strange. dbm.dumb should be a native python implementation if dbm, not affected by the OS
490 2016-09-28T18:52:30  * jonasschnelli doublechecking
491 2016-09-28T18:53:20  <jonasschnelli> Yes. I did change to "import dbm.dumb" in qa/rpc-tests/test_framework/blockstore.py
492 2016-09-28T18:53:22  <wumpus> note that there are three occurences of "dbm.nbdm" in that file
493 2016-09-28T18:53:32  <jonasschnelli> ah..
494 2016-09-28T18:53:47  <luke-jr> uh, the Python tests are trying to use BDB directly? that's unlikely to be 4.8 even on Linux, right? O.o
495 2016-09-28T18:54:18  <wumpus> luke-jr: that doesn't matter, they're not used to communicate with bitcoind nor used for a wallet
496 2016-09-28T18:54:35  <luke-jr> oh
497 2016-09-28T18:54:38  <wumpus> and no, they don't request bdb, they request *a* ndbm implementation, which happens to be bdb on linux
498 2016-09-28T18:54:59  <wumpus> but on BSDs its the libc ndm implementation, which differs
499 2016-09-28T18:55:14  <wumpus> anyhow please read back, I feel I'm repeating myself
500 2016-09-28T18:55:51  <wumpus> jonasschnelli: same problem on freebsd (without .dumb patch) - the file becomes huge
501 2016-09-28T18:56:03  <jonasschnelli> 1GB now...
502 2016-09-28T18:56:15  <jonasschnelli> (during pass 94)
503 2016-09-28T18:56:20  <jonasschnelli> test 94
504 2016-09-28T18:56:24  <wumpus> that's normal size
505 2016-09-28T18:57:20  <jonasschnelli> I don't have the insight here. Is dbm BDB? Whats the difference between dumb and ndbm?
506 2016-09-28T18:57:25  <jonasschnelli> (maybe I should read back)
507 2016-09-28T18:59:17  <luke-jr> jonasschnelli: completely different implementation of a key-value database afaik (and different format ofc)
508 2016-09-28T18:59:25  *** h1d has joined #bitcoin-core-dev
509 2016-09-28T18:59:27  <jonasschnelli> https://docs.python.org/dev/library/dbm.html
510 2016-09-28T18:59:42  <wumpus> they're all key-value stores, which offer the same high-level interface but differ under the hood
511 2016-09-28T18:59:50  <jonasschnelli> Is the dumb module much slower? If no, why not use dump for the tests?
512 2016-09-28T19:00:06  <jonasschnelli> *why not use dumb
513 2016-09-28T19:00:07  <wumpus> I think dumb is much slower, that is what the manual says at least
514 2016-09-28T19:00:10  <wumpus> I haven't *tried*
515 2016-09-28T19:00:14  <wumpus> but it seems it's the safe choice
516 2016-09-28T19:00:36  <jonasschnelli> Theres also a gnu version...
517 2016-09-28T19:00:48  <luke-jr> gnu is GPL, but I'm not sure we care for Python tests
518 2016-09-28T19:01:06  <wumpus> gah, we're not going to include it in our source tree
519 2016-09-28T19:01:15  <wumpus> although BSDs generally don't haev that installed by default
520 2016-09-28T19:01:24  <wumpus> but dumb works fine as a fallback
521 2016-09-28T19:01:41  <wumpus> I think we should default to the current db for Linux, and on all BSD variants use dumb
522 2016-09-28T19:01:54  <wumpus> or use dumb everywhere if the perfrmance is acceptable for this use
523 2016-09-28T19:02:35  <jonasschnelli> Can you easy switch between dumb/db based on the platform in python? Probably yes.
524 2016-09-28T19:02:37  <GitHub123> [bitcoin] MarcoFalke opened pull request #8832: [rpc] throw JSONRPCError when utxo set can not be read (master...Mf1610-rpcUtxoFail) https://github.com/bitcoin/bitcoin/pull/8832
525 2016-09-28T19:02:39  * luke-jr wonders how badly things would blow up if we used a map :p
526 2016-09-28T19:03:05  <wumpus> jonasschnelli: yes, they are just modules that have the same interface
527 2016-09-28T19:03:16  <wumpus> easy enough to import as <something>
528 2016-09-28T19:04:10  *** aureianimus has quit IRC
529 2016-09-28T19:04:20  *** aureianimus has joined #bitcoin-core-dev
530 2016-09-28T19:07:45  <wumpus> updated the issue https://github.com/bitcoin/bitcoin/issues/8605
531 2016-09-28T19:08:06  *** aureianimus has quit IRC
532 2016-09-28T19:08:18  *** aureianimus has joined #bitcoin-core-dev
533 2016-09-28T19:12:29  <jonasschnelli> wumpus: re UTXO set streaming over rest
534 2016-09-28T19:12:36  <jonasschnelli> I get an assertion:
535 2016-09-28T19:12:37  <jonasschnelli> Assertion failed: (valid_), function key, file leveldb/db/db_iter.cc, line 67.
536 2016-09-28T19:13:18  <wumpus> jonasschnelli: ok, maybe someting broke in the rebase, haven't looked at it much, will look at that later
537 2016-09-28T19:13:32  <jonasschnelli> Thanks...
538 2016-09-28T19:19:15  *** shesek has quit IRC
539 2016-09-28T19:23:29  *** jnewbery has quit IRC
540 2016-09-28T19:27:18  *** jnewbery has joined #bitcoin-core-dev
541 2016-09-28T19:29:09  *** aureianimus has quit IRC
542 2016-09-28T19:29:16  *** aureianimus has joined #bitcoin-core-dev
543 2016-09-28T19:29:22  *** shesek has joined #bitcoin-core-dev
544 2016-09-28T19:30:47  *** droark has joined #bitcoin-core-dev
545 2016-09-28T19:36:30  *** Chris_Stewart_5 has quit IRC
546 2016-09-28T19:41:30  *** Chris_Stewart_5 has joined #bitcoin-core-dev
547 2016-09-28T19:44:34  *** donaloconnor has joined #bitcoin-core-dev
548 2016-09-28T19:51:39  *** Chris_Stewart_5 has quit IRC
549 2016-09-28T19:52:52  *** Chris_Stewart_5 has joined #bitcoin-core-dev
550 2016-09-28T20:01:03  *** cryptapus has quit IRC
551 2016-09-28T20:06:13  *** jnewbery has quit IRC
552 2016-09-28T20:07:37  *** jnewbery has joined #bitcoin-core-dev
553 2016-09-28T20:10:54  *** Taek42 is now known as Taek
554 2016-09-28T20:13:52  *** aureianimus_ has joined #bitcoin-core-dev
555 2016-09-28T20:14:08  *** aureianimus has quit IRC
556 2016-09-28T20:22:02  *** laurentmt has joined #bitcoin-core-dev
557 2016-09-28T20:22:05  *** laurentmt has quit IRC
558 2016-09-28T20:23:54  *** shesek has quit IRC
559 2016-09-28T20:29:06  *** shesek has joined #bitcoin-core-dev
560 2016-09-28T20:33:40  *** murch has quit IRC
561 2016-09-28T20:45:13  *** aureianimus has joined #bitcoin-core-dev
562 2016-09-28T20:45:25  *** aureianimus_ has quit IRC
563 2016-09-28T20:46:54  *** shesek has quit IRC
564 2016-09-28T21:04:51  *** aureianimus_ has joined #bitcoin-core-dev
565 2016-09-28T21:04:52  *** aureianimus has quit IRC
566 2016-09-28T21:06:08  *** h1d has quit IRC
567 2016-09-28T21:06:08  *** h1d has joined #bitcoin-core-dev
568 2016-09-28T21:15:12  *** droark has quit IRC
569 2016-09-28T21:21:39  *** jnewbery has quit IRC
570 2016-09-28T21:24:24  *** aureianimus_ has quit IRC
571 2016-09-28T21:24:34  *** aureianimus has joined #bitcoin-core-dev
572 2016-09-28T21:30:35  *** h1d has quit IRC
573 2016-09-28T21:34:05  *** MarcoFalke has left #bitcoin-core-dev
574 2016-09-28T21:43:24  *** aureianimus has quit IRC
575 2016-09-28T21:43:35  *** aureianimus has joined #bitcoin-core-dev
576 2016-09-28T21:46:02  *** pedrobranco has joined #bitcoin-core-dev
577 2016-09-28T21:49:46  <pedrobranco> hi wumpus, so jonasschnelli has able to reproduce the same issue right?
578 2016-09-28T21:51:15  *** donaloconnor has quit IRC
579 2016-09-28T21:55:45  *** aureianimus has quit IRC
580 2016-09-28T21:55:50  *** aureianimus_ has joined #bitcoin-core-dev
581 2016-09-28T22:05:39  *** Guyver2 has quit IRC
582 2016-09-28T22:06:39  *** aureianimus_ has quit IRC
583 2016-09-28T22:06:50  *** aureianimus has joined #bitcoin-core-dev
584 2016-09-28T22:07:40  *** aureianimus has quit IRC
585 2016-09-28T22:07:48  *** aureianimus has joined #bitcoin-core-dev
586 2016-09-28T22:08:49  *** aureianimus_ has joined #bitcoin-core-dev
587 2016-09-28T22:16:29  *** pedrobranco has quit IRC
588 2016-09-28T22:17:04  *** pedrobranco has joined #bitcoin-core-dev
589 2016-09-28T22:20:15  *** aureianimus has joined #bitcoin-core-dev
590 2016-09-28T22:20:16  *** aureianimus_ has quit IRC
591 2016-09-28T22:21:39  *** pedrobranco has quit IRC
592 2016-09-28T22:27:41  *** moli has quit IRC
593 2016-09-28T22:39:43  *** pedrobranco has joined #bitcoin-core-dev
594 2016-09-28T22:44:18  *** pedrobranco has quit IRC
595 2016-09-28T22:49:47  *** zooko has joined #bitcoin-core-dev
596 2016-09-28T22:58:56  *** shesek has joined #bitcoin-core-dev
597 2016-09-28T23:01:00  *** aureianimus has quit IRC
598 2016-09-28T23:01:12  *** aureianimus has joined #bitcoin-core-dev
599 2016-09-28T23:02:11  *** zooko has quit IRC
600 2016-09-28T23:06:44  *** wumpus has quit IRC
601 2016-09-28T23:09:40  *** droark has joined #bitcoin-core-dev
602 2016-09-28T23:12:05  *** aureianimus has quit IRC
603 2016-09-28T23:12:15  *** aureianimus has joined #bitcoin-core-dev
604 2016-09-28T23:13:28  *** donaloconnor has joined #bitcoin-core-dev
605 2016-09-28T23:13:37  *** davec has quit IRC
606 2016-09-28T23:14:19  *** davec has joined #bitcoin-core-dev
607 2016-09-28T23:15:24  *** donaloconnor has quit IRC
608 2016-09-28T23:19:23  *** sipax has joined #bitcoin-core-dev
609 2016-09-28T23:22:30  *** jnewbery has joined #bitcoin-core-dev
610 2016-09-28T23:27:27  *** jnewbery has quit IRC
611 2016-09-28T23:33:23  *** jnewbery has joined #bitcoin-core-dev
612 2016-09-28T23:38:49  *** sipax has quit IRC
613 2016-09-28T23:42:24  *** shesek has quit IRC
614 2016-09-28T23:43:29  *** aureianimus has quit IRC
615 2016-09-28T23:43:39  *** aureianimus has joined #bitcoin-core-dev
616 2016-09-28T23:54:25  *** aureianimus has quit IRC
617 2016-09-28T23:54:36  *** aureianimus has joined #bitcoin-core-dev
618 2016-09-28T23:56:06  *** jnewbery has quit IRC
619 2016-09-28T23:57:41  *** wumpus has joined #bitcoin-core-dev