From aedbc21085b8b81ad9baf2f244062e0c411f2521 Mon Sep 17 00:00:00 2001 From: Michael Jeanson Date: Fri, 16 Apr 2021 16:06:11 -0400 Subject: [PATCH] Remove -Wsystem-headers from tests The objective of add -Wsystem-headers to the tests Makefiles was to catch warnings in the tracepoint headers when they are installed in the compiler system header path. However the tests are always built against the headers inside the tree which are not in the system header path. A side effect is that it causes problems on some system where the system headers generate warnings and thus prevent us from building with warnings treated as errors. Change-Id: I65529c50cf48e49ba34692e6d501e3218be8d41c Signed-off-by: Michael Jeanson Signed-off-by: Mathieu Desnoyers --- tests/benchmark/Makefile.am | 2 +- tests/compile/ctf-types/Makefile.am | 2 +- tests/compile/hello-many/Makefile.am | 2 +- tests/compile/hello.cxx/Makefile.am | 2 +- tests/compile/hello/Makefile.am | 2 +- tests/compile/same_line_tracepoint/Makefile.am | 2 +- tests/compile/test-app-ctx/Makefile.am | 2 +- 7 files changed, 7 insertions(+), 7 deletions(-) diff --git a/tests/benchmark/Makefile.am b/tests/benchmark/Makefile.am index efa09cef..7a71fdc5 100644 --- a/tests/benchmark/Makefile.am +++ b/tests/benchmark/Makefile.am @@ -1,6 +1,6 @@ # SPDX-License-Identifier: LGPL-2.1-only -AM_CPPFLAGS += -I$(srcdir) -Wsystem-headers +AM_CPPFLAGS += -I$(srcdir) noinst_PROGRAMS = bench1 bench2 bench1_SOURCES = bench.c tp.c ust_tests_benchmark.h diff --git a/tests/compile/ctf-types/Makefile.am b/tests/compile/ctf-types/Makefile.am index eea45dea..cb268fe4 100644 --- a/tests/compile/ctf-types/Makefile.am +++ b/tests/compile/ctf-types/Makefile.am @@ -1,6 +1,6 @@ # SPDX-License-Identifier: LGPL-2.1-only -AM_CPPFLAGS += -I$(srcdir) -Wsystem-headers +AM_CPPFLAGS += -I$(srcdir) noinst_PROGRAMS = ctf-types ctf_types_SOURCES = ctf-types.c tp.c ust_tests_ctf_types.h diff --git a/tests/compile/hello-many/Makefile.am b/tests/compile/hello-many/Makefile.am index a6fc3c3f..58701fb2 100644 --- a/tests/compile/hello-many/Makefile.am +++ b/tests/compile/hello-many/Makefile.am @@ -1,6 +1,6 @@ # SPDX-License-Identifier: LGPL-2.1-only -AM_CPPFLAGS += -I$(srcdir) -Wsystem-headers +AM_CPPFLAGS += -I$(srcdir) noinst_PROGRAMS = hello-many hello_many_SOURCES = hello-many.c tp.c ust_tests_hello_many.h diff --git a/tests/compile/hello.cxx/Makefile.am b/tests/compile/hello.cxx/Makefile.am index 153ca1e1..c89c27e7 100644 --- a/tests/compile/hello.cxx/Makefile.am +++ b/tests/compile/hello.cxx/Makefile.am @@ -1,6 +1,6 @@ # SPDX-License-Identifier: LGPL-2.1-only -AM_CPPFLAGS += -I$(srcdir) -Wsystem-headers +AM_CPPFLAGS += -I$(srcdir) noinst_PROGRAMS = hello hello_SOURCES = hello.cpp tp-cpp.cpp ust_tests_hello.h diff --git a/tests/compile/hello/Makefile.am b/tests/compile/hello/Makefile.am index 2ff99328..e212d922 100644 --- a/tests/compile/hello/Makefile.am +++ b/tests/compile/hello/Makefile.am @@ -1,6 +1,6 @@ # SPDX-License-Identifier: LGPL-2.1-only -AM_CPPFLAGS += -I$(srcdir) -Wsystem-headers +AM_CPPFLAGS += -I$(srcdir) noinst_PROGRAMS = hello hello_SOURCES = hello.c tp.c ust_tests_hello.h diff --git a/tests/compile/same_line_tracepoint/Makefile.am b/tests/compile/same_line_tracepoint/Makefile.am index ef4d6dae..6d3ac3e1 100644 --- a/tests/compile/same_line_tracepoint/Makefile.am +++ b/tests/compile/same_line_tracepoint/Makefile.am @@ -1,6 +1,6 @@ # SPDX-License-Identifier: LGPL-2.1-only -AM_CPPFLAGS += -I$(srcdir) -Wsystem-headers +AM_CPPFLAGS += -I$(srcdir) noinst_PROGRAMS = same_line_tracepoint same_line_tracepoint_SOURCES = same_line_tracepoint.c ust_tests_sameline.h diff --git a/tests/compile/test-app-ctx/Makefile.am b/tests/compile/test-app-ctx/Makefile.am index c515859a..c8b6281c 100644 --- a/tests/compile/test-app-ctx/Makefile.am +++ b/tests/compile/test-app-ctx/Makefile.am @@ -1,6 +1,6 @@ # SPDX-License-Identifier: LGPL-2.1-only -AM_CPPFLAGS += -I$(srcdir) -Wsystem-headers +AM_CPPFLAGS += -I$(srcdir) noinst_PROGRAMS = hello hello_SOURCES = hello.c tp.c ust_tests_hello.h -- 2.34.1