Bug 74602 - Build for AIX of fcstat.c fails with undefined struct statvsf
Summary: Build for AIX of fcstat.c fails with undefined struct statvsf
Status: RESOLVED FIXED
Alias: None
Product: fontconfig
Classification: Unclassified
Component: library (show other bugs)
Version: unspecified
Hardware: PowerPC AIX
: medium normal
Assignee: Akira TAGOH
QA Contact: Behdad Esfahbod
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-02-06 07:47 UTC by Dirk
Modified: 2014-02-06 09:00 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:

Attachments

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.