[Libguestfs] [PATCH v2 1/2] daemon: Reimplement statvfs API in OCaml.

Pino Toscano ptoscano at redhat.com
Wed Oct 4 13:17:48 UTC 2017


On Tuesday, 3 October 2017 14:24:13 CEST Richard W.M. Jones wrote:
> common/mlutils: Unix_utils.StatVFS.statvfs: This commit implements a
> full-featured binding for the statvfs(3) function.
> 
> We then use this to reimplement the daemon statvfs API in OCaml.
> 
> Note that the Gnulib fallback is fixed in this commit.  It previously
> referenced non-existent field names in the fs_usage struct.

TBH I'd just drop the gnulib fallback (i.e. the fsusage module):
- (f)statvfs is POSIX since 2001
- Linux, macOS, and *BSDs have been supporting it already for years
  (decades, even)
- Windows (even if untested for long) has the fallback code already

This way there's less dead code around, and one gnulib module less.

-- 
Pino Toscano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part.
URL: <http://listman.redhat.com/archives/libguestfs/attachments/20171004/2c709ca8/attachment.sig>


More information about the Libguestfs mailing list