]> Savannah Git Hosting - gnulib.git/commitdiff
test-pselect, test-select: use different ports
authorPaul Eggert <eggert@cs.ucla.edu>
Sun, 19 Mar 2023 23:06:34 +0000 (16:06 -0700)
committerPaul Eggert <eggert@cs.ucla.edu>
Sun, 19 Mar 2023 23:07:03 +0000 (16:07 -0700)
I have served rare and hard-to-reproduce problems with the GNU
grep release candidate with ‘make -j5 check’ on Fedora 37 x86-64.
One possibility is that test-pselect and test-select interfere
with each other somehow when run simultaneously, as they use the
same port.  Work around this possibility by using different ports
from each other, and from test-poll (which also uses 12345).
Of course it’d be better if all these tests used system-assigned
ports, but I assume that’d take more work.
* tests/test-pselect.c, tests/test-select.c (TEST_PORT): New macro.
* tests/test-select.h (TEST_PORT): Remove.

ChangeLog
tests/test-pselect.c
tests/test-select.c
tests/test-select.h

index dfce1a8df745f0dcb04a5c3158cfb5838e6f58d0..005b4a3c1ef0a800bb1ef9f533ad1207c6ce6e6e 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,17 @@
+2023-03-19  Paul Eggert  <eggert@cs.ucla.edu>
+
+       test-pselect, test-select: use different ports
+       I have served rare and hard-to-reproduce problems with the GNU
+       grep release candidate with ‘make -j5 check’ on Fedora 37 x86-64.
+       One possibility is that test-pselect and test-select interfere
+       with each other somehow when run simultaneously, as they use the
+       same port.  Work around this possibility by using different ports
+       from each other, and from test-poll (which also uses 12345).
+       Of course it’d be better if all these tests used system-assigned
+       ports, but I assume that’d take more work.
+       * tests/test-pselect.c, tests/test-select.c (TEST_PORT): New macro.
+       * tests/test-select.h (TEST_PORT): Remove.
+
 2023-03-19  Bruno Haible  <bruno@clisp.org>
 
        Update MODULES.html.sh.
index 41468684c5d3b17e13eaa7d1831ff90c786f4a3d..a383f1d1b271f2f6d20f5927d6d3a931e8eefb8f 100644 (file)
@@ -24,6 +24,7 @@ SIGNATURE_CHECK (pselect, int,
                  (int, fd_set *restrict, fd_set *restrict, fd_set *restrict,
                   struct timespec const *restrict, const sigset_t *restrict));
 
+#define TEST_PORT 12347
 #include "test-select.h"
 
 static int
index d04be584188b2c150fefe30082ceba503f537979..b460060351c43d4a509a11c771e6c292e0db7011 100644 (file)
@@ -25,6 +25,7 @@
 SIGNATURE_CHECK (select, int, (int, fd_set *, fd_set *, fd_set *,
                                struct timeval *));
 
+#define TEST_PORT 12346
 #include "test-select.h"
 
 int
index afa996f40c3ab5f71399d9f34f6e13b23471ce7d..ceeb4854713b0d8e87622c6197dc69157c153f6b 100644 (file)
@@ -37,8 +37,6 @@
 # include <sys/wait.h>
 #endif
 
-#define TEST_PORT       12345
-
 
 typedef int (*select_fn) (int, fd_set *, fd_set *, fd_set *, struct timeval *);