From 46e65370c6c2c69c219ebf09e627a18907869b95 Mon Sep 17 00:00:00 2001 From: Peter Hutterer Date: Wed, 2 Mar 2022 16:20:33 +1000 Subject: [PATCH] gitlab CI: add a -Werror job (allowed to fail) Most of the time we don't pay attention to the details in the actual jobs, them passing is good enough. So let's have at least one job with -Werror that fails if our CI has warnings we don't have locally. Allowed to fail because we don't want to be beholden to compiler changes (though having this fail for a long period makes the job less useful again). Signed-off-by: Peter Hutterer --- .gitlab-ci.yml | 7 +++++++ .gitlab-ci/ci.template | 7 +++++++ 2 files changed, 14 insertions(+) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 2b5e63f..0dedd31 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -256,6 +256,13 @@ valgrind@fedora:35: before_script: - dnf install -y valgrind +werror@fedora:35: + extends: + - .fedora-build@template + variables: + MESON_ARGS: '-Dwerror=true' + allow_failure: true + ################################################################# # # diff --git a/.gitlab-ci/ci.template b/.gitlab-ci/ci.template index 28a341f..81c5806 100644 --- a/.gitlab-ci/ci.template +++ b/.gitlab-ci/ci.template @@ -277,6 +277,13 @@ valgrind@{{distro.name}}:{{version}}: before_script: - dnf install -y valgrind +werror@{{distro.name}}:{{version}}: + extends: + - .{{distro.name}}-build@template + variables: + MESON_ARGS: '-Dwerror=true' + allow_failure: true + {% endfor %} #################################################################