[Top][All Lists]

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

Re: [Emacs-diffs] master 716b468: Extend `file-notify-test02-rm-watch'

From: Michael Albinus
Subject: Re: [Emacs-diffs] master 716b468: Extend `file-notify-test02-rm-watch'
Date: Mon, 01 May 2017 20:45:15 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/26.0.50 (gnu/linux)

Ken Brown <address@hidden> writes:

> Hi Michael,

Hi Ken,

> On my systems I seem to need a much larger sit-for timeout (on the
> order of 0.1 or more rather than 0.001) in order for the test to pass.
> But maybe this is not worth worrying about, since it's clear that this
> is a timing issue rather than a file-notify problem.

I've moved the `sit-for' call to `file-notify--test-read-event' by
intention, so we have just one place to change. You could do something

(if (eq system-type 'cygwin)
  (sit-for 0.1 'nodisp))

The other backends do not need them as far as I'm aware of, so do it at
your convenience. Maybe also with a comment ...

>> One point still irritates me: the tests run under cygwin change the
>> results after a while. Several tests which pass after a reboot of a
>> machine, will fail when I use the machine for some hours. It isn't the
>> first time I've observed this, so I need to reboot my machine again and
>> again when working for cygwin.
>> Do you know this behaviour?
> No, I don't see this.  I hardly ever reboot my computer.  But I mostly
> run 64-bit Cygwin.  Are you running 32-bit Cygwin?  And are the
> problems related to fork failures?  (You would see error messages to
> this effect on the terminal or in the logs.)  If so, I might be able
> to help.  But in that case we should probably continue the discussion
> off-list.

Likely 32-bit. But I use it very rarely, so I don't want to waste your
time with debugging when you don't know it from somewhere else.

> Regards,
> Ken

Best regards, Michael.

reply via email to

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