diff options
| author | Thomas Haller <thaller@redhat.com> | 2024-05-06 08:32:16 (GMT) |
|---|---|---|
| committer | Thomas Haller <thaller@redhat.com> | 2024-05-06 09:28:09 (GMT) |
| commit | 15d90cbf8ced5b89a596e6435b29fb350e4db9db (patch) | |
| tree | 5e62833a9e1c8dd0f0cefcc70f94e694388f9d5c | |
| parent | 8a5f671a0ae6701c1617d8db6552fa5d0d6ee381 (diff) | |
| download | libnl-15d90cbf8ced5b89a596e6435b29fb350e4db9db.zip libnl-15d90cbf8ced5b89a596e6435b29fb350e4db9db.tar.gz libnl-15d90cbf8ced5b89a596e6435b29fb350e4db9db.tar.bz2 | |
include: add _NL_NO_WARN_DEPRECATED_HEADER for suppressing warning about deprecated headers
Header files are part of a stable API. Warning about using deprecated
API is cumbersome, albeit often useful to force the user to migrate
away from the API.
But it also requires that the unit test disables "-Wcpp". That is
problematic on its own, because we don't want to disable any warnings.
As we only have 4 such header files, instead guard the warnings with
a
#ifndef _NL_NO_WARN_DEPRECATED_HEADER
| -rw-r--r-- | Makefile.am | 2 | ||||
| -rw-r--r-- | include/netlink/cache-api.h | 2 | ||||
| -rw-r--r-- | include/netlink/route/link/api.h | 2 | ||||
| -rw-r--r-- | include/netlink/route/link/info-api.h | 2 | ||||
| -rw-r--r-- | include/netlink/route/tc-api.h | 2 |
5 files changed, 9 insertions, 1 deletions
diff --git a/Makefile.am b/Makefile.am index f205024..e86921d 100644 --- a/Makefile.am +++ b/Makefile.am @@ -1139,7 +1139,7 @@ EXTRA_DIST += \ printf "#include <$$(echo "$<" | sed 's|.*\<include/netlink/|netlink/|')>\nint main(void) { return 0; }" > $@ %.build-headers-test.o: %.build-headers-test.c - $(COMPILE) -Wall -Werror -Wno-error=cpp -I$(srcdir)/include -I$(builddir)/include -c -o $@ $< + $(COMPILE) -Wall -Werror -D_NL_NO_WARN_DEPRECATED_HEADER -I$(srcdir)/include -I$(builddir)/include -c -o $@ $< BUILD_HEADERS_OBJS = $(patsubst %,%.build-headers-test.o,$(public_headers)) diff --git a/include/netlink/cache-api.h b/include/netlink/cache-api.h index 851eca0..e8a4170 100644 --- a/include/netlink/cache-api.h +++ b/include/netlink/cache-api.h @@ -9,6 +9,8 @@ #include <netlink/netlink.h> #include <netlink/cache.h> +#ifndef _NL_NO_WARN_DEPRECATED_HEADER #warning "You are including a deprecated header file, include <netlink/cache.h>." +#endif #endif diff --git a/include/netlink/route/link/api.h b/include/netlink/route/link/api.h index abdd8b2..ef11d2f 100644 --- a/include/netlink/route/link/api.h +++ b/include/netlink/route/link/api.h @@ -9,6 +9,8 @@ #include <netlink/netlink.h> #include <netlink/route/link.h> +#ifndef _NL_NO_WARN_DEPRECATED_HEADER #warning "You are including a deprecated header file, include <netlink/route/link.h>." +#endif #endif diff --git a/include/netlink/route/link/info-api.h b/include/netlink/route/link/info-api.h index 11cffcf..88461bb 100644 --- a/include/netlink/route/link/info-api.h +++ b/include/netlink/route/link/info-api.h @@ -9,6 +9,8 @@ #include <netlink/netlink.h> #include <netlink/route/link.h> +#ifndef _NL_NO_WARN_DEPRECATED_HEADER #warning "You are including a deprecated header file, include <netlink/route/link.h>." +#endif #endif diff --git a/include/netlink/route/tc-api.h b/include/netlink/route/tc-api.h index 3f400ba..36b1b0f 100644 --- a/include/netlink/route/tc-api.h +++ b/include/netlink/route/tc-api.h @@ -10,6 +10,8 @@ #include <netlink/msg.h> #include <netlink/route/tc.h> +#ifndef _NL_NO_WARN_DEPRECATED_HEADER #warning "You are including a deprecated header file, include <netlink/route/tc.h>." +#endif #endif |