GNU bug report logs - #22534
File notify broken on Windows

Previous Next

Full log

Message #59 received at 22534 <at> debbugs.gnu.org (full text, mbox):

From: Fabrice Popineau <fabrice.popineau <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 22534 <at> debbugs.gnu.org, Michael Albinus <michael.albinus <at> gmx.de>
Subject: Re: bug#22534: File notify broken on Windows
Date: Sun, 7 Feb 2016 14:37:34 +0100

[Message part 1 (text/plain, inline)]

The situation has improved especially with file-notify-test06-many-events,
but I still see problems in batch mode.
The remaining problems are in file-notify-test02-events.

In the 'check for attribute change', I see only 2 'changed' notifications,
and not 4
as expected by the test (around line 512: w32notify does not distinguish
between
'changed' and 'attribute-changed').

So I need to apply :

diff --git a/test/automated/file-notify-tests.el
b/test/automated/file-notify-tests.el
index 5fc4ff8..943bd7e 100644
--- a/test/automated/file-notify-tests.el
+++ b/test/automated/file-notify-tests.el
@@ -507,7 +512,7 @@ file-notify--test-with-events
                ;; w32notify does not distinguish between `changed' and
                ;; `attribute-changed'.
                ((string-equal (file-notify--test-library) "w32notify")
-               '(changed changed changed changed))
+               '(changed changed))
                ;; For kqueue and in the remote case, `write-region'
                ;; raises also an `attribute-changed' event.
                ((or (string-equal (file-notify--test-library) "kqueue")

For the rest of this test, whereas the test passes in interactive mode, it
fails in batch mode with
several 'deleted' notifications. Namely, I need to apply the following for
the test to pass.

diff --git a/test/automated/file-notify-tests.el
b/test/automated/file-notify-tests.el
index 5fc4ff8..943bd7e 100644
--- a/test/automated/file-notify-tests.el
+++ b/test/automated/file-notify-tests.el
@@ -396,7 +397,8 @@ file-notify--test-with-events
                ;; w32notify does raise a `stopped' event when a
                ;; watched directory is deleted.
                ((string-equal (file-notify--test-library) "w32notify")
-               '(created changed deleted))
+               '(created changed ; deleted
+                          ))
                ;; cygwin recognizes only `deleted' and `stopped' events.
                ((eq system-type 'cygwin)
                '(deleted stopped))
@@ -430,7 +432,8 @@ file-notify--test-with-events
                ;; `attribute-changed'.
                ((string-equal (file-notify--test-library) "w32notify")
                '(created changed created changed changed changed changed
-                 deleted deleted))
+                          ;; deleted deleted
+                          ))
                ;; cygwin recognizes only `deleted' and `stopped' events.
                ((eq system-type 'cygwin)
                '(deleted stopped))
@@ -479,6 +482,8 @@ file-notify--test-with-events
                ;; the directory.  Except for kqueue.
                ((string-equal (file-notify--test-library) "kqueue")
                '(created changed renamed deleted stopped))
+               ((string-equal (file-notify--test-library) "w32notify")
+               '(created changed renamed))
                (t '(created changed renamed deleted deleted stopped)))
             (read-event nil nil 0.1)
             (write-region


Fabrice

2016-02-06 20:55 GMT+01:00 Eli Zaretskii <eliz <at> gnu.org>:

> > From: Michael Albinus <michael.albinus <at> gmx.de>
> > Cc: 22534 <at> debbugs.gnu.org,  fabrice.popineau <at> gmail.com
> > Date: Sat, 06 Feb 2016 20:24:32 +0100
> >
> > Please install the patch
>
> Done.
>

[Message part 2 (text/html, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.