[PATCH] inotify support for systems without sys/inotify.h

Michael Biebl mbiebl at gmail.com
Tue Mar 13 02:12:11 PDT 2007


2007/3/12, David Zeuthen <david at fubar.dk>:
> On Mon, 2007-03-12 at 22:22 +0100, Michael Biebl wrote:
> > 2007/3/7, Dan Nicholson <dbn.lists at gmail.com>:
> > > On 3/6/07, Michael Biebl <mbiebl at gmail.com> wrote:
> > > >
> > > > attached is a patch (based on Kay's udev patch, but extended), which
> > > > adds a configure check for sys/inotify.h. If not available,
> > > > hald/linux/osspec.c includes a local stub of inotify.h.
> > >
> > > Comments from the peanut gallery, but I think it would be nice if the
> > > file was called hal-inotify.h or inotify-local.h or something so as
> > > not to have it be confused with the real header from
> > > glibc/kernel-headers. This is what I've typically seen in other
> > > applications that ship a local header such as gnome-vfs:
> > >
> > > http://svn.gnome.org/viewcvs/gnome-vfs/trunk/modules/local_inotify.h?view=markup
> > >
> >
> > No further comments?
>
> Sorry, I missed this one!
>
> > If renaming inotify.h to inotify-local.h is the only issue, than I'm
> > happy to change that.
>
> Yeah, that sounds good. Thanks.
>

Updated patch attached.

Cheers,
Michael
-------------- next part --------------
A non-text attachment was scrubbed...
Name: hal_inotify_local.h
Type: text/x-chdr
Size: 8214 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/hal/attachments/20070313/b9f0b4a1/hal_inotify_local-0001.h


More information about the hal mailing list