Tarantool development patches archive
 help / color / mirror / Atom feed
From: "Sergei Voronezhskii" <sergw@tarantool.org>
To: tarantool-patches@freelists.org
Cc: "Alexander Turenko" <alexander.turenko@tarantool.org>,
	"Kirill Yukhin" <kyukhin@tarantool.org>
Subject: [tarantool-patches] Re: [tarantool-patches] [PATCH] Add Linux/clang CI target
Date: Mon, 08 Oct 2018 06:51:43 +0300	[thread overview]
Message-ID: <1538970703.343755675@f533.i.mail.ru> (raw)
In-Reply-To: <b1b1d58b4f112b1adb9b52a8e09d11cdb76535f1.1537576753.git.alexander.turenko@tarantool.org>

[-- Attachment #1: Type: text/plain, Size: 4035 bytes --]

LGTM


>Суббота, 22 сентября 2018, 3:40 +03:00 от Alexander Turenko <alexander.turenko@tarantool.org>:
>
>Replaced targets generation using a matrix expansion + exclusion list
>with the explicit targets list. Gave meagingful names for targets.
>
>Fixes #3673.
>---
>
>branch:  https://github.com/tarantool/tarantool/tree/Totktonada/gh-3673-add-linux-clang-ci-target
>issue:  https://github.com/tarantool/tarantool/issues/3673
>
> .travis.yml | 94 +++++++++++++++++++++--------------------------------
> 1 file changed, 37 insertions(+), 57 deletions(-)
>
>diff --git a/.travis.yml b/.travis.yml
>index bcb018be8..a9d1036da 100644
>--- a/.travis.yml
>+++ b/.travis.yml
>@@ -4,9 +4,9 @@ services:
> 
> language: cpp
> 
>-os:
>-  - linux
>-  - osx
>+# default values
>+os: linux
>+compiler: gcc
> 
> osx_image: xcode9
> 
>@@ -17,64 +17,44 @@ cache:
> git:
>     depth: 100500
> 
>-env:
>-    matrix:
>-      - TARGET=source
>-      - TARGET=test
>-      - TARGET=coverage
>-      - OS=el DIST=6
>-      - OS=el DIST=7
>-      - OS=fedora DIST=26
>-      - OS=fedora DIST=27
>-      - OS=ubuntu DIST=artful
>-      - OS=ubuntu DIST=trusty
>-      - OS=ubuntu DIST=xenial
>-      - OS=ubuntu DIST=bionic
>-      - OS=debian DIST=wheezy
>-      - OS=debian DIST=jessie
>-      - OS=debian DIST=stretch
>-
> matrix:
>     allow_failures:
>       - TARGET=coverage
>-#      - env: OS=el DIST=6
>-#      - env: OS=el DIST=7
>-#      - env: OS=fedora DIST=26
>-#      - env: OS=fedora DIST=27
>-#      - env: OS=ubuntu DIST=artful
>-#      - env: OS=ubuntu DIST=trusty
>-#      - env: OS=ubuntu DIST=xenial
>-#      - env: OS=ubuntu DIST=bionic
>-#      - env: OS=debian DIST=wheezy
>-#      - env: OS=debian DIST=jessie
>-#      - env: OS=debian DIST=stretch
>-    exclude:
>-      - env: OS=el DIST=6
>-        os: osx
>-      - env: OS=el DIST=7
>-        os: osx
>-      - env: OS=fedora DIST=26
>-        os: osx
>-      - env: OS=fedora DIST=27
>-        os: osx
>-      - env: OS=ubuntu DIST=artful
>-        os: osx
>-      - env: OS=ubuntu DIST=trusty
>-        os: osx
>-      - env: OS=ubuntu DIST=xenial
>-        os: osx
>-      - env: OS=ubuntu DIST=bionic
>-        os: osx
>-      - env: OS=debian DIST=wheezy
>-        os: osx
>-      - env: OS=debian DIST=jessie
>-        os: osx
>-      - env: OS=debian DIST=stretch
>-        os: osx
>-      - env: TARGET=source
>-        os: osx
>-      - env: TARGET=coverage
>+    include:
>+      - name: Create and deploy tarball
>+        env: TARGET=source
>+      - name: RelWithDebInfoWError build + test (Linux, gcc)
>+        env: TARGET=test
>+      - name: RelWithDebInfoWError build + test (Linux, clang)
>+        env: TARGET=test
>+        compiler: clang
>+      - name: RelWithDebInfoWError build + test (OS X)
>+        env: TARGET=test
>         os: osx
>+      - name: Debug build + test + coverage (Linux, gcc)
>+        env: TARGET=coverage
>+      - name: CentOS 6 build + test + deploy RPM
>+        env: OS=el DIST=6
>+      - name: CentOS 7 build + test + deploy RPM
>+        env: OS=el DIST=7
>+      - name: Fedora 26 build + test + deploy RPM
>+        env: OS=fedora DIST=26
>+      - name: Fedora 27 build + test + deploy RPM
>+        env: OS=fedora DIST=27
>+      - name: Ubuntu Trusty (14.04) build + deploy DEB
>+        env: OS=ubuntu DIST=trusty
>+      - name: Ubuntu Xenial (16.04) build + deploy DEB
>+        env: OS=ubuntu DIST=xenial
>+      - name: Ubuntu Artful (17.10) build + deploy DEB
>+        env: OS=ubuntu DIST=artful
>+      - name: Ubuntu Bionic (18.04) build + deploy DEB
>+        env: OS=ubuntu DIST=bionic
>+      - name: Debian Wheezy (7) build + deploy DEB
>+        env: OS=debian DIST=wheezy
>+      - name: Debian Jessie (8) build + deploy DEB
>+        env: OS=debian DIST=jessie
>+      - name: Debian Stretch (9) build + deploy DEB
>+        env: OS=debian DIST=stretch
> 
> script:
>   - make -f .travis.mk ${TARGET}
>-- 
>2.19.0
>
>


-- 
Sergei Voronezhskii

[-- Attachment #2: Type: text/html, Size: 5272 bytes --]

  reply	other threads:[~2018-10-08  3:51 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-09-22  0:40 Alexander Turenko
2018-10-08  3:51 ` Sergei Voronezhskii [this message]
2018-10-12 11:52 ` [tarantool-patches] " Kirill Yukhin

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=1538970703.343755675@f533.i.mail.ru \
    --to=sergw@tarantool.org \
    --cc=alexander.turenko@tarantool.org \
    --cc=kyukhin@tarantool.org \
    --cc=tarantool-patches@freelists.org \
    --subject='[tarantool-patches] Re: [tarantool-patches] [PATCH] Add Linux/clang CI target' \
    /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