diff options
author | Jonathan Lebon <jlebon@redhat.com> | 2017-06-28 08:59:36 -0700 |
---|---|---|
committer | Atomic Bot <atomic-devel@projectatomic.io> | 2017-06-28 16:37:15 +0000 |
commit | 9d10bdfd0d90d6ea6a31bbd89243f3b23ff51e8d (patch) | |
tree | e57c7bfa8f8d26c4a5549e0937e06b6a70b93dd9 /ci | |
parent | 373dc4b66c7966bd917039addf134a1dc556db3f (diff) | |
download | ostree-9d10bdfd0d90d6ea6a31bbd89243f3b23ff51e8d.tar.gz |
ci: unconditionally turn on -Werror
Closes: #971
Approved by: cgwalters
Diffstat (limited to 'ci')
-rwxr-xr-x | ci/build-check.sh | 6 | ||||
-rwxr-xr-x | ci/build.sh | 3 |
2 files changed, 6 insertions, 3 deletions
diff --git a/ci/build-check.sh b/ci/build-check.sh index da03f3cc..677515b3 100755 --- a/ci/build-check.sh +++ b/ci/build-check.sh @@ -16,9 +16,9 @@ fi if test -x /usr/bin/clang; then git clean -dfx && git submodule foreach git clean -dfx - # And now a clang build to find unused variables; perhaps - # in the future these could parallelize + # And now a clang build to find unused variables because it does a better + # job than gcc for vars with cleanups; perhaps in the future these could + # parallelize export CC=clang - export CFLAGS='-Werror=unused-variable' build fi diff --git a/ci/build.sh b/ci/build.sh index eefb7c3c..a9a8ac3f 100755 --- a/ci/build.sh +++ b/ci/build.sh @@ -12,6 +12,9 @@ pkg_install sudo which attr fuse \ elfutils pkg_install_if_os fedora gjs gnome-desktop-testing parallel coccinelle clang +# always fail on warnings; https://github.com/ostreedev/ostree/pull/971 +export CFLAGS="-Werror ${CFLAGS:-}" + DETECTED_CONFIGOPTS= if test -x /usr/bin/gnome-desktop-testing-runner; then DETECTED_CONFIGOPTS="${DETECTED_CONFIGOPTS} --enable-installed-tests=exclusive" |