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

[Martin L Roth (Code Review)]

Attention is currently required from: Nico Huber, Edward O'Callaghan, Anastasia Klimchuk.
Martin L Roth has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/62745 )

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

Patch Set 2:

(1 comment)

Patchset:

PS2:
> Isn't it just repeating the Makefile this way? Does it make a difference […]

I'm not sure I understand your question.

I think we want warnings as errors enabled for most of the jenkins builds.  I just want a way to override it.  Jenkins uses this script for the build, and because it's specified on the command line inside the script, I wasn't able to change it. so that I could get a complete build with -fanalyzer turned on.

I mean, we could get rid of the script completely and just hardcode each of the jenkins build tasks to do what this does if that's what you're suggesting, but then members of the project wouldn't be able to update the build task.