all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* Emacs 27.2 RC2
@ 2021-03-19  7:35 Eli Zaretskii
  2021-03-19 10:42 ` Robert Pluim
  2021-03-19 13:42 ` bug#47262: [platform-testers] " Dagobert Michelsen
  0 siblings, 2 replies; 30+ messages in thread
From: Eli Zaretskii @ 2021-03-19  7:35 UTC (permalink / raw)
  To: emacs-devel

The second release candidate for what will be the 27.2 release of Emacs
(the extensible text editor) is now available at

  https://alpha.gnu.org/gnu/emacs/pretest/emacs-27.2-rc2.tar.xz

The tarball is signed; you can get the PGP signature file at

  https://alpha.gnu.org/gnu/emacs/pretest/emacs-27.2-rc2.tar.xz.sig

Please give it as much testing as you can.

As always, if you encounter problems building or using Emacs,
send a report to bug-gnu-emacs@gnu.org with full details
(if possible, use M-x report-emacs-bug).

Thanks for helping to test Emacs.



^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: Emacs 27.2 RC2
  2021-03-19  7:35 Emacs 27.2 RC2 Eli Zaretskii
@ 2021-03-19 10:42 ` Robert Pluim
  2021-03-19 12:19   ` Eli Zaretskii
  2021-03-19 13:42 ` bug#47262: [platform-testers] " Dagobert Michelsen
  1 sibling, 1 reply; 30+ messages in thread
From: Robert Pluim @ 2021-03-19 10:42 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: emacs-devel

>>>>> On Fri, 19 Mar 2021 09:35:55 +0200, Eli Zaretskii <eliz@gnu.org> said:

    Eli> The second release candidate for what will be the 27.2 release of Emacs
    Eli> (the extensible text editor) is now available at

    Eli>   https://alpha.gnu.org/gnu/emacs/pretest/emacs-27.2-rc2.tar.xz

    Eli> The tarball is signed; you can get the PGP signature file at

    Eli>   https://alpha.gnu.org/gnu/emacs/pretest/emacs-27.2-rc2.tar.xz.sig

    Eli> Please give it as much testing as you can.

The tarball builds fine, but building the corresponding tag from git:

make[2]: Entering directory '/home/rpluim/repos/emacs-27/lisp'
  ELC      cedet/semantic/wisent/java-tags-wy.elc

In toplevel form:
cedet/semantic/wisent/java-tags-wy.el:58:17:Error: Bad input grammar
make[2]: *** [Makefile:296: cedet/semantic/wisent/java-tags-wy.elc] Error 1
make[2]: Leaving directory '/home/rpluim/repos/emacs-27/lisp'
make[1]: *** [Makefile:319: compile-main] Error 2
make[1]: Leaving directory '/home/rpluim/repos/emacs-27/lisp'
make: *** [Makefile:411: lisp] Error 2

This is 'make bootstrap'. Is there some extra 'make clean' command I
need to run?

Robert
-- 



^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: Emacs 27.2 RC2
  2021-03-19 10:42 ` Robert Pluim
@ 2021-03-19 12:19   ` Eli Zaretskii
  2021-03-19 17:50     ` Robert Pluim
  0 siblings, 1 reply; 30+ messages in thread
From: Eli Zaretskii @ 2021-03-19 12:19 UTC (permalink / raw)
  To: Robert Pluim; +Cc: emacs-devel

> From: Robert Pluim <rpluim@gmail.com>
> Cc: emacs-devel@gnu.org
> Date: Fri, 19 Mar 2021 11:42:53 +0100
> 
> The tarball builds fine, but building the corresponding tag from git:
> 
> make[2]: Entering directory '/home/rpluim/repos/emacs-27/lisp'
>   ELC      cedet/semantic/wisent/java-tags-wy.elc
> 
> In toplevel form:
> cedet/semantic/wisent/java-tags-wy.el:58:17:Error: Bad input grammar
> make[2]: *** [Makefile:296: cedet/semantic/wisent/java-tags-wy.elc] Error 1
> make[2]: Leaving directory '/home/rpluim/repos/emacs-27/lisp'
> make[1]: *** [Makefile:319: compile-main] Error 2
> make[1]: Leaving directory '/home/rpluim/repos/emacs-27/lisp'
> make: *** [Makefile:411: lisp] Error 2
> 
> This is 'make bootstrap'. Is there some extra 'make clean' command I
> need to run?

I have no idea.  The bootstrap ran successfully for me from a pristine
checkout, AFAIR.

There's no file cedet/semantic/wisent/java-tags-wy.el in my
repository, and it isn't present in the tarball.  Is that a versioned
file in your case, or just a left-over from some distant past?



^ permalink raw reply	[flat|nested] 30+ messages in thread

* bug#47262: [platform-testers] Emacs 27.2 RC2
  2021-03-19  7:35 Emacs 27.2 RC2 Eli Zaretskii
  2021-03-19 10:42 ` Robert Pluim
@ 2021-03-19 13:42 ` Dagobert Michelsen
  2021-03-19 14:05   ` Eli Zaretskii
                     ` (3 more replies)
  1 sibling, 4 replies; 30+ messages in thread
From: Dagobert Michelsen @ 2021-03-19 13:42 UTC (permalink / raw)
  To: eliz, 47262

Hi Eli,

Am 19.03.2021 um 08:35 schrieb Eli Zaretskii <eliz@gnu.org>:
> The second release candidate for what will be the 27.2 release of Emacs
> (the extensible text editor) is now available at
> 
>  https://alpha.gnu.org/gnu/emacs/pretest/emacs-27.2-rc2.tar.xz
> 
> The tarball is signed; you can get the PGP signature file at
> 
>  https://alpha.gnu.org/gnu/emacs/pretest/emacs-27.2-rc2.tar.xz.sig
> 
> Please give it as much testing as you can.
> 
> As always, if you encounter problems building or using Emacs,
> send a report to bug-gnu-emacs@gnu.org with full details
> (if possible, use M-x report-emacs-bug).
> 
> Thanks for helping to test Emacs.

I have a couple of test failures with Solaris 10 Sparc and Sun Studio:

SUMMARY OF TEST RESULTS
-----------------------
Files examined: 267
Ran 3829 tests, 77 failed to run, 3716 results as expected, 23 unexpected, 90 skipped
3 files did not finish:
  src/emacs-module-tests.log
  lisp/progmodes/sql-tests.log
  lisp/auth-source-tests.log
8 files contained unexpected results:
  lisp/vc/diff-mode-tests.log
  lisp/progmodes/xref-tests.log
  lisp/progmodes/flymake-tests.log
  lisp/net/nsm-tests.log
  lisp/net/network-stream-tests.log
  lisp/files-tests.log
  lisp/filenotify-tests.log
  lisp/cedet/semantic-utest-ia.log
gmake[2]: *** [Makefile:318: check-doit] Error 2

All logfiles are available from the buildfarm at
  http://buildfarm.opencsw.org/~dam/emacs-27.2-rc2-logs.tar.gz


Best regards

  — Dago

-- 
"You don't become great by trying to be great, you become great by wanting to do something,
and then doing it so hard that you become great in the process." - xkcd #896






^ permalink raw reply	[flat|nested] 30+ messages in thread

* bug#47262: [platform-testers] Emacs 27.2 RC2
  2021-03-19 13:42 ` bug#47262: [platform-testers] " Dagobert Michelsen
@ 2021-03-19 14:05   ` Eli Zaretskii
  2021-03-19 18:03   ` Robert Pluim
                     ` (2 subsequent siblings)
  3 siblings, 0 replies; 30+ messages in thread
From: Eli Zaretskii @ 2021-03-19 14:05 UTC (permalink / raw)
  To: Dagobert Michelsen; +Cc: 47262

> From: Dagobert Michelsen <dam@opencsw.org>
> Date: Fri, 19 Mar 2021 14:42:23 +0100
> 
> All logfiles are available from the buildfarm at
>   http://buildfarm.opencsw.org/~dam/emacs-27.2-rc2-logs.tar.gz

Thanks.





^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: Emacs 27.2 RC2
  2021-03-19 12:19   ` Eli Zaretskii
@ 2021-03-19 17:50     ` Robert Pluim
  2021-03-19 19:44       ` Eli Zaretskii
  0 siblings, 1 reply; 30+ messages in thread
From: Robert Pluim @ 2021-03-19 17:50 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: emacs-devel

>>>>> On Fri, 19 Mar 2021 14:19:56 +0200, Eli Zaretskii <eliz@gnu.org> said:

    >> This is 'make bootstrap'. Is there some extra 'make clean' command I
    >> need to run?

    Eli> I have no idea.  The bootstrap ran successfully for me from a pristine
    Eli> checkout, AFAIR.

    Eli> There's no file cedet/semantic/wisent/java-tags-wy.el in my
    Eli> repository, and it isn't present in the tarball.  Is that a versioned
    Eli> file in your case, or just a left-over from some distant past?

Itʼs a generated file. If I remove it, and also
lisp/cedet/srecode/srecode-template-wy.el then the build succeeds. As
you say, probably a leftover from before some cedet changes.

Robert
-- 



^ permalink raw reply	[flat|nested] 30+ messages in thread

* bug#47262: [platform-testers] Emacs 27.2 RC2
  2021-03-19 13:42 ` bug#47262: [platform-testers] " Dagobert Michelsen
  2021-03-19 14:05   ` Eli Zaretskii
@ 2021-03-19 18:03   ` Robert Pluim
  2021-03-19 20:09     ` Eli Zaretskii
  2021-03-20  9:04   ` Michael Albinus
  2021-10-21 21:19   ` Stefan Kangas
  3 siblings, 1 reply; 30+ messages in thread
From: Robert Pluim @ 2021-03-19 18:03 UTC (permalink / raw)
  To: Dagobert Michelsen; +Cc: 47262

>>>>> On Fri, 19 Mar 2021 14:42:23 +0100, Dagobert Michelsen <dam@opencsw.org> said:

    Dagobert>   lisp/net/nsm-tests.log

This is Bug#45798, I think, fixed on master by 6dc4fc7d62. Itʼs a
trivial change, I guess we could cherry-pick it to emacs-27, but itʼs
in a code path that no sane user would enable.

    Dagobert>   lisp/net/network-stream-tests.log

And this is 90ac405702, and by extension Bug#46709 and
Bug#46641. Thatʼs probably a bit much to put into emacs-27 (the
underlying code is mostly fine, itʼs the tests that are wrong)

Robert
-- 





^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: Emacs 27.2 RC2
  2021-03-19 17:50     ` Robert Pluim
@ 2021-03-19 19:44       ` Eli Zaretskii
  0 siblings, 0 replies; 30+ messages in thread
From: Eli Zaretskii @ 2021-03-19 19:44 UTC (permalink / raw)
  To: Robert Pluim; +Cc: emacs-devel

> From: Robert Pluim <rpluim@gmail.com>
> Cc: emacs-devel@gnu.org
> Date: Fri, 19 Mar 2021 18:50:17 +0100
> 
>     Eli> There's no file cedet/semantic/wisent/java-tags-wy.el in my
>     Eli> repository, and it isn't present in the tarball.  Is that a versioned
>     Eli> file in your case, or just a left-over from some distant past?
> 
> Itʼs a generated file. If I remove it, and also
> lisp/cedet/srecode/srecode-template-wy.el then the build succeeds. As
> you say, probably a leftover from before some cedet changes.

OK, then we are good.  (That other file is also absent in my tree.)




^ permalink raw reply	[flat|nested] 30+ messages in thread

* bug#47262: [platform-testers] Emacs 27.2 RC2
  2021-03-19 18:03   ` Robert Pluim
@ 2021-03-19 20:09     ` Eli Zaretskii
  0 siblings, 0 replies; 30+ messages in thread
From: Eli Zaretskii @ 2021-03-19 20:09 UTC (permalink / raw)
  To: Robert Pluim; +Cc: dam, 47262

> From: Robert Pluim <rpluim@gmail.com>
> Cc: eliz@gnu.org,  47262@debbugs.gnu.org
> Date: Fri, 19 Mar 2021 19:03:09 +0100
> 
> >>>>> On Fri, 19 Mar 2021 14:42:23 +0100, Dagobert Michelsen <dam@opencsw.org> said:
> 
>     Dagobert>   lisp/net/nsm-tests.log
> 
> This is Bug#45798, I think, fixed on master by 6dc4fc7d62. Itʼs a
> trivial change, I guess we could cherry-pick it to emacs-27, but itʼs
> in a code path that no sane user would enable.
> 
>     Dagobert>   lisp/net/network-stream-tests.log
> 
> And this is 90ac405702, and by extension Bug#46709 and
> Bug#46641. Thatʼs probably a bit much to put into emacs-27 (the
> underlying code is mostly fine, itʼs the tests that are wrong)

Thanks.  I think we are good for 27.2.  There's always one more bug.





^ permalink raw reply	[flat|nested] 30+ messages in thread

* bug#47262: [platform-testers] Emacs 27.2 RC2
  2021-03-19 13:42 ` bug#47262: [platform-testers] " Dagobert Michelsen
  2021-03-19 14:05   ` Eli Zaretskii
  2021-03-19 18:03   ` Robert Pluim
@ 2021-03-20  9:04   ` Michael Albinus
  2021-03-20 14:14     ` Dagobert Michelsen
  2021-10-21 21:19   ` Stefan Kangas
  3 siblings, 1 reply; 30+ messages in thread
From: Michael Albinus @ 2021-03-20  9:04 UTC (permalink / raw)
  To: Dagobert Michelsen; +Cc: 47262

[-- Attachment #1: Type: text/plain, Size: 865 bytes --]

Dagobert Michelsen <dam@opencsw.org> writes:

Hi Dago,

> I have a couple of test failures with Solaris 10 Sparc and Sun Studio:

Thanks a lot!

>   lisp/filenotify-tests.log

These errors are only in the filenotify-tests.el. There's no reason to
stop Emacs 27.2 because of this.

Running 8 tests (2021-03-19 11:06:22+0100, selector `(not (or (tag :expensive-test) (tag :unstable)))')
Library: `gfilenotify'
Monitor: `GFamDirectoryMonitor'

Wow! I know that the GFamDirectoryMonitor exists (it is in the gio
sources), but I couldn't test it ever. Solaris.

Could you pls apply the appended patch to filenotify-tests.el and show
the changed filenotify-tests.log file? Many thanks in advance (and be
prepared, I will exploit you iteratively until all tests pass. If you
agree :-)

> Best regards
>
>   — Dago

Best regards, Michael.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: Type: text/x-patch, Size: 1882 bytes --]

*** /tmp/ediffGmYi43	2021-03-20 09:57:10.563305971 +0100
--- /home/albinus/src/emacs-27/test/lisp/filenotify-tests.el	2021-03-20 09:55:13.121660549 +0100
***************
*** 109,115 ****
      ;; at least on Cygwin.
      ((and (string-equal (file-notify--test-library) "gfilenotify")
            (memq (file-notify--test-monitor)
!                 '(GFamFileMonitor GPollFileMonitor)))
       7)
      ((string-equal (file-notify--test-library) "gvfs-monitor-dir.exe") 1)
      ((file-remote-p temporary-file-directory) 0.1)
--- 109,115 ----
      ;; at least on Cygwin.
      ((and (string-equal (file-notify--test-library) "gfilenotify")
            (memq (file-notify--test-monitor)
!                 '(GFamFileMonitor GFamDirectoryMonitor GPollFileMonitor)))
       7)
      ((string-equal (file-notify--test-library) "gvfs-monitor-dir.exe") 1)
      ((file-remote-p temporary-file-directory) 0.1)
***************
*** 1021,1026 ****
--- 1021,1029 ----
  	     ((string-equal (file-notify--test-library) "gvfs-monitor-dir.exe")
  	      '((deleted stopped)
  		(changed deleted stopped)))
+ 	     ;; GFamDirectoryMonitor does not report the `changed' event.
+ 	     ((equal (file-notify--test-monitor) 'GFamDirectoryMonitor)
+ 	      '(deleted stopped))
  	     ;; There could be one or two `changed' events.
  	     (t '((changed deleted stopped)
  		  (changed changed deleted stopped))))
***************
*** 1286,1291 ****
--- 1289,1297 ----
               ;; On cygwin we only get the `changed' event.
               ((eq system-type 'cygwin)
                '(changed))
+ 	     ;; GFamDirectoryMonitor reports only the `changed' event.
+ 	     ((equal (file-notify--test-monitor) 'GFamDirectoryMonitor)
+ 	      '(changed))
               (t '(renamed created changed)))
            ;; The file is renamed when creating a backup.  It shall
            ;; still be watched.

^ permalink raw reply	[flat|nested] 30+ messages in thread

* bug#47262: [platform-testers] Emacs 27.2 RC2
  2021-03-20  9:04   ` Michael Albinus
@ 2021-03-20 14:14     ` Dagobert Michelsen
  2021-03-20 15:52       ` Michael Albinus
  0 siblings, 1 reply; 30+ messages in thread
From: Dagobert Michelsen @ 2021-03-20 14:14 UTC (permalink / raw)
  To: Michael Albinus; +Cc: 47262

[-- Attachment #1: Type: text/plain, Size: 408 bytes --]

Hi Michael,

Am 20.03.2021 um 10:04 schrieb Michael Albinus <michael.albinus@gmx.de>:
> Dagobert Michelsen <dam@opencsw.org> writes:
> Wow! I know that the GFamDirectoryMonitor exists (it is in the gio
> sources), but I couldn't test it ever. Solaris.
> 
> Could you pls apply the appended patch to filenotify-tests.el and show
> the changed filenotify-tests.log file?

Sure, please find the file attached.


[-- Attachment #2: filenotify-tests.log --]
[-- Type: application/octet-stream, Size: 3338 bytes --]

Running 8 tests (2021-03-20 14:07:20+0100, selector `(not (or (tag :expensive-test) (tag :unstable)))')
Library: `gfilenotify'
Monitor: `GFamDirectoryMonitor'
   passed  1/8  file-notify-test00-availability (0.114761 sec)
   passed  2/8  file-notify-test01-add-watch (0.021881 sec)
   passed  3/8  file-notify-test02-rm-watch (0.983746 sec)
Test file-notify-test03-events backtrace:
  signal(ert-test-failed (((should (file-notify--test-with-actions-che
  ert-fail(((should (file-notify--test-with-actions-check actions)) :f
  #f(compiled-function () #<bytecode 0x165123>)()
  ert--run-test-internal(#s(ert--test-execution-info #s(ert-test file-
  ert-run-test(#s(ert-test file-notify-test03-events "Check file creat
  ert-run-or-rerun-test(#s(ert--stats (not ...) [#s(ert-test file-noti
  ert-run-tests((not (or (tag :expensive-test) (tag :unstable))) #f(co
  ert-run-tests-batch((not (or (tag :expensive-test) (tag :unstable)))
  ert-run-tests-batch-and-exit((not (or (tag :expensive-test) (tag :un
  eval((ert-run-tests-batch-and-exit '(not (or (tag :expensive-test) (
  command-line-1(("-L" ":." "-l" "ert" "-l" "lisp/filenotify-tests" "-
  command-line()
  normal-top-level()
Test file-notify-test03-events condition:
    (ert-test-failed
     ((should
       (file-notify--test-with-actions-check actions))
      :form
      (file-notify--test-with-actions-check
       ((created changed deleted stopped)))
      :value nil :explanation "Received actions do not match expected actions
(created deleted stopped)
(created changed deleted stopped)"))
   FAILED  4/8  file-notify-test03-events (17.010461 sec)
Reverting buffer `file-notify-test9LC2IC'.
Reverting buffer `file-notify-test9LC2IC'.
   passed  5/8  file-notify-test04-autorevert (18.041446 sec)
Test file-notify-test05-file-validity backtrace:
  signal(ert-test-failed (((should (file-notify--test-with-actions-che
  ert-fail(((should (file-notify--test-with-actions-check actions)) :f
  #f(compiled-function () #<bytecode 0x1655d5>)()
  ert--run-test-internal(#s(ert--test-execution-info #s(ert-test file-
  ert-run-test(#s(ert-test file-notify-test05-file-validity "Check `fi
  ert-run-or-rerun-test(#s(ert--stats (not ...) [#s(ert-test file-noti
  ert-run-tests((not (or (tag :expensive-test) (tag :unstable))) #f(co
  ert-run-tests-batch((not (or (tag :expensive-test) (tag :unstable)))
  ert-run-tests-batch-and-exit((not (or (tag :expensive-test) (tag :un
  eval((ert-run-tests-batch-and-exit '(not (or (tag :expensive-test) (
  command-line-1(("-L" ":." "-l" "ert" "-l" "lisp/filenotify-tests" "-
  command-line()
  normal-top-level()
Test file-notify-test05-file-validity condition:
    (ert-test-failed
     ((should
       (file-notify--test-with-actions-check actions))
      :form
      (file-notify--test-with-actions-check
       ((deleted stopped)))
      :value nil :explanation "Received actions do not match expected actions
(changed deleted stopped)
(deleted stopped)"))
   FAILED  6/8  file-notify-test05-file-validity (17.017923 sec)
   passed  7/8  file-notify-test06-dir-validity (3.011006 sec)
   passed  8/8  file-notify-test08-backup (26.051246 sec)

Ran 8 tests, 6 results as expected, 2 unexpected (2021-03-20 14:08:43+0100, 83.188041 sec)

2 unexpected results:
   FAILED  file-notify-test03-events
   FAILED  file-notify-test05-file-validity


[-- Attachment #3: Type: text/plain, Size: 353 bytes --]



> Many thanks in advance (and be
> prepared, I will exploit you iteratively until all tests pass. If you
> agree :-)

Of course :-)


Best regards

  — Dago

-- 
"You don't become great by trying to be great, you become great by wanting to do something,
and then doing it so hard that you become great in the process." - xkcd #896


^ permalink raw reply	[flat|nested] 30+ messages in thread

* bug#47262: [platform-testers] Emacs 27.2 RC2
  2021-03-20 14:14     ` Dagobert Michelsen
@ 2021-03-20 15:52       ` Michael Albinus
  2021-03-20 20:02         ` Dagobert Michelsen
  0 siblings, 1 reply; 30+ messages in thread
From: Michael Albinus @ 2021-03-20 15:52 UTC (permalink / raw)
  To: Dagobert Michelsen; +Cc: 47262

[-- Attachment #1: Type: text/plain, Size: 613 bytes --]

Dagobert Michelsen <dam@opencsw.org> writes:

> Hi Michael,

Hi Dago,

>> Many thanks in advance (and be
>> prepared, I will exploit you iteratively until all tests pass. If you
>> agree :-)
>
> Of course :-)

Well, here's another patch you could apply instead. I've changed the
tests as indicated in the previous log file. And in order to cut the
number of iterations, I don't let fail the tests any longer, but let
them trace what happens. Could you, pls, send me the next log file,
produced after applying this patch?

> Best regards
>
>   — Dago

Thanks, and best regards, Michael.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: Type: text/x-patch, Size: 5046 bytes --]

*** /tmp/ediffumMrVc	2021-03-20 16:50:58.959414691 +0100
--- /home/albinus/src/emacs-27/test/lisp/filenotify-tests.el	2021-03-20 16:44:46.138512407 +0100
***************
*** 109,115 ****
      ;; at least on Cygwin.
      ((and (string-equal (file-notify--test-library) "gfilenotify")
            (memq (file-notify--test-monitor)
!                 '(GFamFileMonitor GPollFileMonitor)))
       7)
      ((string-equal (file-notify--test-library) "gvfs-monitor-dir.exe") 1)
      ((file-remote-p temporary-file-directory) 0.1)
--- 109,115 ----
      ;; at least on Cygwin.
      ((and (string-equal (file-notify--test-library) "gfilenotify")
            (memq (file-notify--test-monitor)
!                 '(GFamFileMonitor GFamDirectoryMonitor GPollFileMonitor)))
       7)
      ((string-equal (file-notify--test-library) "gvfs-monitor-dir.exe") 1)
      ((file-remote-p temporary-file-directory) 0.1)
***************
*** 607,613 ****
           (ert-fail
            (cadr (ert-test-result-with-condition-condition result)))))
       ;; One of the possible event sequences shall match.
!      (should (file-notify--test-with-actions-check actions))))

  (ert-deftest file-notify-test03-events ()
    "Check file creation/change/removal notifications."
--- 607,617 ----
           (ert-fail
            (cadr (ert-test-result-with-condition-condition result)))))
       ;; One of the possible event sequences shall match.
!      (if (file-notify--test-with-actions-check actions)
!          (message
!           "Success!\n%s"
!           (mapcar #'file-notify--test-event-action file-notify--test-events))
!        (message (file-notify--test-with-actions-explainer actions)))))

  (ert-deftest file-notify-test03-events ()
    "Check file creation/change/removal notifications."
***************
*** 635,640 ****
--- 639,647 ----
               ;; cygwin does not raise a `changed' event.
               ((eq system-type 'cygwin)
                '(created deleted stopped))
+ 	     ;; GFamDirectoryMonitor does not report the `changed' event.
+ 	     ((equal (file-notify--test-monitor) 'GFamDirectoryMonitor)
+ 	      '(created deleted stopped))
               (t '(created changed deleted stopped)))
            (write-region
             "another text" nil file-notify--test-tmpfile nil 'no-message)
***************
*** 666,672 ****
  	      '((deleted stopped)
  		(changed deleted stopped)))
  	     ;; There could be one or two `changed' events.
! 	     (t '((changed deleted stopped)
  		  (changed changed deleted stopped))))
            (write-region
             "another text" nil file-notify--test-tmpfile nil 'no-message)
--- 673,682 ----
  	      '((deleted stopped)
  		(changed deleted stopped)))
  	     ;; There could be one or two `changed' events.
! 	     ;; GFamDirectoryMonitor does not report the `changed' event.
! 	     ((equal (file-notify--test-monitor) 'GFamDirectoryMonitor)
! 	      '(deleted stopped))
!  	     (t '((changed deleted stopped)
  		  (changed changed deleted stopped))))
            (write-region
             "another text" nil file-notify--test-tmpfile nil 'no-message)
***************
*** 713,719 ****
  	     ;; cygwin does not raise a `changed' event.
  	     ((eq system-type 'cygwin)
  	      '(created deleted stopped))
! 	     ((string-equal (file-notify--test-library) "kqueue")
  	      '(created changed deleted stopped))
  	     (t '(created changed deleted deleted stopped)))
  	  (write-region
--- 723,732 ----
  	     ;; cygwin does not raise a `changed' event.
  	     ((eq system-type 'cygwin)
  	      '(created deleted stopped))
! 	     ;; GFamDirectoryMonitor does not report the `changed' event.
! 	     ((equal (file-notify--test-monitor) 'GFamDirectoryMonitor)
! 	      '(created deleted stopped))
!  	     ((string-equal (file-notify--test-library) "kqueue")
  	      '(created changed deleted stopped))
  	     (t '(created changed deleted deleted stopped)))
  	  (write-region
***************
*** 1021,1026 ****
--- 1034,1042 ----
  	     ((string-equal (file-notify--test-library) "gvfs-monitor-dir.exe")
  	      '((deleted stopped)
  		(changed deleted stopped)))
+ 	     ;; GFamDirectoryMonitor does not report the `changed' event.
+ 	     ((equal (file-notify--test-monitor) 'GFamDirectoryMonitor)
+ 	      '(deleted stopped))
  	     ;; There could be one or two `changed' events.
  	     (t '((changed deleted stopped)
  		  (changed changed deleted stopped))))
***************
*** 1286,1291 ****
--- 1302,1310 ----
               ;; On cygwin we only get the `changed' event.
               ((eq system-type 'cygwin)
                '(changed))
+ 	     ;; GFamDirectoryMonitor reports only the `changed' event.
+ 	     ((equal (file-notify--test-monitor) 'GFamDirectoryMonitor)
+ 	      '(changed))
               (t '(renamed created changed)))
            ;; The file is renamed when creating a backup.  It shall
            ;; still be watched.

^ permalink raw reply	[flat|nested] 30+ messages in thread

* bug#47262: [platform-testers] Emacs 27.2 RC2
  2021-03-20 15:52       ` Michael Albinus
@ 2021-03-20 20:02         ` Dagobert Michelsen
  2021-03-21 12:48           ` Michael Albinus
  0 siblings, 1 reply; 30+ messages in thread
From: Dagobert Michelsen @ 2021-03-20 20:02 UTC (permalink / raw)
  To: Michael Albinus; +Cc: 47262

[-- Attachment #1: Type: text/plain, Size: 456 bytes --]

Hallo Michael,

Am 20.03.2021 um 16:52 schrieb Michael Albinus <michael.albinus@gmx.de>:
> Well, here's another patch you could apply instead. I've changed the
> tests as indicated in the previous log file. And in order to cut the
> number of iterations, I don't let fail the tests any longer, but let
> them trace what happens. Could you, pls, send me the next log file,
> produced after applying this patch?

Here is the log file from the recent patch:


[-- Attachment #2: filenotify-tests.log --]
[-- Type: application/octet-stream, Size: 1624 bytes --]

Running 8 tests (2021-03-20 18:16:55+0100, selector `(not (or (tag :expensive-test) (tag :unstable)))')
Library: `gfilenotify'
Monitor: `GFamDirectoryMonitor'
   passed  1/8  file-notify-test00-availability (0.103754 sec)
   passed  2/8  file-notify-test01-add-watch (0.014330 sec)
   passed  3/8  file-notify-test02-rm-watch (1.003603 sec)
Success!
(created deleted stopped)
Received actions do not match expected actions
(changed deleted stopped)
(deleted stopped)
Success!
(created deleted stopped)
Received actions do not match expected actions
(created created changed changed deleted stopped)
(created changed created changed deleted deleted deleted stopped)
Received actions do not match expected actions
(created created deleted deleted stopped)
(created changed renamed deleted deleted stopped)
Received actions do not match expected actions
nil
(attribute-changed attribute-changed)
   passed  4/8  file-notify-test03-events (144.107015 sec)
Reverting buffer `file-notify-testoR7LG5'.
Reverting buffer `file-notify-testoR7LG5'.
   passed  5/8  file-notify-test04-autorevert (18.063908 sec)
Received actions do not match expected actions
(changed deleted stopped)
(deleted stopped)
Received actions do not match expected actions
(created deleted stopped)
(created changed deleted deleted stopped)
   passed  6/8  file-notify-test05-file-validity (34.039652 sec)
   passed  7/8  file-notify-test06-dir-validity (3.012760 sec)
Success!
(changed)
Success!
(changed)
   passed  8/8  file-notify-test08-backup (26.072395 sec)

Ran 8 tests, 8 results as expected, 0 unexpected (2021-03-20 18:20:42+0100, 226.429326 sec)


[-- Attachment #3: Type: text/plain, Size: 214 bytes --]




Best regards

  — Dago

-- 
"You don't become great by trying to be great, you become great by wanting to do something,
and then doing it so hard that you become great in the process." - xkcd #896


^ permalink raw reply	[flat|nested] 30+ messages in thread

* bug#47262: [platform-testers] Emacs 27.2 RC2
  2021-03-20 20:02         ` Dagobert Michelsen
@ 2021-03-21 12:48           ` Michael Albinus
  2021-03-22 14:17             ` Dagobert Michelsen
  0 siblings, 1 reply; 30+ messages in thread
From: Michael Albinus @ 2021-03-21 12:48 UTC (permalink / raw)
  To: Dagobert Michelsen; +Cc: 47262

[-- Attachment #1: Type: text/plain, Size: 796 bytes --]

Dagobert Michelsen <dam@opencsw.org> writes:

> Hallo Michael,

Hi Dago,

> Here is the log file from the recent patch:

Thanks. I've appended another patch, which should fix all open problems
on Solaris. Could you pls check?

Furthermore, I have seen that you didn't run the expensive tests
file-notify-test07-many-events and
file-notify-test09-watched-file-in-watched-dir. I expect further
problems there, so please call

# make -C test filenotify-tests.log

Likely, there will be other errors. In that case, set file-notify-debug
to t (line 201), and rerun that expensive test. There will be similar
traces in the log file as you have sent already, which shall allow me to
continue to work.

> Best regards
>
>   — Dago

Many thanks, and best regards, Michael.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: Type: text/x-patch, Size: 13038 bytes --]

*** /tmp/ediffpZId9s	2021-03-21 13:38:20.876372597 +0100
--- /usr/local/src/emacs-27/test/lisp/filenotify-tests.el	2021-03-21 13:36:24.497655703 +0100
***************
*** 105,115 ****
     (cond
      ;; gio/gpollfilemonitor.c declares POLL_TIME_SECS 5.  So we must
      ;; wait at least this time in the GPollFileMonitor case.  A
!     ;; similar timeout seems to be needed in the GFamFileMonitor case,
!     ;; at least on Cygwin.
      ((and (string-equal (file-notify--test-library) "gfilenotify")
            (memq (file-notify--test-monitor)
!                 '(GFamFileMonitor GPollFileMonitor)))
       7)
      ((string-equal (file-notify--test-library) "gvfs-monitor-dir.exe") 1)
      ((file-remote-p temporary-file-directory) 0.1)
--- 105,115 ----
     (cond
      ;; gio/gpollfilemonitor.c declares POLL_TIME_SECS 5.  So we must
      ;; wait at least this time in the GPollFileMonitor case.  A
!     ;; similar timeout seems to be needed in the GFamFileMonitor and
!     ;; GFamDirectoryMonitor case, at least on cygwin.
      ((and (string-equal (file-notify--test-library) "gfilenotify")
            (memq (file-notify--test-monitor)
!                 '(GFamFileMonitor GFamDirectoryMonitor GPollFileMonitor)))
       7)
      ((string-equal (file-notify--test-library) "gvfs-monitor-dir.exe") 1)
      ((file-remote-p temporary-file-directory) 0.1)
***************
*** 455,461 ****

    (unwind-protect
        ;; Check, that removing watch descriptors out of order do not
!       ;; harm.  This fails on Cygwin because of timing issues unless a
        ;; long `sit-for' is added before the call to
        ;; `file-notify--test-read-event'.
        (unless (eq system-type 'cygwin)
--- 455,461 ----

    (unwind-protect
        ;; Check, that removing watch descriptors out of order do not
!       ;; harm.  This fails on cygwin because of timing issues unless a
        ;; long `sit-for' is added before the call to
        ;; `file-notify--test-read-event'.
        (unless (eq system-type 'cygwin)
***************
*** 542,547 ****
--- 542,551 ----
  	    file-notify--test-results
  	    (append file-notify--test-results `(,result))))))

+ (defun file-notify--test-event-actions ()
+   "Helper function to return retrieved actions, as list."
+   (mapcar #'file-notify--test-event-action file-notify--test-events))
+
  (defun file-notify--test-with-actions-check (actions)
    "Check whether received actions match one of the ACTIONS alternatives."
    (let (result)
***************
*** 550,571 ****
              (or result
                  (if (eq (car elt) :random)
                      (equal (sort (cdr elt) 'string-lessp)
!                            (sort (mapcar #'file-notify--test-event-action
!                                          file-notify--test-events)
                                   'string-lessp))
!                   (equal elt (mapcar #'file-notify--test-event-action
!                                      file-notify--test-events))))))))

  (defun file-notify--test-with-actions-explainer (actions)
    "Explain why `file-notify--test-with-actions-check' fails."
    (if (null (cdr actions))
        (format "Received actions do not match expected actions\n%s\n%s"
!               (mapcar #'file-notify--test-event-action file-notify--test-events)
!               (car actions))
      (format
       "Received actions do not match any sequence of expected actions\n%s\n%s"
!      (mapcar #'file-notify--test-event-action file-notify--test-events)
!      actions)))

  (put 'file-notify--test-with-actions-check 'ert-explainer
       'file-notify--test-with-actions-explainer)
--- 554,578 ----
              (or result
                  (if (eq (car elt) :random)
                      (equal (sort (cdr elt) 'string-lessp)
!                            (sort (file-notify--test-event-actions)
                                   'string-lessp))
!                   (equal elt (file-notify--test-event-actions))))))
!     ;; Do not report result in case we debug.  Write messages instead.
!     (if file-notify-debug
!         (prog1 t
!           (if result
!               (message "Success\n%s" (file-notify--test-event-actions))
!             (message (file-notify--test-with-actions-explainer actions))))
!       result)))

  (defun file-notify--test-with-actions-explainer (actions)
    "Explain why `file-notify--test-with-actions-check' fails."
    (if (null (cdr actions))
        (format "Received actions do not match expected actions\n%s\n%s"
!               (file-notify--test-event-actions) (car actions))
      (format
       "Received actions do not match any sequence of expected actions\n%s\n%s"
!      (file-notify--test-event-actions) actions)))

  (put 'file-notify--test-with-actions-check 'ert-explainer
       'file-notify--test-with-actions-explainer)
***************
*** 635,640 ****
--- 642,650 ----
               ;; cygwin does not raise a `changed' event.
               ((eq system-type 'cygwin)
                '(created deleted stopped))
+ 	     ;; GFamDirectoryMonitor does not report the `changed' event.
+ 	     ((equal (file-notify--test-monitor) 'GFamDirectoryMonitor)
+ 	      '(created deleted stopped))
               (t '(created changed deleted stopped)))
            (write-region
             "another text" nil file-notify--test-tmpfile nil 'no-message)
***************
*** 665,670 ****
--- 675,685 ----
  	     ((string-equal (file-notify--test-library) "gvfs-monitor-dir.exe")
  	      '((deleted stopped)
  		(changed deleted stopped)))
+ 	     ;; GFamDirectoryMonitor does not detect the `changed'
+              ;; event reliably.
+ 	     ((equal (file-notify--test-monitor) 'GFamDirectoryMonitor)
+ 	      '((deleted stopped)
+ 		(changed deleted stopped)))
  	     ;; There could be one or two `changed' events.
  	     (t '((changed deleted stopped)
  		  (changed changed deleted stopped))))
***************
*** 713,718 ****
--- 728,736 ----
  	     ;; cygwin does not raise a `changed' event.
  	     ((eq system-type 'cygwin)
  	      '(created deleted stopped))
+ 	     ;; GFamDirectoryMonitor does not report the `changed' event.
+ 	     ((equal (file-notify--test-monitor) 'GFamDirectoryMonitor)
+ 	      '(created deleted stopped))
  	     ((string-equal (file-notify--test-library) "kqueue")
  	      '(created changed deleted stopped))
  	     (t '(created changed deleted deleted stopped)))
***************
*** 755,763 ****
  	      '((deleted stopped)
  		(created created deleted stopped)))
  	     ;; There are three `deleted' events, for two files and
! 	     ;; for the directory.  Except for cygwin and kqueue.
  	     ((eq system-type 'cygwin)
  	      '(created created changed changed deleted stopped))
  	     ((string-equal (file-notify--test-library) "kqueue")
  	      '(created changed created changed deleted stopped))
               ;; On emba, `deleted' and `stopped' events of the
--- 773,784 ----
  	      '((deleted stopped)
  		(created created deleted stopped)))
  	     ;; There are three `deleted' events, for two files and
! 	     ;; for the directory.  Except for cygwin,
! 	     ;; GFamDirectoryMonitor and kqueue.
  	     ((eq system-type 'cygwin)
  	      '(created created changed changed deleted stopped))
+ 	     ((equal (file-notify--test-monitor) 'GFamDirectoryMonitor)
+ 	      '(created created changed changed deleted stopped))
  	     ((string-equal (file-notify--test-library) "kqueue")
  	      '(created changed created changed deleted stopped))
               ;; On emba, `deleted' and `stopped' events of the
***************
*** 813,823 ****
               ((getenv "EMACS_EMBA_CI")
                '(created changed renamed deleted))
  	     ;; There are two `deleted' events, for the file and for
! 	     ;; the directory.  Except for cygwin and kqueue.  And
! 	     ;; cygwin raises `created' and `deleted' events instead
! 	     ;; of a `renamed' event.
  	     ((eq system-type 'cygwin)
  	      '(created created deleted deleted stopped))
  	     ((string-equal (file-notify--test-library) "kqueue")
  	      '(created changed renamed deleted stopped))
  	     (t '(created changed renamed deleted deleted stopped)))
--- 834,847 ----
               ((getenv "EMACS_EMBA_CI")
                '(created changed renamed deleted))
  	     ;; There are two `deleted' events, for the file and for
! 	     ;; the directory.  Except for cygwin,
! 	     ;; GFamDirectoryMonitor and kqueue.  And cygwin and
! 	     ;; GFamDirectoryMonitor raise `created' and `deleted'
! 	     ;; events instead of a `renamed' event.
  	     ((eq system-type 'cygwin)
  	      '(created created deleted deleted stopped))
+ 	     ((equal (file-notify--test-monitor) 'GFamDirectoryMonitor)
+ 	      '(created created deleted deleted stopped))
  	     ((string-equal (file-notify--test-library) "kqueue")
  	      '(created changed renamed deleted stopped))
  	     (t '(created changed renamed deleted deleted stopped)))
***************
*** 837,844 ****
      (file-notify--test-cleanup))

    (unwind-protect
!       ;; Check attribute change.  Does not work for cygwin.
!       (unless (eq system-type 'cygwin)
  	(setq file-notify--test-tmpfile (file-notify--test-make-temp-name))
  	(write-region
  	 "any text" nil file-notify--test-tmpfile nil 'no-message)
--- 861,870 ----
      (file-notify--test-cleanup))

    (unwind-protect
!       ;; Check attribute change.  Does not work for cygwin and
!       ;; GFamDirectoryMonitor.
!       (unless (or (eq system-type 'cygwin)
!                   (equal (file-notify--test-monitor) 'GFamDirectoryMonitor))
  	(setq file-notify--test-tmpfile (file-notify--test-make-temp-name))
  	(write-region
  	 "any text" nil file-notify--test-tmpfile nil 'no-message)
***************
*** 951,957 ****

  	    ;; Modify file.  We wait for two seconds, in order to
  	    ;; have another timestamp.  One second seems to be too
!             ;; short.  And Cygwin sporadically requires more than two.
              (ert-with-message-capture captured-messages
                (sleep-for (if (eq system-type 'cygwin) 3 2))
                (write-region
--- 977,983 ----

  	    ;; Modify file.  We wait for two seconds, in order to
  	    ;; have another timestamp.  One second seems to be too
!             ;; short.  And cygwin sporadically requires more than two.
              (ert-with-message-capture captured-messages
                (sleep-for (if (eq system-type 'cygwin) 3 2))
                (write-region
***************
*** 1021,1026 ****
--- 1047,1057 ----
  	     ((string-equal (file-notify--test-library) "gvfs-monitor-dir.exe")
  	      '((deleted stopped)
  		(changed deleted stopped)))
+ 	     ;; GFamDirectoryMonitor does not detect the `changed'
+              ;; event reliably.
+ 	     ((equal (file-notify--test-monitor) 'GFamDirectoryMonitor)
+ 	      '((deleted stopped)
+ 	        (changed deleted stopped)))
  	     ;; There could be one or two `changed' events.
  	     (t '((changed deleted stopped)
  		  (changed changed deleted stopped))))
***************
*** 1064,1073 ****
  	        '((deleted stopped)
  	          (created deleted stopped)))
  	       ;; There are two `deleted' events, for the file and for
! 	       ;; the directory.  Except for cygwin and kqueue.  And
! 	       ;; cygwin does not raise a `changed' event.
  	       ((eq system-type 'cygwin)
  	        '(created deleted stopped))
  	       ((string-equal (file-notify--test-library) "kqueue")
  	        '(created changed deleted stopped))
  	       (t '(created changed deleted deleted stopped)))
--- 1095,1107 ----
  	        '((deleted stopped)
  	          (created deleted stopped)))
  	       ;; There are two `deleted' events, for the file and for
! 	       ;; the directory.  Except for cygwin,
! 	       ;; GFamDirectoryMonitor and kqueue.  And cygwin and
! 	       ;; GFamDirectoryMonitor do not raise a `changed' event.
  	       ((eq system-type 'cygwin)
  	        '(created deleted stopped))
+ 	       ((equal (file-notify--test-monitor) 'GFamDirectoryMonitor)
+ 	        '(created deleted stopped))
  	       ((string-equal (file-notify--test-library) "kqueue")
  	        '(created changed deleted stopped))
  	       (t '(created changed deleted deleted stopped)))
***************
*** 1286,1291 ****
--- 1320,1328 ----
               ;; On cygwin we only get the `changed' event.
               ((eq system-type 'cygwin)
                '(changed))
+ 	     ;; GFamDirectoryMonitor reports only the `changed' event.
+ 	     ((equal (file-notify--test-monitor) 'GFamDirectoryMonitor)
+ 	      '(changed))
               (t '(renamed created changed)))
            ;; The file is renamed when creating a backup.  It shall
            ;; still be watched.

^ permalink raw reply	[flat|nested] 30+ messages in thread

* bug#47262: [platform-testers] Emacs 27.2 RC2
  2021-03-21 12:48           ` Michael Albinus
@ 2021-03-22 14:17             ` Dagobert Michelsen
  2021-03-22 14:47               ` Michael Albinus
  0 siblings, 1 reply; 30+ messages in thread
From: Dagobert Michelsen @ 2021-03-22 14:17 UTC (permalink / raw)
  To: Michael Albinus; +Cc: 47262

Hi Michael,

Am 21.03.2021 um 13:48 schrieb Michael Albinus <michael.albinus@gmx.de>:
> Thanks. I've appended another patch, which should fix all open problems
> on Solaris. Could you pls check?
> 
> Furthermore, I have seen that you didn't run the expensive tests
> file-notify-test07-many-events and
> file-notify-test09-watched-file-in-watched-dir. I expect further
> problems there, so please call
> 
> # make -C test filenotify-tests.log

This does not terminate in test #7 and I manually needed to kill it:

/bin/bash: line 2: 26390 Terminated              HOME=/nonexistent EMACSLOADPATH= LC_ALL=C EMACS_TEST_DIRECTORY=/home/dam/mgar/pkg/emacs/trunk/work/solaris10-sparc/build-isa-sparcv8plus-features-athena/emacs-27.2/test "../src/emacs" --module-assertions --no-init-file --no-site-file --no-site-lisp -L ":." -l ert -l lisp/filenotify-tests.el --batch --eval '(ert-run-tests-batch-and-exit (quote (not (tag :unstable))))' > lisp/filenotify-tests.log 2>&1


Best regards

  — Dago

-- 
"You don't become great by trying to be great, you become great by wanting to do something,
and then doing it so hard that you become great in the process." - xkcd #896






^ permalink raw reply	[flat|nested] 30+ messages in thread

* bug#47262: [platform-testers] Emacs 27.2 RC2
  2021-03-22 14:17             ` Dagobert Michelsen
@ 2021-03-22 14:47               ` Michael Albinus
  2021-03-23  7:38                 ` Dagobert Michelsen
  0 siblings, 1 reply; 30+ messages in thread
From: Michael Albinus @ 2021-03-22 14:47 UTC (permalink / raw)
  To: Dagobert Michelsen; +Cc: 47262

[-- Attachment #1: Type: text/plain, Size: 525 bytes --]

Dagobert Michelsen <dam@opencsw.org> writes:

> Hi Michael,

Hi Dago,

>> # make -C test filenotify-tests.log
>
> This does not terminate in test #7 and I manually needed to kill it:

Oops, yes. file-notify-test07-many-events and
file-notify-test09-watched-file-in-watched-dir run large loops. I've
reduced the number of iterations in both tests, see appended patch. It
contains also some other changes I have applied last days, testing on cygwin.

> Best regards
>
>   — Dago

Best regards, Michael.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: Type: text/x-patch, Size: 19582 bytes --]

*** /tmp/ediffmmNEZ6	2021-03-22 15:43:13.464939528 +0100
--- /home/albinus/src/emacs-27/test/lisp/filenotify-tests.el	2021-03-22 15:41:35.740131740 +0100
***************
*** 105,115 ****
     (cond
      ;; gio/gpollfilemonitor.c declares POLL_TIME_SECS 5.  So we must
      ;; wait at least this time in the GPollFileMonitor case.  A
!     ;; similar timeout seems to be needed in the GFamFileMonitor case,
!     ;; at least on Cygwin.
!     ((and (string-equal (file-notify--test-library) "gfilenotify")
!           (memq (file-notify--test-monitor)
!                 '(GFamFileMonitor GPollFileMonitor)))
       7)
      ((string-equal (file-notify--test-library) "gvfs-monitor-dir.exe") 1)
      ((file-remote-p temporary-file-directory) 0.1)
--- 105,114 ----
     (cond
      ;; gio/gpollfilemonitor.c declares POLL_TIME_SECS 5.  So we must
      ;; wait at least this time in the GPollFileMonitor case.  A
!     ;; similar timeout seems to be needed in the
!     ;; GFam{File,Directory}Monitor case.
!     ((memq (file-notify--test-monitor)
!            '(GFamFileMonitor GFamDirectoryMonitor GPollFileMonitor))
       7)
      ((string-equal (file-notify--test-library) "gvfs-monitor-dir.exe") 1)
      ((file-remote-p temporary-file-directory) 0.1)
***************
*** 263,274 ****
    ;; `gfile-monitor-name' does not return a proper result anymore.
    ;; But we still need this information.
    (unless (file-remote-p temporary-file-directory)
!     (or (cdr (assq file-notify--test-desc file-notify--test-monitors))
          (when (functionp 'gfile-monitor-name)
            (add-to-list 'file-notify--test-monitors
                         (cons file-notify--test-desc
                               (gfile-monitor-name file-notify--test-desc)))
!           (cdr (assq file-notify--test-desc file-notify--test-monitors))))))

  (defmacro file-notify--deftest-remote (test docstring &optional unstable)
    "Define ert `TEST-remote' for remote files.
--- 262,273 ----
    ;; `gfile-monitor-name' does not return a proper result anymore.
    ;; But we still need this information.
    (unless (file-remote-p temporary-file-directory)
!     (or (alist-get file-notify--test-desc file-notify--test-monitors)
          (when (functionp 'gfile-monitor-name)
            (add-to-list 'file-notify--test-monitors
                         (cons file-notify--test-desc
                               (gfile-monitor-name file-notify--test-desc)))
!           (alist-get file-notify--test-desc file-notify--test-monitors)))))

  (defmacro file-notify--deftest-remote (test docstring &optional unstable)
    "Define ert `TEST-remote' for remote files.
***************
*** 455,461 ****

    (unwind-protect
        ;; Check, that removing watch descriptors out of order do not
!       ;; harm.  This fails on Cygwin because of timing issues unless a
        ;; long `sit-for' is added before the call to
        ;; `file-notify--test-read-event'.
        (unless (eq system-type 'cygwin)
--- 454,460 ----

    (unwind-protect
        ;; Check, that removing watch descriptors out of order do not
!       ;; harm.  This fails on cygwin because of timing issues unless a
        ;; long `sit-for' is added before the call to
        ;; `file-notify--test-read-event'.
        (unless (eq system-type 'cygwin)
***************
*** 542,547 ****
--- 541,550 ----
  	    file-notify--test-results
  	    (append file-notify--test-results `(,result))))))

+ (defun file-notify--test-event-actions ()
+   "Helper function to return retrieved actions, as list."
+   (mapcar #'file-notify--test-event-action file-notify--test-events))
+
  (defun file-notify--test-with-actions-check (actions)
    "Check whether received actions match one of the ACTIONS alternatives."
    (let (result)
***************
*** 550,571 ****
              (or result
                  (if (eq (car elt) :random)
                      (equal (sort (cdr elt) 'string-lessp)
!                            (sort (mapcar #'file-notify--test-event-action
!                                          file-notify--test-events)
                                   'string-lessp))
!                   (equal elt (mapcar #'file-notify--test-event-action
!                                      file-notify--test-events))))))))

  (defun file-notify--test-with-actions-explainer (actions)
    "Explain why `file-notify--test-with-actions-check' fails."
    (if (null (cdr actions))
        (format "Received actions do not match expected actions\n%s\n%s"
!               (mapcar #'file-notify--test-event-action file-notify--test-events)
!               (car actions))
      (format
       "Received actions do not match any sequence of expected actions\n%s\n%s"
!      (mapcar #'file-notify--test-event-action file-notify--test-events)
!      actions)))

  (put 'file-notify--test-with-actions-check 'ert-explainer
       'file-notify--test-with-actions-explainer)
--- 553,577 ----
              (or result
                  (if (eq (car elt) :random)
                      (equal (sort (cdr elt) 'string-lessp)
!                            (sort (file-notify--test-event-actions)
                                   'string-lessp))
!                   (equal elt (file-notify--test-event-actions))))))
!     ;; Do not report result in case we debug.  Write messages instead.
!     (if file-notify-debug
!         (prog1 t
!           (if result
!               (message "Success\n%s" (file-notify--test-event-actions))
!             (message (file-notify--test-with-actions-explainer actions))))
!       result)))

  (defun file-notify--test-with-actions-explainer (actions)
    "Explain why `file-notify--test-with-actions-check' fails."
    (if (null (cdr actions))
        (format "Received actions do not match expected actions\n%s\n%s"
!               (file-notify--test-event-actions) (car actions))
      (format
       "Received actions do not match any sequence of expected actions\n%s\n%s"
!      (file-notify--test-event-actions) actions)))

  (put 'file-notify--test-with-actions-check 'ert-explainer
       'file-notify--test-with-actions-explainer)
***************
*** 632,640 ****
  	       (file-notify--test-library) "gvfs-monitor-dir.exe")
  	      '((deleted stopped)
  	        (created deleted stopped)))
!              ;; cygwin does not raise a `changed' event.
!              ((eq system-type 'cygwin)
!               '(created deleted stopped))
               (t '(created changed deleted stopped)))
            (write-region
             "another text" nil file-notify--test-tmpfile nil 'no-message)
--- 638,647 ----
  	       (file-notify--test-library) "gvfs-monitor-dir.exe")
  	      '((deleted stopped)
  	        (created deleted stopped)))
! 	     ;; GFam{File,Directory}Monitor do not report the `changed' event.
! 	     ((memq (file-notify--test-monitor)
!                     '(GFamFileMonitor GFamDirectoryMonitor))
! 	      '(created deleted stopped))
               (t '(created changed deleted stopped)))
            (write-region
             "another text" nil file-notify--test-tmpfile nil 'no-message)
***************
*** 665,670 ****
--- 672,683 ----
  	     ((string-equal (file-notify--test-library) "gvfs-monitor-dir.exe")
  	      '((deleted stopped)
  		(changed deleted stopped)))
+ 	     ;; GFam{File,Directory}Monitor do not detect the
+              ;; `changed' event reliably.
+ 	     ((memq (file-notify--test-monitor)
+                     '(GFamFileMonitor GFamDirectoryMonitor))
+ 	      '((deleted stopped)
+ 		(changed deleted stopped)))
  	     ;; There could be one or two `changed' events.
  	     (t '((changed deleted stopped)
  		  (changed changed deleted stopped))))
***************
*** 709,717 ****
               ((getenv "EMACS_EMBA_CI")
                '(created changed deleted))
  	     ;; There are two `deleted' events, for the file and for
! 	     ;; the directory.  Except for cygwin and kqueue.  And
! 	     ;; cygwin does not raise a `changed' event.
! 	     ((eq system-type 'cygwin)
  	      '(created deleted stopped))
  	     ((string-equal (file-notify--test-library) "kqueue")
  	      '(created changed deleted stopped))
--- 722,732 ----
               ((getenv "EMACS_EMBA_CI")
                '(created changed deleted))
  	     ;; There are two `deleted' events, for the file and for
! 	     ;; the directory.  Except for GFam{File,Directory}Monitor
! 	     ;; and kqueue.  And GFam{File,Directory}Monitor do not
! 	     ;; raise a `changed' event.
! 	     ((memq (file-notify--test-monitor)
!                     '(GFamFileMonitor GFamDirectoryMonitor))
  	      '(created deleted stopped))
  	     ((string-equal (file-notify--test-library) "kqueue")
  	      '(created changed deleted stopped))
***************
*** 755,762 ****
  	      '((deleted stopped)
  		(created created deleted stopped)))
  	     ;; There are three `deleted' events, for two files and
! 	     ;; for the directory.  Except for cygwin and kqueue.
! 	     ((eq system-type 'cygwin)
  	      '(created created changed changed deleted stopped))
  	     ((string-equal (file-notify--test-library) "kqueue")
  	      '(created changed created changed deleted stopped))
--- 770,779 ----
  	      '((deleted stopped)
  		(created created deleted stopped)))
  	     ;; There are three `deleted' events, for two files and
! 	     ;; for the directory.  Except for
! 	     ;; GFam{File,Directory}Monitor and kqueue.
! 	     ((memq (file-notify--test-monitor)
!                     '(GFamFileMonitor GFamDirectoryMonitor))
  	      '(created created changed changed deleted stopped))
  	     ((string-equal (file-notify--test-library) "kqueue")
  	      '(created changed created changed deleted stopped))
***************
*** 813,822 ****
               ((getenv "EMACS_EMBA_CI")
                '(created changed renamed deleted))
  	     ;; There are two `deleted' events, for the file and for
! 	     ;; the directory.  Except for cygwin and kqueue.  And
! 	     ;; cygwin raises `created' and `deleted' events instead
! 	     ;; of a `renamed' event.
! 	     ((eq system-type 'cygwin)
  	      '(created created deleted deleted stopped))
  	     ((string-equal (file-notify--test-library) "kqueue")
  	      '(created changed renamed deleted stopped))
--- 830,841 ----
               ((getenv "EMACS_EMBA_CI")
                '(created changed renamed deleted))
  	     ;; There are two `deleted' events, for the file and for
! 	     ;; the directory.  Except for GFam{File,Directory}Monitor
! 	     ;; and kqueue.  And GFam{File,Directory}Monitor raise
! 	     ;; `created' and `deleted' events instead of a `renamed'
! 	     ;; event.
! 	     ((memq (file-notify--test-monitor)
!                     '(GFamFileMonitor GFamDirectoryMonitor))
  	      '(created created deleted deleted stopped))
  	     ((string-equal (file-notify--test-library) "kqueue")
  	      '(created changed renamed deleted stopped))
***************
*** 837,844 ****
      (file-notify--test-cleanup))

    (unwind-protect
!       ;; Check attribute change.  Does not work for cygwin.
!       (unless (eq system-type 'cygwin)
  	(setq file-notify--test-tmpfile (file-notify--test-make-temp-name))
  	(write-region
  	 "any text" nil file-notify--test-tmpfile nil 'no-message)
--- 856,863 ----
      (file-notify--test-cleanup))

    (unwind-protect
!       ;; Check attribute change.  Does not work for GFam{File,Directory}Monitor.
!       (progn
  	(setq file-notify--test-tmpfile (file-notify--test-make-temp-name))
  	(write-region
  	 "any text" nil file-notify--test-tmpfile nil 'no-message)
***************
*** 847,875 ****
  	       (file-notify--test-add-watch
  		file-notify--test-tmpfile
  		'(attribute-change) #'file-notify--test-event-handler)))
! 	(file-notify--test-with-actions
! 	    (cond
! 	     ;; w32notify does not distinguish between `changed' and
! 	     ;; `attribute-changed'.  Under MS Windows 7, we get four
! 	     ;; `changed' events, and under MS Windows 10 just two.
! 	     ;; Strange.
! 	     ((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")
! 		  (file-remote-p temporary-file-directory))
! 	      '(attribute-changed attribute-changed attribute-changed))
! 	     (t '(attribute-changed attribute-changed)))
! 	  (write-region
! 	   "any text" nil file-notify--test-tmpfile nil 'no-message)
! 	  (file-notify--test-read-event)
! 	  (set-file-modes file-notify--test-tmpfile 000)
! 	  (file-notify--test-read-event)
! 	  (set-file-times file-notify--test-tmpfile '(0 0))
! 	  (file-notify--test-read-event)
! 	  (delete-file file-notify--test-tmpfile))
          (file-notify-rm-watch file-notify--test-desc)

          ;; The environment shall be cleaned up.
--- 866,896 ----
  	       (file-notify--test-add-watch
  		file-notify--test-tmpfile
  		'(attribute-change) #'file-notify--test-event-handler)))
! 	(unless (memq (file-notify--test-monitor)
!                       '(GFamFileMonitor GFamDirectoryMonitor))
! 	  (file-notify--test-with-actions
! 	      (cond
! 	       ;; w32notify does not distinguish between `changed' and
! 	       ;; `attribute-changed'.  Under MS Windows 7, we get
! 	       ;; four `changed' events, and under MS Windows 10 just
! 	       ;; two.  Strange.
! 	       ((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")
! 		    (file-remote-p temporary-file-directory))
! 		'(attribute-changed attribute-changed attribute-changed))
! 	       (t '(attribute-changed attribute-changed)))
! 	    (write-region
! 	     "any text" nil file-notify--test-tmpfile nil 'no-message)
! 	    (file-notify--test-read-event)
! 	    (set-file-modes file-notify--test-tmpfile 000)
! 	    (file-notify--test-read-event)
! 	    (set-file-times file-notify--test-tmpfile '(0 0))
! 	    (file-notify--test-read-event)
! 	    (delete-file file-notify--test-tmpfile)))
          (file-notify-rm-watch file-notify--test-desc)

          ;; The environment shall be cleaned up.
***************
*** 951,957 ****

  	    ;; Modify file.  We wait for two seconds, in order to
  	    ;; have another timestamp.  One second seems to be too
!             ;; short.  And Cygwin sporadically requires more than two.
              (ert-with-message-capture captured-messages
                (sleep-for (if (eq system-type 'cygwin) 3 2))
                (write-region
--- 972,978 ----

  	    ;; Modify file.  We wait for two seconds, in order to
  	    ;; have another timestamp.  One second seems to be too
!             ;; short.  And cygwin sporadically requires more than two.
              (ert-with-message-capture captured-messages
                (sleep-for (if (eq system-type 'cygwin) 3 2))
                (write-region
***************
*** 1021,1026 ****
--- 1042,1053 ----
  	     ((string-equal (file-notify--test-library) "gvfs-monitor-dir.exe")
  	      '((deleted stopped)
  		(changed deleted stopped)))
+ 	     ;; GFam{File,Directory}Monitor do not detect the
+              ;; `changed' event reliably.
+ 	     ((memq (file-notify--test-monitor)
+                     '(GFamFileMonitor GFamDirectoryMonitor))
+ 	      '((deleted stopped)
+ 	        (changed deleted stopped)))
  	     ;; There could be one or two `changed' events.
  	     (t '((changed deleted stopped)
  		  (changed changed deleted stopped))))
***************
*** 1064,1072 ****
  	        '((deleted stopped)
  	          (created deleted stopped)))
  	       ;; There are two `deleted' events, for the file and for
! 	       ;; the directory.  Except for cygwin and kqueue.  And
! 	       ;; cygwin does not raise a `changed' event.
! 	       ((eq system-type 'cygwin)
  	        '(created deleted stopped))
  	       ((string-equal (file-notify--test-library) "kqueue")
  	        '(created changed deleted stopped))
--- 1091,1102 ----
  	        '((deleted stopped)
  	          (created deleted stopped)))
  	       ;; There are two `deleted' events, for the file and for
! 	       ;; the directory.  Except for
! 	       ;; GFam{File,Directory}Monitor and kqueue.  And
! 	       ;; GFam{File,Directory}Monitor do not raise a `changed'
! 	       ;; event.
! 	       ((memq (file-notify--test-monitor)
!                       '(GFamFileMonitor GFamDirectoryMonitor))
  	        '(created deleted stopped))
  	       ((string-equal (file-notify--test-library) "kqueue")
  	        '(created changed deleted stopped))
***************
*** 1169,1175 ****
  	  file-notify--test-tmpfile
  	  '(change) #'file-notify--test-event-handler)))
    (unwind-protect
!       (let ((n 1000)
              source-file-list target-file-list
              (default-directory file-notify--test-tmpfile))
          (dotimes (i n)
--- 1199,1205 ----
  	  file-notify--test-tmpfile
  	  '(change) #'file-notify--test-event-handler)))
    (unwind-protect
!       (let ((n 10);00)
              source-file-list target-file-list
              (default-directory file-notify--test-tmpfile))
          (dotimes (i n)
***************
*** 1283,1291 ****
          (should (file-notify-valid-p file-notify--test-desc))
          (file-notify--test-with-actions
              (cond
!              ;; On cygwin we only get the `changed' event.
!              ((eq system-type 'cygwin)
!               '(changed))
               (t '(renamed created changed)))
            ;; The file is renamed when creating a backup.  It shall
            ;; still be watched.
--- 1313,1322 ----
          (should (file-notify-valid-p file-notify--test-desc))
          (file-notify--test-with-actions
              (cond
! 	     ;; GFam{File,Directory}Monitor report only the `changed' event.
! 	     ((memq (file-notify--test-monitor)
!                     '(GFamFileMonitor GFamDirectoryMonitor))
! 	      '(changed))
               (t '(renamed created changed)))
            ;; The file is renamed when creating a backup.  It shall
            ;; still be watched.
***************
*** 1355,1361 ****
          (should (file-notify-valid-p file-notify--test-desc1))
          (should (file-notify-valid-p file-notify--test-desc2))
          (should-not (equal file-notify--test-desc1 file-notify--test-desc2))
!         (let ((n 100))
            ;; Run the test.
            (file-notify--test-with-actions
                ;; There could be one or two `changed' events.
--- 1386,1392 ----
          (should (file-notify-valid-p file-notify--test-desc1))
          (should (file-notify-valid-p file-notify--test-desc2))
          (should-not (equal file-notify--test-desc1 file-notify--test-desc2))
!         (let ((n 10));0))
            ;; Run the test.
            (file-notify--test-with-actions
                ;; There could be one or two `changed' events.

^ permalink raw reply	[flat|nested] 30+ messages in thread

* bug#47262: [platform-testers] Emacs 27.2 RC2
  2021-03-22 14:47               ` Michael Albinus
@ 2021-03-23  7:38                 ` Dagobert Michelsen
  2021-03-23  8:17                   ` Michael Albinus
  0 siblings, 1 reply; 30+ messages in thread
From: Dagobert Michelsen @ 2021-03-23  7:38 UTC (permalink / raw)
  To: Michael Albinus; +Cc: 47262

[-- Attachment #1: Type: text/plain, Size: 467 bytes --]

Hi Michael,

Am 22.03.2021 um 15:47 schrieb Michael Albinus <michael.albinus@gmx.de>:
> Oops, yes. file-notify-test07-many-events and
> file-notify-test09-watched-file-in-watched-dir run large loops. I've
> reduced the number of iterations in both tests, see appended patch. It
> contains also some other changes I have applied last days, testing on cygwin.

Please find the logfiles for the latest patch attached. This looks good,
now all tests pass :-)


[-- Attachment #2: filenotify-tests.log --]
[-- Type: application/octet-stream, Size: 828 bytes --]

Running 8 tests (2021-03-22 17:48:46+0100, selector `(not (or (tag :expensive-test) (tag :unstable)))')
Library: `gfilenotify'
Monitor: `GFamDirectoryMonitor'
   passed  1/8  file-notify-test00-availability (0.102576 sec)
   passed  2/8  file-notify-test01-add-watch (0.014409 sec)
   passed  3/8  file-notify-test02-rm-watch (1.007739 sec)
   passed  4/8  file-notify-test03-events (113.057275 sec)
Reverting buffer `file-notify-testN00ey6'.
Reverting buffer `file-notify-testN00ey6'.
   passed  5/8  file-notify-test04-autorevert (18.041154 sec)
   passed  6/8  file-notify-test05-file-validity (34.025455 sec)
   passed  7/8  file-notify-test06-dir-validity (3.011377 sec)
   passed  8/8  file-notify-test08-backup (26.050180 sec)

Ran 8 tests, 8 results as expected, 0 unexpected (2021-03-22 17:52:02+0100, 195.320520 sec)


[-- Attachment #3: Type: text/plain, Size: 214 bytes --]




Best regards

  — Dago

-- 
"You don't become great by trying to be great, you become great by wanting to do something,
and then doing it so hard that you become great in the process." - xkcd #896


^ permalink raw reply	[flat|nested] 30+ messages in thread

* bug#47262: [platform-testers] Emacs 27.2 RC2
  2021-03-23  7:38                 ` Dagobert Michelsen
@ 2021-03-23  8:17                   ` Michael Albinus
  2021-03-23  8:42                     ` Dagobert Michelsen
  0 siblings, 1 reply; 30+ messages in thread
From: Michael Albinus @ 2021-03-23  8:17 UTC (permalink / raw)
  To: Dagobert Michelsen; +Cc: 47262

Dagobert Michelsen <dam@opencsw.org> writes:

> Hi Michael,

Hi Dago,

> Please find the logfiles for the latest patch attached. This looks good,
> now all tests pass :-)

Thanks a lot! This is good progress.

> Running 8 tests (2021-03-22 17:48:46+0100, selector `(not (or (tag :expensive-test) (tag :unstable)))')

These are the not expensive tests. Do you see a chance to run

# make -C test filenotify-tests.log SELECTOR='(not "remote")'

? This runs all local tests, expensive ones included, but avoids the
remote tests (which might be problematic). And in case there are errors
with the expensive tests, could you rerun this with file-notify-debug
set to t in the test file?

> Best regards
>
>   — Dago

Thanks, and best regards, Michael.





^ permalink raw reply	[flat|nested] 30+ messages in thread

* bug#47262: [platform-testers] Emacs 27.2 RC2
  2021-03-23  8:17                   ` Michael Albinus
@ 2021-03-23  8:42                     ` Dagobert Michelsen
  2021-03-23  9:32                       ` Michael Albinus
  0 siblings, 1 reply; 30+ messages in thread
From: Dagobert Michelsen @ 2021-03-23  8:42 UTC (permalink / raw)
  To: Michael Albinus; +Cc: 47262

[-- Attachment #1: Type: text/plain, Size: 551 bytes --]

Hi Michael,

Am 23.03.2021 um 09:17 schrieb Michael Albinus <michael.albinus@gmx.de>:
> These are the not expensive tests.

Ahh sorry, I tend to forget to execute this.

> Do you see a chance to run
> 
> # make -C test filenotify-tests.log SELECTOR='(not "remote")'
> 
> ? This runs all local tests, expensive ones included, but avoids the
> remote tests (which might be problematic). And in case there are errors
> with the expensive tests, could you rerun this with file-notify-debug
> set to t in the test file?

Please find the new log attached.


[-- Attachment #2: filenotify-tests.log --]
[-- Type: application/octet-stream, Size: 4953 bytes --]

Running 11 tests (2021-03-23 09:25:41+0100, selector `(not "remote")')
Library: `gfilenotify'
Monitor: `GFamDirectoryMonitor'
   passed   1/11  file-notify-test00-availability (0.314023 sec)
   passed   2/11  file-notify-test01-add-watch (0.015639 sec)
   passed   3/11  file-notify-test02-rm-watch (1.115934 sec)
   passed   4/11  file-notify-test03-events (113.061687 sec)
Reverting buffer `file-notify-test0ZX8Re'.
Reverting buffer `file-notify-test0ZX8Re'.
   passed   5/11  file-notify-test04-autorevert (18.041596 sec)
   passed   6/11  file-notify-test05-file-validity (34.026242 sec)
   passed   7/11  file-notify-test06-dir-validity (3.011524 sec)
Test file-notify-test07-many-events backtrace:
  signal(ert-test-failed (((should (file-notify--test-with-actions-che
  ert-fail(((should (file-notify--test-with-actions-check actions)) :f
  (if (unwind-protect (setq value-338 (apply fn-336 args-337)) (setq f
  (let (form-description-340) (if (unwind-protect (setq value-338 (app
  (let ((value-338 'ert-form-evaluation-aborted-339)) (let (form-descr
  (let* ((fn-336 #'file-notify--test-with-actions-check) (args-337 (co
  (let* ((actions (if (consp (car (cond (... ...) (... ...) (t ...))))
  (let ((n 10) source-file-list target-file-list (default-directory fi
  (unwind-protect (let ((n 10) source-file-list target-file-list (defa
  (closure (t) nil (let* ((fn-322 #'file-notify--test-local-enabled) (
  ert--run-test-internal(#s(ert--test-execution-info :test #s(ert-test
  ert-run-test(#s(ert-test :name file-notify-test07-many-events :docum
  ert-run-or-rerun-test(#s(ert--stats :selector ... :tests ... :test-m
  ert-run-tests((not "remote") #f(compiled-function (event-type &rest 
  ert-run-tests-batch((not "remote"))
  ert-run-tests-batch-and-exit((not "remote"))
  eval((ert-run-tests-batch-and-exit '(not "remote")) t)
  command-line-1(("-L" ":." "-l" "ert" "-l" "lisp/filenotify-tests.el"
  command-line()
  normal-top-level()
Test file-notify-test07-many-events condition:
    (ert-test-failed
     ((should
       (file-notify--test-with-actions-check actions))
      :form
      (file-notify--test-with-actions-check
       ((renamed renamed renamed renamed renamed renamed renamed renamed renamed renamed)))
      :value nil :explanation "Received actions do not match expected actions
(changed deleted changed deleted changed deleted changed deleted changed deleted changed deleted changed deleted changed deleted changed deleted changed deleted)
(renamed renamed renamed renamed renamed renamed renamed renamed renamed renamed)"))
   FAILED   8/11  file-notify-test07-many-events (230.052797 sec)
   passed   9/11  file-notify-test08-backup (26.051370 sec)
Test file-notify-test09-watched-file-in-watched-dir backtrace:
  signal(ert-test-failed (((should (file-notify--test-with-actions-che
  ert-fail(((should (file-notify--test-with-actions-check actions)) :f
  (if (unwind-protect (setq value-445 (apply fn-443 args-444)) (setq f
  (let (form-description-447) (if (unwind-protect (setq value-445 (app
  (let ((value-445 'ert-form-evaluation-aborted-446)) (let (form-descr
  (let* ((fn-443 #'file-notify--test-with-actions-check) (args-444 (co
  (let* ((actions (if (consp (car (cond (... ...) (t ...)))) (cond ((o
  (progn (let ((value-404 (gensym "ert-form-evaluation-aborted-"))) (l
  (let* ((--cl-dir-callback-- #'(lambda (event file) (let (...) (file-
  (unwind-protect (let* ((--cl-dir-callback-- #'(lambda (event file) (
  (closure (t) nil (let* ((fn-395 #'file-notify--test-local-enabled) (
  ert--run-test-internal(#s(ert--test-execution-info :test #s(ert-test
  ert-run-test(#s(ert-test :name file-notify-test09-watched-file-in-wa
  ert-run-or-rerun-test(#s(ert--stats :selector ... :tests ... :test-m
  ert-run-tests((not "remote") #f(compiled-function (event-type &rest 
  ert-run-tests-batch((not "remote"))
  ert-run-tests-batch-and-exit((not "remote"))
  eval((ert-run-tests-batch-and-exit '(not "remote")) t)
  command-line-1(("-L" ":." "-l" "ert" "-l" "lisp/filenotify-tests.el"
  command-line()
  normal-top-level()
Test file-notify-test09-watched-file-in-watched-dir condition:
    (ert-test-failed
     ((should
       (file-notify--test-with-actions-check actions))
      :form
      (file-notify--test-with-actions-check
       ((deleted deleted deleted deleted deleted deleted stopped)))
      :value nil :explanation "Received actions do not match expected actions
(deleted stopped)
(deleted deleted deleted deleted deleted deleted stopped)"))
   FAILED  10/11  file-notify-test09-watched-file-in-watched-dir (100.035757 sec)
  skipped  11/11  file-notify-test10-sufficient-resources (0.001596 sec)

Ran 11 tests, 8 results as expected, 2 unexpected, 1 skipped (2021-03-23 09:34:34+0100, 533.237428 sec)

2 unexpected results:
   FAILED  file-notify-test07-many-events
   FAILED  file-notify-test09-watched-file-in-watched-dir

1 skipped results:
  SKIPPED  file-notify-test10-sufficient-resources


[-- Attachment #3: Type: text/plain, Size: 214 bytes --]




Best regards

  — Dago

-- 
"You don't become great by trying to be great, you become great by wanting to do something,
and then doing it so hard that you become great in the process." - xkcd #896


^ permalink raw reply	[flat|nested] 30+ messages in thread

* bug#47262: [platform-testers] Emacs 27.2 RC2
  2021-03-23  8:42                     ` Dagobert Michelsen
@ 2021-03-23  9:32                       ` Michael Albinus
  2021-03-23 11:52                         ` Dagobert Michelsen
  0 siblings, 1 reply; 30+ messages in thread
From: Michael Albinus @ 2021-03-23  9:32 UTC (permalink / raw)
  To: Dagobert Michelsen; +Cc: 47262

[-- Attachment #1: Type: text/plain, Size: 402 bytes --]

Dagobert Michelsen <dam@opencsw.org> writes:

> Hi Michael,

Hi Dago,

> Please find the new log attached.

Thanks! Appended the next patch iteration. And I'm quite optimistic that
it is the last one :-)

Could you pls check? As a reminder, pls call

# make -C test filenotify-tests.log SELECTOR='(not "remote")'

> Best regards
>
>   — Dago

Thanks, and best regards, Michael.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: Type: text/x-patch, Size: 21426 bytes --]

*** /tmp/ediffpkAKG4	2021-03-23 10:28:18.611288593 +0100
--- /usr/local/src/emacs-27/test/lisp/filenotify-tests.el	2021-03-23 10:10:45.362714979 +0100
***************
*** 105,115 ****
     (cond
      ;; gio/gpollfilemonitor.c declares POLL_TIME_SECS 5.  So we must
      ;; wait at least this time in the GPollFileMonitor case.  A
!     ;; similar timeout seems to be needed in the GFamFileMonitor case,
!     ;; at least on Cygwin.
!     ((and (string-equal (file-notify--test-library) "gfilenotify")
!           (memq (file-notify--test-monitor)
!                 '(GFamFileMonitor GPollFileMonitor)))
       7)
      ((string-equal (file-notify--test-library) "gvfs-monitor-dir.exe") 1)
      ((file-remote-p temporary-file-directory) 0.1)
--- 105,114 ----
     (cond
      ;; gio/gpollfilemonitor.c declares POLL_TIME_SECS 5.  So we must
      ;; wait at least this time in the GPollFileMonitor case.  A
!     ;; similar timeout seems to be needed in the
!     ;; GFam{File,Directory}Monitor case.
!     ((memq (file-notify--test-monitor)
!            '(GFamFileMonitor GFamDirectoryMonitor GPollFileMonitor))
       7)
      ((string-equal (file-notify--test-library) "gvfs-monitor-dir.exe") 1)
      ((file-remote-p temporary-file-directory) 0.1)
***************
*** 263,274 ****
    ;; `gfile-monitor-name' does not return a proper result anymore.
    ;; But we still need this information.
    (unless (file-remote-p temporary-file-directory)
!     (or (cdr (assq file-notify--test-desc file-notify--test-monitors))
          (when (functionp 'gfile-monitor-name)
            (add-to-list 'file-notify--test-monitors
                         (cons file-notify--test-desc
                               (gfile-monitor-name file-notify--test-desc)))
!           (cdr (assq file-notify--test-desc file-notify--test-monitors))))))

  (defmacro file-notify--deftest-remote (test docstring &optional unstable)
    "Define ert `TEST-remote' for remote files.
--- 262,273 ----
    ;; `gfile-monitor-name' does not return a proper result anymore.
    ;; But we still need this information.
    (unless (file-remote-p temporary-file-directory)
!     (or (alist-get file-notify--test-desc file-notify--test-monitors)
          (when (functionp 'gfile-monitor-name)
            (add-to-list 'file-notify--test-monitors
                         (cons file-notify--test-desc
                               (gfile-monitor-name file-notify--test-desc)))
!           (alist-get file-notify--test-desc file-notify--test-monitors)))))

  (defmacro file-notify--deftest-remote (test docstring &optional unstable)
    "Define ert `TEST-remote' for remote files.
***************
*** 455,461 ****

    (unwind-protect
        ;; Check, that removing watch descriptors out of order do not
!       ;; harm.  This fails on Cygwin because of timing issues unless a
        ;; long `sit-for' is added before the call to
        ;; `file-notify--test-read-event'.
        (unless (eq system-type 'cygwin)
--- 454,460 ----

    (unwind-protect
        ;; Check, that removing watch descriptors out of order do not
!       ;; harm.  This fails on cygwin because of timing issues unless a
        ;; long `sit-for' is added before the call to
        ;; `file-notify--test-read-event'.
        (unless (eq system-type 'cygwin)
***************
*** 542,547 ****
--- 541,550 ----
  	    file-notify--test-results
  	    (append file-notify--test-results `(,result))))))

+ (defun file-notify--test-event-actions ()
+   "Helper function to return retrieved actions, as list."
+   (mapcar #'file-notify--test-event-action file-notify--test-events))
+
  (defun file-notify--test-with-actions-check (actions)
    "Check whether received actions match one of the ACTIONS alternatives."
    (let (result)
***************
*** 550,571 ****
              (or result
                  (if (eq (car elt) :random)
                      (equal (sort (cdr elt) 'string-lessp)
!                            (sort (mapcar #'file-notify--test-event-action
!                                          file-notify--test-events)
                                   'string-lessp))
!                   (equal elt (mapcar #'file-notify--test-event-action
!                                      file-notify--test-events))))))))

  (defun file-notify--test-with-actions-explainer (actions)
    "Explain why `file-notify--test-with-actions-check' fails."
    (if (null (cdr actions))
        (format "Received actions do not match expected actions\n%s\n%s"
!               (mapcar #'file-notify--test-event-action file-notify--test-events)
!               (car actions))
      (format
       "Received actions do not match any sequence of expected actions\n%s\n%s"
!      (mapcar #'file-notify--test-event-action file-notify--test-events)
!      actions)))

  (put 'file-notify--test-with-actions-check 'ert-explainer
       'file-notify--test-with-actions-explainer)
--- 553,577 ----
              (or result
                  (if (eq (car elt) :random)
                      (equal (sort (cdr elt) 'string-lessp)
!                            (sort (file-notify--test-event-actions)
                                   'string-lessp))
!                   (equal elt (file-notify--test-event-actions))))))
!     ;; Do not report result in case we debug.  Write messages instead.
!     (if file-notify-debug
!         (prog1 t
!           (if result
!               (message "Success\n%s" (file-notify--test-event-actions))
!             (message (file-notify--test-with-actions-explainer actions))))
!       result)))

  (defun file-notify--test-with-actions-explainer (actions)
    "Explain why `file-notify--test-with-actions-check' fails."
    (if (null (cdr actions))
        (format "Received actions do not match expected actions\n%s\n%s"
!               (file-notify--test-event-actions) (car actions))
      (format
       "Received actions do not match any sequence of expected actions\n%s\n%s"
!      (file-notify--test-event-actions) actions)))

  (put 'file-notify--test-with-actions-check 'ert-explainer
       'file-notify--test-with-actions-explainer)
***************
*** 632,640 ****
  	       (file-notify--test-library) "gvfs-monitor-dir.exe")
  	      '((deleted stopped)
  	        (created deleted stopped)))
!              ;; cygwin does not raise a `changed' event.
!              ((eq system-type 'cygwin)
!               '(created deleted stopped))
               (t '(created changed deleted stopped)))
            (write-region
             "another text" nil file-notify--test-tmpfile nil 'no-message)
--- 638,647 ----
  	       (file-notify--test-library) "gvfs-monitor-dir.exe")
  	      '((deleted stopped)
  	        (created deleted stopped)))
! 	     ;; GFam{File,Directory}Monitor do not report the `changed' event.
! 	     ((memq (file-notify--test-monitor)
!                     '(GFamFileMonitor GFamDirectoryMonitor))
! 	      '(created deleted stopped))
               (t '(created changed deleted stopped)))
            (write-region
             "another text" nil file-notify--test-tmpfile nil 'no-message)
***************
*** 665,670 ****
--- 672,683 ----
  	     ((string-equal (file-notify--test-library) "gvfs-monitor-dir.exe")
  	      '((deleted stopped)
  		(changed deleted stopped)))
+ 	     ;; GFam{File,Directory}Monitor do not detect the
+              ;; `changed' event reliably.
+ 	     ((memq (file-notify--test-monitor)
+                     '(GFamFileMonitor GFamDirectoryMonitor))
+ 	      '((deleted stopped)
+ 		(changed deleted stopped)))
  	     ;; There could be one or two `changed' events.
  	     (t '((changed deleted stopped)
  		  (changed changed deleted stopped))))
***************
*** 709,717 ****
               ((getenv "EMACS_EMBA_CI")
                '(created changed deleted))
  	     ;; There are two `deleted' events, for the file and for
! 	     ;; the directory.  Except for cygwin and kqueue.  And
! 	     ;; cygwin does not raise a `changed' event.
! 	     ((eq system-type 'cygwin)
  	      '(created deleted stopped))
  	     ((string-equal (file-notify--test-library) "kqueue")
  	      '(created changed deleted stopped))
--- 722,732 ----
               ((getenv "EMACS_EMBA_CI")
                '(created changed deleted))
  	     ;; There are two `deleted' events, for the file and for
! 	     ;; the directory.  Except for GFam{File,Directory}Monitor
! 	     ;; and kqueue.  And GFam{File,Directory}Monitor do not
! 	     ;; raise a `changed' event.
! 	     ((memq (file-notify--test-monitor)
!                     '(GFamFileMonitor GFamDirectoryMonitor))
  	      '(created deleted stopped))
  	     ((string-equal (file-notify--test-library) "kqueue")
  	      '(created changed deleted stopped))
***************
*** 755,762 ****
  	      '((deleted stopped)
  		(created created deleted stopped)))
  	     ;; There are three `deleted' events, for two files and
! 	     ;; for the directory.  Except for cygwin and kqueue.
! 	     ((eq system-type 'cygwin)
  	      '(created created changed changed deleted stopped))
  	     ((string-equal (file-notify--test-library) "kqueue")
  	      '(created changed created changed deleted stopped))
--- 770,779 ----
  	      '((deleted stopped)
  		(created created deleted stopped)))
  	     ;; There are three `deleted' events, for two files and
! 	     ;; for the directory.  Except for
! 	     ;; GFam{File,Directory}Monitor and kqueue.
! 	     ((memq (file-notify--test-monitor)
!                     '(GFamFileMonitor GFamDirectoryMonitor))
  	      '(created created changed changed deleted stopped))
  	     ((string-equal (file-notify--test-library) "kqueue")
  	      '(created changed created changed deleted stopped))
***************
*** 813,822 ****
               ((getenv "EMACS_EMBA_CI")
                '(created changed renamed deleted))
  	     ;; There are two `deleted' events, for the file and for
! 	     ;; the directory.  Except for cygwin and kqueue.  And
! 	     ;; cygwin raises `created' and `deleted' events instead
! 	     ;; of a `renamed' event.
! 	     ((eq system-type 'cygwin)
  	      '(created created deleted deleted stopped))
  	     ((string-equal (file-notify--test-library) "kqueue")
  	      '(created changed renamed deleted stopped))
--- 830,841 ----
               ((getenv "EMACS_EMBA_CI")
                '(created changed renamed deleted))
  	     ;; There are two `deleted' events, for the file and for
! 	     ;; the directory.  Except for GFam{File,Directory}Monitor
! 	     ;; and kqueue.  And GFam{File,Directory}Monitor raise
! 	     ;; `created' and `deleted' events instead of a `renamed'
! 	     ;; event.
! 	     ((memq (file-notify--test-monitor)
!                     '(GFamFileMonitor GFamDirectoryMonitor))
  	      '(created created deleted deleted stopped))
  	     ((string-equal (file-notify--test-library) "kqueue")
  	      '(created changed renamed deleted stopped))
***************
*** 837,844 ****
      (file-notify--test-cleanup))

    (unwind-protect
!       ;; Check attribute change.  Does not work for cygwin.
!       (unless (eq system-type 'cygwin)
  	(setq file-notify--test-tmpfile (file-notify--test-make-temp-name))
  	(write-region
  	 "any text" nil file-notify--test-tmpfile nil 'no-message)
--- 856,863 ----
      (file-notify--test-cleanup))

    (unwind-protect
!       ;; Check attribute change.  Does not work for GFam{File,Directory}Monitor.
!       (progn
  	(setq file-notify--test-tmpfile (file-notify--test-make-temp-name))
  	(write-region
  	 "any text" nil file-notify--test-tmpfile nil 'no-message)
***************
*** 847,875 ****
  	       (file-notify--test-add-watch
  		file-notify--test-tmpfile
  		'(attribute-change) #'file-notify--test-event-handler)))
! 	(file-notify--test-with-actions
! 	    (cond
! 	     ;; w32notify does not distinguish between `changed' and
! 	     ;; `attribute-changed'.  Under MS Windows 7, we get four
! 	     ;; `changed' events, and under MS Windows 10 just two.
! 	     ;; Strange.
! 	     ((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")
! 		  (file-remote-p temporary-file-directory))
! 	      '(attribute-changed attribute-changed attribute-changed))
! 	     (t '(attribute-changed attribute-changed)))
! 	  (write-region
! 	   "any text" nil file-notify--test-tmpfile nil 'no-message)
! 	  (file-notify--test-read-event)
! 	  (set-file-modes file-notify--test-tmpfile 000)
! 	  (file-notify--test-read-event)
! 	  (set-file-times file-notify--test-tmpfile '(0 0))
! 	  (file-notify--test-read-event)
! 	  (delete-file file-notify--test-tmpfile))
          (file-notify-rm-watch file-notify--test-desc)

          ;; The environment shall be cleaned up.
--- 866,896 ----
  	       (file-notify--test-add-watch
  		file-notify--test-tmpfile
  		'(attribute-change) #'file-notify--test-event-handler)))
! 	(unless (memq (file-notify--test-monitor)
!                       '(GFamFileMonitor GFamDirectoryMonitor))
! 	  (file-notify--test-with-actions
! 	      (cond
! 	       ;; w32notify does not distinguish between `changed' and
! 	       ;; `attribute-changed'.  Under MS Windows 7, we get
! 	       ;; four `changed' events, and under MS Windows 10 just
! 	       ;; two.  Strange.
! 	       ((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")
! 		    (file-remote-p temporary-file-directory))
! 		'(attribute-changed attribute-changed attribute-changed))
! 	       (t '(attribute-changed attribute-changed)))
! 	    (write-region
! 	     "any text" nil file-notify--test-tmpfile nil 'no-message)
! 	    (file-notify--test-read-event)
! 	    (set-file-modes file-notify--test-tmpfile 000)
! 	    (file-notify--test-read-event)
! 	    (set-file-times file-notify--test-tmpfile '(0 0))
! 	    (file-notify--test-read-event)
! 	    (delete-file file-notify--test-tmpfile)))
          (file-notify-rm-watch file-notify--test-desc)

          ;; The environment shall be cleaned up.
***************
*** 951,957 ****

  	    ;; Modify file.  We wait for two seconds, in order to
  	    ;; have another timestamp.  One second seems to be too
!             ;; short.  And Cygwin sporadically requires more than two.
              (ert-with-message-capture captured-messages
                (sleep-for (if (eq system-type 'cygwin) 3 2))
                (write-region
--- 972,978 ----

  	    ;; Modify file.  We wait for two seconds, in order to
  	    ;; have another timestamp.  One second seems to be too
!             ;; short.  And cygwin sporadically requires more than two.
              (ert-with-message-capture captured-messages
                (sleep-for (if (eq system-type 'cygwin) 3 2))
                (write-region
***************
*** 1021,1026 ****
--- 1042,1053 ----
  	     ((string-equal (file-notify--test-library) "gvfs-monitor-dir.exe")
  	      '((deleted stopped)
  		(changed deleted stopped)))
+ 	     ;; GFam{File,Directory}Monitor do not detect the
+              ;; `changed' event reliably.
+ 	     ((memq (file-notify--test-monitor)
+                     '(GFamFileMonitor GFamDirectoryMonitor))
+ 	      '((deleted stopped)
+ 	        (changed deleted stopped)))
  	     ;; There could be one or two `changed' events.
  	     (t '((changed deleted stopped)
  		  (changed changed deleted stopped))))
***************
*** 1064,1072 ****
  	        '((deleted stopped)
  	          (created deleted stopped)))
  	       ;; There are two `deleted' events, for the file and for
! 	       ;; the directory.  Except for cygwin and kqueue.  And
! 	       ;; cygwin does not raise a `changed' event.
! 	       ((eq system-type 'cygwin)
  	        '(created deleted stopped))
  	       ((string-equal (file-notify--test-library) "kqueue")
  	        '(created changed deleted stopped))
--- 1091,1102 ----
  	        '((deleted stopped)
  	          (created deleted stopped)))
  	       ;; There are two `deleted' events, for the file and for
! 	       ;; the directory.  Except for
! 	       ;; GFam{File,Directory}Monitor and kqueue.  And
! 	       ;; GFam{File,Directory}Monitor do not raise a `changed'
! 	       ;; event.
! 	       ((memq (file-notify--test-monitor)
!                       '(GFamFileMonitor GFamDirectoryMonitor))
  	        '(created deleted stopped))
  	       ((string-equal (file-notify--test-library) "kqueue")
  	        '(created changed deleted stopped))
***************
*** 1169,1175 ****
  	  file-notify--test-tmpfile
  	  '(change) #'file-notify--test-event-handler)))
    (unwind-protect
!       (let ((n 1000)
              source-file-list target-file-list
              (default-directory file-notify--test-tmpfile))
          (dotimes (i n)
--- 1199,1205 ----
  	  file-notify--test-tmpfile
  	  '(change) #'file-notify--test-event-handler)))
    (unwind-protect
!       (let ((n 10);00)
              source-file-list target-file-list
              (default-directory file-notify--test-tmpfile))
          (dotimes (i n)
***************
*** 1198,1206 ****
  		(dotimes (_i n)
  		  (setq r (append '(deleted renamed) r)))
  		r))
! 	     ;; cygwin fires `changed' and `deleted' events, sometimes
! 	     ;; in random order.
! 	     ((eq system-type 'cygwin)
  	      (let (r)
  		(dotimes (_i n)
  		  (setq r (append '(changed deleted) r)))
--- 1228,1237 ----
  		(dotimes (_i n)
  		  (setq r (append '(deleted renamed) r)))
  		r))
! 	     ;; GFam{File,Directory}Monitor fire `changed' and
! 	     ;; `deleted' events, sometimes in random order.
! 	     ((memq (file-notify--test-monitor)
!                     '(GFamFileMonitor GFamDirectoryMonitor))
  	      (let (r)
  		(dotimes (_i n)
  		  (setq r (append '(changed deleted) r)))
***************
*** 1283,1291 ****
          (should (file-notify-valid-p file-notify--test-desc))
          (file-notify--test-with-actions
              (cond
!              ;; On cygwin we only get the `changed' event.
!              ((eq system-type 'cygwin)
!               '(changed))
               (t '(renamed created changed)))
            ;; The file is renamed when creating a backup.  It shall
            ;; still be watched.
--- 1314,1323 ----
          (should (file-notify-valid-p file-notify--test-desc))
          (file-notify--test-with-actions
              (cond
! 	     ;; GFam{File,Directory}Monitor report only the `changed' event.
! 	     ((memq (file-notify--test-monitor)
!                     '(GFamFileMonitor GFamDirectoryMonitor))
! 	      '(changed))
               (t '(renamed created changed)))
            ;; The file is renamed when creating a backup.  It shall
            ;; still be watched.
***************
*** 1355,1361 ****
          (should (file-notify-valid-p file-notify--test-desc1))
          (should (file-notify-valid-p file-notify--test-desc2))
          (should-not (equal file-notify--test-desc1 file-notify--test-desc2))
!         (let ((n 100))
            ;; Run the test.
            (file-notify--test-with-actions
                ;; There could be one or two `changed' events.
--- 1387,1393 ----
          (should (file-notify-valid-p file-notify--test-desc1))
          (should (file-notify-valid-p file-notify--test-desc2))
          (should-not (equal file-notify--test-desc1 file-notify--test-desc2))
!         (let ((n 10));0))
            ;; Run the test.
            (file-notify--test-with-actions
                ;; There could be one or two `changed' events.
***************
*** 1407,1416 ****
          ;; Now we delete the directory.
          (file-notify--test-with-actions
              (cond
!              ;; In kqueue and for cygwin, just one `deleted' event for
!              ;; the directory is received.
!              ((or (eq system-type 'cygwin)
! 		  (string-equal (file-notify--test-library) "kqueue"))
                '(deleted stopped))
               (t (append
                   ;; The directory monitor raises a `deleted' event for
--- 1439,1450 ----
          ;; Now we delete the directory.
          (file-notify--test-with-actions
              (cond
!              ;; GFam{File,Directory}Monitor and kqueue raise just one
!              ;; `deleted' event for the directory.
! 	     ((memq (file-notify--test-monitor)
!                     '(GFamFileMonitor GFamDirectoryMonitor))
!               '(deleted stopped))
! 	     ((string-equal (file-notify--test-library) "kqueue")
                '(deleted stopped))
               (t (append
                   ;; The directory monitor raises a `deleted' event for

^ permalink raw reply	[flat|nested] 30+ messages in thread

* bug#47262: [platform-testers] Emacs 27.2 RC2
  2021-03-23  9:32                       ` Michael Albinus
@ 2021-03-23 11:52                         ` Dagobert Michelsen
  2021-03-23 12:08                           ` Michael Albinus
  0 siblings, 1 reply; 30+ messages in thread
From: Dagobert Michelsen @ 2021-03-23 11:52 UTC (permalink / raw)
  To: Michael Albinus; +Cc: 47262

[-- Attachment #1: Type: text/plain, Size: 335 bytes --]

Hi Michael,

Am 23.03.2021 um 10:32 schrieb Michael Albinus <michael.albinus@gmx.de>:
> Thanks! Appended the next patch iteration. And I'm quite optimistic that
> it is the last one :-)
> 
> Could you pls check? As a reminder, pls call
> 
> # make -C test filenotify-tests.log SELECTOR='(not "remote")'

Please find the log attached.


[-- Attachment #2: filenotify-tests.log --]
[-- Type: application/octet-stream, Size: 1116 bytes --]

Running 11 tests (2021-03-23 12:17:03+0100, selector `(not "remote")')
Library: `gfilenotify'
Monitor: `GFamDirectoryMonitor'
   passed   1/11  file-notify-test00-availability (0.104734 sec)
   passed   2/11  file-notify-test01-add-watch (0.015828 sec)
   passed   3/11  file-notify-test02-rm-watch (1.126690 sec)
   passed   4/11  file-notify-test03-events (113.101214 sec)
Reverting buffer `file-notify-testNLyzfF'.
Reverting buffer `file-notify-testNLyzfF'.
   passed   5/11  file-notify-test04-autorevert (18.061405 sec)
   passed   6/11  file-notify-test05-file-validity (34.041062 sec)
   passed   7/11  file-notify-test06-dir-validity (3.018815 sec)
   passed   8/11  file-notify-test07-many-events (313.080747 sec)
   passed   9/11  file-notify-test08-backup (26.063838 sec)
   passed  10/11  file-notify-test09-watched-file-in-watched-dir (100.049620 sec)
  skipped  11/11  file-notify-test10-sufficient-resources (0.003505 sec)

Ran 11 tests, 10 results as expected, 0 unexpected, 1 skipped (2021-03-23 12:27:12+0100, 608.684862 sec)

1 skipped results:
  SKIPPED  file-notify-test10-sufficient-resources


[-- Attachment #3: Type: text/plain, Size: 214 bytes --]




Best regards

  — Dago

-- 
"You don't become great by trying to be great, you become great by wanting to do something,
and then doing it so hard that you become great in the process." - xkcd #896


^ permalink raw reply	[flat|nested] 30+ messages in thread

* bug#47262: [platform-testers] Emacs 27.2 RC2
  2021-03-23 11:52                         ` Dagobert Michelsen
@ 2021-03-23 12:08                           ` Michael Albinus
  0 siblings, 0 replies; 30+ messages in thread
From: Michael Albinus @ 2021-03-23 12:08 UTC (permalink / raw)
  To: Dagobert Michelsen; +Cc: 47262

Dagobert Michelsen <dam@opencsw.org> writes:

> Hi Michael,

Hi Dago,

>> Thanks! Appended the next patch iteration. And I'm quite optimistic that
>> it is the last one :-)
>
> Ran 11 tests, 10 results as expected, 0 unexpected, 1 skipped (2021-03-23 12:27:12+0100, 608.684862 sec)

So that's it, all tests pass :-)

I'll push the changed filenotify-tests.el to the emacs-27 branch after
the release of Emacs 27.2.

> Best regards
>
>   — Dago

Thanks again for your continuous support in testing!

Best regards, Michael.





^ permalink raw reply	[flat|nested] 30+ messages in thread

* bug#47262: [platform-testers] Emacs 27.2 RC2
  2021-03-19 13:42 ` bug#47262: [platform-testers] " Dagobert Michelsen
                     ` (2 preceding siblings ...)
  2021-03-20  9:04   ` Michael Albinus
@ 2021-10-21 21:19   ` Stefan Kangas
  2021-10-22  6:32     ` Dagobert Michelsen
  3 siblings, 1 reply; 30+ messages in thread
From: Stefan Kangas @ 2021-10-21 21:19 UTC (permalink / raw)
  To: Dagobert Michelsen; +Cc: 47262

Dagobert Michelsen <dam@opencsw.org> writes:

> Am 19.03.2021 um 08:35 schrieb Eli Zaretskii <eliz@gnu.org>:
>> The second release candidate for what will be the 27.2 release of Emacs
>> (the extensible text editor) is now available at
[...]
>
> I have a couple of test failures with Solaris 10 Sparc and Sun Studio:
>
> SUMMARY OF TEST RESULTS
> -----------------------
> Files examined: 267
> Ran 3829 tests, 77 failed to run, 3716 results as expected, 23 unexpected, 90 skipped
> 3 files did not finish:
>   src/emacs-module-tests.log
>   lisp/progmodes/sql-tests.log
>   lisp/auth-source-tests.log
> 8 files contained unexpected results:
>   lisp/vc/diff-mode-tests.log
>   lisp/progmodes/xref-tests.log
>   lisp/progmodes/flymake-tests.log
>   lisp/net/nsm-tests.log
>   lisp/net/network-stream-tests.log
>   lisp/files-tests.log
>   lisp/filenotify-tests.log
>   lisp/cedet/semantic-utest-ia.log
> gmake[2]: *** [Makefile:318: check-doit] Error 2
>
> All logfiles are available from the buildfarm at
>   http://buildfarm.opencsw.org/~dam/emacs-27.2-rc2-logs.tar.gz

Thank you for this.  We will release a new prerelease of Emacs soon, for
what will become the new Emacs 28.1, and if you could repeat these tests
for that version, that will be greatly appreciated.

Meanwhile, does it make sense to keep this bug report open?  I skimmed
the discussion and it seems like several issues were fixed, but perhaps
many of them still remain?





^ permalink raw reply	[flat|nested] 30+ messages in thread

* bug#47262: [platform-testers] Emacs 27.2 RC2
  2021-10-21 21:19   ` Stefan Kangas
@ 2021-10-22  6:32     ` Dagobert Michelsen
  2021-12-02 22:41       ` Stefan Kangas
  0 siblings, 1 reply; 30+ messages in thread
From: Dagobert Michelsen @ 2021-10-22  6:32 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: 47262

Hi Stefan,

Am 21.10.2021 um 23:19 schrieb Stefan Kangas <stefan@marxist.se>:
> Thank you for this.  We will release a new prerelease of Emacs soon, for
> what will become the new Emacs 28.1, and if you could repeat these tests
> for that version, that will be greatly appreciated.

Sure, just drop me a note prior to release.

> Meanwhile, does it make sense to keep this bug report open?  I skimmed
> the discussion and it seems like several issues were fixed, but perhaps
> many of them still remain?

Probably, maybe until it is verified that all issues are vanished in the
upcoming release.


Best regards

  — Dago 


-- 
"You don't become great by trying to be great, you become great by wanting to do something,
and then doing it so hard that you become great in the process." - xkcd #896






^ permalink raw reply	[flat|nested] 30+ messages in thread

* bug#47262: [platform-testers] Emacs 27.2 RC2
  2021-10-22  6:32     ` Dagobert Michelsen
@ 2021-12-02 22:41       ` Stefan Kangas
  2021-12-05 10:09         ` Dagobert Michelsen
  0 siblings, 1 reply; 30+ messages in thread
From: Stefan Kangas @ 2021-12-02 22:41 UTC (permalink / raw)
  To: Dagobert Michelsen; +Cc: 47262

Hi Dagobert,

Dagobert Michelsen <dam@opencsw.org> writes:

>> Thank you for this.  We will release a new prerelease of Emacs soon, for
>> what will become the new Emacs 28.1, and if you could repeat these tests
>> for that version, that will be greatly appreciated.
>
> Sure, just drop me a note prior to release.

I just sent the announcement for the first pretest of Emacs 28.1:

  The first pretest for what will be the 28.1 release of Emacs (the
  extensible text editor) is available at:

    https://alpha.gnu.org/gnu/emacs/pretest/emacs-28.0.90.tar.xz

  The tarball is signed; you can get the PGP signature file at:

    https://alpha.gnu.org/gnu/emacs/pretest/emacs-28.0.90.tar.xz.sig

Any testing you can provide is most appreciated.  Thanks in advance.





^ permalink raw reply	[flat|nested] 30+ messages in thread

* bug#47262: [platform-testers] Emacs 27.2 RC2
  2021-12-02 22:41       ` Stefan Kangas
@ 2021-12-05 10:09         ` Dagobert Michelsen
  2021-12-05 10:42           ` Eli Zaretskii
  0 siblings, 1 reply; 30+ messages in thread
From: Dagobert Michelsen @ 2021-12-05 10:09 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: 47262

Hi Stefan,

Am 02.12.2021 um 23:41 schrieb Stefan Kangas <stefan@marxist.se>:
> Dagobert Michelsen <dam@opencsw.org> writes:
> 
>>> Thank you for this.  We will release a new prerelease of Emacs soon, for
>>> what will become the new Emacs 28.1, and if you could repeat these tests
>>> for that version, that will be greatly appreciated.
>> 
>> Sure, just drop me a note prior to release.
> 
> I just sent the announcement for the first pretest of Emacs 28.1:
> 
>  The first pretest for what will be the 28.1 release of Emacs (the
>  extensible text editor) is available at:
> 
>    https://alpha.gnu.org/gnu/emacs/pretest/emacs-28.0.90.tar.xz
> 
>  The tarball is signed; you can get the PGP signature file at:
> 
>    https://alpha.gnu.org/gnu/emacs/pretest/emacs-28.0.90.tar.xz.sig
> 
> Any testing you can provide is most appreciated.  Thanks in advance.

I have a build failure on Solaris 10 x86 with this version:

gmake[1]: Entering directory '/home/dam/mgar/pkg/emacs/trunk/work/solaris10-i386/build-isa-pentium_pro-features-athena/emacs-28.0.90/lib-src'
/opt/csw/bin/gcc-5.5   -fno-common -Wall -Wdate-time -Wdisabled-optimization -Wdouble-promotion -Wextra -Wformat-signedness -Winit-self -Winvalid-pch -Wlogical-op -Wmissing-declarations -Wmissing-include-dirs -Wmissing-prototypes -Wnested-externs -Wold-style-definition -Wopenmp-simd -Wpacked -Wpointer-arith -Wstrict-prototypes -Wsuggest-attribute=format -Wsuggest-attribute=noreturn -Wsuggest-final-methods -Wsuggest-final-types -Wtrampolines -Wuninitialized -Wunknown-pragmas -Wunused-macros -Wvariadic-macros -Wvector-operation-performance -Warray-bounds=2 -Wformat=2 -Wredundant-decls -Wno-missing-field-initializers -Wno-override-init -Wno-sign-compare -Wno-type-limits -Wno-unused-parameter -Wno-format-nonliteral  -I. -I../src -I../lib -I. -I./../src -I./../lib  -m32 -march=pentiumpro -L/opt/csw/lib -L/usr/openwin/lib -I/opt/csw/include -O2 -pipe -fno-omit-frame-pointer -m32 -march=pentiumpro emacsclient.c \
    ../lib/libgnu.a -lsocket -lnsl \
    -lgen  -o emacsclient
Undefined                       first referenced
 symbol                             in file
acl_trivial                         ../lib/libgnu.a(file-has-acl.o)
ld: fatal: symbol referencing errors. No output written to emacsclient
collect2: error: ld returned 1 exit status
gmake[1]: *** [Makefile:401: emacsclient] Error 1


Best regards

  — Dago






^ permalink raw reply	[flat|nested] 30+ messages in thread

* bug#47262: [platform-testers] Emacs 27.2 RC2
  2021-12-05 10:09         ` Dagobert Michelsen
@ 2021-12-05 10:42           ` Eli Zaretskii
  2021-12-05 11:34             ` Dagobert Michelsen
  0 siblings, 1 reply; 30+ messages in thread
From: Eli Zaretskii @ 2021-12-05 10:42 UTC (permalink / raw)
  To: Dagobert Michelsen; +Cc: 47262, stefan

> From: Dagobert Michelsen <dam@opencsw.org>
> Date: Sun, 5 Dec 2021 11:09:22 +0100
> Cc: Eli Zaretskii <eliz@gnu.org>,
>  47262@debbugs.gnu.org
> 
> I have a build failure on Solaris 10 x86 with this version:
> 
> gmake[1]: Entering directory '/home/dam/mgar/pkg/emacs/trunk/work/solaris10-i386/build-isa-pentium_pro-features-athena/emacs-28.0.90/lib-src'
> /opt/csw/bin/gcc-5.5   -fno-common -Wall -Wdate-time -Wdisabled-optimization -Wdouble-promotion -Wextra -Wformat-signedness -Winit-self -Winvalid-pch -Wlogical-op -Wmissing-declarations -Wmissing-include-dirs -Wmissing-prototypes -Wnested-externs -Wold-style-definition -Wopenmp-simd -Wpacked -Wpointer-arith -Wstrict-prototypes -Wsuggest-attribute=format -Wsuggest-attribute=noreturn -Wsuggest-final-methods -Wsuggest-final-types -Wtrampolines -Wuninitialized -Wunknown-pragmas -Wunused-macros -Wvariadic-macros -Wvector-operation-performance -Warray-bounds=2 -Wformat=2 -Wredundant-decls -Wno-missing-field-initializers -Wno-override-init -Wno-sign-compare -Wno-type-limits -Wno-unused-parameter -Wno-format-nonliteral  -I. -I../src -I../lib -I. -I./../src -I./../lib  -m32 -march=pentiumpro -L/o
 pt/csw/lib -L/usr/openwin/lib -I/opt/csw/include -O2 -pipe -fno-omit-frame-pointer -m32 -march=pentiumpro emacsclient.c \
>     ../lib/libgnu.a -lsocket -lnsl \
>     -lgen  -o emacsclient
> Undefined                       first referenced
>  symbol                             in file
> acl_trivial                         ../lib/libgnu.a(file-has-acl.o)
> ld: fatal: symbol referencing errors. No output written to emacsclient
> collect2: error: ld returned 1 exit status
> gmake[1]: *** [Makefile:401: emacsclient] Error 1

Please try the latest emacs-28 branch (if you can), it's possible that
this was recently fixed.





^ permalink raw reply	[flat|nested] 30+ messages in thread

* bug#47262: [platform-testers] Emacs 27.2 RC2
  2021-12-05 10:42           ` Eli Zaretskii
@ 2021-12-05 11:34             ` Dagobert Michelsen
  2021-12-05 13:51               ` Eli Zaretskii
  0 siblings, 1 reply; 30+ messages in thread
From: Dagobert Michelsen @ 2021-12-05 11:34 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 47262, stefan

Hi Eli,

Am 05.12.2021 um 11:42 schrieb Eli Zaretskii <eliz@gnu.org>:
>> I have a build failure on Solaris 10 x86 with this version:
>> 
>> gmake[1]: Entering directory '/home/dam/mgar/pkg/emacs/trunk/work/solaris10-i386/build-isa-pentium_pro-features-athena/emacs-28.0.90/lib-src'
>> /opt/csw/bin/gcc-5.5   -fno-common -Wall -Wdate-time -Wdisabled-optimization -Wdouble-promotion -Wextra -Wformat-signedness -Winit-self -Winvalid-pch -Wlogical-op -Wmissing-declarations -Wmissing-include-dirs -Wmissing-prototypes -Wnested-externs -Wold-style-definition -Wopenmp-simd -Wpacked -Wpointer-arith -Wstrict-prototypes -Wsuggest-attribute=format -Wsuggest-attribute=noreturn -Wsuggest-final-methods -Wsuggest-final-types -Wtrampolines -Wuninitialized -Wunknown-pragmas -Wunused-macros -Wvariadic-macros -Wvector-operation-performance -Warray-bounds=2 -Wformat=2 -Wredundant-decls -Wno-missing-field-initializers -Wno-override-init -Wno-sign-compare -Wno-type-limits -Wno-unused-parameter -Wno-format-nonliteral  -I. -I../src -I../lib -I. -I./../src -I./../lib  -m32 -march=pentiumpro -L/opt/csw/lib -L/usr/openwin/lib -I/opt/csw/include -O2 -pipe -fno-omit-frame-pointer -m32 -march=pentiumpro emacsclient.c \
>>    ../lib/libgnu.a -lsocket -lnsl \
>>    -lgen  -o emacsclient
>> Undefined                       first referenced
>> symbol                             in file
>> acl_trivial                         ../lib/libgnu.a(file-has-acl.o)
>> ld: fatal: symbol referencing errors. No output written to emacsclient
>> collect2: error: ld returned 1 exit status
>> gmake[1]: *** [Makefile:401: emacsclient] Error 1
> 
> Please try the latest emacs-28 branch (if you can), it's possible that
> this was recently fixed.

Unfortunately this requires bootstrapping which is pretty hard in my
environment. Is there a chance for a bootstrapped tarball?


Best regards

  — Dago

-- 
"You don't become great by trying to be great, you become great by wanting to do something,
and then doing it so hard that you become great in the process." - xkcd #896






^ permalink raw reply	[flat|nested] 30+ messages in thread

* bug#47262: [platform-testers] Emacs 27.2 RC2
  2021-12-05 11:34             ` Dagobert Michelsen
@ 2021-12-05 13:51               ` Eli Zaretskii
  2022-06-25 12:00                 ` Lars Ingebrigtsen
  0 siblings, 1 reply; 30+ messages in thread
From: Eli Zaretskii @ 2021-12-05 13:51 UTC (permalink / raw)
  To: Dagobert Michelsen; +Cc: 47262, stefan

> From: Dagobert Michelsen <dam@opencsw.org>
> Date: Sun, 5 Dec 2021 12:34:41 +0100
> Cc: stefan@marxist.se,
>  47262@debbugs.gnu.org
> 
> Am 05.12.2021 um 11:42 schrieb Eli Zaretskii <eliz@gnu.org>:
> >> I have a build failure on Solaris 10 x86 with this version:
> >> 
> >> gmake[1]: Entering directory '/home/dam/mgar/pkg/emacs/trunk/work/solaris10-i386/build-isa-pentium_pro-features-athena/emacs-28.0.90/lib-src'
> >> /opt/csw/bin/gcc-5.5   -fno-common -Wall -Wdate-time -Wdisabled-optimization -Wdouble-promotion -Wextra -Wformat-signedness -Winit-self -Winvalid-pch -Wlogical-op -Wmissing-declarations -Wmissing-include-dirs -Wmissing-prototypes -Wnested-externs -Wold-style-definition -Wopenmp-simd -Wpacked -Wpointer-arith -Wstrict-prototypes -Wsuggest-attribute=format -Wsuggest-attribute=noreturn -Wsuggest-final-methods -Wsuggest-final-types -Wtrampolines -Wuninitialized -Wunknown-pragmas -Wunused-macros -Wvariadic-macros -Wvector-operation-performance -Warray-bounds=2 -Wformat=2 -Wredundant-decls -Wno-missing-field-initializers -Wno-override-init -Wno-sign-compare -Wno-type-limits -Wno-unused-parameter -Wno-format-nonliteral  -I. -I../src -I../lib -I. -I./../src -I./../lib  -m32 -march=pentiumpro -
 L/opt/csw/lib -L/usr/openwin/lib -I/opt/csw/include -O2 -pipe -fno-omit-frame-pointer -m32 -march=pentiumpro emacsclient.c \
> >>    ../lib/libgnu.a -lsocket -lnsl \
> >>    -lgen  -o emacsclient
> >> Undefined                       first referenced
> >> symbol                             in file
> >> acl_trivial                         ../lib/libgnu.a(file-has-acl.o)
> >> ld: fatal: symbol referencing errors. No output written to emacsclient
> >> collect2: error: ld returned 1 exit status
> >> gmake[1]: *** [Makefile:401: emacsclient] Error 1
> > 
> > Please try the latest emacs-28 branch (if you can), it's possible that
> > this was recently fixed.
> 
> Unfortunately this requires bootstrapping which is pretty hard in my
> environment. Is there a chance for a bootstrapped tarball?

Sorry, I cannot produce one.  maybe you could just apply the patches
from these two commits:

  fed35a89517aa4e282273f7e3c75bafd4e698ce4
  9c222b9c1a7f91497a37567b4d7de3a511fff069

Please apply them in the order shown, lest Patch fails.





^ permalink raw reply	[flat|nested] 30+ messages in thread

* bug#47262: [platform-testers] Emacs 27.2 RC2
  2021-12-05 13:51               ` Eli Zaretskii
@ 2022-06-25 12:00                 ` Lars Ingebrigtsen
  0 siblings, 0 replies; 30+ messages in thread
From: Lars Ingebrigtsen @ 2022-06-25 12:00 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: Dagobert Michelsen, 47262, stefan

Eli Zaretskii <eliz@gnu.org> writes:

> Sorry, I cannot produce one.  maybe you could just apply the patches
> from these two commits:
>
>   fed35a89517aa4e282273f7e3c75bafd4e698ce4
>   9c222b9c1a7f91497a37567b4d7de3a511fff069
>
> Please apply them in the order shown, lest Patch fails.

This bug report was about build errors of the previous release, and
skimming it, it seems like most of them are probably fixed in the
current release?  So I'm closing this bug report, but feel free to open
a new one if there are problems in 28.1.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 30+ messages in thread

end of thread, other threads:[~2022-06-25 12:00 UTC | newest]

Thread overview: 30+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-03-19  7:35 Emacs 27.2 RC2 Eli Zaretskii
2021-03-19 10:42 ` Robert Pluim
2021-03-19 12:19   ` Eli Zaretskii
2021-03-19 17:50     ` Robert Pluim
2021-03-19 19:44       ` Eli Zaretskii
2021-03-19 13:42 ` bug#47262: [platform-testers] " Dagobert Michelsen
2021-03-19 14:05   ` Eli Zaretskii
2021-03-19 18:03   ` Robert Pluim
2021-03-19 20:09     ` Eli Zaretskii
2021-03-20  9:04   ` Michael Albinus
2021-03-20 14:14     ` Dagobert Michelsen
2021-03-20 15:52       ` Michael Albinus
2021-03-20 20:02         ` Dagobert Michelsen
2021-03-21 12:48           ` Michael Albinus
2021-03-22 14:17             ` Dagobert Michelsen
2021-03-22 14:47               ` Michael Albinus
2021-03-23  7:38                 ` Dagobert Michelsen
2021-03-23  8:17                   ` Michael Albinus
2021-03-23  8:42                     ` Dagobert Michelsen
2021-03-23  9:32                       ` Michael Albinus
2021-03-23 11:52                         ` Dagobert Michelsen
2021-03-23 12:08                           ` Michael Albinus
2021-10-21 21:19   ` Stefan Kangas
2021-10-22  6:32     ` Dagobert Michelsen
2021-12-02 22:41       ` Stefan Kangas
2021-12-05 10:09         ` Dagobert Michelsen
2021-12-05 10:42           ` Eli Zaretskii
2021-12-05 11:34             ` Dagobert Michelsen
2021-12-05 13:51               ` Eli Zaretskii
2022-06-25 12:00                 ` Lars Ingebrigtsen

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.