unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Emacs pretest 28.0.91 is out
@ 2022-01-10 22:39 Stefan Kangas
  0 siblings, 0 replies; 17+ messages in thread
From: Stefan Kangas @ 2022-01-10 22:39 UTC (permalink / raw)
  To: emacs-devel

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

Hi!

The second 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.91.tar.xz

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

  https://alpha.gnu.org/gnu/emacs/pretest/emacs-28.0.91.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.

--------------------------------------

To verify that the tarball is intact, download both the .sig and
the tarball, and run this command:

  gpg --verify emacs-28.0.91.tar.xz.sig

If that command fails because you don't have the required public key,
run this command to import it:

  gpg --keyserver pgp.mit.edu --recv-keys \
    CEA1DE21AB108493CC9C65742E82323B8F4353EE

You can also run sha1sum or sha256sum and confirm that these
checksums match:

SHA1  emacs-28.0.91.tar.xz
1da31cce4002ae5a7a662fedd27add98fa8b8738

SHA256  emacs-28.0.91.tar.xz
7a9088567511d88a06dc1e1b12f748a0a26e9dcea594549ab855375225345057

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 511 bytes --]

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

* Re: Emacs pretest 28.0.91 is out
       [not found] <CADwFkmm1-qstLz2B80_wEZH3FjvAfdg-+2W1eMQg6J-6_wC7mw__6229.00487376086$1641854896$gmane$org@mail.gmail.com>
@ 2022-01-11 11:43 ` Robert Pluim
  2022-01-11 12:50   ` Michael Albinus
  2022-01-11 14:14   ` Robert Pluim
  0 siblings, 2 replies; 17+ messages in thread
From: Robert Pluim @ 2022-01-11 11:43 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: emacs-devel

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

>>>>> On Mon, 10 Jan 2022 16:39:40 -0600, Stefan Kangas <stefan@marxist.se> said:

    Stefan> Hi!
    Stefan> The second pretest for what will be the 28.1 release of Emacs (the
    Stefan> extensible text editor) is available at:

    Stefan>   https://alpha.gnu.org/gnu/emacs/pretest/emacs-28.0.91.tar.xz

I get three test failures on macOS with this (and zero on
GNU/Linux). The spelling test one is because I think my hunspell
installation is broken, but we can guard against it (see attached
patch). The other two I suspect are because 'gcc' here is 'clang', not
real gcc, so maybe those tests need to check for that. Logs attached.


1 files did not contain any tests:
  lisp/so-long-tests/spelling-tests.log
2 files contained unexpected results:
  lisp/progmodes/flymake-tests.log
  lisp/cedet/semantic/bovine/gcc-tests.log
make[2]: *** [check-doit] Error 2
make[1]: *** [check] Error 2
make: *** [check] Error 2


Robert
-- 


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-spelling-tests.el-trap-error.patch --]
[-- Type: text/x-diff, Size: 1240 bytes --]

From f7129d54b258cfaf5de374e7cc2c678c5bf490eb Mon Sep 17 00:00:00 2001
From: Robert Pluim <rpluim@gmail.com>
Date: Tue, 11 Jan 2022 12:20:44 +0100
Subject: [PATCH] spelling-tests.el: trap error
To: emacs-devel@gnu.org

'ispell-valid-dictionary-list' can signal an error, so guard against
that.

* test/lisp/so-long-tests/spelling-tests.el (so-long-spelling): Wrap
check for ispell-valid-dictionary-list in condition-case.
---
 test/lisp/so-long-tests/spelling-tests.el | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/test/lisp/so-long-tests/spelling-tests.el b/test/lisp/so-long-tests/spelling-tests.el
index eb1014dd8a..837ccd5600 100644
--- a/test/lisp/so-long-tests/spelling-tests.el
+++ b/test/lisp/so-long-tests/spelling-tests.el
@@ -40,7 +40,9 @@
            (condition-case ()
                (progn (ispell-check-version) t)
              (error nil))
-           (member "british" (ispell-valid-dictionary-list)))
+           (condition-case ()
+               (member "british" (ispell-valid-dictionary-list))
+             (error nil)))
   (ert-deftest so-long-spelling ()
     "Check the spelling in the source code."
     :tags '(:unstable) ;; It works for me, but I'm not sure about others.
-- 
2.34.0


[-- Attachment #3: spelling-tests.log --]
[-- Type: application/octet-stream, Size: 1437 bytes --]

Debugger entered--Lisp error: (error "Can't find Hunspell dictionary with a .aff affix f...")
  signal(error ("Can't find Hunspell dictionary with a .aff affix f..."))
  error("Can't find Hunspell dictionary with a .aff affix f...")
  ispell-find-hunspell-dictionaries()
  ispell-set-spellchecker-params()
  ispell-valid-dictionary-list()
  byte-code("\301\302!\210\301\303!\210\301\304!\210\10\203>\0\305\10!\203>\0\3061\"\0\307 \210\3100\202&\0\210\202>\0\203>\0\311\312 \235\203>\0\313\314..." [ispell-program-name require ert ispell cl-lib executable-find (error) ispell-check-version t "british" ispell-valid-dictionary-list ert-set-test so-long-spelling record ert-test "Check the spelling in the source code." #f(compiled-function () #<bytecode 0x63d714e81979f10>) nil :passed (:unstable)] 10)
  load("lisp/so-long-tests/spelling-tests" nil t)
  command-line-1(("-L" ":." "-L" "./../../elpa/packages/url-http-ntlm/" "-l" "./../../elpa/packages/url-http-ntlm/url-http-ntlm...." "-L" "./../../elpa/packages/web-server/" "-l" "./../../elpa/packages/web-server/web-server.el" "-l" "ert" "-L" "./../../elpa/packages/url-http-ntlm/" "-l" "./../../elpa/packages/url-http-ntlm/url-http-ntlm...." "-L" "./../../elpa/packages/web-server/" "-l" "./../../elpa/packages/web-server/web-server.el" "-l" "lisp/so-long-tests/spelling-tests" "--eval" "(ert-run-tests-batch-and-exit (quote (not (or (tag..."))
  command-line()
  normal-top-level()


[-- Attachment #4: flymake-tests.log --]
[-- Type: application/octet-stream, Size: 3141 bytes --]

Running 9 tests (2022-01-11 12:26:52+0100, selector `(not (or (tag :expensive-test) (tag :unstable) (tag :nativecomp)))')
Test different-diagnostic-types backtrace:
  signal(ert-test-failed (((should (eq 'flymake-error (face-at-point))
  ert-fail(((should (eq 'flymake-error (face-at-point))) :form (eq fly
  #f(compiled-function () #<bytecode -0x899aa44f89dbfa>)()
  flymake-tests--call-with-fixture(#f(compiled-function () #<bytecode 
  #f(compiled-function () #<bytecode -0x6f84e02efe25893>)()
  ert--run-test-internal(#s(ert--test-execution-info :test #s(ert-test
  ert-run-test(#s(ert-test :name different-diagnostic-types :documenta
  ert-run-or-rerun-test(#s(ert--stats :selector (not (or ... ... ...))
  ert-run-tests((not (or (tag :expensive-test) (tag :unstable) (tag :n
  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" "./../../elpa/packages/url-http-ntlm/
  command-line()
  normal-top-level()
Test different-diagnostic-types condition:
    (ert-test-failed
     ((should
       (eq 'flymake-error
	    (face-at-point)))
      :form
      (eq flymake-error flymake-warning)
      :value nil))
   FAILED  1/9  different-diagnostic-types (1.172453 sec)
   passed  2/9  dummy-backends (1.004546 sec)
   passed  3/9  eob-region-and-trailing-newline (0.000231 sec)
Test included-c-header-files backtrace:
  signal(ert-test-failed (((should (eq 'flymake-error (face-at-point))
  ert-fail(((should (eq 'flymake-error (face-at-point))) :form (eq fly
  #f(compiled-function () #<bytecode 0x6b624c6b834498c>)()
  flymake-tests--call-with-fixture(#f(compiled-function () #<bytecode 
  #f(compiled-function () #<bytecode 0x1b9738f3fe49e06d>)()
  ert--run-test-internal(#s(ert--test-execution-info :test #s(ert-test
  ert-run-test(#s(ert-test :name included-c-header-files :documentatio
  ert-run-or-rerun-test(#s(ert--stats :selector ... :tests ... :test-m
  ert-run-tests((not (or (tag :expensive-test) (tag :unstable) (tag :n
  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" "./../../elpa/packages/url-http-ntlm/
  command-line()
  normal-top-level()
Test included-c-header-files condition:
    (ert-test-failed
     ((should
       (eq 'flymake-error
	    (face-at-point)))
      :form
      (eq flymake-error flymake-warning)
      :value nil))
   FAILED  4/9  included-c-header-files (1.061344 sec)
   passed  5/9  perl-backend (1.037435 sec)
   passed  6/9  recurrent-backend (1.011210 sec)
   passed  7/9  ruby-backend (1.049725 sec)
   passed  8/9  warning-predicate-function-gcc (1.075085 sec)
   passed  9/9  warning-predicate-rx-gcc (1.063238 sec)

Ran 9 tests, 7 results as expected, 2 unexpected (2022-01-11 12:27:01+0100, 8.685472 sec)

2 unexpected results:
   FAILED  different-diagnostic-types
   FAILED  included-c-header-files


[-- Attachment #5: gcc-tests.log --]
[-- Type: application/octet-stream, Size: 1641 bytes --]

Running 2 tests (2022-01-11 12:25:00+0100, selector `(not (or (tag :expensive-test) (tag :unstable) (tag :nativecomp)))')
   passed  1/2  semantic-gcc-test-output-parser (0.000334 sec)
Test semantic-gcc-test-output-parser-this-machine backtrace:
  signal(error ("Test failed on \"Configured with: --prefix=/Library..
  error("Test failed on %S\nV H P:\n%S %S %S" "Configured with: --pref
  semantic-gcc-test-output-parser()
  #f(compiled-function () #<bytecode 0x1f3bfaba4f5f647f>)()
  ert--run-test-internal(#s(ert--test-execution-info :test #s(ert-test
  ert-run-test(#s(ert-test :name semantic-gcc-test-output-parser-this-
  ert-run-or-rerun-test(#s(ert--stats :selector (not (or (tag :expensi
  ert-run-tests((not (or (tag :expensive-test) (tag :unstable) (tag :n
  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" "./../../elpa/packages/url-http-ntlm/
  command-line()
  normal-top-level()
Test semantic-gcc-test-output-parser-this-machine condition:
    (error "Test failed on \"Configured with: --prefix=/Library/Developer/CommandLineTools/usr --with-gxx-include-dir=/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/c++/4.2.1\"\nV H P:\nnil \"x86_64-apple-darwin20.6.0\" nil")
   FAILED  2/2  semantic-gcc-test-output-parser-this-machine (0.029720 sec)

Ran 2 tests, 1 results as expected, 1 unexpected (2022-01-11 12:25:01+0100, 0.114861 sec)

1 unexpected results:
   FAILED  semantic-gcc-test-output-parser-this-machine


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

* Re: Emacs pretest 28.0.91 is out
  2022-01-11 11:43 ` Emacs pretest 28.0.91 is out Robert Pluim
@ 2022-01-11 12:50   ` Michael Albinus
  2022-01-11 13:25     ` Robert Pluim
  2022-01-11 14:14   ` Robert Pluim
  1 sibling, 1 reply; 17+ messages in thread
From: Michael Albinus @ 2022-01-11 12:50 UTC (permalink / raw)
  To: Robert Pluim; +Cc: Stefan Kangas, emacs-devel

Robert Pluim <rpluim@gmail.com> writes:

> +           (condition-case ()
> +               (member "british" (ispell-valid-dictionary-list))
> +             (error nil)))

aka (ignore-errors ...)

Best regards, Michael.



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

* Re: Emacs pretest 28.0.91 is out
  2022-01-11 12:50   ` Michael Albinus
@ 2022-01-11 13:25     ` Robert Pluim
  0 siblings, 0 replies; 17+ messages in thread
From: Robert Pluim @ 2022-01-11 13:25 UTC (permalink / raw)
  To: Michael Albinus; +Cc: Stefan Kangas, emacs-devel

>>>>> On Tue, 11 Jan 2022 13:50:23 +0100, Michael Albinus <michael.albinus@gmx.de> said:

    Michael> Robert Pluim <rpluim@gmail.com> writes:
    >> +           (condition-case ()
    >> +               (member "british" (ispell-valid-dictionary-list))
    >> +             (error nil)))

    Michael> aka (ignore-errors ...)

Sure, but this is an emacs-28 pre-test, so I donʼt want to start
making cleanups :-)

Robert
-- 



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

* Re: Emacs pretest 28.0.91 is out
  2022-01-11 11:43 ` Emacs pretest 28.0.91 is out Robert Pluim
  2022-01-11 12:50   ` Michael Albinus
@ 2022-01-11 14:14   ` Robert Pluim
  2022-01-11 17:06     ` Eli Zaretskii
  1 sibling, 1 reply; 17+ messages in thread
From: Robert Pluim @ 2022-01-11 14:14 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: emacs-devel

>>>>> On Tue, 11 Jan 2022 12:43:55 +0100, Robert Pluim <rpluim@gmail.com> said:

>>>>> On Mon, 10 Jan 2022 16:39:40 -0600, Stefan Kangas <stefan@marxist.se> said:
    Stefan> Hi!
    Stefan> The second pretest for what will be the 28.1 release of Emacs (the
    Stefan> extensible text editor) is available at:

    Stefan> https://alpha.gnu.org/gnu/emacs/pretest/emacs-28.0.91.tar.xz

    Robert> I get three test failures on macOS with this (and zero on
    Robert> GNU/Linux). The spelling test one is because I think my hunspell
    Robert> installation is broken, but we can guard against it (see attached
    Robert> patch). The other two I suspect are because 'gcc' here is 'clang', not
    Robert> real gcc, so maybe those tests need to check for that. Logs attached.

I see the latter two are fixed on master already. I guess since this
is macOS specific weʼre not going to backport those to emacs-28.

Robert
-- 



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

* Re: Emacs pretest 28.0.91 is out
  2022-01-11 14:14   ` Robert Pluim
@ 2022-01-11 17:06     ` Eli Zaretskii
  2022-01-11 17:45       ` Robert Pluim
  0 siblings, 1 reply; 17+ messages in thread
From: Eli Zaretskii @ 2022-01-11 17:06 UTC (permalink / raw)
  To: Robert Pluim; +Cc: stefan, emacs-devel

> From: Robert Pluim <rpluim@gmail.com>
> Date: Tue, 11 Jan 2022 15:14:33 +0100
> Cc: emacs-devel@gnu.org
> 
> I see the latter two are fixed on master already. I guess since this
> is macOS specific weʼre not going to backport those to emacs-28.

If the fixes are simple enough, we could backport them  The test suite
is not really critical for the success of a release, so we can install
changes in it which we won't in the "main" code.



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

* Re: Emacs pretest 28.0.91 is out
  2022-01-11 17:06     ` Eli Zaretskii
@ 2022-01-11 17:45       ` Robert Pluim
  2022-01-11 18:38         ` Eli Zaretskii
  2022-01-12 11:24         ` Michael Albinus
  0 siblings, 2 replies; 17+ messages in thread
From: Robert Pluim @ 2022-01-11 17:45 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: stefan, emacs-devel

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

>>>>> On Tue, 11 Jan 2022 19:06:44 +0200, Eli Zaretskii <eliz@gnu.org> said:

    >> From: Robert Pluim <rpluim@gmail.com>
    >> Date: Tue, 11 Jan 2022 15:14:33 +0100
    >> Cc: emacs-devel@gnu.org
    >> 
    >> I see the latter two are fixed on master already. I guess since this
    >> is macOS specific weʼre not going to backport those to emacs-28.

    Eli> If the fixes are simple enough, we could backport them  The test suite
    Eli> is not really critical for the success of a release, so we can install
    Eli> changes in it which we won't in the "main" code.

It would basically be the attached, although Iʼll note that the tests
theyʼre using to determine if gcc is clang are different.

Robert
-- 


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-Don-t-fail-flymake-tests-if-gcc-actually-is-Clang.patch --]
[-- Type: text/x-diff, Size: 1995 bytes --]

From b4632b2afb0aff4d1b2d1c7fd40d7ee26b228eed Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Mattias=20Engdeg=C3=A5rd?= <mattiase@acm.org>
Date: Sun, 2 Jan 2022 13:00:13 +0100
Subject: [PATCH 1/2] Don't fail flymake-tests if `gcc` actually is Clang
To: emacs-devel@gnu.org

* test/lisp/progmodes/flymake-tests.el (flymake-tests--gcc-is-clang)
(different-diagnostic-types, included-c-header-files): Skip tests that
depend on the `gcc` command really being GCC and not Clang.

(cherry picked from commit b2167d98432a78442522b7564e22f47d75a98b6f)
---
 test/lisp/progmodes/flymake-tests.el | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/test/lisp/progmodes/flymake-tests.el b/test/lisp/progmodes/flymake-tests.el
index 2eb4004f10..45cabf2efb 100644
--- a/test/lisp/progmodes/flymake-tests.el
+++ b/test/lisp/progmodes/flymake-tests.el
@@ -140,9 +140,15 @@ ruby-backend
             (should (eq 'flymake-error (face-at-point)))))
       (delete-directory tempdir t))))
 
+(defun flymake-tests--gcc-is-clang ()
+  "Whether the `gcc' command actually runs the Clang compiler."
+  (string-match "[Cc]lang version "
+                (shell-command-to-string "gcc --version")))
+
 (ert-deftest different-diagnostic-types ()
   "Test GCC warning via function predicate."
   (skip-unless (and (executable-find "gcc")
+                    (not (flymake-tests--gcc-is-clang))
                     (version<=
                      "5" (string-trim
                           (shell-command-to-string "gcc -dumpversion")))
@@ -166,7 +172,9 @@ different-diagnostic-types
 
 (ert-deftest included-c-header-files ()
   "Test inclusion of .h header files."
-  (skip-unless (and (executable-find "gcc") (executable-find "make")))
+  (skip-unless (and (executable-find "gcc")
+                    (not (flymake-tests--gcc-is-clang))
+                    (executable-find "make")))
   (let ((flymake-wrap-around nil))
     (flymake-tests--with-flymake
         ("some-problems.h")
-- 
2.34.0


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #3: 0002-Fix-test-lisp-cedet-semantic-bovine-gcc-tests-on-mac.patch --]
[-- Type: text/x-diff, Size: 1452 bytes --]

From d6c33d55def9f1e3021a418e6e3f3cf4be44cc68 Mon Sep 17 00:00:00 2001
From: Philipp Stephani <phst@google.com>
Date: Tue, 28 Dec 2021 21:05:23 +0100
Subject: [PATCH 2/2] Fix test lisp/cedet/semantic/bovine/gcc-tests on macOS
 (Bug#52431)
To: emacs-devel@gnu.org

* test/lisp/cedet/semantic/bovine/gcc-tests.el
(semantic-gcc-test-output-parser-this-machine): Also detect Apple
clang on macOS Monterey.

(cherry picked from commit 6e52becfbe2a33c025b8c4838b3c8f06ba5a6fb8)
---
 test/lisp/cedet/semantic/bovine/gcc-tests.el | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/test/lisp/cedet/semantic/bovine/gcc-tests.el b/test/lisp/cedet/semantic/bovine/gcc-tests.el
index 2ebd991679..2e61f91e58 100644
--- a/test/lisp/cedet/semantic/bovine/gcc-tests.el
+++ b/test/lisp/cedet/semantic/bovine/gcc-tests.el
@@ -127,8 +127,9 @@ semantic-gcc-test-output-parser-this-machine
     ;; Some macOS machines run llvm when you type gcc.  (!)
     ;; We can't even check if it's a symlink; it's a binary placed in
     ;; "/usr/bin/gcc".  So check the output and just skip this test if
-    ;; it says "Apple LLVM".
-    (unless (string-match "Apple LLVM" (car semantic-gcc-test-strings))
+    ;; it looks like that's the case.
+    (unless (string-match "Apple \\(LLVM\\|clang\\)\\|Xcode\\.app"
+                          (car semantic-gcc-test-strings))
         (semantic-gcc-test-output-parser))))
 
 ;;; gcc-tests.el ends here
-- 
2.34.0


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

* Re: Emacs pretest 28.0.91 is out
  2022-01-11 17:45       ` Robert Pluim
@ 2022-01-11 18:38         ` Eli Zaretskii
  2022-01-12  8:28           ` Robert Pluim
  2022-01-12 11:24         ` Michael Albinus
  1 sibling, 1 reply; 17+ messages in thread
From: Eli Zaretskii @ 2022-01-11 18:38 UTC (permalink / raw)
  To: Robert Pluim; +Cc: stefan, emacs-devel

> From: Robert Pluim <rpluim@gmail.com>
> Cc: stefan@marxist.se,  emacs-devel@gnu.org
> Date: Tue, 11 Jan 2022 18:45:09 +0100
> 
>     Eli> If the fixes are simple enough, we could backport them  The test suite
>     Eli> is not really critical for the success of a release, so we can install
>     Eli> changes in it which we won't in the "main" code.
> 
> It would basically be the attached, although Iʼll note that the tests
> theyʼre using to determine if gcc is clang are different.

I don't object to backporting those.



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

* Re: Emacs pretest 28.0.91 is out
  2022-01-11 18:38         ` Eli Zaretskii
@ 2022-01-12  8:28           ` Robert Pluim
  2022-01-12 13:35             ` Eli Zaretskii
  0 siblings, 1 reply; 17+ messages in thread
From: Robert Pluim @ 2022-01-12  8:28 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: stefan, emacs-devel

>>>>> On Tue, 11 Jan 2022 20:38:59 +0200, Eli Zaretskii <eliz@gnu.org> said:

    >> From: Robert Pluim <rpluim@gmail.com>
    >> Cc: stefan@marxist.se,  emacs-devel@gnu.org
    >> Date: Tue, 11 Jan 2022 18:45:09 +0100
    >> 
    Eli> If the fixes are simple enough, we could backport them  The test suite
    Eli> is not really critical for the success of a release, so we can install
    Eli> changes in it which we won't in the "main" code.
    >> 
    >> It would basically be the attached, although Iʼll note that the tests
    >> theyʼre using to determine if gcc is clang are different.

    Eli> I don't object to backporting those.

Done. What about trapping errors from ispell-valid-dictionary-list in
lisp/so-long-tests/spelling-tests.el? I mean, my hunspell installation
is borked, because brew on macOS doesnʼt install any dictionaries by
default, so itʼs not entirely a test suite issue.

Robert
-- 



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

* Re: Emacs pretest 28.0.91 is out
  2022-01-11 17:45       ` Robert Pluim
  2022-01-11 18:38         ` Eli Zaretskii
@ 2022-01-12 11:24         ` Michael Albinus
  2022-01-12 12:59           ` Robert Pluim
  1 sibling, 1 reply; 17+ messages in thread
From: Michael Albinus @ 2022-01-12 11:24 UTC (permalink / raw)
  To: Robert Pluim; +Cc: Eli Zaretskii, stefan, emacs-devel

Robert Pluim <rpluim@gmail.com> writes:

Hi Robert,

> +(defun flymake-tests--gcc-is-clang ()
> +  "Whether the `gcc' command actually runs the Clang compiler."
> +  (string-match "[Cc]lang version "
> +                (shell-command-to-string "gcc --version")))

I don't know whether this can happen here, but *if* default-directory is
remote, the test runs on a wrong machine. Therefore, I wrap such calls
always with

(let ((default-directory temporary-file-directory))
  ...)

Best regards, Michael.



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

* Re: Emacs pretest 28.0.91 is out
  2022-01-12 11:24         ` Michael Albinus
@ 2022-01-12 12:59           ` Robert Pluim
  2022-01-12 13:11             ` Michael Albinus
  0 siblings, 1 reply; 17+ messages in thread
From: Robert Pluim @ 2022-01-12 12:59 UTC (permalink / raw)
  To: Michael Albinus; +Cc: Eli Zaretskii, stefan, emacs-devel

>>>>> On Wed, 12 Jan 2022 12:24:28 +0100, Michael Albinus <michael.albinus@gmx.de> said:

    Michael> Robert Pluim <rpluim@gmail.com> writes:
    Michael> Hi Robert,

    >> +(defun flymake-tests--gcc-is-clang ()
    >> +  "Whether the `gcc' command actually runs the Clang compiler."
    >> +  (string-match "[Cc]lang version "
    >> +                (shell-command-to-string "gcc --version")))

    Michael> I don't know whether this can happen here, but *if* default-directory is
    Michael> remote, the test runs on a wrong machine. Therefore, I wrap such calls
    Michael> always with

    Michael> (let ((default-directory temporary-file-directory))
    Michael>   ...)

I canʼt think of a way that default-directory would be remote when
running the test suite (except inside tramp's tests).

Robert
-- 



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

* Re: Emacs pretest 28.0.91 is out
  2022-01-12 12:59           ` Robert Pluim
@ 2022-01-12 13:11             ` Michael Albinus
  0 siblings, 0 replies; 17+ messages in thread
From: Michael Albinus @ 2022-01-12 13:11 UTC (permalink / raw)
  To: Robert Pluim; +Cc: Eli Zaretskii, stefan, emacs-devel

Robert Pluim <rpluim@gmail.com> writes:

>     Michael> I don't know whether this can happen here, but *if* default-directory is
>     Michael> remote, the test runs on a wrong machine. Therefore, I wrap such calls
>     Michael> always with
>
>     Michael> (let ((default-directory temporary-file-directory))
>     Michael>   ...)
>
> I canʼt think of a way that default-directory would be remote when
> running the test suite (except inside tramp's tests).

Somebody could call the tests interactively, from a buffer with a remote
default-directory. Unlikely, but possible.

> Robert

Best regards, Michael.



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

* Re: Emacs pretest 28.0.91 is out
  2022-01-12  8:28           ` Robert Pluim
@ 2022-01-12 13:35             ` Eli Zaretskii
  2022-01-12 14:15               ` Robert Pluim
  0 siblings, 1 reply; 17+ messages in thread
From: Eli Zaretskii @ 2022-01-12 13:35 UTC (permalink / raw)
  To: Robert Pluim; +Cc: stefan, emacs-devel

> From: Robert Pluim <rpluim@gmail.com>
> Cc: stefan@marxist.se,  emacs-devel@gnu.org
> Date: Wed, 12 Jan 2022 09:28:43 +0100
> 
>     Eli> I don't object to backporting those.
> 
> Done.

Thanks.

> What about trapping errors from ispell-valid-dictionary-list in
> lisp/so-long-tests/spelling-tests.el? I mean, my hunspell installation
> is borked, because brew on macOS doesnʼt install any dictionaries by
> default, so itʼs not entirely a test suite issue.

I'm not sure I understand what errors you allude to.  Can you tell
more?  (spelling-tests.el skips all tests on my machine, it seems
because the test expects Ispell-style dictionary names, which doesn't
work with Hunspell.)



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

* Re: Emacs pretest 28.0.91 is out
  2022-01-12 13:35             ` Eli Zaretskii
@ 2022-01-12 14:15               ` Robert Pluim
  2022-01-12 14:22                 ` Eli Zaretskii
  0 siblings, 1 reply; 17+ messages in thread
From: Robert Pluim @ 2022-01-12 14:15 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: stefan, emacs-devel

>>>>> On Wed, 12 Jan 2022 15:35:42 +0200, Eli Zaretskii <eliz@gnu.org> said:

    >> What about trapping errors from ispell-valid-dictionary-list in
    >> lisp/so-long-tests/spelling-tests.el? I mean, my hunspell installation
    >> is borked, because brew on macOS doesnʼt install any dictionaries by
    >> default, so itʼs not entirely a test suite issue.

    Eli> I'm not sure I understand what errors you allude to.  Can you tell
    Eli> more?  (spelling-tests.el skips all tests on my machine, it seems
    Eli> because the test expects Ispell-style dictionary names, which doesn't
    Eli> work with Hunspell.)

From my original message:

Debugger entered--Lisp error: (error "Can't find Hunspell dictionary with a .aff affix f...")
  signal(error ("Can't find Hunspell dictionary with a .aff affix f..."))
  error("Can't find Hunspell dictionary with a .aff affix f...")
  ispell-find-hunspell-dictionaries()
  ispell-set-spellchecker-params()
  ispell-valid-dictionary-list()
  byte-code("\301\302!\210\301\303!\210\301\304!\210\10\203>\0\305\10!\203>\0\3061\"\0\307 \210\3100\202&\0\210\202>\0\203>\0\311\312 \235\203>\0\313\314..." [ispell-program-name require ert ispell cl-lib executable-find (error) ispell-check-version t "british" ispell-valid-dictionary-list ert-set-test so-long-spelling record ert-test "Check the spelling in the source code." #f(compiled-function () #<bytecode 0x63d714e81979f10>) nil :passed (:unstable)] 10)
  load("lisp/so-long-tests/spelling-tests" nil t)

This is because I have no hunspell dictionaries at all (and it would
be cleaner to define the test unconditionally, but skip it when
needed, but thatʼs another discussion).

Robert
-- 



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

* Re: Emacs pretest 28.0.91 is out
  2022-01-12 14:15               ` Robert Pluim
@ 2022-01-12 14:22                 ` Eli Zaretskii
  2022-01-12 14:33                   ` Robert Pluim
  0 siblings, 1 reply; 17+ messages in thread
From: Eli Zaretskii @ 2022-01-12 14:22 UTC (permalink / raw)
  To: Robert Pluim; +Cc: stefan, emacs-devel

> From: Robert Pluim <rpluim@gmail.com>
> Cc: stefan@marxist.se,  emacs-devel@gnu.org
> Date: Wed, 12 Jan 2022 15:15:08 +0100
> 
> Debugger entered--Lisp error: (error "Can't find Hunspell dictionary with a .aff affix f...")
>   signal(error ("Can't find Hunspell dictionary with a .aff affix f..."))
>   error("Can't find Hunspell dictionary with a .aff affix f...")
>   ispell-find-hunspell-dictionaries()
>   ispell-set-spellchecker-params()
>   ispell-valid-dictionary-list()
>   byte-code("\301\302!\210\301\303!\210\301\304!\210\10\203>\0\305\10!\203>\0\3061\"\0\307 \210\3100\202&\0\210\202>\0\203>\0\311\312 \235\203>\0\313\314..." [ispell-program-name require ert ispell cl-lib executable-find (error) ispell-check-version t "british" ispell-valid-dictionary-list ert-set-test so-long-spelling record ert-test "Check the spelling in the source code." #f(compiled-function () #<bytecode 0x63d714e81979f10>) nil :passed (:unstable)] 10)
>   load("lisp/so-long-tests/spelling-tests" nil t)
> 
> This is because I have no hunspell dictionaries at all (and it would
> be cleaner to define the test unconditionally, but skip it when
> needed, but thatʼs another discussion).

So you want to fix this?  I don't mind, but since it's (AFAIU)
something specific to your installation, is it important to fix on the
release branch?



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

* Re: Emacs pretest 28.0.91 is out
  2022-01-12 14:22                 ` Eli Zaretskii
@ 2022-01-12 14:33                   ` Robert Pluim
  2022-01-12 15:03                     ` Eli Zaretskii
  0 siblings, 1 reply; 17+ messages in thread
From: Robert Pluim @ 2022-01-12 14:33 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: stefan, emacs-devel

>>>>> On Wed, 12 Jan 2022 16:22:53 +0200, Eli Zaretskii <eliz@gnu.org> said:

    >> From: Robert Pluim <rpluim@gmail.com>
    >> Cc: stefan@marxist.se,  emacs-devel@gnu.org
    >> Date: Wed, 12 Jan 2022 15:15:08 +0100
    >> 
    >> Debugger entered--Lisp error: (error "Can't find Hunspell dictionary with a .aff affix f...")
    >> signal(error ("Can't find Hunspell dictionary with a .aff affix f..."))
    >> error("Can't find Hunspell dictionary with a .aff affix f...")
    >> ispell-find-hunspell-dictionaries()
    >> ispell-set-spellchecker-params()
    >> ispell-valid-dictionary-list()
    >> byte-code("\301\302!\210\301\303!\210\301\304!\210\10\203>\0\305\10!\203>\0\3061\"\0\307
    >> \210\3100\202&\0\210\202>\0\203>\0\311\312 \235\203>\0\313\314..."
    >> [ispell-program-name require ert ispell cl-lib executable-find
    >> (error) ispell-check-version t "british"
    >> ispell-valid-dictionary-list ert-set-test so-long-spelling record
    >> ert-test "Check the spelling in the source code."
    >> #f(compiled-function () #<bytecode 0x63d714e81979f10>) nil :passed
    >> (:unstable)] 10)
    >> load("lisp/so-long-tests/spelling-tests" nil t)
    >> 
    >> This is because I have no hunspell dictionaries at all (and it would
    >> be cleaner to define the test unconditionally, but skip it when
    >> needed, but thatʼs another discussion).

    Eli> So you want to fix this?  I don't mind, but since it's (AFAIU)
    Eli> something specific to your installation, is it important to fix on the
    Eli> release branch?

Iʼm not sure, which is why Iʼm asking you

Although, given that it works fine on GNU/Linux, perhaps let sleeping
tests lie. Who here hasn't pushed a last minute harmless bug fix that
caused problems? :-)

Robert
-- 



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

* Re: Emacs pretest 28.0.91 is out
  2022-01-12 14:33                   ` Robert Pluim
@ 2022-01-12 15:03                     ` Eli Zaretskii
  0 siblings, 0 replies; 17+ messages in thread
From: Eli Zaretskii @ 2022-01-12 15:03 UTC (permalink / raw)
  To: Robert Pluim; +Cc: stefan, emacs-devel

> From: Robert Pluim <rpluim@gmail.com>
> Cc: stefan@marxist.se,  emacs-devel@gnu.org
> Date: Wed, 12 Jan 2022 15:33:56 +0100
> 
>     Eli> So you want to fix this?  I don't mind, but since it's (AFAIU)
>     Eli> something specific to your installation, is it important to fix on the
>     Eli> release branch?
> 
> Iʼm not sure, which is why Iʼm asking you
> 
> Although, given that it works fine on GNU/Linux, perhaps let sleeping
> tests lie. Who here hasn't pushed a last minute harmless bug fix that
> caused problems? :-)

Yes, probably.

Thanks.



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

end of thread, other threads:[~2022-01-12 15:03 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <CADwFkmm1-qstLz2B80_wEZH3FjvAfdg-+2W1eMQg6J-6_wC7mw__6229.00487376086$1641854896$gmane$org@mail.gmail.com>
2022-01-11 11:43 ` Emacs pretest 28.0.91 is out Robert Pluim
2022-01-11 12:50   ` Michael Albinus
2022-01-11 13:25     ` Robert Pluim
2022-01-11 14:14   ` Robert Pluim
2022-01-11 17:06     ` Eli Zaretskii
2022-01-11 17:45       ` Robert Pluim
2022-01-11 18:38         ` Eli Zaretskii
2022-01-12  8:28           ` Robert Pluim
2022-01-12 13:35             ` Eli Zaretskii
2022-01-12 14:15               ` Robert Pluim
2022-01-12 14:22                 ` Eli Zaretskii
2022-01-12 14:33                   ` Robert Pluim
2022-01-12 15:03                     ` Eli Zaretskii
2022-01-12 11:24         ` Michael Albinus
2022-01-12 12:59           ` Robert Pluim
2022-01-12 13:11             ` Michael Albinus
2022-01-10 22:39 Stefan Kangas

Code repositories for project(s) associated with this public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).