Christian Hesse <list@xxxxxxxx> writes: > Christian Hesse <list@xxxxxxxx> on Wed, 2023/05/17 22:12: >> > OK, so the fix seems to make sense, but the justification for the >> > change needs to be rewritten, I think. >> > >> > We make liberal use of the strbuf API functions and types, but >> > the inclusion of <strbuf.h> comes indirectly by including >> > <http.h>, which does not happen if you build with NO_CURL. >> > >> > or something like that? >> >> Fine with me! >> Do you want me to re-send the patch or do you modify this on the fly? > > Found it in next branch already. Thanks a lot! We made moderate amount of these kind of header shuffling this cycle, and it is inevitable to encounter a gotcha like this when building with anything less than the most common configurations. Thanks for reporting and fixing; very much appreciated.