unofficial mirror of meta@public-inbox.org
 help / color / mirror / Atom feed
* [PATCH] t/run.perl: fix "make check-run" on FreeBSD 11.x
@ 2021-08-16  7:29 Eric Wong
  0 siblings, 0 replies; only message in thread
From: Eric Wong @ 2021-08-16  7:29 UTC (permalink / raw)
  To: meta

Persistent lei-daemon still leads to ECONNRESET client errors on
FreeBSD, and maxing out the kern.ipc.soacceptqueue sysctl (as
documented in the FreeBSD listen(2) manpage) doesn't seem to
help.

"make check-run" is still 4-5s faster than "make check" on my
FreeBSD VM even after this change, so it's still a worthwhile
improvement.
---
 t/run.perl | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/t/run.perl b/t/run.perl
index acd60ff5..f6eb0ad3 100755
--- a/t/run.perl
+++ b/t/run.perl
@@ -52,7 +52,11 @@ $run_log->autoflush(1); # one reader, many writers
 key2sub($_) for @tests; # precache
 
 my ($for_destroy, $lei_env, $lei_daemon_pid, $owner_pid);
-if (!$ENV{TEST_LEI_DAEMON_PERSIST_DIR} &&
+
+# I get ECONNRESET from lei on FreeBSD 11.3 even with
+# kern.ipc.soacceptqueue=1073741823, so persistent lei-daemon in tests
+# is Linux-only for now:
+if ($^O eq 'linux' && !$ENV{TEST_LEI_DAEMON_PERSIST_DIR} &&
 		(PublicInbox::Spawn->can('recv_cmd4') ||
 			eval { require Socket::MsgHdr })) {
 	$lei_env = {};

^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2021-08-16  7:29 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-16  7:29 [PATCH] t/run.perl: fix "make check-run" on FreeBSD 11.x Eric Wong

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).