packaging
/
ostree
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

ci: Enable -Werror for clang 

I hit an unused-variable warning with `GLNX_AUTO_PREFIX_ERROR` for
rpm-ostree and led me to wonder why ostree didn't fail, then I noticed
we had lost the special `-Werror=unused-variable` bit.  Let's go
ahead and use `-Werror` for clang too.

Closes: #1023
Approved by: jlebon
This commit is contained in:
Colin Walters 2017-07-20 15:34:15 -04:00 committed by Atomic Bot
parent ed99b4169e
commit 0ded552cf1
2 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
.papr.yml
View File

@ -31,6 +31,7 @@ timeout: 30m
artifacts: artifacts:
- test-suite.log - test-suite.log
- config.log
--- ---
context: c7-primary context: c7-primary

3
ci/build-check.sh
View File

@ -15,6 +15,9 @@ if test -x /usr/bin/gnome-desktop-testing-runner; then
fi fi
if test -x /usr/bin/clang; then if test -x /usr/bin/clang; then
# always fail on warnings; https://github.com/ostreedev/ostree/pull/971
# Except for clang-4.0: error: argument unused during compilation: '-specs=/usr/lib/rpm/redhat/redhat-hardened-cc1' [-Werror,-Wunused-command-line-argument]
export CFLAGS="-Wno-error=unused-command-line-argument -Werror ${CFLAGS:-}"
git clean -dfx && git submodule foreach git clean -dfx git clean -dfx && git submodule foreach git clean -dfx
# And now a clang build to find unused variables because it does a better # 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 # job than gcc for vars with cleanups; perhaps in the future these could