Change in flashrom[master]: test_build.sh: Allow WARNERROR to be overridden

[Nico Huber (Code Review)]

Nico Huber has submitted this change. ( https://review.coreboot.org/c/flashrom/+/62745 )

Change subject: test_build.sh: Allow WARNERROR to be overridden
......................................................................

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.  Removing the WARNERROR line
from the script allows the variable to be overridden from an environment
variable.

Signed-off-by: Martin Roth <[e-mail]>
Change-Id: Iea931e57f2a6992762566dc3dbaae8bb8df5b226
Reviewed-on: https://review.coreboot.org/c/flashrom/+/62745
Reviewed-by: Felix Singer <[e-mail]>
Reviewed-by: Nico Huber <[e-mail]>
Tested-by: build bot (Jenkins) <[e-mail]>
---
M test_build.sh
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  build bot (Jenkins): Verified
  Nico Huber: Looks good to me, approved
  Felix Singer: Looks good to me, approved



diff --git a/test_build.sh b/test_build.sh
index dc286f8..6c0c093 100755
--- a/test_build.sh
+++ b/test_build.sh
@@ -1,7 +1,7 @@
 #!/usr/bin/env sh
 set -e
 
-make CONFIG_EVERYTHING=yes WARNERROR=yes
+make CONFIG_EVERYTHING=yes
 
 
 builddir=out

-- 
To view, visit https://review.coreboot.org/c/flashrom/+/62745
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Iea931e57f2a6992762566dc3dbaae8bb8df5b226
Gerrit-Change-Number: 62745
Gerrit-PatchSet: 6
Gerrit-Owner: Martin L Roth <[e-mail]>
Gerrit-Reviewer: Anastasia Klimchuk <[e-mail]>
Gerrit-Reviewer: Angel Pons <[e-mail]>
Gerrit-Reviewer: Edward O'Callaghan <[e-mail]>
Gerrit-Reviewer: Felix Singer <[e-mail]>
Gerrit-Reviewer: Nico Huber <[e-mail]>
Gerrit-Reviewer: build bot (Jenkins) <[e-mail]>
Gerrit-CC: Paul Menzel <[e-mail]>
Gerrit-MessageType: merged