From: Philipp Stephani <phst@google.com>
Date: Wed, 30 Dec 2020 23:05:14 +0000 (+0100)
Subject: Fix a potential unit test breakage on GNU/Linux.
X-Git-Tag: emacs-28.0.90~4471
X-Git-Url: http://git.eshelyaron.com/gitweb/?a=commitdiff_plain;h=f43f1e32e92ef341077b4ffdfdd743b1a6d9d284;p=emacs.git

Fix a potential unit test breakage on GNU/Linux.

* test/src/process-tests.el
(process-tests/fd-setsize-no-crash/make-process): Allow special exit
codes that can happen if terminal setup fails in the child process.
---

diff --git a/test/src/process-tests.el b/test/src/process-tests.el
index 3a19b6c6f14..590f72f9b03 100644
--- a/test/src/process-tests.el
+++ b/test/src/process-tests.el
@@ -525,7 +525,14 @@ FD_SETSIZE file descriptors (Bug#24325)."
               (dolist (process processes)
                 (while (accept-process-output process))
                 (should (eq (process-status process) 'exit))
-                (should (eql (process-exit-status process) 0))))))))))
+                ;; If there's an error between fork and exec, Emacs
+                ;; will use exit statuses between 125 and 127, see
+                ;; process.h.  This can happen if the child process
+                ;; tries to set up terminal device but fails due to
+                ;; file number limits.  We don't treat this as an
+                ;; error.
+                (should (memql (process-exit-status process)
+                               '(0 125 126 127)))))))))))
 
 (ert-deftest process-tests/fd-setsize-no-crash/make-pipe-process ()
   "Check that Emacs doesn't crash when trying to use more than