Skip to content

Commit 30bced3

Browse files
phil-blaingitster
authored andcommitted
imap-send: add missing "strbuf.h" include under NO_CURL
Building with NO_CURL is currently broken since imap-send.c uses things defined in "strbuf.h" wihtout including it. The inclusion of that header was removed in eea0e59 (treewide: remove unnecessary includes in source files, 2023-12-23), which failed to notice that "strbuf.h" was transitively included in imap-send.c via "http.h", but only if USE_CURL_FOR_IMAP_SEND is defined. Add back the missing include. Note that it was explicitely added in 3307f7d (imap-send: include strbuf.h, 2023-05-17) after a similar breakage in ba3d1c7 (treewide: remove unnecessary cache.h includes, 2023-02-24) - see the thread starting at [1]. It can be verified by inspection that this is the only case where a header we include is dependent on a Makefile knob in the files modified in eea0e59. [1] https://lore.kernel.org/git/[email protected]/ Signed-off-by: Philippe Blain <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent d57c671 commit 30bced3

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

imap-send.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@
2828
#include "run-command.h"
2929
#include "parse-options.h"
3030
#include "setup.h"
31+
#include "strbuf.h"
3132
#if defined(NO_OPENSSL) && !defined(HAVE_OPENSSL_CSPRNG)
3233
typedef void *SSL;
3334
#endif

0 commit comments

Comments
 (0)