1 2018-07-27T00:06:21  *** StopAndDecrypt has quit IRC
  2 2018-07-27T00:07:14  *** StopAndDecrypt has joined #bitcoin-core-dev
  3 2018-07-27T00:10:00  *** grafcaps has quit IRC
  4 2018-07-27T00:13:30  *** dqx__ has quit IRC
  5 2018-07-27T00:15:35  *** dqx_ has joined #bitcoin-core-dev
  6 2018-07-27T00:16:04  *** dqx_ has quit IRC
  7 2018-07-27T00:56:35  *** AaronvanW has quit IRC
  8 2018-07-27T00:57:12  *** AaronvanW has joined #bitcoin-core-dev
  9 2018-07-27T01:02:09  *** AaronvanW has quit IRC
 10 2018-07-27T01:05:38  *** jpe_ has joined #bitcoin-core-dev
 11 2018-07-27T01:08:38  *** jpe__ has quit IRC
 12 2018-07-27T01:17:48  *** savil has joined #bitcoin-core-dev
 13 2018-07-27T01:26:52  *** Victorsueca has quit IRC
 14 2018-07-27T01:28:08  *** Victorsueca has joined #bitcoin-core-dev
 15 2018-07-27T01:49:43  *** leishman__ has quit IRC
 16 2018-07-27T02:02:06  *** jarthur has joined #bitcoin-core-dev
 17 2018-07-27T02:08:06  *** AaronvanW has joined #bitcoin-core-dev
 18 2018-07-27T02:12:35  *** AaronvanW has quit IRC
 19 2018-07-27T02:34:54  *** savil has quit IRC
 20 2018-07-27T03:32:10  *** justan0theruser has quit IRC
 21 2018-07-27T03:32:27  *** justanotheruser has joined #bitcoin-core-dev
 22 2018-07-27T04:00:55  *** grubles has quit IRC
 23 2018-07-27T04:08:44  *** AaronvanW has joined #bitcoin-core-dev
 24 2018-07-27T04:13:21  *** AaronvanW has quit IRC
 25 2018-07-27T04:29:18  *** jarthur has quit IRC
 26 2018-07-27T04:35:16  *** grubles has joined #bitcoin-core-dev
 27 2018-07-27T04:46:35  *** grubles has quit IRC
 28 2018-07-27T04:56:18  *** schmidty_ has quit IRC
 29 2018-07-27T04:56:50  *** grubles has joined #bitcoin-core-dev
 30 2018-07-27T05:09:43  *** BillSmith4lyfe has joined #bitcoin-core-dev
 31 2018-07-27T05:42:27  *** jtimon has quit IRC
 32 2018-07-27T05:53:19  *** leishman has joined #bitcoin-core-dev
 33 2018-07-27T05:55:23  *** leishman has quit IRC
 34 2018-07-27T06:09:28  *** AaronvanW has joined #bitcoin-core-dev
 35 2018-07-27T06:14:27  *** AaronvanW has quit IRC
 36 2018-07-27T06:32:53  *** no_input_found has quit IRC
 37 2018-07-27T06:33:14  *** no_input_found has joined #bitcoin-core-dev
 38 2018-07-27T07:21:47  *** SopaXorzTaker has joined #bitcoin-core-dev
 39 2018-07-27T07:40:53  *** Victorsueca has quit IRC
 40 2018-07-27T07:42:08  *** Victorsueca has joined #bitcoin-core-dev
 41 2018-07-27T07:52:21  *** murrayn has quit IRC
 42 2018-07-27T07:54:28  *** ken2812221 has quit IRC
 43 2018-07-27T07:54:34  *** timothy has joined #bitcoin-core-dev
 44 2018-07-27T08:01:03  *** csknk has joined #bitcoin-core-dev
 45 2018-07-27T08:02:47  *** csknk has quit IRC
 46 2018-07-27T08:08:59  *** setpill has joined #bitcoin-core-dev
 47 2018-07-27T08:13:48  *** polydin has quit IRC
 48 2018-07-27T08:15:05  <jonasschnelli> sipa: would adding 'h' as equivalent to ' in the descriptor xpub/xpriv range definition makes sense? >'< in conjunction with JSON and the GUI/bitcoin-cli/shell seems to be a hassle
 49 2018-07-27T08:18:30  *** csknk has joined #bitcoin-core-dev
 50 2018-07-27T08:21:10  *** csknk has joined #bitcoin-core-dev
 51 2018-07-27T08:22:48  *** csknk has quit IRC
 52 2018-07-27T08:22:58  *** csknk has joined #bitcoin-core-dev
 53 2018-07-27T08:30:57  *** promag has joined #bitcoin-core-dev
 54 2018-07-27T08:37:01  *** vicenteH has joined #bitcoin-core-dev
 55 2018-07-27T09:37:27  *** go1111111 has quit IRC
 56 2018-07-27T09:54:58  *** AaronvanW has joined #bitcoin-core-dev
 57 2018-07-27T10:08:13  *** SopaXorzTaker has quit IRC
 58 2018-07-27T10:14:22  *** csknk has quit IRC
 59 2018-07-27T11:45:37  *** exit70 has quit IRC
 60 2018-07-27T11:45:49  <sipa> jonasschnelli: yes perhaps; but for compatibility with existing RPCs i stayed with '
 61 2018-07-27T11:46:53  *** exit70 has joined #bitcoin-core-dev
 62 2018-07-27T11:53:38  *** promag has quit IRC
 63 2018-07-27T11:54:36  <jonasschnelli> sipa: is think it should not be OR, it should be AND
 64 2018-07-27T11:54:51  <jonasschnelli> support ' AND h
 65 2018-07-27T11:56:42  *** vicenteH has quit IRC
 66 2018-07-27T11:57:10  *** vicenteH has joined #bitcoin-core-dev
 67 2018-07-27T12:00:50  *** jtimon has joined #bitcoin-core-dev
 68 2018-07-27T12:37:00  *** csknk has joined #bitcoin-core-dev
 69 2018-07-27T13:03:05  *** Chris_Stewart_5 has joined #bitcoin-core-dev
 70 2018-07-27T13:06:03  *** rafalcpp has quit IRC
 71 2018-07-27T13:17:13  *** rafalcpp has joined #bitcoin-core-dev
 72 2018-07-27T13:20:10  *** promag has joined #bitcoin-core-dev
 73 2018-07-27T13:23:16  *** jpe_ has quit IRC
 74 2018-07-27T13:23:34  *** jpe has joined #bitcoin-core-dev
 75 2018-07-27T13:27:34  *** setpill has quit IRC
 76 2018-07-27T13:29:33  *** setpill has joined #bitcoin-core-dev
 77 2018-07-27T13:30:05  *** Chris_Stewart_5 has quit IRC
 78 2018-07-27T13:52:20  *** profmac has quit IRC
 79 2018-07-27T14:20:46  *** AaronvanW has quit IRC
 80 2018-07-27T14:26:17  <brighton36> so, I'm looking to find out a few things about the bitcoincore.org site. Chiefly, content changes are delegated. Who can I speak to about that?
 81 2018-07-27T14:26:23  <brighton36> *how
 82 2018-07-27T14:26:38  *** SopaXorzTaker has joined #bitcoin-core-dev
 83 2018-07-27T14:27:19  <brighton36> Is there a policy somewhere perhaps?
 84 2018-07-27T14:29:55  *** csknk has quit IRC
 85 2018-07-27T14:31:25  *** AaronvanW has joined #bitcoin-core-dev
 86 2018-07-27T14:35:57  *** AaronvanW has quit IRC
 87 2018-07-27T14:40:53  *** Chris_Stewart_5 has joined #bitcoin-core-dev
 88 2018-07-27T14:48:26  <harding> brighton36: I help maintain the site.  I don't relaly understand your question.  There's a basic contributing policy at https://github.com/bitcoin-core/bitcoincore.org/blob/master/CONTRIBUTING.md but the basic process is the same for any open source project: people open PRs proposing changes, other people provide feedback, once the feedback is generally positive, the PR is merged.  If the PR never gets to that stage, it's
 89 2018-07-27T14:48:27  <harding> eventually closed.  If there's something wonky going on (like lots of comments from unknown accounts), that's factored into the merge/no-merge decision.
 90 2018-07-27T15:08:36  <brighton36> thank-you harding :)
 91 2018-07-27T15:08:48  <brighton36> I get that this is a heavily politicized issue. But, I'd like to know what the process is
 92 2018-07-27T15:08:56  <brighton36> I think a mission statement would help greatly
 93 2018-07-27T15:09:12  <brighton36> less would obviously be more, but, just being able to point at a mission would help us out greatly
 94 2018-07-27T15:09:52  <brighton36> Borrowing one from any other prominent open source project, and redacting specifics would seem productive. Less is more.
 95 2018-07-27T15:10:25  *** harrymm has quit IRC
 96 2018-07-27T15:10:26  <brighton36> Is there a process by which the stakeholders could decide that, or is a PR the process?
 97 2018-07-27T15:10:38  <harding> brighton36: I'm not sure what you're talking about, but it feels a bit off topic for this channel.  Would you mind joining #bitcoin-core-website to discuss further?  (Or you can PM me if you're a registered user.)
 98 2018-07-27T15:10:46  <brighton36> yep
 99 2018-07-27T15:10:51  <brighton36> thank-you sir
100 2018-07-27T15:12:13  *** cryptapus has quit IRC
101 2018-07-27T15:20:26  *** marcinja_ has quit IRC
102 2018-07-27T15:20:47  *** AaronvanW has joined #bitcoin-core-dev
103 2018-07-27T15:25:15  *** savil has joined #bitcoin-core-dev
104 2018-07-27T15:34:48  *** vicenteH has quit IRC
105 2018-07-27T15:44:08  *** Chris_Stewart_5 has quit IRC
106 2018-07-27T15:45:52  *** Chris_Stewart_5 has joined #bitcoin-core-dev
107 2018-07-27T15:48:27  *** Emcy has quit IRC
108 2018-07-27T15:49:20  *** |EHG| has quit IRC
109 2018-07-27T15:50:39  *** Emcy has joined #bitcoin-core-dev
110 2018-07-27T15:55:09  *** |EHG| has joined #bitcoin-core-dev
111 2018-07-27T15:55:49  *** |EHG| has left #bitcoin-core-dev
112 2018-07-27T16:00:53  *** promag has quit IRC
113 2018-07-27T16:01:27  *** promag has joined #bitcoin-core-dev
114 2018-07-27T16:02:19  *** masonicboom has joined #bitcoin-core-dev
115 2018-07-27T16:06:11  *** promag has quit IRC
116 2018-07-27T16:06:15  *** dqx_ has joined #bitcoin-core-dev
117 2018-07-27T16:12:29  *** dqx__ has joined #bitcoin-core-dev
118 2018-07-27T16:12:52  *** dqx__ has quit IRC
119 2018-07-27T16:15:51  *** dqx_ has quit IRC
120 2018-07-27T16:31:52  *** setpill has quit IRC
121 2018-07-27T16:34:06  *** jarthur has joined #bitcoin-core-dev
122 2018-07-27T16:40:57  *** leishman has joined #bitcoin-core-dev
123 2018-07-27T16:52:59  *** leishman___ has joined #bitcoin-core-dev
124 2018-07-27T16:53:06  *** leishman___ has left #bitcoin-core-dev
125 2018-07-27T17:26:27  *** rex4539 has quit IRC
126 2018-07-27T17:26:39  *** jarthur has quit IRC
127 2018-07-27T17:32:34  *** vicenteH has joined #bitcoin-core-dev
128 2018-07-27T17:36:14  *** timothy has quit IRC
129 2018-07-27T17:40:02  *** masonicboom has quit IRC
130 2018-07-27T17:45:48  *** masonicboom has joined #bitcoin-core-dev
131 2018-07-27T17:56:00  *** rex4539 has joined #bitcoin-core-dev
132 2018-07-27T18:23:35  <sipa> MarcoFalke: AssertionError: not(0.00700000 == 0.007)
133 2018-07-27T18:23:42  <sipa> any idea how to deal with that in tests?
134 2018-07-27T18:24:48  <luke-jr> sipa: maybe assert_approx?
135 2018-07-27T18:25:09  <sipa> i don't see that anywhere in our source code
136 2018-07-27T18:25:18  <sipa> this can't be the only place where this is an issue
137 2018-07-27T18:25:23  <sipa> so i must be doing something wrong
138 2018-07-27T18:25:36  <sipa> (it's ValueFromAmount output in RPC
139 2018-07-27T18:26:12  <sipa> i'll just try to stick to integer amounts
140 2018-07-27T18:26:15  <luke-jr> sipa: test/functional/wallet_groups.py
141 2018-07-27T18:30:09  <sdaftuar> sipa: are both values coming from rpc calls, or is one hardcoded in?  i think we use Decimal() in the test framework a bunch, presumably to solve that problem
142 2018-07-27T18:31:03  <sipa> sdaftuar: ah!
143 2018-07-27T18:31:08  <sipa> one is hardcoded
144 2018-07-27T18:33:43  <jonasschnelli> sipa: re xpub range in descriptors...
145 2018-07-27T18:33:46  <gmaxwell> :( even we can't get it right.
146 2018-07-27T18:34:14  <jonasschnelli> sipa: if you disagree with the index <= vs <, then I guess you need to +1 when populating the range from the JSON or change the doc
147 2018-07-27T18:34:31  <jonasschnelli> Because the doc states: "\"range\": n,             (numeric, optional) Up to what child index HD chains should be explored (default: 1000)\n""
148 2018-07-27T18:34:44  <jonasschnelli> if you pass in 1000, it will only derive up to 999
149 2018-07-27T18:35:17  <jonasschnelli> I think the doc is correct and would recommend to +1 somewhere else
150 2018-07-27T18:37:22  <sipa> jonasschnelli: good point
151 2018-07-27T18:38:16  <jonasschnelli> IMO if one puts a range of 1500, then one probably expect that key at index 1500 will be included...
152 2018-07-27T18:39:23  <jonasschnelli> sipa: why not a real range X-Y? instead of pure Y?
153 2018-07-27T18:39:23  <sipa> ok, will fix
154 2018-07-27T18:39:39  <sipa> jonasschnelli: sounds like something for a later extension?
155 2018-07-27T18:39:40  <jonasschnelli> (although could be changed later)
156 2018-07-27T18:39:44  <jonasschnelli> agree
157 2018-07-27T18:42:20  <sipa> GAH
158 2018-07-27T18:42:21  <sipa> AssertionError: not(0.00200000 == 0.00200000000000000004163336342344337026588618755340576171875)
159 2018-07-27T18:44:53  <sdaftuar> sipa: i think you need to do `Decimal("0.002")`?
160 2018-07-27T18:45:20  <sdaftuar> at least, that's what i see all over our python test code
161 2018-07-27T18:46:35  <sipa> sdaftuar: thanks! that works
162 2018-07-27T18:47:39  *** grubles has quit IRC
163 2018-07-27T18:50:27  *** jtimon has quit IRC
164 2018-07-27T18:54:07  *** rex4539 has quit IRC
165 2018-07-27T18:56:03  <sipa> jonasschnelli: regarding support for "h" instead of "'", should it roundtrip?
166 2018-07-27T18:56:21  <jonasschnelli> no... I think only input
167 2018-07-27T18:56:37  <sipa> ok, that's easy enough then
168 2018-07-27T18:56:51  <jonasschnelli> Yes.
169 2018-07-27T18:57:08  <jonasschnelli> I guess no-one will complain about the missing roundtripness...
170 2018-07-27T18:57:22  <jonasschnelli> IMO it's mainly useful to bypass shellish input sources
171 2018-07-27T18:57:39  <jonasschnelli> *shellish input source restrictions
172 2018-07-27T18:58:55  *** satwo has joined #bitcoin-core-dev
173 2018-07-27T19:04:44  <satwo> Has the bitcoind config param -rpcclienttimeout been removed? I'm getting errors trying to set it in both the command line and in bitcoin.conf.
174 2018-07-27T19:04:46  *** rex4539 has joined #bitcoin-core-dev
175 2018-07-27T19:05:37  <sipa> satwo: no, but it's a bitcoin-cli command line option; not a bitcoind one
176 2018-07-27T19:06:39  <satwo> Thanks sipa. This doc may need updating then: https://github.com/bitcoin/bitcoin/blob/master/contrib/debian/examples/bitcoin.conf
177 2018-07-27T19:09:18  <satwo> Would it be appropriate to remove the section for rpcclienttimeout there? Or am I missing something?
178 2018-07-27T19:09:57  *** Chris_Stewart_5 has quit IRC
179 2018-07-27T19:10:00  <sipa> in bitcoin.conf it should work fine; that file should be read by both bitcoind and bitcoin-cli
180 2018-07-27T19:10:51  *** go1111111 has joined #bitcoin-core-dev
181 2018-07-27T19:11:43  <satwo> That's what I thought, but I got the following error when starting bitcoind: Error reading configuration file: Invalid configuration value rpcclienttimeout
182 2018-07-27T19:12:46  <sipa> oh, of course
183 2018-07-27T19:13:04  <sipa> right; you can't put it in bitcoin.conf anymore, because it's not a bitcoind option
184 2018-07-27T19:13:16  <sipa> and since 0.17 bitcoind will give an error if an invalid option is passed
185 2018-07-27T19:13:34  <sipa> we could add an exception to ignore it
186 2018-07-27T19:16:22  <satwo> For just that one param?
187 2018-07-27T19:16:36  <sipa> no, this is true for every parameter
188 2018-07-27T19:16:52  <sipa> in earlier versions bitcoind would just ignore unknown options; now it will give an error
189 2018-07-27T19:20:02  <satwo> Got it. What do you mean exactly by adding an exception to ignore it? Show the error/warning but allow bitcoind to start anyway?
190 2018-07-27T19:20:08  <sipa> but we should probably make it ignore bitcoin-cli only options like this one, when present in bitcoin.conf
191 2018-07-27T19:20:11  <wumpus> it's good that it now gives errors on unkonwn options, this was a long-running issue
192 2018-07-27T19:20:33  <wumpus> but yes, that's firly annoying too...
193 2018-07-27T19:21:09  <wumpus> sipa: though that means that all executables need to have knowledge of all other executables' flags
194 2018-07-27T19:21:28  <wumpus> which is exactly what we were trying to avoid by modularizing the options system
195 2018-07-27T19:21:38  <sipa> wumpus: or we need a way to specify bitcoin.conf options that only apply to a particular binary
196 2018-07-27T19:21:39  <wumpus> it's kind of annoying, maybe cli-specific options could have their own sections?
197 2018-07-27T19:21:44  <sipa> like the network sections
198 2018-07-27T19:21:45  <wumpus> right
199 2018-07-27T19:22:00  <wumpus> for 0.17 I agree we could just make it a warning
200 2018-07-27T19:22:04  <wumpus> if an unknown option is specified
201 2018-07-27T19:22:11  <wumpus> then for 0.18 think about for to sensibly enforce this
202 2018-07-27T19:23:07  <sipa> that's one possibility
203 2018-07-27T19:23:14  <sipa> or we could use the deprecation mechanism
204 2018-07-27T19:23:57  <sipa> (by default it's enforced, unless you pass -ignoreunknownconfigoptions or so; and then it still only applies to bitcoin.conf and not to command line options)
205 2018-07-27T19:24:55  <wumpus> yes
206 2018-07-27T19:27:58  <luke-jr> wumpus: actually, it might be annoying if users get errors for Knots configs in bitcoin.conf
207 2018-07-27T19:28:09  <luke-jr> perhaps there should be a way to explicitly say "don't error if you don't understand this"
208 2018-07-27T19:28:20  <wumpus> sipa has a good point
209 2018-07-27T19:28:32  <luke-jr> per-option seems safer IMO
210 2018-07-27T19:28:33  <wumpus> make it an error if an unknown option on the command line
211 2018-07-27T19:28:36  <luke-jr> though we could do both
212 2018-07-27T19:28:42  <wumpus> but a warning if in bitcoin.conf
213 2018-07-27T19:28:46  <luke-jr> sure
214 2018-07-27T19:39:32  <satwo> Happy to attempt an implementation of this, if no one else wants to knock it out.
215 2018-07-27T19:42:06  <sipa> i vaguely remember we already have an exception for some option, but i can't check right now
216 2018-07-27T19:43:49  *** grubles has joined #bitcoin-core-dev
217 2018-07-27T19:49:34  *** polydin has joined #bitcoin-core-dev
218 2018-07-27T19:53:38  *** owowo has quit IRC
219 2018-07-27T19:59:08  *** owowo has joined #bitcoin-core-dev
220 2018-07-27T20:00:18  *** Victorsueca has quit IRC
221 2018-07-27T20:01:31  *** Victorsueca has joined #bitcoin-core-dev
222 2018-07-27T20:04:36  *** savil has quit IRC
223 2018-07-27T20:16:42  <wumpus> satwo: would be awesome
224 2018-07-27T20:17:21  *** SopaXorzTaker has quit IRC
225 2018-07-27T20:27:36  *** Chris_Stewart_5 has joined #bitcoin-core-dev
226 2018-07-27T20:46:57  *** Chris_Stewart_5 has quit IRC
227 2018-07-27T20:47:51  <satwo> wumpus: Will work on it this weekend. Still learning C++ but I'll do the best I can
228 2018-07-27T20:57:38  *** booyah is now known as booyah_
229 2018-07-27T21:00:18  <cfields> jonasschnelli: 0.16.2 macOS signature ping :)
230 2018-07-27T21:05:25  *** michaelsdunn1 has joined #bitcoin-core-dev
231 2018-07-27T21:15:36  *** Chris_Stewart_5 has joined #bitcoin-core-dev
232 2018-07-27T21:18:23  <jonasschnelli> cfields: Oh. Yes. Just started the build,... will PR the detatched sig tomorrow (in 10-12h).
233 2018-07-27T21:23:19  <cfields> jonasschnelli: np, thanks
234 2018-07-27T21:27:20  *** masonicboom has quit IRC
235 2018-07-27T21:38:51  *** masonicboom has joined #bitcoin-core-dev
236 2018-07-27T21:39:14  *** savil has joined #bitcoin-core-dev
237 2018-07-27T21:42:41  *** Victorsueca has quit IRC
238 2018-07-27T21:44:11  *** Victorsueca has joined #bitcoin-core-dev
239 2018-07-27T21:45:05  *** Chris_Stewart_5 has quit IRC
240 2018-07-27T22:07:18  *** LKbTkXFn has joined #bitcoin-core-dev
241 2018-07-27T22:14:55  *** lnostdal has quit IRC
242 2018-07-27T22:37:34  *** jhfrontz has joined #bitcoin-core-dev
243 2018-07-27T22:39:36  *** spinza has quit IRC
244 2018-07-27T22:55:54  <jhfrontz> Is there anything that needs to be done to enable use of the  0.16.2 version of contrib/gitian-descriptors/gitian-linux.yml (when using a docker-based gitian build)?
245 2018-07-27T22:56:53  <jhfrontz> I'm getting errors about missing symbols (like described here: https://github.com/bitcoin/bitcoin/issues/12511#issuecomment-386740137 ).
246 2018-07-27T23:01:06  *** michaelsdunn1 has quit IRC
247 2018-07-27T23:03:06  <jhfrontz> It looks like all the signers of 0.16.2  for linux so far have used "trusty" (instead of the 0.16.2-gitian-linux.yml-spec'd "bionic").  So maybe I shouldn't be using the new version?
248 2018-07-27T23:03:30  <wumpus> yes 0.16.2 is built with trusty
249 2018-07-27T23:03:59  <wumpus> no releases yet have been built with bionic, though this is the plan for 0.17
250 2018-07-27T23:05:48  <sipa> jhfrontz: if you build on bionic (even if it works), you won't get the same binaries as others
251 2018-07-27T23:06:09  *** Chris_Stewart_5 has joined #bitcoin-core-dev
252 2018-07-27T23:06:24  *** CubicEarth has quit IRC
253 2018-07-27T23:06:48  <jhfrontz> OK, tnx.  I must've pulled down the wrong version?
254 2018-07-27T23:07:04  <wumpus> I guess so? I'm not sure how you end up in that situation at all, the descriptors on the 0.16 branch shouldn't point to bionic
255 2018-07-27T23:07:04  <jhfrontz> of gitian-linux.yml, I mean
256 2018-07-27T23:07:42  <jhfrontz> doh, wrong branch…
257 2018-07-27T23:07:47  <sipa> https://github.com/bitcoin/bitcoin/blob/v0.16.2/contrib/gitian-descriptors/gitian-linux.yml <- specifies trusty
258 2018-07-27T23:08:18  *** CubicEarth has joined #bitcoin-core-dev
259 2018-07-27T23:29:59  *** leishman has quit IRC
260 2018-07-27T23:44:46  *** owowo has quit IRC
261 2018-07-27T23:49:32  *** owowo has joined #bitcoin-core-dev