]> git.dujemihanovic.xyz Git - linux.git/commit
selftests/bpf: Filter out _GNU_SOURCE when compiling test_cpp
authorStanislav Fomichev <sdf@fomichev.me>
Thu, 25 Jul 2024 21:40:29 +0000 (14:40 -0700)
committerAndrii Nakryiko <andrii@kernel.org>
Mon, 29 Jul 2024 20:43:31 +0000 (13:43 -0700)
commit41c24102af7b6236277a214428b203d51a3462df
tree88899ab944af9bac21c481a0130ddc8277801cc2
parent039564d2fd37b122ec0d268e2ee6334e7169e225
selftests/bpf: Filter out _GNU_SOURCE when compiling test_cpp

Jakub reports build failures when merging linux/master with net tree:

CXX      test_cpp
In file included from <built-in>:454:
<command line>:2:9: error: '_GNU_SOURCE' macro redefined [-Werror,-Wmacro-redefined]
    2 | #define _GNU_SOURCE
      |         ^
<built-in>:445:9: note: previous definition is here
  445 | #define _GNU_SOURCE 1

The culprit is commit cc937dad85ae ("selftests: centralize -D_GNU_SOURCE= to
CFLAGS in lib.mk") which unconditionally added -D_GNU_SOUCE to CLFAGS.
Apparently clang++ also unconditionally adds it for the C++ targets [0]
which causes a conflict. Add small change in the selftests makefile
to filter it out for test_cpp.

Not sure which tree it should go via, targeting bpf for now, but net
might be better?

0: https://stackoverflow.com/questions/11670581/why-is-gnu-source-defined-by-default-and-how-to-turn-it-off

Signed-off-by: Stanislav Fomichev <sdf@fomichev.me>
Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
Acked-by: Jiri Olsa <jolsa@kernel.org>
Link: https://lore.kernel.org/bpf/20240725214029.1760809-1-sdf@fomichev.me
tools/testing/selftests/bpf/Makefile