diff options
| author | Thomas Heijligen <thomas.heijligen@secunet.com> | 2021-10-26 11:26:32 +0200 |
|---|---|---|
| committer | Nico Huber <nico.h@gmx.de> | 2021-11-01 14:25:46 +0000 |
| commit | 36fa20d1764acd5d84e35e66e8c7efb1e7c523ad (patch) | |
| tree | 05b02b5b0a215aeb2df7cb3d4ea98f39c51ce051 /Makefile.include | |
| parent | fac2bfe54caa162f5c6109c5f5c471471e81cc22 (diff) | |
| download | flashrom-36fa20d1764acd5d84e35e66e8c7efb1e7c523ad.tar.gz flashrom-36fa20d1764acd5d84e35e66e8c7efb1e7c523ad.tar.bz2 flashrom-36fa20d1764acd5d84e35e66e8c7efb1e7c523ad.zip | |
Makefile: Revise C compiler check
Clean up the compiler target by outsourcing the test to an own variable.
Change the print output and don't write to the build-details file.
This is in preparation for further changes.
Change-Id: I3d6f08ef030744c772b4ec0dc2c9e614fb90461d
Signed-off-by: Thomas Heijligen <thomas.heijligen@secunet.com>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/58616
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Diffstat (limited to 'Makefile.include')
| -rw-r--r-- | Makefile.include | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/Makefile.include b/Makefile.include index ca70d0514..2b5a3447d 100644 --- a/Makefile.include +++ b/Makefile.include @@ -36,15 +36,9 @@ define c_macro_test $(strip $(shell $(CC) -E $1 2>/dev/null | tail -1 | tr -d '"')) endef -define COMPILER_TEST -int main(int argc, char **argv) -{ - (void) argc; - (void) argv; - return 0; -} +define c_compile_test +$(shell $(CC) -c -Wall -Werror -o /dev/null $1 2>/dev/null && echo yes || echo no) endef -export COMPILER_TEST define LIBPCI_TEST /* Avoid a failing test due to libpci header symbol shadowing breakage */ |