From d5ec498e26c26fc175b7349b6041738c9ff811e3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ond=C5=99ej=20Sur=C3=BD?= Date: Thu, 19 Jul 2018 11:32:32 -0400 Subject: [PATCH 1/4] Add Debian jessie and stretch images to the build to catch errors with older but supported releases --- .gitlab-ci.yml | 38 ++++++++++++++++++++++++++++++++++++++ 1 file changed, 38 insertions(+) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index c071b7c440..0b46276d75 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -214,6 +214,20 @@ build:clang:debian:sid:amd64: <<: *debian_sid_amd64_image <<: *build_job +build:debian:jessie:amd64: + variables: + CC: gcc + CFLAGS: "-Wall -Wextra -O2 -g" + <<: *debian_jessie_amd64_image + <<: *build_job + +build:debian:stretch:amd64: + variables: + CC: gcc + CFLAGS: "-Wall -Wextra -O2 -g" + <<: *debian_stretch_amd64_image + <<: *build_job + build:debian:sid:amd64: variables: CC: gcc @@ -247,6 +261,18 @@ unittest:centos:centos7:amd64: dependencies: - build:centos:centos7:amd64 +unittest:debian:jessie:amd64: + <<: *debian_jessie_amd64_image + <<: *unit_test_job + dependencies: + - build:debian:jessie:amd64 + +unittest:debian:stretch:amd64: + <<: *debian_stretch_amd64_image + <<: *unit_test_job + dependencies: + - build:debian:stretch:amd64 + unittest:debian:sid:amd64: <<: *debian_sid_amd64_image <<: *unit_test_job @@ -277,6 +303,18 @@ systemtest:centos:centos7:amd64: dependencies: - build:centos:centos7:amd64 +systemtest:debian:jessie:amd64: + <<: *debian_jessie_amd64_image + <<: *system_test_job + dependencies: + - build:debian:jessie:amd64 + +systemtest:debian:stretch:amd64: + <<: *debian_stretch_amd64_image + <<: *system_test_job + dependencies: + - build:debian:stretch:amd64 + systemtest:debian:sid:amd64: <<: *debian_sid_amd64_image <<: *system_test_job From 1d8a62713d6828674f0f22e5c2ac127e5d63ce5d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ond=C5=99ej=20Sur=C3=BD?= Date: Thu, 19 Jul 2018 11:34:52 -0400 Subject: [PATCH 2/4] Disable autogen.sh in precheck, it needs newer autoconf and it breaks the older builds --- .gitlab-ci.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 0b46276d75..88da5fcf85 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -112,7 +112,7 @@ stages: stage: build before_script: - test -w "${CCACHE_DIR}" && export PATH="/usr/lib/ccache:${PATH}" - - ./autogen.sh +# - ./autogen.sh script: - ./configure --enable-developer --with-libtool --disable-static --with-atf=/usr --with-libidn2 "${EXTRA_CONFIGURE}" - make -j${PARALLEL_JOBS_BUILD:-1} -k all V=1 From f5cb3d707a590ae5a45b6b03d8f91e7735e0f2d4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ond=C5=99ej=20Sur=C3=BD?= Date: Thu, 19 Jul 2018 12:22:55 -0400 Subject: [PATCH 3/4] Make libidn2 build optional --- .gitlab-ci.yml | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 88da5fcf85..633870d986 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -114,7 +114,7 @@ stages: - test -w "${CCACHE_DIR}" && export PATH="/usr/lib/ccache:${PATH}" # - ./autogen.sh script: - - ./configure --enable-developer --with-libtool --disable-static --with-atf=/usr --with-libidn2 "${EXTRA_CONFIGURE}" + - ./configure --enable-developer --with-libtool --disable-static --with-atf=/usr "${EXTRA_CONFIGURE}" - make -j${PARALLEL_JOBS_BUILD:-1} -k all V=1 artifacts: expire_in: '1 hour' @@ -196,7 +196,7 @@ build:centos:centos6:amd64: variables: CC: gcc CFLAGS: "-Wall -Wextra -O2 -g" - EXTRA_CONFIGURE: "--disable-warn-error" + EXTRA_CONFIGURE: "--with-libidn2 --disable-warn-error" <<: *centos_centos6_amd64_image <<: *build_job @@ -204,6 +204,7 @@ build:centos:centos7:amd64: variables: CC: gcc CFLAGS: "-Wall -Wextra -O2 -g" + EXTRA_CONFIGURE: "--with-libidn2" <<: *centos_centos7_amd64_image <<: *build_job @@ -211,6 +212,7 @@ build:clang:debian:sid:amd64: variables: CC: clang-6.0 CFLAGS: "-Wall -Wextra -Wenum-conversion -O2 -g" + EXTRA_CONFIGURE: "--with-libidn2" <<: *debian_sid_amd64_image <<: *build_job @@ -232,6 +234,7 @@ build:debian:sid:amd64: variables: CC: gcc CFLAGS: "-Wall -Wextra -O2 -g" + EXTRA_CONFIGURE: "--with-libidn2" <<: *debian_sid_amd64_image <<: *build_job @@ -239,6 +242,7 @@ build:clang:debian:sid:i386: variables: CC: clang-6.0 CFLAGS: "-Wall -Wextra -Wenum-conversion -O2 -g" + EXTRA_CONFIGURE: "--with-libidn2" <<: *debian_sid_i386_image <<: *build_job @@ -246,6 +250,7 @@ build:debian:sid:i386: variables: CC: gcc CFLAGS: "-Wall -Wextra -O2 -g" + EXTRA_CONFIGURE: "--with-libidn2" <<: *debian_sid_i386_image <<: *build_job From d34aa0ccd0f94491491c13a6bf1c101ab7bc8fc8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ond=C5=99ej=20Sur=C3=BD?= Date: Thu, 19 Jul 2018 11:24:20 -0400 Subject: [PATCH 4/4] Remove quotes around $EXTRA_CONFIGURE to break out more options into ... more options and not a single one --- .gitlab-ci.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 633870d986..37f19b68f7 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -114,7 +114,7 @@ stages: - test -w "${CCACHE_DIR}" && export PATH="/usr/lib/ccache:${PATH}" # - ./autogen.sh script: - - ./configure --enable-developer --with-libtool --disable-static --with-atf=/usr "${EXTRA_CONFIGURE}" + - ./configure --enable-developer --with-libtool --disable-static --with-atf=/usr $EXTRA_CONFIGURE - make -j${PARALLEL_JOBS_BUILD:-1} -k all V=1 artifacts: expire_in: '1 hour'