diff options
| author | Stefano Brivio <sbrivio@redhat.com> | 2024-10-24 23:10:09 +0200 |
|---|---|---|
| committer | Stefano Brivio <sbrivio@redhat.com> | 2024-10-30 08:21:19 +0100 |
| commit | 988a4d75f89473cbf76e09852a03f21658859710 (patch) | |
| tree | 48c70a4fbefb85e23312d86087c1e2ae4756cab9 | |
| parent | ba38e67cf405c5fd4c0fc043af453fa23a55fb35 (diff) | |
| download | passt-988a4d75f89473cbf76e09852a03f21658859710.tar passt-988a4d75f89473cbf76e09852a03f21658859710.tar.gz passt-988a4d75f89473cbf76e09852a03f21658859710.tar.bz2 passt-988a4d75f89473cbf76e09852a03f21658859710.tar.lz passt-988a4d75f89473cbf76e09852a03f21658859710.tar.xz passt-988a4d75f89473cbf76e09852a03f21658859710.tar.zst passt-988a4d75f89473cbf76e09852a03f21658859710.zip | |
Makefile: Exclude qrap.c from clang-tidy checks
We'll deprecate qrap(1) soon, and warnings reported by clang-tidy as
of LLVM versions 16 and later would need a bunch of changes there to
be addressed, mostly around CERT C rule ERR33-C and checking return
code from snprintf().
It makes no sense to fix warnings in qrap just for the sake of it, so
officially declare the bitrotting season open.
Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
Reviewed-by: David Gibson <david@gibson.dropbear.id.au>
| -rw-r--r-- | Makefile | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -256,7 +256,7 @@ docs: README.md # weird for cases like standalone constants, and causes other # awkwardness for a bunch of cases we use -clang-tidy: $(SRCS) $(HEADERS) +clang-tidy: $(filter-out qrap.c,$(SRCS)) $(HEADERS) clang-tidy -checks=*,-modernize-*,\ -clang-analyzer-valist.Uninitialized,\ -cppcoreguidelines-init-variables,\ @@ -283,7 +283,7 @@ clang-tidy: $(SRCS) $(HEADERS) -misc-include-cleaner,\ -cppcoreguidelines-macro-to-enum \ -config='{CheckOptions: [{key: bugprone-suspicious-string-compare.WarnOnImplicitComparison, value: "false"}]}' \ - --warnings-as-errors=* $(SRCS) -- $(filter-out -pie,$(FLAGS) $(CFLAGS) $(CPPFLAGS)) -DCLANG_TIDY_58992 + --warnings-as-errors=* $(filter-out qrap.c,$(SRCS)) -- $(filter-out -pie,$(FLAGS) $(CFLAGS) $(CPPFLAGS)) -DCLANG_TIDY_58992 SYSTEM_INCLUDES := /usr/include $(wildcard /usr/include/$(TARGET)) ifeq ($(shell $(CC) -v 2>&1 | grep -c "gcc version"),1) |