Re: [PATCH] linux/inotify.h: do not include <linux/fcntl.h> inuserspace

From: Adrian Bunk
Date: Tue Sep 16 2008 - 12:09:30 EST


On Tue, Sep 16, 2008 at 07:10:25AM -0700, Ulrich Drepper wrote:
> Kirill A. Shutemov wrote:
> >> What is the error message?
> >
> > /usr/include/asm-generic/fcntl.h:117: error: redefinition of 'struct
> > flock'
>
> And? None of these programs should use <linux/inotify.h>. There has
> for the longest time been a <sys/inotify.h> header which doesn't need
> any kernel headers. In fact, <linux/inotify.h> should not be exported.

Even if userspace applications shouldn't use it directly this doesn't
sound right:

We shouldn't force all libc implementations to copy the contents into a
private header.

cu
Adrian

--

"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/