Re: [PATCH] Added basic file system watching support.

[Michael Albinus]

Stefan Monnier <address@hidden> writes:

> If there's a good chance this won't work without breaking compatibility,
> maybe a better option is to provide a low-level API that maps very
> closely to inotify and then an Elisp layer on top which abstracts away
> differences between different systems.  In that case we can install the
> inotify support right away while we're still experimenting with the
> higher-level abstraction.

This has inspired me to check, whether we could extend inotify support
for remote files, via Tramp. I know it might have performance issues,
but I'm curious :-)

A first shot to write a file handler for `inotify-add-watch' was quite
easy. It works pretty well, and would need only some polishing before
being installed.

For `inotify-rm-watch' that's not possible right now. It takes as
argument WATCH-DESCRIPTOR, which is not a file name, and which does not
contain a file name.

Could we extend the interface of `inotify-rm-watch' to add a file name?
It would be sufficient already, if WATCH-DESCRIPTOR would contain a file
name as first element of its structure.

>         Stefan

Best regards, Michael.