Attention is currently required from: Julius Werner. Jakub Czapiga has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57009 )
Change subject: tests: Fix function mocking for clang ......................................................................
Patch Set 1:
(2 comments)
File tests/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/57009/comment/0928850c_03ce1a7f PS1, Line 60: TEST_CFLAGS += $(if $(filter clang,$(shell $(HOSTCC) --version)),,-Werror)
We could add a new diagnostic flag e.g. "-Wno-weak-identifier-undeclared" and add check: […]
Oh, that's true. People are still using old compilers.I guess we are not able to do anything about it anyway.
https://review.coreboot.org/c/coreboot/+/57009/comment/8d3ca094_8b3faba0 PS1, Line 139: printf '#ifdef __TEST_SRCOBJ__\n' >> $$@; I found a somehow-working solution.
printf '#pragma clang system_header\n' >> $$@;
This should disable warnings for config file. Of course, this line has to be added only for clang (GCC throws an error: unknown pragma)