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

Michael Biebl mbiebl at gmail.com
Mon Mar 12 14:22:54 PDT 2007


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?

If renaming inotify.h to inotify-local.h is the only issue, than I'm
happy to change that.

Cheers,
Michael


More information about the hal mailing list