From 684d62276af37c260671024ce17cb281f493b289 Mon Sep 17 00:00:00 2001 From: Andrea Bolognani Date: Wed, 25 Oct 2017 17:16:14 +0200 Subject: *.pc: Fix include path in Cflags The include path in glusterfs-api.pc looks like -I${includedir}/glusterfs However, client code will include the glusterfs headers using #include rather than #include which makes the "/glusterfs" part entirely unnecessary. More importantly, on some platforms such as FreeBSD, the header files for glusterfs will be installed in /usr/local/include, which is *not* part of the compiler's default include path, so compilation will fail with something like fatal error: 'glusterfs/api/glfs.h' file not found #include ^~~~~~~~~~~~~~~~~~~~~~ The fix is to simply drop the extra "/glusterfs". The same change is applied to other *.pc files as well, althought I haven't actually tested those. A test program (gfapi-load-volfile) and the glfsxmp example application were using the wrong include paths, so they had to be fixed as well. Change-Id: I9a16de47fee7ab9c12d1cb823bbe061a69352670 BUG: 1508947 Signed-off-by: Andrea Bolognani --- libgfchangelog.pc.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'libgfchangelog.pc.in') diff --git a/libgfchangelog.pc.in b/libgfchangelog.pc.in index 91c85e6f1f8..e2ff1fb6214 100644 --- a/libgfchangelog.pc.in +++ b/libgfchangelog.pc.in @@ -9,4 +9,4 @@ Description: GlusterFS Changelog Consumer Library Version: @LIBGFCHANGELOG_VERSION@ Requires: @PKGCONFIG_UUID@ Libs: -L${libdir} -lgfchangelog -lglusterfs -Cflags: -I${includedir}/glusterfs/gfchangelog -D_FILE_OFFSET_BITS=64 -D__USE_FILE_OFFSET64 +Cflags: -I${includedir} -D_FILE_OFFSET_BITS=64 -D__USE_FILE_OFFSET64 -- cgit