Subject: Re: [PATCH 1/1] imap-send: include strbuf.h

From: Junio C Hamano <[email-masked]>
Date: Thu, 18 May 2023 08:56:55 -0700

Christian Hesse <[email-masked]> writes:

> Christian Hesse <[email-masked]> 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.