Re: [Tarantool-patches] [PATCH 2/3] build: enable smtp in libCURL

[Alexander Turenko via Tarantool-patches <tarantool-patches@dev.tarantool.org>]

On Fri, Mar 19, 2021 at 05:13:07PM +0300, Roman Khabibov wrote:
> Enable smtp protocol in bundled libCURL.
> ---
>  cmake/BuildLibCURL.cmake         |  2 +-
>  test/box-tap/curl-build.test.lua | 38 +++++++++++++++++++++++++++++++-
>  2 files changed, 38 insertions(+), 2 deletions(-)
> 
> diff --git a/cmake/BuildLibCURL.cmake b/cmake/BuildLibCURL.cmake
> index 5f8b15a63..313adfb97 100644
> --- a/cmake/BuildLibCURL.cmake
> +++ b/cmake/BuildLibCURL.cmake
> @@ -124,6 +124,7 @@ macro(curl_build)
>                  --enable-http-auth
>                  --enable-mime
>                  --enable-dateparse
> +                --enable-smtp
>  
>                  ${DISABLED_DNS_OPT}
>                  --disable-ftp
> @@ -137,7 +138,6 @@ macro(curl_build)
>                  --disable-pop3
>                  --disable-imap
>                  --disable-smb
> -                --disable-smtp
>                  --disable-gopher
>                  --disable-manual
>                  --disable-sspi

It seems, it is quite straightforward backport of [1], so it is okay
after comments from the original patch ([2]).

Don't miss changelog entries on 1.10.

[1]: https://lists.tarantool.org/pipermail/tarantool-patches/2021-March/022797.html
[2]: https://lists.tarantool.org/pipermail/tarantool-patches/2021-March/023177.html

I performed tests 2 and 3 from [2] and the result looks the same. It
seems, we don't enable or disable something unintentionally here as
well.

WBR, Alexander Turenko.