Re: [Tarantool-patches] [PATCH 0/3] Export curl symbols, enable smtp and install headers

[Leonid Vasiliev via Tarantool-patches <tarantool-patches@dev.tarantool.org>]

Hi! Thank you for the patchset.
Generally LGTM.
See several commits below:
1) Add changelogs.
2) Have you checked the patch on some deb and rpm based distributions?


On 3/19/21 5:13 PM, Roman Khabibov via Tarantool-patches wrote:
> The patchset is for 1.10 only.
> 
> Issue: https://github.com/tarantool/smtp/issues/24
> Branch: https://github.com/tarantool/tarantool/tree/romanhabibov/curl-smtp_1_10
> 
> Roman Khabibov (3):
>    build: export libCURL symbols
>    build: enable smtp in libCURL
>    build: install libCURL headers
> 
>   CMakeLists.txt                   |  17 ++++
>   cmake/BuildLibCURL.cmake         |   2 +-
>   extra/curl_symbols               |  81 ++++++++++++++++
>   extra/mkexports                  |  14 ++-
>   rpm/tarantool.spec               |   9 ++
>   src/CMakeLists.txt               |   5 +-
>   test/box-tap/curl-build.test.lua | 160 +++++++++++++++++++++++++++++++
>   7 files changed, 282 insertions(+), 6 deletions(-)
>   create mode 100755 extra/curl_symbols
>   create mode 100755 test/box-tap/curl-build.test.lua
>