Tarantool development patches archive
 help / color / mirror / Atom feed
From: "Alexander V. Tikhonov" <avtikhon@tarantool.org>
To: Oleg Piskunov <o.piskunov@tarantool.org>,
	Sergey Bronnikov <sergeyb@tarantool.org>
Cc: tarantool-patches@dev.tarantool.org,
	Alexander Turenko <alexander.turenko@tarantool.org>
Subject: [Tarantool-patches] [PATCH v2 0/7] Implement OOS build
Date: Tue, 26 May 2020 17:42:31 +0300	[thread overview]
Message-ID: <cover.1590503508.git.avtikhon@tarantool.org> (raw)

Implemented cmake for curl build and setup OOS build with it.
Described in short the patches as it were committed.

7.  gitlab-ci: add out-of-source build
    
    Implemented out-of-source build at cmake files.
    Added out of source build make targets and added
    test job to gitlab-ci.
    
    Closes #4874

6.  Skip failing test from out-of-source build
    
    Skipped failing test app-tap/http_client.test.lua
    from out-of-source build. Left #4258 issue opened
    till it will be fixed.
    
    Part of #4874

5.  build: change autoconf to cmake in curl build
    
    Completely changed autoconf to cmake in curl build. After curl
    sources were changed to be able to be build since 2.8 version
    the old OS like CentOS 6/7 and Ubuntu 14.04 became available
    for curl build using cmake.
    
    Autoconf part completely removed and code cleaned up for cmake.
    
    1. Fixed issue with building on CentOS 6:

       It was fixed with added "-lrt" flag to CMAKE_C_FLAGS and
       CMAKE_CXX_FLAGS build flags, when cmake version is lower
       than 3.0 and RT library had needed function.
    
    2. Fixed issue with building FreeBSD 12: app/socket test failed.

       It was fixed with added "-DLDFLAGS=" flag to cmake call.
    
    3. Fixed issue with static build using CentOS 7, where SSL cmake rule
       failed.
    
       It was fixed in issue #5019.
    
    Closes #4968
    Closes #5020

4.  Fix curl repository for out-of-source build
    
    Temporary added branch in curl repository
    with fix for out-of-source build.
    
    Part of #4874

3.  build: enable cmake in curl build
    
    Changed autoconf tools in Curl build to cmake use for builds where
    cmake major starts from 3.

    Part of #4968

2.  build: fix static build w/ dockerfile with cmake
    
    Fixed cmake OS build file:
    
      /usr/share/cmake/Modules/FindOpenSSL.cmake
    
    it had:
           REGEX "^#define[\t ]+OPENSSL_VERSION_NUMBER[\t ]+0x([0-9a-fA-F])+.*")
    
    changed to:
           REGEX "^#[\t ]*define[\t ]+OPENSSL_VERSION_NUMBER[\t ]+0x([0-9a-fA-F])+.*")
    
    Closes #5019

1.  build: static build needs more cleanup in sources
    
    Added cleanup for test/small test/luajit-tap directories.
    
    Closes #5025

---

Alexander V. Tikhonov (7):
  build: static build needs more cleanup in sources
  build: fix static build w/ dockerfile with cmake
  build: enable cmake in curl build
  Fix curl repository for out-of-source build
  build: change autoconf to cmake in curl build
  Skip failing test from out-of-source build
  gitlab-ci: add out-of-source build

 .gitlab-ci.yml                    |   7 ++
 .travis.mk                        |  26 ++++
 Dockerfile.staticbuild            |   5 +-
 cmake/BuildLibCURL.cmake          | 201 +++++++++++-------------------
 cmake/utils.cmake                 |   4 +-
 src/box/CMakeLists.txt            |   1 +
 test/app-tap/http_client.skipcond |   5 +
 third_party/curl                  |   2 +-
 8 files changed, 119 insertions(+), 132 deletions(-)

-- 
2.17.1

             reply	other threads:[~2020-05-26 14:42 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-05-26 14:42 Alexander V. Tikhonov [this message]
2020-05-26 14:42 ` [Tarantool-patches] [PATCH v2 1/7] build: static build needs more cleanup in sources Alexander V. Tikhonov
2020-06-26 13:51   ` Alexander Turenko
2020-07-03 11:13   ` Kirill Yukhin
2020-05-26 14:42 ` [Tarantool-patches] [PATCH v2 2/7] build: fix static build w/ dockerfile with cmake Alexander V. Tikhonov
2020-05-26 14:42 ` [Tarantool-patches] [PATCH v2 3/7] build: enable cmake in curl build Alexander V. Tikhonov
2020-05-26 14:42 ` [Tarantool-patches] [PATCH v2 4/7] Fix curl repository for out-of-source build Alexander V. Tikhonov
2020-05-26 14:42 ` [Tarantool-patches] [PATCH v2 5/7] build: change autoconf to cmake in curl build Alexander V. Tikhonov
2020-05-26 14:42 ` [Tarantool-patches] [PATCH v2 6/7] Skip failing test from out-of-source build Alexander V. Tikhonov
2020-05-26 14:42 ` [Tarantool-patches] [PATCH v2 7/7] gitlab-ci: add " Alexander V. Tikhonov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cover.1590503508.git.avtikhon@tarantool.org \
    --to=avtikhon@tarantool.org \
    --cc=alexander.turenko@tarantool.org \
    --cc=o.piskunov@tarantool.org \
    --cc=sergeyb@tarantool.org \
    --cc=tarantool-patches@dev.tarantool.org \
    --subject='Re: [Tarantool-patches] [PATCH v2 0/7] Implement OOS build' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox