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

David Zeuthen david at fubar.dk
Mon Mar 12 14:27:11 PDT 2007


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.

    David




More information about the hal mailing list