Martin Roth has uploaded this change for review.

View Change

test_build.sh: Allow WARNERROR to be overridden

Currently, WARNERROR is hardcoded to 'yes' which is what we want most of
the time, but there are cases (such as when building with -fanalyzer)
that we don't want it enabled, so we see the entire output of the build,
and it doesn't halt at the first error.

After the flashrom build passes with the -fanalyzer setting set, we
can go back to setting WARNERROR=yes for that build in jenkins.

Signed-off-by: Martin Roth <gaumless@gmail.com>
Change-Id: Iea931e57f2a6992762566dc3dbaae8bb8df5b226
---
M test_build.sh
1 file changed, 1 insertion(+), 2 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/45/62745/1
diff --git a/test_build.sh b/test_build.sh
index dc286f8..1d60377 100755
--- a/test_build.sh
+++ b/test_build.sh
@@ -1,8 +1,7 @@
#!/usr/bin/env sh
set -e

-make CONFIG_EVERYTHING=yes WARNERROR=yes
-
+make CONFIG_EVERYTHING=yes WARNERROR="${WARNERROR-yes}"

builddir=out
meson $builddir

To view, visit change 62745. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Iea931e57f2a6992762566dc3dbaae8bb8df5b226
Gerrit-Change-Number: 62745
Gerrit-PatchSet: 1
Gerrit-Owner: Martin Roth <martinroth@google.com>
Gerrit-MessageType: newchange