Bug 74602

Summary:	Build for AIX of fcstat.c fails with undefined struct statvsf
Product:	fontconfig	Reporter:	Dirk <tomsies>
Component:	library	Assignee:	Akira TAGOH <akira>
Status:	RESOLVED FIXED	QA Contact:	Behdad Esfahbod <freedesktop>
Severity:	normal
Priority:	medium	CC:	akira, fontconfig-bugs
Version:	unspecified
Hardware:	PowerPC
OS:	AIX
Whiteboard:
i915 platform:		i915 features:

Description Dirk 2014-02-06 07:47:53 UTC

configure is (correctly) checking for function fstatvfs(), and finding it. However there is also a configure test for vsf.h which sets HAVE_SYS_VFS_H, and results in 
#include <sys/vsf.h>

By inspection, windows target does not provide for fstatvsf(), and both AIX and linux provide this prototype in <sys/statvsf.h>, not in <sys/vsf.h>. I dunno which other OS targets you guys have on your radar where perhaps vsf.h is the correct answer?

I have altered the include to be <sys/statvsf.h> and all is well. I am happy to provide a patch for the c code, but am reluctant to fiddle with automake / autoconf tests - my version of auto-tools is very different from that shipped with fontconfig.

Comment 1 Akira TAGOH 2014-02-06 08:28:25 UTC

You mean vfs.h

Well, it is not for fstatvfs() but statfs().
Anyway, it is just a typo while it is used for checking members for struct statvfs. I'll fix it in git soon.

Comment 2 Dirk 2014-02-06 08:58:54 UTC

Sorry, yes vfs.h not vsf.h ......

Thanks!

Comment 3 Akira TAGOH 2014-02-06 09:00:33 UTC

fixed in git 7d756532.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.