>From 34af5f3be8b04443dc151e6d070aa49cbe59b7fc Mon Sep 17 00:00:00 2001 From: Jim Meyering <meyering(a)redhat.com&gt; Date: Fri, 20 Nov 2009 11:50:54 +0100 Subject: [PATCH libguestfs] maint: remove unnecessary include of openat.h * daemon/realpath.c: Don't include "openat.h". not used. --- daemon/realpath.c | 2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/daemon/realpath.c b/daemon/realpath.c index 17e74ea..750cadb 100644 --- a/daemon/realpath.c +++ b/daemon/realpath.c @@ -27,8 +27,6 @@ #include <sys/types.h> #include <dirent.h> -#include "openat.h" - #include "daemon.h" #include "actions.h"

Richard W.M. Jones wrote: > On Fri, Nov 20, 2009 at 11:51:20AM +0100, Jim Meyering wrote: >> >From 34af5f3be8b04443dc151e6d070aa49cbe59b7fc Mon Sep 17 00:00:00 2001 >> From: Jim Meyering <meyering(a)redhat.com&gt; >> Date: Fri, 20 Nov 2009 11:50:54 +0100 >> Subject: [PATCH libguestfs] maint: remove unnecessary include of openat.h >> >> * daemon/realpath.c: Don't include "openat.h". not used. >> --- >> daemon/realpath.c | 2 -- >> 1 files changed, 0 insertions(+), 2 deletions(-) >> >> diff --git a/daemon/realpath.c b/daemon/realpath.c >> index 17e74ea..750cadb 100644 >> --- a/daemon/realpath.c >> +++ b/daemon/realpath.c >> @@ -27,8 +27,6 @@ >> #include <sys/types.h> >> #include <dirent.h> >> >> -#include "openat.h" >> - >> #include "daemon.h" >> #include "actions.h" With the latest standard-conforming headers, (and gnulib provides that), openat is declared in fcntl.h, so openat.h really is not required. Assuming you're using gnulib's fcntl module, which guarantees a conforming <fcntl.h> at worst, via lib/fcntl.h.