提交 a7851864 编写于 作者: M Michal Privoznik

tools: Enable warnings for more binaries/libs

Because WARN_CFLAGS and COVERAGE_CFLAGS are not set globally, we
rely on each binary built to include WARN_CFLAGS/COVERAGE_CFLAGS.
But it is easy to forget those - e.g. libvirt_shell.la. However,
don't enable WARN_FLAGS (i.e. don't include AM_CFLAGS) for
wireshark plugin - parts of that code are generated and trigger
some warnings.
Signed-off-by: NMichal Privoznik <mprivozn@redhat.com>
Reviewed-by: NErik Skultety <eskultet@redhat.com>
上级 15e661d5
...@@ -25,6 +25,13 @@ INCLUDES = \ ...@@ -25,6 +25,13 @@ INCLUDES = \
WARN_CFLAGS += $(STRICT_FRAME_LIMIT_CFLAGS) WARN_CFLAGS += $(STRICT_FRAME_LIMIT_CFLAGS)
AM_CFLAGS = \
$(WARN_CFLAGS) \
$(COVERAGE_CFLAGS) \
$(PIE_CFLAGS) \
$(LIBXML_CFLAGS) \
$(NULL)
AM_LDFLAGS = \ AM_LDFLAGS = \
$(RELRO_LDFLAGS) \ $(RELRO_LDFLAGS) \
$(NO_INDIRECT_LDFLAGS) \ $(NO_INDIRECT_LDFLAGS) \
...@@ -122,9 +129,7 @@ virt-sanlock-cleanup: virt-sanlock-cleanup.in Makefile ...@@ -122,9 +129,7 @@ virt-sanlock-cleanup: virt-sanlock-cleanup.in Makefile
noinst_LTLIBRARIES = libvirt_shell.la noinst_LTLIBRARIES = libvirt_shell.la
libvirt_shell_la_CFLAGS = \ libvirt_shell_la_CFLAGS = \
$(AM_CFLAGS) \ $(AM_CFLAGS) \
$(PIE_CFLAGS) \
$(READLINE_CFLAGS) \ $(READLINE_CFLAGS) \
$(LIBXML_CFLAGS) \
$(NULL) $(NULL)
libvirt_shell_la_LDFLAGS = \ libvirt_shell_la_LDFLAGS = \
$(AM_LDFLAGS) \ $(AM_LDFLAGS) \
...@@ -182,10 +187,7 @@ virt_host_validate_LDADD = \ ...@@ -182,10 +187,7 @@ virt_host_validate_LDADD = \
$(NULL) $(NULL)
virt_host_validate_CFLAGS = \ virt_host_validate_CFLAGS = \
$(LIBXML_CFLAGS) \ $(AM_CFLAGS) \
$(WARN_CFLAGS) \
$(PIE_CFLAGS) \
$(COVERAGE_CFLAGS) \
$(NULL) $(NULL)
# Since virt-login-shell will be setuid, we must do everything # Since virt-login-shell will be setuid, we must do everything
...@@ -208,10 +210,8 @@ virt_login_shell_LDADD = \ ...@@ -208,10 +210,8 @@ virt_login_shell_LDADD = \
virt_login_shell_CFLAGS = \ virt_login_shell_CFLAGS = \
-DLIBVIRT_SETUID_RPC_CLIENT \ -DLIBVIRT_SETUID_RPC_CLIENT \
$(LIBXML_CFLAGS) \ $(AM_CFLAGS) \
$(WARN_CFLAGS) \ $(NULL)
$(PIE_CFLAGS) \
$(COVERAGE_CFLAGS)
virsh_SOURCES = \ virsh_SOURCES = \
virsh.c virsh.h \ virsh.c virsh.h \
...@@ -241,10 +241,7 @@ virsh_LDADD = \ ...@@ -241,10 +241,7 @@ virsh_LDADD = \
../src/libvirt-qemu.la \ ../src/libvirt-qemu.la \
libvirt_shell.la libvirt_shell.la
virsh_CFLAGS = \ virsh_CFLAGS = \
$(WARN_CFLAGS) \ $(AM_CFLAGS) \
$(PIE_CFLAGS) \
$(COVERAGE_CFLAGS) \
$(LIBXML_CFLAGS) \
$(READLINE_CFLAGS) $(READLINE_CFLAGS)
virt_admin_SOURCES = \ virt_admin_SOURCES = \
...@@ -263,10 +260,7 @@ virt_admin_LDADD = \ ...@@ -263,10 +260,7 @@ virt_admin_LDADD = \
$(LIBXML_LIBS) \ $(LIBXML_LIBS) \
$(NULL) $(NULL)
virt_admin_CFLAGS = \ virt_admin_CFLAGS = \
$(WARN_CFLAGS) \ $(AM_CFLAGS) \
$(PIE_CFLAGS) \
$(COVERAGE_CFLAGS) \
$(LIBXML_CFLAGS) \
$(READLINE_CFLAGS) $(READLINE_CFLAGS)
BUILT_SOURCES = BUILT_SOURCES =
...@@ -502,10 +496,7 @@ nss_libnss_libvirt_impl_la_SOURCES = \ ...@@ -502,10 +496,7 @@ nss_libnss_libvirt_impl_la_SOURCES = \
nss_libnss_libvirt_impl_la_CFLAGS = \ nss_libnss_libvirt_impl_la_CFLAGS = \
-DLIBVIRT_NSS \ -DLIBVIRT_NSS \
$(AM_CFLAGS) \ $(AM_CFLAGS) \
$(WARN_CFLAGS) \ $(NULL)
$(PIE_CFLAGS) \
$(COVERAGE_CFLAGS) \
$(LIBXML_CFLAGS)
nss_libnss_libvirt_impl_la_LIBADD = \ nss_libnss_libvirt_impl_la_LIBADD = \
../gnulib/lib/libgnu.la \ ../gnulib/lib/libgnu.la \
...@@ -532,10 +523,7 @@ nss_libnss_libvirt_guest_impl_la_CFLAGS = \ ...@@ -532,10 +523,7 @@ nss_libnss_libvirt_guest_impl_la_CFLAGS = \
-DLIBVIRT_NSS \ -DLIBVIRT_NSS \
-DLIBVIRT_NSS_GUEST \ -DLIBVIRT_NSS_GUEST \
$(AM_CFLAGS) \ $(AM_CFLAGS) \
$(WARN_CFLAGS) \ $(NULL)
$(PIE_CFLAGS) \
$(COVERAGE_CFLAGS) \
$(LIBXML_CFLAGS)
nss_libnss_libvirt_guest_impl_la_LIBADD = \ nss_libnss_libvirt_guest_impl_la_LIBADD = \
../gnulib/lib/libgnu.la \ ../gnulib/lib/libgnu.la \
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册