1 2020-08-07T00:00:03 *** Guest45455 has quit IRC
2 2020-08-07T00:01:47 *** Chris_Stewart_5 has quit IRC
3 2020-08-07T00:05:49 *** mrostecki has quit IRC
4 2020-08-07T00:05:59 *** mrostecki has joined #bitcoin-core-dev
5 2020-08-07T00:06:43 *** Pavlenex has quit IRC
6 2020-08-07T00:10:32 *** Chris_Stewart_5 has joined #bitcoin-core-dev
7 2020-08-07T00:13:43 *** tryphe_ has joined #bitcoin-core-dev
8 2020-08-07T00:14:25 *** tryphe has quit IRC
9 2020-08-07T00:14:39 *** promag_ has quit IRC
10 2020-08-07T00:20:51 *** afk11 has joined #bitcoin-core-dev
11 2020-08-07T00:22:25 *** tekojo has joined #bitcoin-core-dev
12 2020-08-07T00:31:32 *** luke-jr has quit IRC
13 2020-08-07T00:32:58 *** luke-jr has joined #bitcoin-core-dev
14 2020-08-07T01:00:05 *** mol_ has quit IRC
15 2020-08-07T01:02:02 *** arowser_ has quit IRC
16 2020-08-07T01:02:29 *** arowser_ has joined #bitcoin-core-dev
17 2020-08-07T01:18:24 *** proofofkeags has quit IRC
18 2020-08-07T01:19:25 *** arowser_ has quit IRC
19 2020-08-07T01:20:01 *** arowser_ has joined #bitcoin-core-dev
20 2020-08-07T01:25:22 *** proofofkeags has joined #bitcoin-core-dev
21 2020-08-07T01:25:41 *** mol has joined #bitcoin-core-dev
22 2020-08-07T01:40:40 *** mdunnio has joined #bitcoin-core-dev
23 2020-08-07T01:42:05 *** proofofkeags has quit IRC
24 2020-08-07T01:42:40 *** proofofkeags has joined #bitcoin-core-dev
25 2020-08-07T01:43:50 *** arowser_ has quit IRC
26 2020-08-07T01:44:53 *** mdunnio has quit IRC
27 2020-08-07T01:45:54 *** mdunnio has joined #bitcoin-core-dev
28 2020-08-07T01:46:51 *** mdrollette has quit IRC
29 2020-08-07T01:46:59 *** proofofkeags has quit IRC
30 2020-08-07T01:47:10 *** mdrollette has joined #bitcoin-core-dev
31 2020-08-07T01:50:20 *** mdunnio has quit IRC
32 2020-08-07T01:51:24 *** shesek has quit IRC
33 2020-08-07T01:53:35 *** gzhao408 has joined #bitcoin-core-dev
34 2020-08-07T02:06:14 *** arowser_ has joined #bitcoin-core-dev
35 2020-08-07T02:13:44 *** proofofkeags has joined #bitcoin-core-dev
36 2020-08-07T02:18:07 *** proofofkeags has quit IRC
37 2020-08-07T02:20:10 *** mattdrollette has joined #bitcoin-core-dev
38 2020-08-07T02:21:26 *** mdrollette has quit IRC
39 2020-08-07T02:26:51 *** jarthur has quit IRC
40 2020-08-07T02:31:07 *** bitcoin-git has joined #bitcoin-core-dev
41 2020-08-07T02:31:08 <bitcoin-git> [bitcoin] andrewtoth closed pull request #19271: validation: Warm coins cache during prevalidation to connect blocks faster (master...warm-coinscache) https://github.com/bitcoin/bitcoin/pull/19271
42 2020-08-07T02:31:20 *** bitcoin-git has left #bitcoin-core-dev
43 2020-08-07T02:44:10 *** dr-orlovsky has quit IRC
44 2020-08-07T02:50:26 *** proofofkeags has joined #bitcoin-core-dev
45 2020-08-07T02:55:06 *** proofofkeags has quit IRC
46 2020-08-07T02:58:52 *** proofofkeags has joined #bitcoin-core-dev
47 2020-08-07T03:00:02 *** tekojo has quit IRC
48 2020-08-07T03:05:04 *** watersnake1 has quit IRC
49 2020-08-07T03:08:25 *** arowser_ has quit IRC
50 2020-08-07T03:14:58 *** arowser_ has joined #bitcoin-core-dev
51 2020-08-07T03:20:25 *** Highway61 has quit IRC
52 2020-08-07T03:21:26 *** edunham1 has joined #bitcoin-core-dev
53 2020-08-07T03:38:41 <sipa> i don't understand this
54 2020-08-07T03:38:43 <sipa> https://travis-ci.org/github/bitcoin/bitcoin/builds/715710056
55 2020-08-07T03:39:02 <sipa> it's failing consistently, in the same way, for all platforms that run functional tests & have wallet enabled
56 2020-08-07T03:39:26 <sipa> but i can't reproduce it locally, not even after git clean -dfx && ccache -C
57 2020-08-07T03:40:14 <sipa> can anyone try to build & run #17977, and run tests/functional/feature_taproot.py ?
58 2020-08-07T03:40:16 <gribble> https://github.com/bitcoin/bitcoin/issues/17977 | Implement BIP 340-342 validation (Schnorr/taproot/tapscript) by sipa · Pull Request #17977 · bitcoin/bitcoin · GitHub
59 2020-08-07T03:40:20 *** bitcoin-git has joined #bitcoin-core-dev
60 2020-08-07T03:40:20 <bitcoin-git> [bitcoin] JeremyRubin closed pull request #19531: add static_check_equal for easier to read compiler errors (master...static_check) https://github.com/bitcoin/bitcoin/pull/19531
61 2020-08-07T03:40:21 *** bitcoin-git has left #bitcoin-core-dev
62 2020-08-07T03:40:40 *** bitcoin-git has joined #bitcoin-core-dev
63 2020-08-07T03:40:40 <bitcoin-git> [bitcoin] JeremyRubin reopened pull request #19531: add static_check_equal for easier to read compiler errors (master...static_check) https://github.com/bitcoin/bitcoin/pull/19531
64 2020-08-07T03:40:41 *** bitcoin-git has left #bitcoin-core-dev
65 2020-08-07T03:43:28 *** Emcy_ has left #bitcoin-core-dev
66 2020-08-07T03:43:41 *** Emcy_ has quit IRC
67 2020-08-07T03:47:02 <sipa> oh, maybe a recent merge
68 2020-08-07T03:49:27 <troygiorshev> sipa: test passed for me
69 2020-08-07T03:50:08 <sipa> troygiorshev: thanks for trying - i think i know what caused it
70 2020-08-07T03:50:22 <sipa> the merge of #19620
71 2020-08-07T03:50:26 <gribble> https://github.com/bitcoin/bitcoin/issues/19620 | Add txids with non-standard inputs to reject filter by sdaftuar · Pull Request #19620 · bitcoin/bitcoin · GitHub
72 2020-08-07T03:50:59 *** Emcy has joined #bitcoin-core-dev
73 2020-08-07T03:51:03 <troygiorshev> ah great
74 2020-08-07T03:51:13 <troygiorshev> fwiw too, I think this test should be moved to the "less than 2m" section
75 2020-08-07T03:51:24 <troygiorshev> Took 1:12 on my i7-8750H
76 2020-08-07T03:51:25 <aj> sipa: that seems to match the error in the travis log
77 2020-08-07T03:52:28 <sipa> troygiorshev: thanks, doing that
78 2020-08-07T03:53:04 <sipa> aj: yeah, i can reproduce after a rebase on master
79 2020-08-07T04:00:46 <fanquake> sipa: gotta keep up :p
80 2020-08-07T04:02:01 <fanquake> bitcoin/bitcoin is a fast moving repository
81 2020-08-07T04:02:29 <sipa> fanquake: i started this set of changes over a week ago, then when i was done rebased on master, and right then "someone" merges probably the only open PR in the whole repo that breaks the tests without causing a compilation failure :p
82 2020-08-07T04:15:38 *** arowser_ has quit IRC
83 2020-08-07T04:29:17 *** gzhao408 has quit IRC
84 2020-08-07T04:31:03 *** da2ce7_ has quit IRC
85 2020-08-07T04:37:57 *** da2ce7 has joined #bitcoin-core-dev
86 2020-08-07T04:38:14 *** arowser_ has joined #bitcoin-core-dev
87 2020-08-07T04:41:10 *** justan0theruser has joined #bitcoin-core-dev
88 2020-08-07T04:43:12 *** justanotheruser has quit IRC
89 2020-08-07T04:46:15 *** justanotheruser has joined #bitcoin-core-dev
90 2020-08-07T04:49:06 *** justan0theruser has quit IRC
91 2020-08-07T04:51:51 *** TallTim has quit IRC
92 2020-08-07T05:00:24 *** arowser_ has quit IRC
93 2020-08-07T05:01:02 *** arowser_ has joined #bitcoin-core-dev
94 2020-08-07T05:10:11 *** arowser_ has quit IRC
95 2020-08-07T05:10:31 *** arowser_ has joined #bitcoin-core-dev
96 2020-08-07T05:15:27 *** justanotheruser has quit IRC
97 2020-08-07T05:20:18 *** justanotheruser has joined #bitcoin-core-dev
98 2020-08-07T05:23:16 *** bitcoin-git has joined #bitcoin-core-dev
99 2020-08-07T05:23:17 <bitcoin-git> [bitcoin] JeremyRubin opened pull request #19677: WIP: Switch BlockMap to use an unordered_set under the hood (master...refactor-blockmap-blockset) https://github.com/bitcoin/bitcoin/pull/19677
100 2020-08-07T05:23:17 *** bitcoin-git has left #bitcoin-core-dev
101 2020-08-07T05:25:09 *** arowser_ has quit IRC
102 2020-08-07T05:25:33 *** arowser_ has joined #bitcoin-core-dev
103 2020-08-07T05:44:19 *** justan0theruser has joined #bitcoin-core-dev
104 2020-08-07T05:45:49 *** justanotheruser has quit IRC
105 2020-08-07T05:50:18 *** justan0theruser has quit IRC
106 2020-08-07T06:00:02 *** edunham1 has quit IRC
107 2020-08-07T06:08:25 *** proofofkeags has quit IRC
108 2020-08-07T06:08:59 *** proofofkeags has joined #bitcoin-core-dev
109 2020-08-07T06:09:00 *** bitcoin-git has joined #bitcoin-core-dev
110 2020-08-07T06:09:00 <bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/6d8543504d8c...4b705b1c98f6
111 2020-08-07T06:09:01 <bitcoin-git> bitcoin/master edc3160 Russell Yanofsky: test: Remove duplicate NodeContext hacks
112 2020-08-07T06:09:01 <bitcoin-git> bitcoin/master 4b705b1 MarcoFalke: Merge #19098: test: Remove duplicate NodeContext hacks
113 2020-08-07T06:09:03 *** bitcoin-git has left #bitcoin-core-dev
114 2020-08-07T06:09:45 *** bitcoin-git has joined #bitcoin-core-dev
115 2020-08-07T06:09:46 <bitcoin-git> [bitcoin] MarcoFalke merged pull request #19098: test: Remove duplicate NodeContext hacks (master...pr/qtx) https://github.com/bitcoin/bitcoin/pull/19098
116 2020-08-07T06:09:47 *** bitcoin-git has left #bitcoin-core-dev
117 2020-08-07T06:13:26 *** proofofkeags has quit IRC
118 2020-08-07T06:16:44 *** justanotheruser has joined #bitcoin-core-dev
119 2020-08-07T06:18:48 *** justanotheruser has joined #bitcoin-core-dev
120 2020-08-07T06:21:50 *** Esky33 has joined #bitcoin-core-dev
121 2020-08-07T06:22:02 *** pehjota1 has joined #bitcoin-core-dev
122 2020-08-07T06:22:11 *** arowser_ has quit IRC
123 2020-08-07T06:22:40 *** arowser_ has joined #bitcoin-core-dev
124 2020-08-07T06:29:42 *** justanotheruser has quit IRC
125 2020-08-07T06:30:01 *** justanotheruser has joined #bitcoin-core-dev
126 2020-08-07T06:33:59 *** davec has quit IRC
127 2020-08-07T06:37:07 *** Esky33 has quit IRC
128 2020-08-07T06:41:12 *** arowser_ has quit IRC
129 2020-08-07T06:41:33 *** arowser_ has joined #bitcoin-core-dev
130 2020-08-07T06:42:37 *** davec has joined #bitcoin-core-dev
131 2020-08-07T06:43:25 *** davterra has quit IRC
132 2020-08-07T06:57:09 *** arowser_ has quit IRC
133 2020-08-07T06:57:29 *** arowser_ has joined #bitcoin-core-dev
134 2020-08-07T06:59:10 *** davec has quit IRC
135 2020-08-07T07:07:16 *** marcoagner has joined #bitcoin-core-dev
136 2020-08-07T07:08:03 *** ghost43 has joined #bitcoin-core-dev
137 2020-08-07T07:10:33 *** Emcy has quit IRC
138 2020-08-07T07:12:21 *** Emcy has joined #bitcoin-core-dev
139 2020-08-07T07:12:34 *** davec has joined #bitcoin-core-dev
140 2020-08-07T07:14:50 *** arowser_ has quit IRC
141 2020-08-07T07:21:42 *** arowser_ has joined #bitcoin-core-dev
142 2020-08-07T07:29:12 *** arowser_ has quit IRC
143 2020-08-07T07:29:36 *** arowser_ has joined #bitcoin-core-dev
144 2020-08-07T07:33:23 *** Dean_Guss has quit IRC
145 2020-08-07T07:43:16 *** marcoagner has quit IRC
146 2020-08-07T07:53:03 *** vasild has quit IRC
147 2020-08-07T07:56:12 *** arowser_ has quit IRC
148 2020-08-07T07:56:31 *** arowser_ has joined #bitcoin-core-dev
149 2020-08-07T08:04:51 *** bitcoin-git has joined #bitcoin-core-dev
150 2020-08-07T08:04:51 <bitcoin-git> [bitcoin] MarcoFalke closed pull request #15045: [test] Apply maximal flags to tx_valid tests and minimal flags to tx_invalid tests (master...min_txtests_flags) https://github.com/bitcoin/bitcoin/pull/15045
151 2020-08-07T08:04:52 *** bitcoin-git has left #bitcoin-core-dev
152 2020-08-07T08:05:40 *** promag has quit IRC
153 2020-08-07T08:11:40 *** tryphe_ has quit IRC
154 2020-08-07T08:11:44 *** tryphe__ has joined #bitcoin-core-dev
155 2020-08-07T08:19:14 *** vincenzopalazzo has joined #bitcoin-core-dev
156 2020-08-07T08:21:27 *** palazzovincenzo has joined #bitcoin-core-dev
157 2020-08-07T08:22:16 *** palazzovincenzo has joined #bitcoin-core-dev
158 2020-08-07T08:23:09 *** Davterra has joined #bitcoin-core-dev
159 2020-08-07T08:23:25 *** ghost43_ has joined #bitcoin-core-dev
160 2020-08-07T08:24:03 *** ghost43 has quit IRC
161 2020-08-07T08:24:11 *** vincenzopalazzo has quit IRC
162 2020-08-07T08:24:36 *** palazzovincenzo has quit IRC
163 2020-08-07T08:25:03 *** promag has joined #bitcoin-core-dev
164 2020-08-07T08:25:44 *** mrostecki has quit IRC
165 2020-08-07T08:26:07 *** mrostecki has joined #bitcoin-core-dev
166 2020-08-07T08:31:16 *** vincenzopalazzo has joined #bitcoin-core-dev
167 2020-08-07T08:31:35 *** promag_ has joined #bitcoin-core-dev
168 2020-08-07T08:32:36 *** promag has quit IRC
169 2020-08-07T08:34:04 *** vasild has joined #bitcoin-core-dev
170 2020-08-07T08:36:26 *** pehjota1 has quit IRC
171 2020-08-07T08:38:13 *** tryphe_ has joined #bitcoin-core-dev
172 2020-08-07T08:38:26 *** tryphe__ has quit IRC
173 2020-08-07T08:42:25 *** Pavlenex has joined #bitcoin-core-dev
174 2020-08-07T08:54:54 *** Alucard has joined #bitcoin-core-dev
175 2020-08-07T08:55:54 *** K4TJAN has joined #bitcoin-core-dev
176 2020-08-07T08:55:56 *** tryphe has joined #bitcoin-core-dev
177 2020-08-07T08:55:59 *** K4TJAN has quit IRC
178 2020-08-07T08:56:21 *** K4TJANG95 has joined #bitcoin-core-dev
179 2020-08-07T08:56:35 *** dr-orlovsky has joined #bitcoin-core-dev
180 2020-08-07T08:56:46 *** K4TJANG95 has quit IRC
181 2020-08-07T08:57:52 *** tryphe_ has quit IRC
182 2020-08-07T09:10:51 *** dr-orlovsky has quit IRC
183 2020-08-07T09:16:10 *** davec has quit IRC
184 2020-08-07T09:22:40 *** davec has joined #bitcoin-core-dev
185 2020-08-07T09:33:00 *** Bullitje has left #bitcoin-core-dev
186 2020-08-07T09:41:44 *** syperf has joined #bitcoin-core-dev
187 2020-08-07T09:46:13 *** EagleTM has joined #bitcoin-core-dev
188 2020-08-07T09:47:08 *** arowser_ has quit IRC
189 2020-08-07T09:47:54 *** arowser_ has joined #bitcoin-core-dev
190 2020-08-07T09:49:08 *** dr-orlovsky has joined #bitcoin-core-dev
191 2020-08-07T09:52:00 *** andreacab has joined #bitcoin-core-dev
192 2020-08-07T10:03:28 *** Shanie14Harris has joined #bitcoin-core-dev
193 2020-08-07T10:11:49 *** andreacab has quit IRC
194 2020-08-07T10:12:15 *** andreacab has joined #bitcoin-core-dev
195 2020-08-07T10:16:19 *** andreacab has quit IRC
196 2020-08-07T10:16:26 *** andreacab has joined #bitcoin-core-dev
197 2020-08-07T10:16:52 *** andreacab has quit IRC
198 2020-08-07T10:19:32 *** andreacab has joined #bitcoin-core-dev
199 2020-08-07T10:27:01 <sdaftuar> sipa: just catching up now, is the test in #19620 not stable?
200 2020-08-07T10:27:03 <gribble> https://github.com/bitcoin/bitcoin/issues/19620 | Add txids with non-standard inputs to reject filter by sdaftuar · Pull Request #19620 · bitcoin/bitcoin · GitHub
201 2020-08-07T10:28:19 *** Kiminuo has joined #bitcoin-core-dev
202 2020-08-07T10:35:52 *** jeremyrubin has quit IRC
203 2020-08-07T10:36:49 *** dr-orlovsky has quit IRC
204 2020-08-07T10:37:08 *** arowser_ has quit IRC
205 2020-08-07T10:37:30 *** arowser_ has joined #bitcoin-core-dev
206 2020-08-07T10:37:50 *** Shanie14Harris has quit IRC
207 2020-08-07T10:39:20 *** dr-orlovsky has joined #bitcoin-core-dev
208 2020-08-07T10:44:44 *** bitcoin-git has joined #bitcoin-core-dev
209 2020-08-07T10:44:44 <bitcoin-git> [bitcoin] fanquake closed pull request #19675: Run clang-tidy -*,performance-* (master...master) https://github.com/bitcoin/bitcoin/pull/19675
210 2020-08-07T10:44:45 *** bitcoin-git has left #bitcoin-core-dev
211 2020-08-07T10:49:58 *** arowser_ has quit IRC
212 2020-08-07T10:53:13 *** andreacab has quit IRC
213 2020-08-07T10:55:33 *** vasild_ has joined #bitcoin-core-dev
214 2020-08-07T10:58:23 *** vasild has quit IRC
215 2020-08-07T10:58:25 *** vasild_ is now known as vasild
216 2020-08-07T11:00:35 *** kristapsk has quit IRC
217 2020-08-07T11:03:42 *** arowser_ has joined #bitcoin-core-dev
218 2020-08-07T11:14:59 *** promag_ has quit IRC
219 2020-08-07T11:21:27 *** promag has joined #bitcoin-core-dev
220 2020-08-07T11:21:58 *** TallTim has joined #bitcoin-core-dev
221 2020-08-07T11:24:05 *** promag has quit IRC
222 2020-08-07T11:28:36 *** morcos has quit IRC
223 2020-08-07T11:29:01 *** morcos has joined #bitcoin-core-dev
224 2020-08-07T11:35:00 *** dr-orlovsky has quit IRC
225 2020-08-07T11:35:36 *** dr-orlovsky has joined #bitcoin-core-dev
226 2020-08-07T11:41:52 *** syperf has quit IRC
227 2020-08-07T11:49:52 *** dr-orlovsky has quit IRC
228 2020-08-07T11:51:52 *** vincenzopalazzo has quit IRC
229 2020-08-07T11:56:50 *** dr-orlovsky has joined #bitcoin-core-dev
230 2020-08-07T11:58:34 *** promag has joined #bitcoin-core-dev
231 2020-08-07T12:02:54 *** promag has quit IRC
232 2020-08-07T12:14:32 *** reallll has joined #bitcoin-core-dev
233 2020-08-07T12:17:41 *** belcher_ has quit IRC
234 2020-08-07T12:18:00 *** reallll is now known as belcher
235 2020-08-07T12:22:02 *** Raccoon has joined #bitcoin-core-dev
236 2020-08-07T12:28:25 *** vincenzopalazzo has joined #bitcoin-core-dev
237 2020-08-07T12:29:32 *** dr_orlovsky has joined #bitcoin-core-dev
238 2020-08-07T12:32:02 *** Kiminuo has quit IRC
239 2020-08-07T12:36:13 *** arowser_ has quit IRC
240 2020-08-07T12:38:04 *** arowser_ has joined #bitcoin-core-dev
241 2020-08-07T12:40:23 *** tryphe_ has joined #bitcoin-core-dev
242 2020-08-07T12:42:35 *** promag has joined #bitcoin-core-dev
243 2020-08-07T12:43:08 *** tryphe has quit IRC
244 2020-08-07T12:52:27 *** promag has quit IRC
245 2020-08-07T12:53:23 *** Kiminuo has joined #bitcoin-core-dev
246 2020-08-07T12:53:52 *** promag has joined #bitcoin-core-dev
247 2020-08-07T12:56:16 *** Guyver2 has joined #bitcoin-core-dev
248 2020-08-07T13:04:23 *** ghost43_ has quit IRC
249 2020-08-07T13:05:29 *** ghost43 has joined #bitcoin-core-dev
250 2020-08-07T13:06:12 *** arowser_ has quit IRC
251 2020-08-07T13:06:31 *** arowser_ has joined #bitcoin-core-dev
252 2020-08-07T13:07:12 *** arowser_ has quit IRC
253 2020-08-07T13:07:31 *** arowser_ has joined #bitcoin-core-dev
254 2020-08-07T13:08:09 *** arowser_ has quit IRC
255 2020-08-07T13:08:46 *** arowser_ has joined #bitcoin-core-dev
256 2020-08-07T13:09:10 *** arowser_ has quit IRC
257 2020-08-07T13:09:37 *** arowser_ has joined #bitcoin-core-dev
258 2020-08-07T13:10:03 *** Highway61 has joined #bitcoin-core-dev
259 2020-08-07T13:10:11 *** arowser_ has quit IRC
260 2020-08-07T13:10:57 *** arowser_ has joined #bitcoin-core-dev
261 2020-08-07T13:11:10 *** arowser_ has quit IRC
262 2020-08-07T13:11:30 *** arowser_ has joined #bitcoin-core-dev
263 2020-08-07T13:12:11 *** arowser_ has quit IRC
264 2020-08-07T13:12:30 *** arowser_ has joined #bitcoin-core-dev
265 2020-08-07T13:15:12 *** promag has quit IRC
266 2020-08-07T13:23:09 *** bitdex has quit IRC
267 2020-08-07T13:24:11 *** arowser_ has quit IRC
268 2020-08-07T13:24:42 *** arowser_ has joined #bitcoin-core-dev
269 2020-08-07T13:27:10 *** Dean_Guss has joined #bitcoin-core-dev
270 2020-08-07T13:28:11 <sipa> sdaftuar: no no; there was an actual interaction between 19620 and 17977, and 17977's tests started failing after merging (as it didn't move taproot outputs out of WITNESS_UNKNOWN, those ended up being nonstandard to spend)
271 2020-08-07T13:31:53 <sdaftuar> ah!
272 2020-08-07T13:32:23 *** Pavlenex has quit IRC
273 2020-08-07T13:41:06 *** arowser_ has quit IRC
274 2020-08-07T13:43:10 *** arowser_ has joined #bitcoin-core-dev
275 2020-08-07T13:44:30 *** dr-orlovsky has quit IRC
276 2020-08-07T13:45:16 *** dr_orlovsky has quit IRC
277 2020-08-07T13:45:33 *** dr-orlovsky has joined #bitcoin-core-dev
278 2020-08-07T13:47:19 *** dr-orlovsky has quit IRC
279 2020-08-07T13:47:52 *** dr-orlovsky has joined #bitcoin-core-dev
280 2020-08-07T13:52:18 *** promag has joined #bitcoin-core-dev
281 2020-08-07T13:52:25 *** promag has quit IRC
282 2020-08-07T14:00:37 *** mdunnio has joined #bitcoin-core-dev
283 2020-08-07T14:04:25 *** proofofkeags has joined #bitcoin-core-dev
284 2020-08-07T14:09:20 *** arowser_ has quit IRC
285 2020-08-07T14:09:49 *** arowser_ has joined #bitcoin-core-dev
286 2020-08-07T14:10:16 *** arowser_ has quit IRC
287 2020-08-07T14:10:40 *** mdunnio has quit IRC
288 2020-08-07T14:10:45 *** arowser_ has joined #bitcoin-core-dev
289 2020-08-07T14:10:54 *** mdunnio has joined #bitcoin-core-dev
290 2020-08-07T14:11:14 *** arowser_ has quit IRC
291 2020-08-07T14:11:42 *** arowser_ has joined #bitcoin-core-dev
292 2020-08-07T14:11:53 *** promag has joined #bitcoin-core-dev
293 2020-08-07T14:12:39 *** sdaftuar has quit IRC
294 2020-08-07T14:13:04 *** sdaftuar has joined #bitcoin-core-dev
295 2020-08-07T14:19:44 *** justanotheruser has quit IRC
296 2020-08-07T14:20:44 *** promag has quit IRC
297 2020-08-07T14:22:07 *** mrostecki has quit IRC
298 2020-08-07T14:30:20 *** Pavlenex has joined #bitcoin-core-dev
299 2020-08-07T14:33:08 *** arowser_ has quit IRC
300 2020-08-07T14:33:46 *** arowser_ has joined #bitcoin-core-dev
301 2020-08-07T14:44:33 *** Highway61 has quit IRC
302 2020-08-07T14:45:11 *** dongcarl has quit IRC
303 2020-08-07T14:45:23 *** Highway61 has joined #bitcoin-core-dev
304 2020-08-07T14:45:26 *** dongcarl has joined #bitcoin-core-dev
305 2020-08-07T14:47:16 *** arowser_ has quit IRC
306 2020-08-07T14:47:45 *** arowser_ has joined #bitcoin-core-dev
307 2020-08-07T14:59:34 *** Talkless has joined #bitcoin-core-dev
308 2020-08-07T15:00:01 *** Raccoon has quit IRC
309 2020-08-07T15:08:24 *** Highway61 has quit IRC
310 2020-08-07T15:09:17 *** arowser_ has quit IRC
311 2020-08-07T15:09:46 *** arowser_ has joined #bitcoin-core-dev
312 2020-08-07T15:21:29 *** gzhao408 has joined #bitcoin-core-dev
313 2020-08-07T15:21:40 *** superfly1 has joined #bitcoin-core-dev
314 2020-08-07T15:23:17 *** arowser_ has quit IRC
315 2020-08-07T15:23:33 *** Pavlenex has joined #bitcoin-core-dev
316 2020-08-07T15:24:03 *** arowser_ has joined #bitcoin-core-dev
317 2020-08-07T15:29:18 *** Talkless has quit IRC
318 2020-08-07T15:31:50 *** Pavlenex has quit IRC
319 2020-08-07T15:32:29 *** bitcoin-git has joined #bitcoin-core-dev
320 2020-08-07T15:32:30 <bitcoin-git> [bitcoin] sdaftuar opened pull request #19680: 0.20: Add txids with non-standard inputs to reject filter (0.20...2020-08-reject-unknown-wit-0.20) https://github.com/bitcoin/bitcoin/pull/19680
321 2020-08-07T15:32:30 *** bitcoin-git has left #bitcoin-core-dev
322 2020-08-07T15:33:24 *** bitcoin-git has joined #bitcoin-core-dev
323 2020-08-07T15:33:25 <bitcoin-git> [bitcoin] sdaftuar opened pull request #19681: 0.19: Add txids with non-standard inputs to reject filter (0.19...2020-08-reject-unknown-wit-0.19) https://github.com/bitcoin/bitcoin/pull/19681
324 2020-08-07T15:33:37 *** bitcoin-git has left #bitcoin-core-dev
325 2020-08-07T15:35:39 *** superfly1 has quit IRC
326 2020-08-07T15:37:18 *** gzhao408 has quit IRC
327 2020-08-07T15:38:12 *** arowser_ has quit IRC
328 2020-08-07T15:38:49 *** arowser_ has joined #bitcoin-core-dev
329 2020-08-07T15:40:33 *** gzhao408 has joined #bitcoin-core-dev
330 2020-08-07T15:40:40 *** bitcoin-git has joined #bitcoin-core-dev
331 2020-08-07T15:40:40 <bitcoin-git> [bitcoin] hebasto closed pull request #19600: doc: Fix log message for -reindex (master...200727-load-log) https://github.com/bitcoin/bitcoin/pull/19600
332 2020-08-07T15:40:41 *** bitcoin-git has left #bitcoin-core-dev
333 2020-08-07T15:45:03 *** andreacab has joined #bitcoin-core-dev
334 2020-08-07T15:52:05 *** justanotheruser has joined #bitcoin-core-dev
335 2020-08-07T15:52:55 *** proofofkeags has quit IRC
336 2020-08-07T15:55:58 *** leoburd has joined #bitcoin-core-dev
337 2020-08-07T15:57:12 *** proofofkeags has joined #bitcoin-core-dev
338 2020-08-07T15:57:13 *** proofofkeags has quit IRC
339 2020-08-07T15:57:25 *** proofofkeags has joined #bitcoin-core-dev
340 2020-08-07T15:58:26 *** justanotheruser has quit IRC
341 2020-08-07T16:03:10 *** arowser_ has quit IRC
342 2020-08-07T16:03:33 *** arowser_ has joined #bitcoin-core-dev
343 2020-08-07T16:04:05 *** luke-jr has quit IRC
344 2020-08-07T16:04:27 *** luke-jr has joined #bitcoin-core-dev
345 2020-08-07T16:13:42 *** justanotheruser has joined #bitcoin-core-dev
346 2020-08-07T16:13:53 *** andreacab has quit IRC
347 2020-08-07T16:14:19 *** andreacab has joined #bitcoin-core-dev
348 2020-08-07T16:18:06 *** justanotheruser has quit IRC
349 2020-08-07T16:19:09 *** andreacab has quit IRC
350 2020-08-07T16:20:06 *** proofofkeags has quit IRC
351 2020-08-07T16:20:40 *** proofofkeags has joined #bitcoin-core-dev
352 2020-08-07T16:25:08 *** proofofkeags has quit IRC
353 2020-08-07T16:31:33 *** marcoagner has joined #bitcoin-core-dev
354 2020-08-07T16:43:35 *** jeremyrubin has joined #bitcoin-core-dev
355 2020-08-07T16:51:14 *** roconnor_ has joined #bitcoin-core-dev
356 2020-08-07T16:52:17 *** roconnor has quit IRC
357 2020-08-07T16:52:21 *** roconnor_ is now known as roconnor
358 2020-08-07T17:00:04 *** jb55 has quit IRC
359 2020-08-07T17:00:32 *** jb55 has joined #bitcoin-core-dev
360 2020-08-07T17:11:13 *** Highway61 has joined #bitcoin-core-dev
361 2020-08-07T17:18:07 *** mattdrollette is now known as MDrollette
362 2020-08-07T17:22:29 *** proofofkeags has joined #bitcoin-core-dev
363 2020-08-07T17:22:53 *** promag_ has joined #bitcoin-core-dev
364 2020-08-07T17:30:41 *** promag_ has quit IRC
365 2020-08-07T17:32:07 *** watersnake1 has joined #bitcoin-core-dev
366 2020-08-07T17:36:50 *** marcoagner has quit IRC
367 2020-08-07T17:41:54 *** bitcoin-git has joined #bitcoin-core-dev
368 2020-08-07T17:41:55 <bitcoin-git> [bitcoin] dongcarl opened pull request #19683: WIP: depends: Explicitly specify clang search paths for darwin host (master...2020-08-clang-search-path-robustness) https://github.com/bitcoin/bitcoin/pull/19683
369 2020-08-07T17:42:07 *** bitcoin-git has left #bitcoin-core-dev
370 2020-08-07T17:45:45 *** marcoagner has joined #bitcoin-core-dev
371 2020-08-07T17:55:25 *** owowo has quit IRC
372 2020-08-07T17:59:56 *** owowo has joined #bitcoin-core-dev
373 2020-08-07T18:00:01 *** leoburd has quit IRC
374 2020-08-07T18:00:06 *** gzhao408 has quit IRC
375 2020-08-07T18:00:51 *** gzhao408 has joined #bitcoin-core-dev
376 2020-08-07T18:10:47 *** Guyver2 has quit IRC
377 2020-08-07T18:13:40 *** Davterra has quit IRC
378 2020-08-07T18:16:36 *** Chris_Stewart_5 has quit IRC
379 2020-08-07T18:22:32 *** drybjed1 has joined #bitcoin-core-dev
380 2020-08-07T18:35:28 *** Chris_Stewart_5 has joined #bitcoin-core-dev
381 2020-08-07T18:38:19 *** watersnake1 has quit IRC
382 2020-08-07T18:38:23 *** sdaftuar has quit IRC
383 2020-08-07T18:38:48 *** sdaftuar has joined #bitcoin-core-dev
384 2020-08-07T18:57:05 *** Highway61 has quit IRC
385 2020-08-07T18:59:18 *** Highway61 has joined #bitcoin-core-dev
386 2020-08-07T19:05:14 *** gzhao408 has quit IRC
387 2020-08-07T19:18:59 *** Dean_Guss has quit IRC
388 2020-08-07T19:20:16 *** jarthur has joined #bitcoin-core-dev
389 2020-08-07T19:29:14 *** cloudhead has joined #bitcoin-core-dev
390 2020-08-07T19:29:23 *** Davterra has joined #bitcoin-core-dev
391 2020-08-07T19:32:51 *** EagleTM has quit IRC
392 2020-08-07T19:34:13 *** Dean_Guss has joined #bitcoin-core-dev
393 2020-08-07T19:41:03 *** mrostecki has joined #bitcoin-core-dev
394 2020-08-07T19:41:03 *** justanotheruser has joined #bitcoin-core-dev
395 2020-08-07T19:48:33 *** filchef has joined #bitcoin-core-dev
396 2020-08-07T19:52:47 *** filchef has quit IRC
397 2020-08-07T20:00:43 *** arowser_ has quit IRC
398 2020-08-07T20:01:07 *** arowser_ has joined #bitcoin-core-dev
399 2020-08-07T20:04:44 *** troygiorshev has quit IRC
400 2020-08-07T20:05:34 *** troygiorshev has joined #bitcoin-core-dev
401 2020-08-07T20:22:07 *** lnostdal has quit IRC
402 2020-08-07T20:22:47 *** troygiorshev has quit IRC
403 2020-08-07T20:23:04 *** troygiorshev has joined #bitcoin-core-dev
404 2020-08-07T20:28:45 *** EagleTM has joined #bitcoin-core-dev
405 2020-08-07T20:35:36 *** alko has quit IRC
406 2020-08-07T20:41:11 *** promag has joined #bitcoin-core-dev
407 2020-08-07T20:45:32 *** promag has quit IRC
408 2020-08-07T20:46:45 *** proofofkeags has quit IRC
409 2020-08-07T20:47:21 *** proofofkeags has joined #bitcoin-core-dev
410 2020-08-07T20:51:45 *** proofofkeags has quit IRC
411 2020-08-07T20:58:03 *** mrostecki has quit IRC
412 2020-08-07T21:00:01 *** drybjed1 has quit IRC
413 2020-08-07T21:05:07 *** bitcoin-git has joined #bitcoin-core-dev
414 2020-08-07T21:05:07 <bitcoin-git> [bitcoin] JeremyRubin closed pull request #18071: Refactoring CHashWriter & Get{Prevouts,Sequence,Outputs}Hash to SHA256 (master...refactoring-hashers) https://github.com/bitcoin/bitcoin/pull/18071
415 2020-08-07T21:05:08 *** bitcoin-git has left #bitcoin-core-dev
416 2020-08-07T21:16:25 <Kiminuo> Hi, I'm trying to slowly move forward this https://github.com/bitcoin/bitcoin/pull/19245. And it seems I need to put somewhere `-lstdc++fs` (as per https://www.bfilipek.com/2019/05/boost-to-stdfs.html) to fix failing travis configurations (https://travis-ci.org/github/bitcoin/bitcoin/builds/715745829). I know everybody is busy but any advice or pointer would help me immensely as this is not something I know much about. :-|
417 2020-08-07T21:18:09 *** Talkless has joined #bitcoin-core-dev
418 2020-08-07T21:22:20 *** ThunderChicken1 has joined #bitcoin-core-dev
419 2020-08-07T21:23:32 <jeremyrubin> hi Kiminuo
420 2020-08-07T21:23:43 <sipa> Kiminuo: is that needed if you're actually building in c++17 mode?
421 2020-08-07T21:23:51 <jeremyrubin> I don't think you need to do -lstdc++fs if the gcc > 9.0
422 2020-08-07T21:24:02 *** andreacab has joined #bitcoin-core-dev
423 2020-08-07T21:24:15 <jeremyrubin> It sounds like you only need it to build against an earlier version
424 2020-08-07T21:24:34 <Kiminuo> well, I can see that #include <filesystem> is not recognized in those tests
425 2020-08-07T21:24:52 <sipa> -l only affects linking
426 2020-08-07T21:24:58 <sipa> it can't affect header includes
427 2020-08-07T21:25:00 <Kiminuo> I'm not entirely sure c++17 is correctly enabled in my branch
428 2020-08-07T21:25:07 <Kiminuo> ok
429 2020-08-07T21:25:20 <sipa> do you build with --enable-c++17 ?
430 2020-08-07T21:25:53 <jeremyrubin> The travis builds won't work until c++17 is default I guess
431 2020-08-07T21:26:34 <jeremyrubin> This patch might just have to sit for a year for that
432 2020-08-07T21:26:35 <Kiminuo> https://github.com/bitcoin/bitcoin/pull/19245/files#diff-309426131fa2cf2ebbc5cde58961b8a9R19 - Marco Falke actually have a PR that I build upon
433 2020-08-07T21:26:49 <Kiminuo> and I can see a few instance of `--enable-c++17`
434 2020-08-07T21:27:07 <Kiminuo> *instances
435 2020-08-07T21:27:16 <jeremyrubin> you need to pass it to ./configure --enable-c++17
436 2020-08-07T21:27:28 <Kiminuo> jeremyrubin, I'm fine with waiting :)
437 2020-08-07T21:28:08 <jeremyrubin> sipa may know otherwise, but I suspect waiting till we are no longer building for c++11 is the most likely case due to the API changes
438 2020-08-07T21:28:15 *** Pavlenex has joined #bitcoin-core-dev
439 2020-08-07T21:28:47 *** andreacab has quit IRC
440 2020-08-07T21:28:49 <gwillen> the change to configure.ac in that PR does appear to make c++17 unconditional in configure
441 2020-08-07T21:29:02 <gwillen> it removes the configure flag and forces it on
442 2020-08-07T21:29:13 *** andreacab has joined #bitcoin-core-dev
443 2020-08-07T21:29:14 <gwillen> (I don't know if there could be other places that were using it that got missed though)
444 2020-08-07T21:29:27 <Kiminuo> https://github.com/bitcoin/bitcoin/pull/19245/files#diff-67e997bcfdac55191033d57a16d1408aR72 - do you mean this, right?
445 2020-08-07T21:30:19 <gwillen> right, yeah. So I'm not sure if $use_cxx17 is referenced anywhere else, or if that change is enough to be the same as passing the flag to configure.
446 2020-08-07T21:30:41 *** andreacab has quit IRC
447 2020-08-07T21:30:48 *** andreacab has joined #bitcoin-core-dev
448 2020-08-07T21:31:00 <gwillen> I don't see any other uses, so off the top of my head I'd expect that to work
449 2020-08-07T21:31:55 <Kiminuo> Well, travis shows:
450 2020-08-07T21:31:56 <Kiminuo> ./fs.h:14:10: fatal error: filesystem: No such file or directory
451 2020-08-07T21:31:56 <Kiminuo> #include <filesystem>
452 2020-08-07T21:31:56 <Kiminuo> ^~~~~~~~~~~~
453 2020-08-07T21:31:59 <Kiminuo> here https://travis-ci.org/github/bitcoin/bitcoin/jobs/715745835
454 2020-08-07T21:32:02 <gwillen> looking at the travis failures it does look like some headers got rearranged in c++17 that you haven't accounted for, but I don't know if you were even looking at that yet (e.g. std::function got moved into #include <functional>, which is causing a ton of errors, but not the ones you're asking about)
455 2020-08-07T21:33:37 <sipa> Kiminuo: but your changes to use std::filesystem instead of boost::filesystem should only be active whenever you're building in C++17 mode
456 2020-08-07T21:33:49 <sipa> i think?
457 2020-08-07T21:34:03 <sipa> ah
458 2020-08-07T21:34:07 <sipa> sorry, just reading
459 2020-08-07T21:34:11 <jeremyrubin> sipa: they're building on mandatory c++17
460 2020-08-07T21:34:17 <Kiminuo> gwillen, I have based my PR on this: https://github.com/bitcoin/bitcoin/pull/19183
461 2020-08-07T21:34:17 <sipa> hmm, ok
462 2020-08-07T21:34:27 <sipa> and does the problem arise on all platforms?
463 2020-08-07T21:34:35 <gwillen> configure does say at the end: CXX = /usr/bin/ccache g++ -m64 -std=c++17
464 2020-08-07T21:34:59 <Kiminuo> sipa, all I know is this result: https://travis-ci.org/github/bitcoin/bitcoin/builds/715745829
465 2020-08-07T21:35:09 <Kiminuo> > but your changes to use std::filesystem instead of boost::filesystem should only be active whenever you're building in C++17 mode
466 2020-08-07T21:35:25 <sipa> yeah, ignore me;
467 2020-08-07T21:35:30 <Kiminuo> that would be preferable I guess, but it's out of scope of my ability
468 2020-08-07T21:36:37 <sipa> erhhh
469 2020-08-07T21:36:49 <gwillen> it seems that the <filesystem> library may not be broadly available yet, even in c++17 mode
470 2020-08-07T21:36:50 <sipa> the windows build fails to include <filesystem>
471 2020-08-07T21:37:04 <sipa> that sounds like we just can't use it
472 2020-08-07T21:37:11 <sipa> even in c++17 mode
473 2020-08-07T21:37:23 <gwillen> it's hard for me to find up-to-date info though
474 2020-08-07T21:37:34 <Kiminuo> Interestingly, I have a C++ project on Windows where <filesystem> works.
475 2020-08-07T21:37:34 <gwillen> it seems likely there is a minimum compiler version to support it
476 2020-08-07T21:37:38 <jeremyrubin> Kiminuo: I would just wait on it till mandatory c++17 goes in
477 2020-08-07T21:37:46 <jeremyrubin> it should be like a year or so
478 2020-08-07T21:37:48 <sipa> jeremyrubin: that won't help
479 2020-08-07T21:37:59 <sipa> because c++17 is enabled in these builds that are failing
480 2020-08-07T21:38:16 <jeremyrubin> sipa: wait on it meaning, not worth spending more time now?
481 2020-08-07T21:38:20 <gwillen> what version of gcc do we use? It appears <filesystem> is introduced without qualifications in gcc 8
482 2020-08-07T21:38:31 <gwillen> before that you can use <experimental/filesystem> with an additional compile flag
483 2020-08-07T21:38:34 <sipa> jeremyrubin: waiting on mandatory c++17 won't change the fact that these builds fail
484 2020-08-07T21:38:52 <sipa> oh
485 2020-08-07T21:38:56 <sipa> ok, i get it
486 2020-08-07T21:39:12 <sipa> i assumed that -std=c++17 would only _work_ at all on compilers that fully support c++17
487 2020-08-07T21:39:32 <gwillen> yeah it seems that it means "give me as much c++17 as you have"
488 2020-08-07T21:39:40 <sipa> right
489 2020-08-07T21:39:46 <jeremyrubin> the shakespeare rule
490 2020-08-07T21:39:54 <gwillen> https://old.reddit.com/r/cpp/comments/7o9kg6/gcc_80_support_stdfilesystem_include_filesystem/
491 2020-08-07T21:40:20 <sipa> so we won't make c++17 mandatory until we drop support for all platforms that don't support it fully (or at least, sufficiently)
492 2020-08-07T21:40:40 <jeremyrubin> Or, we will do that
493 2020-08-07T21:40:48 <jeremyrubin> as long as we have libs we can still use
494 2020-08-07T21:40:55 <jeremyrubin> like boost::filesystem
495 2020-08-07T21:40:55 <Kiminuo> https://travis-ci.org/github/bitcoin/bitcoin/jobs/715745835 - I can see in this build that GCC 7.5 is used. Is enough for <filesystem>?
496 2020-08-07T21:40:55 <sipa> how so?
497 2020-08-07T21:40:56 <gwillen> Kiminuo: if you still want to experiment, understanding it could be quite awhile before this is feasible in production, it looks like you could try including <experimental/filesystem> and linking with "-lstdc++fs"
498 2020-08-07T21:41:13 <gwillen> Kiminuo: no it seems you need 8.0
499 2020-08-07T21:41:26 <Kiminuo> gwillen, so that may be the issue, right?
500 2020-08-07T21:41:31 <gwillen> yeah I think that is indeed the issue
501 2020-08-07T21:41:38 <jeremyrubin> I think c++17 will be fine for even "as much as you have of 17" compilers with the current code
502 2020-08-07T21:41:55 <jeremyrubin> but we may not be able to use *all* c++17 features until then
503 2020-08-07T21:41:58 <gwillen> your local compiler is newer than the one that CI is using, so you have <filesystem> but CI does not
504 2020-08-07T21:42:17 <gwillen> I assume the compiler CI uses is the benchmark for "minimum supported to build bitcoin with"?
505 2020-08-07T21:42:37 <Kiminuo> gwillen, I don't think <experimental/filesystem> is way to go. It will make it more complex and it does not sound good to use something experimental in Bitcoin.
506 2020-08-07T21:43:31 <gwillen> I believe it is the same library, it just moved when it was finally standardized to a new name. But I agree it may not be suitable for inclusion into Bitcoin for real for now.
507 2020-08-07T21:43:48 <gwillen> especially since boost::filesystem is probably _also_ the same library under a different name, more or less
508 2020-08-07T21:45:33 *** andreacab has quit IRC
509 2020-08-07T21:46:35 <Kiminuo> std::filesystem is kind of renamed boost::filesystem with some modifications, but the details are sometimes not that easy to find (at least for me)
510 2020-08-07T21:46:39 *** andreacab has joined #bitcoin-core-dev
511 2020-08-07T21:47:12 <gwillen> yeah, that's usually how it goes with C++ library features being standardized I think -- they start in boost and work their way into the standard library
512 2020-08-07T21:50:00 <sipa> GCC 7 jusy predates the c++17 standard being finalized
513 2020-08-07T21:50:31 <sipa> so i guess that'z why it's only available under experimental - at the time it was written there was no way to know of that would eventually match the spec
514 2020-08-07T21:51:01 <sipa> GCC 7.5 is more than recent enough, but i guess these things are only updated in major releasesm
515 2020-08-07T21:51:39 *** andreacab has quit IRC
516 2020-08-07T21:53:03 <Kiminuo> https://travis-ci.org/github/bitcoin/bitcoin/jobs/715745836 - but here GCC is also 7.5 and the test passes
517 2020-08-07T21:53:16 <sipa> hmm, strange
518 2020-08-07T21:53:33 <Kiminuo> actually it's verbatim the same thing as in the failing test
519 2020-08-07T21:58:09 *** promag has joined #bitcoin-core-dev
520 2020-08-07T22:00:31 <Kiminuo> https://www.diffchecker.com/Q52OO5NI - "Build system information" sections are actually the same for OK test & failing test.
521 2020-08-07T22:01:23 <gwillen> according to https://gcc.gnu.org/onlinedocs/libstdc++/manual/status.html#status.iso.2017 it was not added until 8.1 _and_ it still requires a special link argument, -lstdc++fs
522 2020-08-07T22:01:29 <gwillen> so, there's something weird happening
523 2020-08-07T22:11:02 <Kiminuo> yeah, I have mentioned that earlier
524 2020-08-07T22:11:12 <Kiminuo> (that link argument)
525 2020-08-07T22:12:02 <Kiminuo> I'm not sure how long it would take to wait for GCC 8.1 to be widespread.
526 2020-08-07T22:14:04 <sipa> this is something to report on https://github.com/bitcoin/bitcoin/issues/16684
527 2020-08-07T22:15:22 <sipa> so perhaps the right thing to do is making std::filesystem based builds still configurable, and only enable them on GCC 8.1+ platforms and others that we know support it
528 2020-08-07T22:16:45 <sipa> so that the lack of full std::filesystem support doesn't hold up otherwise migrating to C++17
529 2020-08-07T22:17:21 <sipa> gwillen, Kiminuo: perhaps a discrepancy is due to libstdc++ and gcc not being the exact same thing and available in lockstep?
530 2020-08-07T22:17:34 <sipa> std::filesystem support is purely a libstdc++ side thing, i expect
531 2020-08-07T22:19:04 *** promag has quit IRC
532 2020-08-07T22:19:17 <Kiminuo> sipa, I don't know about that configuration. The APIs of both libraries is not in one-to-one correspondence. It can be done by a wrapper I guess.
533 2020-08-07T22:19:52 <sipa> well, that wrapper already exists
534 2020-08-07T22:19:57 <sipa> this is why we have the fs namespace
535 2020-08-07T22:20:00 *** promag has joined #bitcoin-core-dev
536 2020-08-07T22:20:04 <Kiminuo> True
537 2020-08-07T22:20:13 <Kiminuo> But you would need to add more functions
538 2020-08-07T22:20:52 <sipa> i expect that slight differences between boost::filesystem and std::filesystem make it not exactly a "using fs = boost::filesystem" vs "using fs = std::filesystem", but it shouldn't be too much more than that
539 2020-08-07T22:21:37 <Kiminuo> https://github.com/bitcoin/bitcoin/pull/19245 - it's documented here
540 2020-08-07T22:21:53 <Kiminuo> The differences are not that big
541 2020-08-07T22:22:20 <sipa> ok
542 2020-08-07T22:22:23 *** Guest25949 has quit IRC
543 2020-08-07T22:22:52 <Kiminuo> But if you have a fs::path and you use operator "/", I think it behaves slightly differently. And that may be hard to explain to people
544 2020-08-07T22:23:07 <sipa> what is the difference?
545 2020-08-07T22:23:33 *** per has joined #bitcoin-core-dev
546 2020-08-07T22:23:56 *** per is now known as Guest55343
547 2020-08-07T22:24:48 <Kiminuo> https://github.com/bitcoin/bitcoin/pull/19466#discussion_r451809243
548 2020-08-07T22:24:54 <Kiminuo> there was something like this
549 2020-08-07T22:25:09 <Kiminuo> let me check what that was actually about
550 2020-08-07T22:25:14 *** andreacab has joined #bitcoin-core-dev
551 2020-08-07T22:26:49 <Kiminuo> PR 19466 is based on boost::filesystem and Marco said that in https://github.com/bitcoin/bitcoin/pull/19466#discussion_r451809243 you would get `/tmp//sadf` and in std::filesystem you would get "/foo"
552 2020-08-07T22:27:09 <Kiminuo> I hope I'm saying that correctly
553 2020-08-07T22:27:52 <sipa> heh
554 2020-08-07T22:33:32 <Kiminuo> That's pretty nasty, if true
555 2020-08-07T22:34:04 <Kiminuo> That makes me thing that using boost::filesystem & std::filesystem at the same time may bring troubles
556 2020-08-07T22:37:02 *** Talkless has quit IRC
557 2020-08-07T22:42:57 *** andreacab has quit IRC
558 2020-08-07T22:55:33 *** vasild_ has joined #bitcoin-core-dev
559 2020-08-07T22:58:43 *** vasild has quit IRC
560 2020-08-07T22:58:44 *** vasild_ is now known as vasild
561 2020-08-07T23:04:29 *** promag has quit IRC
562 2020-08-07T23:14:21 *** Pavlenex has joined #bitcoin-core-dev
563 2020-08-07T23:17:35 *** Pavlenex has quit IRC
564 2020-08-07T23:19:25 *** Chris_Stewart_5 has quit IRC
565 2020-08-07T23:29:02 *** marcoagner has quit IRC
566 2020-08-07T23:31:05 *** Davterra has quit IRC
567 2020-08-07T23:34:06 *** arowser_ has quit IRC
568 2020-08-07T23:34:34 *** arowser_ has joined #bitcoin-core-dev
569 2020-08-07T23:34:35 *** promag has joined #bitcoin-core-dev
570 2020-08-07T23:41:06 *** arowser_ has quit IRC
571 2020-08-07T23:41:34 *** arowser_ has joined #bitcoin-core-dev
572 2020-08-07T23:54:06 *** arowser_ has quit IRC
573 2020-08-07T23:54:35 *** arowser_ has joined #bitcoin-core-dev
574 2020-08-07T23:59:26 *** troygiorshev has quit IRC