inotify polling in master branch

Derek Martin invalid at pizzashack.org
Fri Jun 8 18:51:08 UTC 2018


On Fri, Jun 08, 2018 at 02:14:46PM +0200, Vincent Lefevre wrote:
> On 2018-06-05 10:03:55 -0700, Kevin J. McCarthy wrote:
> > Just a heads up that inotify polling was committed to master earlier
> > this week.
> > 
> > There _are_ still a few quirks with it that I'm looking into.
> > Specifically weird behavior with Esc-prefixed commands.  I won't have
> > time to debug that until this weekend though.
> > 
> > If you notice other issues, please let me know.
> 
> I've just tried it, and there is a major regression: it now fails to
> detect new mail at all in the current mailbox, even when moving the
> cursor, viewing messages, modifying the mailbox and synchronizing!
> 
> I'm using the Maildir format.

I haven't looked at the patch, but I have been meaning to point out
that inotify can miss events, under system load (I think it's mostly
kernel memory pressure, but exactly how is probably irrelevant).  So
if Mutt will include such a feature, it would probably be prudent to
not rely on it solely.

I'm a big fan though, I've been meaning to suggest something like this
for a while.  Main reason I haven't is due to the platform-
specificness of inotify, and lack of familiarity with of alternatives
on other platforms.  I would have preferred using a library that
handles that for you (the way libevent does for event-based I/O) if
there were such a thing...  But I'm not aware of such a beast.

-- 
Derek D. Martin    http://www.pizzashack.org/   GPG Key ID: 0xDFBEAD02
-=-=-=-=-
This message is posted from an invalid address.  Replying to it will result in
undeliverable mail due to spam prevention.  Sorry for the inconvenience.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mutt.org/pipermail/mutt-dev/attachments/20180608/dffa48d8/attachment.asc>


More information about the Mutt-dev mailing list