[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20080916130234.GA17082@localhost.localdomain>
Date: Tue, 16 Sep 2008 16:02:36 +0300
From: "Kirill A. Shutemov" <kirill@...temov.name>
To: Adrian Bunk <bunk@...nel.org>
Cc: linux-kernel@...r.kernel.org, Ulrich Drepper <drepper@...hat.com>,
Davide Libenzi <davidel@...ilserver.org>,
David Woodhouse <dwmw2@...radead.org>,
Andrew Morton <akpm@...ux-foundation.org>
Subject: Re: [PATCH] linux/inotify.h: do not include <linux/fcntl.h> in
userspace
On Tue, Sep 16, 2008 at 03:58:02PM +0300, Adrian Bunk wrote:
> On Tue, Sep 16, 2008 at 03:22:43PM +0300, Kirill A. Shutemov wrote:
> > <linux/fcntl.h> conflicts with glibc's <fcntl.h>.
> >
> > It breaks building kdelibs, kdepim and pinot.
>
> We should rather fix the actual bug.
>
> What is the error message?
/usr/include/asm-generic/fcntl.h:117: error: redefinition of 'struct
flock'
/usr/include/bits/fcntl.h:142: error: previous definition of 'struct
flock'
/usr/include/asm-generic/fcntl.h:140: error: redefinition of 'struct
flock64'
/usr/include/bits/fcntl.h:157: error: previous definition of 'struct
flock64'
>
> >...
> > --- a/include/linux/inotify.h
> > +++ b/include/linux/inotify.h
> > @@ -7,8 +7,10 @@
> > #ifndef _LINUX_INOTIFY_H
> > #define _LINUX_INOTIFY_H
> >
> > +#ifdef __KERNEL__
> > /* For O_CLOEXEC and O_NONBLOCK */
> > #include <linux/fcntl.h>
> > +#endif
> >...
>
> This breaks the header for users of IN_CLOEXEC/IN_NONBLOCK.
>
> 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
>
--
Regards, Kirill A. Shutemov
+ Belarus, Minsk
+ ALT Linux Team, http://www.altlinux.com/
Download attachment "signature.asc" of type "application/pgp-signature" (198 bytes)
Powered by blists - more mailing lists