diff options
| author | Philippe Blain <levraiphilippeblain@gmail.com> | 2024-02-02 00:18:33 +0000 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2024-02-01 18:20:41 -0800 |
| commit | 30bced3a6780e4db964fde6ce6d5950ff98bfdde (patch) | |
| tree | 1e8d3a688a808a79122488a9b5cc6cd5b3a7bf35 /imap-send.c | |
| parent | d57c671a511d885a5cd390e3d6064c37af524a91 (diff) | |
| download | git-30bced3a6780e4db964fde6ce6d5950ff98bfdde.tar.gz | |
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 eea0e59ffb (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 3307f7dde2
(imap-send: include strbuf.h, 2023-05-17) after a similar breakage in
ba3d1c73da (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 eea0e59ffb.
[1] https://lore.kernel.org/git/20230517070632.71884-1-list@eworm.de/
Signed-off-by: Philippe Blain <levraiphilippeblain@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'imap-send.c')
| -rw-r--r-- | imap-send.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/imap-send.c b/imap-send.c index 904cb23cf5..9b1806a285 100644 --- a/imap-send.c +++ b/imap-send.c @@ -28,6 +28,7 @@ #include "run-command.h" #include "parse-options.h" #include "setup.h" +#include "strbuf.h" #if defined(NO_OPENSSL) && !defined(HAVE_OPENSSL_CSPRNG) typedef void *SSL; #endif |