rpc: add $(SASL_CFLAGS) to libvirt_net_rpc_server_la_CFLAGS

[libvirt] [PATCH] rpc: add $(SASL_CFLAGS) to libvirt_net_rpc_server_la_CFLAGS

Daniel P. Berrangé posted 1 patch 4 years, 10 months ago

Download mbox

Test syntax-check passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20190621144155.29866-1-berrange@redhat.com

src/rpc/Makefile.inc.am | 1 +
1 file changed, 1 insertion(+)

Expand all Fold all

[libvirt] [PATCH] rpc: add $(SASL_CFLAGS) to libvirt_net_rpc_server_la_CFLAGS

Posted by Daniel P. Berrangé 4 years, 10 months ago

The files for libvirt-net-rpc-server.la refernce the sasl/sasl.h
system header but never used the $(SASL_CFLAGS) variable. This
was never noticed previously because the $(AVAHI_CLFAGS) were
set and these typically pulled in the same include directory.

When mDNS/Avahi support was removed this exposed the bug which
caused FreeBSD builds to break as /usr/local/include was no
longer searched for headers.

Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
---

Pushed as a build fix

 src/rpc/Makefile.inc.am | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/rpc/Makefile.inc.am b/src/rpc/Makefile.inc.am
index e5b830605a..b8ca53c69a 100644
--- a/src/rpc/Makefile.inc.am
+++ b/src/rpc/Makefile.inc.am
@@ -111,6 +111,7 @@ libvirt_net_rpc_server_la_SOURCES = \
 libvirt_net_rpc_server_la_CFLAGS = \
 	$(DBUS_CFLAGS) \
 	$(XDR_CFLAGS) \
+	$(SASL_CFLAGS) \
 	$(AM_CFLAGS) \
 	$(NULL)
 libvirt_net_rpc_server_la_LDFLAGS = \
-- 
2.21.0

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list