emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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


From: Eli Zaretskii
Subject: Re: [PATCH] Added basic file system watching support.
Date: Mon, 01 Oct 2012 09:24:39 +0200

> From: Stefan Monnier <address@hidden>
> Date: Mon, 01 Oct 2012 00:38:09 -0400
> Cc: Leo <address@hidden>, address@hidden
> 
> >> I'm not very familiar with inotify and other file-system watching
> >> facilities, so just as a guideline for others's reviews: I'm OK with
> >> installing in 24.3 a non-complete version of the code, and I'm OK to
> >> install before the freeze a code that needs more testing, *but* only if
> >> those missing functionalities and testing won't require a redesign of
> >> the API.
> > It could require a minor redesign of the API.
> 
> I know of "minor changes" (where backward compatibility is easy to
> preserve) and "redesigns" but I don't know what a "minor redesign" would
> look like.
> 
> > At least I want to gather some  experience using the API first.
> > And maybe porting it to other systems will  require API adjustments.
> > Both BSD's and Windows' filesystem watch APIs are  not as powerful
> > as inotify.
> 
> [ I'm probably rehashing something already discussed at the time.  ]
> If the API is sufficiently general that it can be reused for other
> systems, that's great.

Perhaps the API could be spelled out here, then this question could be
answered for systems without inotify.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]