GNU bug report logs - #5827
[PATCH] tail: include sys/vfs.h

Previous Next

Package: coreutils;

Reported by: Marc Kleine-Budde <mkl <at> pengutronix.de>

Date: Sat, 3 Apr 2010 14:22:01 UTC

Severity: normal

Tags: patch

Done: Jim Meyering <jim <at> meyering.net>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Jim Meyering <jim <at> meyering.net>
To: Marc Kleine-Budde <mkl <at> pengutronix.de>
Cc: 5827 <at> debbugs.gnu.org
Subject: bug#5827: [PATCH] tail: include sys/vfs.h
Date: Sun, 04 Apr 2010 09:16:10 +0200
Marc Kleine-Budde wrote:
> the latest coreutils release 8.4 and current git master fail during the
> (cross-) compilation of tail. The used components are:
>
> gcc-4.3.2
> glibc-2.8
> binutils-2.18
>
> The following patch fixes the commit. It's against git master, but should
> apply on 8.4, too.
...
>  # if HAVE_SYS_STATFS_H
>  #  include <sys/statfs.h>
>  # endif
> +# if HAVE_SYS_VFS_H
> +#  include <sys/vfs.h>
> +# endif
>  #endif

I'd like to take a slightly more conservative approach,
including sys/vfs.h only when sys/statfs.h is not available.
Does this work for you?

From 98dacf492e2e6f2153455cb4179058578cee20ff Mon Sep 17 00:00:00 2001
From: Marc Kleine-Budde <mkl <at> pengutronix.de>
Date: Sun, 4 Apr 2010 09:15:07 +0200
Subject: [PATCH] tail: include sys/vfs.h (if possible) when sys/statfs.h is absent

* src/tail.c [HAVE_INOTIFY && !HAVE_SYS_STATFS_H]: Include <sys/vfs.h>.
---
 src/tail.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/src/tail.c b/src/tail.c
index 02c4a1a..9e95dee 100644
--- a/src/tail.c
+++ b/src/tail.c
@@ -56,6 +56,8 @@
 # include "fs.h"
 # if HAVE_SYS_STATFS_H
 #  include <sys/statfs.h>
+# elif HAVE_SYS_VFS_H
+#  include <sys/vfs.h>
 # endif
 #endif

--
1.7.0.4.529.g78fb




This bug report was last modified 15 years and 54 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.